| To: | Stephen Hemminger <shemminger@xxxxxxxx> |
|---|---|
| Subject: | Re: [PATCH] (3/23) sk98: /proc interface related changes |
| From: | Christoph Hellwig <hch@xxxxxxxxxxxxx> |
| Date: | Fri, 12 Nov 2004 16:26:42 +0000 |
| Cc: | Jeff Garzik <jgarzik@xxxxxxxxx>, Michael Heyse <mhk@xxxxxxxxxxxxxxxxx>, Mirko Lindner <mlindner@xxxxxxxxxxxxx>, netdev@xxxxxxxxxxx |
| In-reply-to: | <20041111155349.4537044c@xxxxxxxxxxxxxxxxx> |
| References: | <4192C60A.1050205@xxxxxxxxxxxxxxxxx> <20041111154225.5cf85567@xxxxxxxxxxxxxxxxx> <20041111155349.4537044c@xxxxxxxxxxxxxxxxx> |
| Sender: | netdev-bounce@xxxxxxxxxxx |
| User-agent: | Mutt/1.4.1i |
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. |
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | [PATCH] [Repost] IPSec: cleartext packets path for 2.6.9, Ilia Sotnikov/ASSTRA/BY |
|---|---|
| Next by Date: | Re: [PATCH] (7/23) sk98: basic ethtool support, Christoph Hellwig |
| Previous by Thread: | [PATCH] (3/23) sk98: /proc interface related changes, Stephen Hemminger |
| Next by Thread: | Re: [PATCH] (3/23) sk98: /proc interface related changes, shemminger |
| Indexes: | [Date] [Thread] [Top] [All Lists] |