Re: [PATCH 3/3] sanitize building of fs/compat_ioctl.c

From: David S. Miller
Date: Tue Dec 13 2005 - 21:23:17 EST


From: Andi Kleen <ak@xxxxxxx>
Date: 14 Dec 2005 02:41:42 +0100

> "David S. Miller" <davem@xxxxxxxxxxxxx> writes:
>
> > What do you really still need it for at this point?
>
> input needs it :/ Take a look at drivers/input/evdev.c:evdev_write_compat
...
> I have given in for now. Assuming the test is done on a flag that is only set
> by the system call entry path. But I still think it will result in
> a lot of ugly code. For for read/write it's hard to avoid because
> there are so many variants and we have too many message passing
> protocols now.

I suppose. We could also funnel down ->compat_{read,write}() and
so on down the call chain, but that would likely be even uglier.

I guess with is_compat_task() we can do the netlink and pfkeyv2 compat
stuff on ia64/x86_64. I don't look forward to reviewing a patch
implementing that, however :-/

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