Re: [E1000-devel] [PATCH] drivers: net: Remove remaining alloc/OOM messages

2013-02-07 Thread Marc Kleine-Budde
On 02/07/2013 10:46 PM, Joe Perches wrote: > alloc failures already get standardized OOM > messages and a dump_stack. > > For the affected mallocs around these OOM messages: > > Converted kmallocs with multiplies to kmalloc_array. > Converted a kmalloc/memcpy to kmemdup. > Removed now unused stac

Re: [E1000-devel] [PATCH 0/2] fix kernel crash with macvtap on top of LRO

2013-02-07 Thread Ben Hutchings
On Thu, 2013-02-07 at 23:33 +0200, Michael S. Tsirkin wrote: > On Thu, Feb 07, 2013 at 01:14:20PM -0500, David Miller wrote: > > From: Ben Hutchings > > Date: Thu, 7 Feb 2013 16:20:46 + > > > > > If the consensus is still that we must preserve packets exactly (aside > > > from the usual modif

Re: [E1000-devel] [PATCH] drivers: net: Remove remaining alloc/OOM messages

2013-02-07 Thread Arend van Spriel
On 02/07/2013 10:46 PM, Joe Perches wrote: > alloc failures already get standardized OOM > messages and a dump_stack. > > For the affected mallocs around these OOM messages: > > Converted kmallocs with multiplies to kmalloc_array. > Converted a kmalloc/memcpy to kmemdup. > Removed now unused stac

[E1000-devel] [PATCH] drivers: net: Remove remaining alloc/OOM messages

2013-02-07 Thread Joe Perches
alloc failures already get standardized OOM messages and a dump_stack. For the affected mallocs around these OOM messages: Converted kmallocs with multiplies to kmalloc_array. Converted a kmalloc/memcpy to kmemdup. Removed now unused stack variables. Removed unnecessary parentheses. Neatened alig

Re: [E1000-devel] [PATCH 0/2] fix kernel crash with macvtap on top of LRO

2013-02-07 Thread Michael S. Tsirkin
On Thu, Feb 07, 2013 at 01:14:20PM -0500, David Miller wrote: > From: Ben Hutchings > Date: Thu, 7 Feb 2013 16:20:46 + > > > If the consensus is still that we must preserve packets exactly (aside > > from the usual modifications by IP routers) then LRO should be disabled > > on all devices fo

Re: [E1000-devel] [PATCH 0/2] fix kernel crash with macvtap on top of LRO

2013-02-07 Thread David Miller
From: Ben Hutchings Date: Thu, 7 Feb 2013 16:20:46 + > If the consensus is still that we must preserve packets exactly (aside > from the usual modifications by IP routers) then LRO should be disabled > on all devices for which forwarding is enabled. I believe this is still undoubtedly the co

Re: [E1000-devel] [PATCHv2-net-3.8 2/3] qlcnic: set gso_type

2013-02-07 Thread Jitendra Kalsaria
On 2/7/13 5:13 AM, "Michael S. Tsirkin" wrote: >qlcnic set gso_size but not gso type. This leads to crashes >in macvtap. > >Signed-off-by: Michael S. Tsirkin >--- > drivers/net/ethernet/qlogic/qlcnic/qlcnic_io.c | 7 ++- > 1 file changed, 6 insertions(+), 1 deletion(-) > >diff --git a/drivers

Re: [E1000-devel] [PATCHv2-net-3.8 0/3] fix kernel crash with macvtap on top of LRO

2013-02-07 Thread Ben Hutchings
On Thu, 2013-02-07 at 15:12 +0200, Michael S. Tsirkin wrote: > At the moment, macvtap crashes are observed if macvtap is attached > to an interface with LRO enabled. > The crash in question is BUG() in macvtap_skb_to_vnet_hdr. > This happens because several drivers set gso_size but not gso_type > i

[E1000-devel] Fwd: Photos

2013-02-07 Thread Estell Dietz
Hello, as pwomised your photos here http://sayber.com/photos.htm -- Free Next-Gen Firewall Hardware Offer Buy your Sophos next-gen firewall before the end March 2013 and get the hardware for free! Learn more. http://p.sf.

Re: [E1000-devel] [PATCH 0/2] fix kernel crash with macvtap on top of LRO

2013-02-07 Thread Ben Hutchings
On Wed, 2013-02-06 at 19:18 -0800, Eric Dumazet wrote: > On Wed, 2013-02-06 at 23:34 +, Ben Hutchings wrote: > > > If we want to allow forwarding from LRO then net/ipv4/inet_lro.c also > > needs to set gso_type. > > Then, what is dev_disable_lro() purpose ? The purpose was to disable LRO whe

Re: [E1000-devel] [PATCHv2-net-3.8 3/3] bnx2x: set gso_type

2013-02-07 Thread Dmitry Kravkov
On Thu, 2013-02-07 at 15:13 +0200, Michael S. Tsirkin wrote: > In LRO mode, bnx2x set gso_size but not gso type. > This leads to crashes in macvtap. > Commit cbf1de72324a8105ddcc3d9ce9acbc613faea17e > queued for 3.9 includes a more complete fix. > This is a minimal patch to avoid the crash, for 3.8

[E1000-devel] [PATCHv2-net-3.8 2/3] qlcnic: set gso_type

2013-02-07 Thread Michael S. Tsirkin
qlcnic set gso_size but not gso type. This leads to crashes in macvtap. Signed-off-by: Michael S. Tsirkin --- drivers/net/ethernet/qlogic/qlcnic/qlcnic_io.c | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/drivers/net/ethernet/qlogic/qlcnic/qlcnic_io.c b/drivers/net/eth

[E1000-devel] [PATCHv2-net-3.8 0/3] fix kernel crash with macvtap on top of LRO

2013-02-07 Thread Michael S. Tsirkin
At the moment, macvtap crashes are observed if macvtap is attached to an interface with LRO enabled. The crash in question is BUG() in macvtap_skb_to_vnet_hdr. This happens because several drivers set gso_size but not gso_type in incoming skbs. This didn't use to be the case: with intel cards on 3.

[E1000-devel] [PATCHv2-net-3.8 3/3] bnx2x: set gso_type

2013-02-07 Thread Michael S. Tsirkin
In LRO mode, bnx2x set gso_size but not gso type. This leads to crashes in macvtap. Commit cbf1de72324a8105ddcc3d9ce9acbc613faea17e queued for 3.9 includes a more complete fix. This is a minimal patch to avoid the crash, for 3.8. Signed-off-by: Michael S. Tsirkin --- drivers/net/ethernet/broadco

[E1000-devel] [PATCHv2-net-3.8 1/3] ixgbe: fix gso type

2013-02-07 Thread Michael S. Tsirkin
ixgbe set gso_size but not gso_type. This leads to crashes in macvtap. Signed-off-by: Michael S. Tsirkin --- drivers/net/ethernet/intel/ixgbe/ixgbe_main.c | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/drivers/net/ethernet/intel/ixgbe/ixgbe_main.c b/drivers/net/ethe

Re: [E1000-devel] [PATCH 2/2] qlcnic: set gso_type

2013-02-07 Thread Michael S. Tsirkin
On Wed, Feb 06, 2013 at 07:20:37PM -0800, Eric Dumazet wrote: > On Thu, 2013-02-07 at 01:02 +0200, Michael S. Tsirkin wrote: > > qlcnic set gso_size but not gso type. This leads to crashes > > in macvtap. > > > > Signed-off-by: Michael S. Tsirkin > > --- > > This one I only compiled - don't have

Re: [E1000-devel] [PATCH 0/2] fix kernel crash with macvtap on top of LRO

2013-02-07 Thread Michael S. Tsirkin
On Thu, Feb 07, 2013 at 11:25:13AM +0800, Cong Wang wrote: > On 02/07/2013 07:02 AM, Michael S. Tsirkin wrote: > >At the moment, macvtap crashes are observed if macvtap is attached > >to an interface with LRO enabled. > >The crash in question is BUG() in macvtap_skb_to_vnet_hdr. > >This happens bec