Re: [E1000-devel] [PATCH] RX initialization sequence fixed - enable RX after corresponding ring initialization only

2012-10-22 Thread Jeff Kirsher
On Fri, 2012-10-19 at 20:19 +0100, Richard Davies wrote: > Jeff Kirsher wrote: > > Dmitry Fleytman wrote: > > > Reported-by: Chris Webb > > > Reported-by: Richard Davies > > > > > > Signed-off-by: Dmitry Fleytman > > > --- > > > drivers/net/ethernet/intel/e1000/e1000_ethtool.c |9 +

Re: [E1000-devel] [PATCH] RX initialization sequence fixed - enable RX after corresponding ring initialization only

2012-10-22 Thread Jeff Kirsher
On Fri, 2012-10-19 at 20:19 +0100, Richard Davies wrote: Jeff Kirsher wrote: Dmitry Fleytman wrote: Reported-by: Chris Webb chris.w...@elastichosts.com Reported-by: Richard Davies richard.dav...@elastichosts.com Signed-off-by: Dmitry Fleytman dmi...@daynix.com ---

Re: [E1000-devel] [PATCH] RX initialization sequence fixed - enable RX after corresponding ring initialization only

2012-10-19 Thread Dmitry Fleytman
Exactly. This patch is incorrect and should be ignored. Dmitry. On Fri, Oct 19, 2012 at 9:19 PM, Richard Davies wrote: > Jeff Kirsher wrote: >> Dmitry Fleytman wrote: >> > Reported-by: Chris Webb >> > Reported-by: Richard Davies >> > >> > Signed-off-by: Dmitry Fleytman >> > --- >> >

Re: [E1000-devel] [PATCH] RX initialization sequence fixed - enable RX after corresponding ring initialization only

2012-10-19 Thread Richard Davies
Jeff Kirsher wrote: > Dmitry Fleytman wrote: > > Reported-by: Chris Webb > > Reported-by: Richard Davies > > > > Signed-off-by: Dmitry Fleytman > > --- > > drivers/net/ethernet/intel/e1000/e1000_ethtool.c |9 + > > drivers/net/ethernet/intel/e1000/e1000_main.c| 18 > >

Re: [E1000-devel] [PATCH] RX initialization sequence fixed - enable RX after corresponding ring initialization only

2012-10-19 Thread Richard Davies
Jeff Kirsher wrote: Dmitry Fleytman wrote: Reported-by: Chris Webb chris.w...@elastichosts.com Reported-by: Richard Davies richard.dav...@elastichosts.com Signed-off-by: Dmitry Fleytman dmi...@daynix.com --- drivers/net/ethernet/intel/e1000/e1000_ethtool.c |9 +

Re: [E1000-devel] [PATCH] RX initialization sequence fixed - enable RX after corresponding ring initialization only

2012-10-19 Thread Dmitry Fleytman
Exactly. This patch is incorrect and should be ignored. Dmitry. On Fri, Oct 19, 2012 at 9:19 PM, Richard Davies richard.dav...@elastichosts.com wrote: Jeff Kirsher wrote: Dmitry Fleytman wrote: Reported-by: Chris Webb chris.w...@elastichosts.com Reported-by: Richard Davies

Re: [E1000-devel] [PATCH] RX initialization sequence fixed - enable RX after corresponding ring initialization only

2012-10-15 Thread Dmitry Fleytman
Thanks, Jeff According to what we see this problem is very old and exists in stable branches at least back to 2.6.x. Since we see this problem a lot on our servers and we cannot recompile some kernels we use, we are highly interested in backporting of this patch to older versions to make it

Re: [E1000-devel] [PATCH] RX initialization sequence fixed - enable RX after corresponding ring initialization only

2012-10-15 Thread Dmitry Fleytman
Thanks, Jeff According to what we see this problem is very old and exists in stable branches at least back to 2.6.x. Since we see this problem a lot on our servers and we cannot recompile some kernels we use, we are highly interested in backporting of this patch to older versions to make it

Re: [E1000-devel] [PATCH] RX initialization sequence fixed - enable RX after corresponding ring initialization only

2012-10-14 Thread Jeff Kirsher
On Sun, 2012-10-14 at 19:19 +0200, Dmitry Fleytman wrote: > Reported-by: Chris Webb > Reported-by: Richard Davies > > Signed-off-by: Dmitry Fleytman > --- > drivers/net/ethernet/intel/e1000/e1000_ethtool.c |9 + > drivers/net/ethernet/intel/e1000/e1000_main.c| 18 >

Re: [E1000-devel] [PATCH] RX initialization sequence fixed - enable RX after corresponding ring initialization only

2012-10-14 Thread Jeff Kirsher
On Sun, 2012-10-14 at 19:19 +0200, Dmitry Fleytman wrote: Reported-by: Chris Webb chris.w...@elastichosts.com Reported-by: Richard Davies richard.dav...@elastichosts.com Signed-off-by: Dmitry Fleytman dmi...@daynix.com --- drivers/net/ethernet/intel/e1000/e1000_ethtool.c |9 +