| To: | "Christoph Hellwig" <hch@xxxxxxxxxxxxx> |
|---|---|
| Subject: | Re: [PATCH] (3/23) sk98: /proc interface related changes |
| From: | shemminger@xxxxxxxx |
| Date: | Fri, 12 Nov 2004 20:34:19 -0800 (PST) |
| Cc: | "Stephen Hemminger" <shemminger@xxxxxxxx>, "Jeff Garzik" <jgarzik@xxxxxxxxx>, "Michael Heyse" <mhk@xxxxxxxxxxxxxxxxx>, "Mirko Lindner" <mlindner@xxxxxxxxxxxxx>, netdev@xxxxxxxxxxx |
| Importance: | Normal |
| In-reply-to: | <20041112162641.GA7066@infradead.org> |
| References: | <4192C60A.1050205@designassembly.de> <20041111154225.5cf85567@zqx3.pdx.osdl.net> <20041111155349.4537044c@zqx3.pdx.osdl.net> <20041112162641.GA7066@infradead.org> |
| Sender: | netdev-bounce@xxxxxxxxxxx |
| User-agent: | SquirrelMail/1.4.2-1_osdl_00 |
> On Thu, Nov 11, 2004 at 03:53:49PM -0800, Stephen Hemminger wrote: >> The /proc intereface can be cleaned up a lot: >> + Pass the device to through the proc data/seq_private hook. >> + No longer needs to have a list of network devices >> + use seq_printf directly rather than indirection into a buffer >> + failure to create proc directory or interface should not >> be fatal. >> + if PROC_FS not configured, let the stubs cause the necessary >> code elimination rather than using #ifdef > > This is still totally broken because dev->name may change between > registration and removal. If a netdriver wants to use procfs file > per-adapter it needs to use an enumeration scheme independant of the > netdevice naming. Or even better stop using procfs at all. I would prefer to get rid of /proc interface altogether. I will unless SysKonnect says they have some application depending on it. There are a few statistics that don't show up in ethtool, but it wouldn't be hard to add those (or put them in sysfs). |
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | Re: A case AGAINST checksum offload, David S. Miller |
|---|---|
| Next by Date: | Re: [PATCH] (7/23) sk98: basic ethtool support, shemminger |
| Previous by Thread: | Re: [PATCH] (3/23) sk98: /proc interface related changes, Christoph Hellwig |
| Next by Thread: | [PATCH] (4/23) sk98: use netdev_priv, Stephen Hemminger |
| Indexes: | [Date] [Thread] [Top] [All Lists] |