[E1000-devel] e1000 rx emulation bug (Was: [PATCH] e1000: Reset rx ring index on receive overrun)

2012-05-18 Thread Samuel Thibault
Hello, There seems to be a bug in qemu in the e1000 emulation, which triggers an issue with the Linux driver. What happens in Linux is the following: - e1000_open - e1000_configure - e1000_setup_rctl - enables E1000_RCTL_EN - e1000_configure_rx - sets RDT/RDH to 0 - al

Re: [E1000-devel] [PATCH] e1000: Reset rx ring index on receive overrun

2012-05-17 Thread Samuel Thibault
Brandeburg, Jesse, le Thu 17 May 2012 17:04:04 -0700, a écrit : > On Thu, 17 May 2012, Samuel Thibault wrote: > > > Samuel Thibault, le Fri 18 May 2012 01:28:21 +0200, a écrit : > > > Dave, Tushar N, le Thu 17 May 2012 23:22:31 +, a écrit : > > > > I am intere

Re: [E1000-devel] [PATCH] e1000: Reset rx ring index on receive overrun

2012-05-17 Thread Samuel Thibault
Samuel Thibault, le Fri 18 May 2012 01:28:21 +0200, a écrit : > Dave, Tushar N, le Thu 17 May 2012 23:22:31 +, a écrit : > > I am interested in to see if you have actual test case and more importantly > > test data that shows that kernel and device indexes are not synchronize

Re: [E1000-devel] [PATCH] e1000: Reset rx ring index on receive overrun

2012-05-17 Thread Samuel Thibault
Dave, Tushar N, le Thu 17 May 2012 23:22:31 +, a écrit : > I am interested in to see if you have actual test case and more importantly > test data that shows that kernel and device indexes are not synchronized any > more. Well, it's not with an actual physical device, but with the kvm emulat

[E1000-devel] [PATCH] e1000: Reset rx ring index on receive overrun

2012-05-17 Thread Samuel Thibault
At high traffic rate, the rx ring may get completely filled before we manage to consume it. After it is filled, the kernel and device indexes are not synchronized any more, so we have to reset them, otherwise the kernel will be stuck waiting for the wrong slot to be filled. Signed-off-by: Samuel