netdev
[Top] [All Lists]

Re: PATCH 2.4.0.9.2: export ethtool interface

To: "David S. Miller" <davem@xxxxxxxxxx>
Subject: Re: PATCH 2.4.0.9.2: export ethtool interface
From: Andi Kleen <ak@xxxxxx>
Date: Thu, 21 Sep 2000 14:30:42 +0200
Cc: ak@xxxxxx, andrewm@xxxxxxxxxx, becker@xxxxxxxxx, havanna_moon@xxxxxxx, netdev@xxxxxxxxxxx
In-reply-to: <200009211159.EAA09358@xxxxxxxxxxxxxxx>; from davem@xxxxxxxxxx on Thu, Sep 21, 2000 at 02:12:25PM +0200
References: <39C883CF.9FB262FC@xxxxxxxxxx> <Pine.LNX.4.10.10009201152510.1031-100000@xxxxxxxxxxxxx> <39C9F123.D8FA4F68@xxxxxxxxxx> <20000921133302.36264@xxxxxxxxxxxx> <200009211159.EAA09358@xxxxxxxxxxxxxxx>
Sender: owner-netdev@xxxxxxxxxxx
On Thu, Sep 21, 2000 at 02:12:25PM +0200, David S. Miller wrote:
>    From: Andi Kleen <ak@xxxxxx>
>    Date:   Thu, 21 Sep 2000 13:33:02 +0200
> 
>    There is (2.4.0test9pre2): 
> 
> Andi, mind taking a quick peek at net/socket.c:sock_ioctl() :-)

I hadn't expected such stupid code to survive in 2.4 @)

Looks rather wastefull. How about adding a inode or file flag for that ? 

Also I would propose to run dev->do_ioctl and probably other device
methods inside the BKL, they are not performance critical anyways and 
it is much safer.


-Andi

-- 
This is like TV. I don't like TV.

<Prev in Thread] Current Thread [Next in Thread>