Re: [PATCH] ethtool.h should use userspace-accessible types

[Eric W. Biederman]

Jeff Garzik <jgarzik@xxxxxxxxx> writes:

> Eric W. Biederman wrote:
> > Eric Brower <ebrower@xxxxxxx> writes:
> >
> >>Attached is a patch to 2.4's ethtool.h to use appropriate,
> userspace-accessible
> 
> >>data types (__u8 and friends, rather than u8 and friends).
> > Why there is no #ifdef __KERNEL__ in this header to make it userspace
> > safe.
> 
> 
> Because it's not needed.

I think we are in agreement.

My intent was to say:  Why change the types when there is no #ifdef
__KERNEL__ in the header.  With no #ifdef __KERNEL__ it exports
definitions that are private to the kernel making it not safe for
userspace to use.  With kernel private definitions in there it will
generate name space pollution if included by user space.

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