lkml.org 
[lkml]   [2004]   [Mar]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH] ethtool.h should use userspace-accessible types
From
Date
Jeff Garzik <jgarzik@pobox.com> writes:

> Eric W. Biederman wrote:
> > Eric Brower <ebrower@usa.net> 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@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2005-03-22 14:01    [W:0.465 / U:0.076 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site