Messages in this thread | | | Subject | Re: Flaws with "UIO: Add the User IO core code" (with patch) | From | Thomas Gleixner <> | Date | Sat, 28 Apr 2007 22:38:35 +0200 |
| |
On Sat, 2007-04-28 at 21:24 +0100, Alan Cox wrote: > > > AFAIK we don't currently have any platform that runs binaries with > > > different sizes of "int" but this is a) an unsigned value anyway, and b) > > > should be a fixed type (eg u32) > > > > I reviewed the code once more and find it OK. There is only one legal > > value for the parameter "count" of uio_read(), and that's sizeof(int). > > If you are a box with multiple supported binary types how big is an > "int". We use explicit sizes to ensure that uio_read() will work when/if > we get platforms which support binaries with differing ideas of the size > of "int". Thus it should use s32 or s64 or similar.
Well, it is kind of academic, as probably half of the user space interfaces will explode, but in theory you are right.
Hans, let's change it to u32 to get this out of the way.
tglx
- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |