Re: patch: signed char issues

From: Philip Blundell (Philip.Blundell@pobox.com)
Date: Mon Apr 24 2000 - 04:41:03 EST


>> Here's a handful of patches for areas of the kernel that assume `char' is
>> a signed type. It was generated against 2.3.99-pre3 but it seems to =
>> apply cleanly against -pre5 as well.
>
>Are you sure this is really needed? On a machine with native unsigned char
>this might be a sizeable performance impact for each use.

Yes, there is a performance impact to using `signed char' on machines where
char is normally unsigned. But the situation without the patch is that the
code in question is completely broken, which is probably even worse.

Most of the signed char issues are in code that isn't performance critical
anyway.

p.

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.rutgers.edu
Please read the FAQ at http://www.tux.org/lkml/



This archive was generated by hypermail 2b29 : Sun Apr 30 2000 - 21:00:07 EST