| To: | Jeff Garzik <jgarzik@xxxxxxxxx> |
|---|---|
| Subject: | Re: [PATCH] netdev_ops |
| From: | Ben Greear <greearb@xxxxxxxxxxxxxxx> |
| Date: | Fri, 11 Jul 2003 13:07:07 -0700 |
| Cc: | Matthew Wilcox <willy@xxxxxxxxxx>, netdev@xxxxxxxxxxx, "David S. Miller" <davem@xxxxxxxxxx>, Arnaldo Carvalho de Melo <acme@xxxxxxxxxxxxxxxx> |
| In-reply-to: | <20030711195856.GB30449@xxxxxxx> |
| Organization: | Candela Technologies |
| References: | <20030708163042.GL23597@xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx> <3F0B2D30.4020102@xxxxxxxxxxxxxxx> <20030708212551.GL1939@xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx> <20030708.150835.78728697.davem@xxxxxxxxxx> <20030709161520.GW1939@xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx> <20030711193215.GH16037@xxxxxxx> <3F0F153C.4040506@xxxxxxxxxxxxxxx> <20030711195856.GB30449@xxxxxxx> |
| Sender: | netdev-bounce@xxxxxxxxxxx |
| User-agent: | Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.4) Gecko/20030529 |
Jeff Garzik wrote: Regardless, addressing your point, I consider ethtool.h a kernel-internal header, that's why it uses internal kernel types. Anybody who copies it to userspace must deal with that. It is _not_ intended to be #included directly from userspace. ethtool (the userland program) purposefully does its own typedefs and stuff. Any particular reason to not include it directly? It seems no more likely to cause problems than to use some potentially out-of-date copy in user-space. (And it might make the compile slightly tougher if you are distributing primarily as source.) Ben -- Ben Greear <greearb@xxxxxxxxxxxxxxx> <Ben_Greear AT excite.com> President of Candela Technologies Inc http://www.candelatech.com ScryMUD: http://scry.wanfear.com http://scry.wanfear.com/~greear |
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | Re: [PATCH] netdev_ops, Jeff Garzik |
|---|---|
| Next by Date: | Re: [PATCH] Move eth_mac_addr and eth_change_mtu, Matthew Wilcox |
| Previous by Thread: | Re: [PATCH] netdev_ops, Jeff Garzik |
| Next by Thread: | Re: [PATCH] netdev_ops, Matthew Wilcox |
| Indexes: | [Date] [Thread] [Top] [All Lists] |