Thread (24 messages) 24 messages, 7 authors, 2010-08-16

Re: [PATCH v2] net: add Fast Ethernet driver for PXA168.

From: Lennert Buytenhek <hidden>
Date: 2010-08-10 18:01:55

On Tue, Aug 10, 2010 at 10:55:02AM -0700, Philip Rakity wrote:
We agree that extra header prepend space is needed.  The solution of
globally defining SKB_NET_PAD to increase this area for all drivers
is where I have some concern.  The solution certainly works but at
the cost of extra space for drivers that do not need to do this.
The solution is fine by me and maybe the best answer is to increase
the value in the standard linux implementation from 32 to 48 or 64 bytes. 
What I said was:

| This seems like something that, should you want to handle it, should be
| handled centrally (e.g. by conditionally increasing NET_SKB_PAD or so)
| and not hardcoded into one specific driver only.

In other words, "don't hardcode it into your driver just because you
think that most users will want to use your driver for bridging/routing
to wlan".

It makes no sense to categorically declare that "pxa168_eth will always
be used for routing to wireless, so we'll add some extra headroom by
default there, but e1000e will not, so there we won't".

Even if this stays a private kernel tree hack, I would still do it
globally rather than locally.

quoted
quoted
11n needs more.
Really?  Pointer?
Still curious about this.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help