Re: [Bridge] [PATCH] net/bridge: use the maximum hard_header_len of ports for bridging device

2009-03-23 Thread Li Yang
On Fri, Mar 20, 2009 at 5:04 PM, Li Yang le...@freescale.com wrote: The bridging device used a constant hard_header_len.  This will cause headroom shortage for ports with additional hardware header.  The patch makes bridging device to use the maximum value of all ports. Signed-off-by: Li Yang

Re: [Bridge] [PATCH] net/bridge: use the maximum hard_header_len of ports for bridging device

2009-03-23 Thread David Miller
From: Li Yang le...@freescale.com Date: Mon, 23 Mar 2009 16:15:34 +0800 I was wondering if this one failed to get your attention. yeah, happens all the time ___ Bridge mailing list Bridge@lists.linux-foundation.org

Re: [Bridge] [PATCH] net/bridge: use the maximum hard_header_len of ports for bridging device

2009-03-23 Thread Stephen Hemminger
On Fri, 20 Mar 2009 17:04:29 +0800 Li Yang le...@freescale.com wrote: The bridging device used a constant hard_header_len. This will cause headroom shortage for ports with additional hardware header. The patch makes bridging device to use the maximum value of all ports. Signed-off-by: Li

Re: [Bridge] [PATCH] net/bridge: use the maximum hard_header_len of ports for bridging device

2009-03-23 Thread David Miller
From: Stephen Hemminger shemmin...@linux-foundation.org Date: Mon, 23 Mar 2009 08:51:22 -0700 That ensures big enough header for locally generated packets, but any drivers that need bigger headroom still must handle bridged packets that come in with smaller space. When bridging packets, the

Re: [Bridge] [PATCH] net/bridge: use the maximum hard_header_len of ports for bridging device

2009-03-23 Thread Stephen Hemminger
On Mon, 23 Mar 2009 15:20:28 -0700 (PDT) David Miller da...@davemloft.net wrote: From: Stephen Hemminger shemmin...@linux-foundation.org Date: Mon, 23 Mar 2009 08:51:22 -0700 That ensures big enough header for locally generated packets, but any drivers that need bigger headroom still must

Re: [Bridge] [PATCH] net/bridge: use the maximum hard_header_len of ports for bridging device

2009-03-23 Thread David Miller
From: Stephen Hemminger shemmin...@linux-foundation.org Date: Mon, 23 Mar 2009 15:45:17 -0700 So you dynamically compute the additional space but if the space was an awkward size, could it cause driver to breaks alignment assumptions? Yes, you'd need to 16-byte align or something like that.