| 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@xxxxxxxxxxxxxxx> |
| Organization: | Open Source Development Lab |
| References: | <424C6089.1080507@xxxxxxxxxxxxxxx> |
| 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> |
|---|---|---|
| ||
| Previous by Date: | Re: RFC: Redirect-Device, David S. Miller |
|---|---|
| Next by Date: | Re: [RFC] netif_rx: receive path optimization, Jamal Hadi Salim |
| Previous by Thread: | Re: RFC: Redirect-Device, David S. Miller |
| Next by Thread: | Re: RFC: Redirect-Device, Ben Greear |
| Indexes: | [Date] [Thread] [Top] [All Lists] |