netdev
[Top] [All Lists]

Re: RFC: Redirect-Device

To: Ben Greear <greearb@xxxxxxxxxxxxxxx>
Subject: Re: RFC: Redirect-Device
From: Stephen Hemminger <shemminger@xxxxxxxx>
Date: Thu, 31 Mar 2005 13:05:12 -0800
Cc: "'netdev@xxxxxxxxxxx'" <netdev@xxxxxxxxxxx>
In-reply-to: <424C6089.1080507@candelatech.com>
Organization: Open Source Development Lab
References: <424C6089.1080507@candelatech.com>
Sender: netdev-bounce@xxxxxxxxxxx
You need to cleanup the style.  
        Lots of dead commented out code.
        Encapsulating locking in macros is ugly and makes maintenance a 
nuisance.
       CamelCaps
        Use prdebug instead of DEBUG
        RCU instead of rwlock's or just use spinlock
        Adding new /proc interfaces is frowned on
        Adding new ioctl's are very frowned on
        global namespace issues with toupper et a..

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