Re: [Qemu-devel] [PATCH V3] e1000: drop check_rxov, always treat RX ring with RDH == RDT as empty

2012-10-19 Thread Dmitry Fleytman
Thanks Stefan, It was my very first idea to drop check_rxov and solve the problem, however for some reason I was sure that it required to emulate real HW behavior. I'm glad we clarified this. Regards, Dmitry Fleytman On Fri, Oct 19, 2012 at 9:52 AM, Stefan Hajnoczi wrote: > On Fri, Oct 19, 201

Re: [Qemu-devel] [PATCH V3] e1000: drop check_rxov, always treat RX ring with RDH == RDT as empty

2012-10-19 Thread Stefan Hajnoczi
On Fri, Oct 19, 2012 at 07:56:55AM +0200, Dmitry Fleytman wrote: > Real HW always treats RX ring with RDH == RDT as empty. > Emulation is supposed to behave the same. > > Reported-by: Chris Webb > Reported-by: Richard Davies > Signed-off-by: Dmitry Fleytman > --- > hw/e1000.c | 7 ++- > 1

[Qemu-devel] [PATCH V3] e1000: drop check_rxov, always treat RX ring with RDH == RDT as empty

2012-10-18 Thread Dmitry Fleytman
Real HW always treats RX ring with RDH == RDT as empty. Emulation is supposed to behave the same. Reported-by: Chris Webb Reported-by: Richard Davies Signed-off-by: Dmitry Fleytman --- hw/e1000.c | 7 ++- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/hw/e1000.c b/hw/e1000.c