There is a reason that we didn't use the ones in the kernel. It is that we need to have portability to other platforms. (There are other issues too...., but for the moment, that is the most important.) This is not to say that I am unhappy to see somebody working on something--but this just isn't a patch that the main tree can use (or that, for GPL vs. LGPL reasons, can really be included at all).
Also, while we are on the subject, util-nexus.c is, if I remember correctly, no just part of the linux kernel module, but it is part of all kernel modules. Please check for things like this before submitting a platform specific (if you know it to be such) patch. Also, please check the mailinglist, as somebody may have discussed patching that code before (and may be in the process of doing it...), or we may have decided not to do that seemingly obvious thing that you are about to patch to for some not so obvious reason. [EMAIL PROTECTED] wrote: > > Hi. I made a change so plex86 uses the Linux kernel's memcpy(), memset(), > etc functions instead of the ones in util-nexus.c. The kernel's versions > are written in x86 assembly, and are optimized based on which CPU type you > select when you build the kernel. I figure they have to be faster than > gcc's version of unoptimized c-code. Also, this is for the kernel module, > so it only makes sense... > > The patch is included. It's against the cvs tree I got last night. Let > me know if you have any problems. > > ------------------------------------------------------------------------ > Name: plex86.2.diff > plex86.2.diff Type: Plain Text (TEXT/PLAIN) > Encoding: BASE64 -- |^^^ | | |^^| |^^^ Drew Northup, N1XIM |^^| | |^^^ \ / /^^\ /^^~ |__ | | | | | |__| | |___ \/ |__| |__ | | | | | | www.plex86.org | | | /\ | | | \ ___| |__| |__| |___ web.syr.edu/~suoc/ | |___ |___ / \ \__/ \__/
