netdev
[Top] [All Lists]

Re: [PATCH] (3/23) sk98: /proc interface related changes

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>