Wine performance improvement
Wolfram Sang
wolfram at the-dreams.de
Wed Dec 9 16:48:14 CST 2009
Stefan Dösinger wrote:
> Am 09.12.2009 um 09:36 schrieb 임은지:
>> Thank you for your comments. I will cleanup the patch as you
>> suggested and send it.
> Did you contact the kernel maintainers regarding the kernel side of
> your changes? I think including the Wine patch doesn't make sense
> until the kernel developers accept the kernel side of it.
In the current form, this is unlikely. There are a number of formal
issues alone:
- needs to be based on a somewhat recent version of the kernel
- too much #ifdef
- just one big patch
- too much debug output
- too much code commented out
- commenting not consistent (//)
The latter three can be easily worked out, of course. The others are a
bit more troublesome, but are major show-stoppers nevertheless.
Also, adding syscalls is a very, very sensitive area. You really need a
gallore of good arguments for that. IMHO I have doubts here, because the
rest of the code looks also a bit intrusive to me (i.e. patching
fork|sched.c).
Then again, was it really the aim to send this upstream? I didn't get
the impression when I read the original posting, more like "We did
something interesting. Maybe you want to try it?"
Kind regards,
Wolfram
More information about the wine-devel
mailing list