Re: [PATCH 2/7] fat: Fix VFAT_IOCTL_READDIR_xxx and cleanup foruserland

From: Christoph Hellwig
Date: Tue Jul 01 2008 - 03:40:47 EST


On Tue, Jul 01, 2008 at 11:57:03AM +0900, OGAWA Hirofumi wrote:
>
> "struct dirent" is a kernel type here, but is a **different type** in
> userspace! This means both the structure and the IOCTL number is wrong!
>
> So, this adds new "struct __fat_dirent" to generate correct IOCTL
> number. And kernel stuff moves to under __KERNEL__.

Given that the current version can't actually work without defininig
it's own dirent and thus ioctl number symbolic name I wonder if these
ioctls are used at all? They must have been completely untested for
a while, and I suspect we'd be better off just removing them.

--
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/