From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1Mp3Sm-0001Yi-Q2 for qemu-devel@nongnu.org; Sat, 19 Sep 2009 13:11:08 -0400 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1Mp3Sh-0001X0-9Z for qemu-devel@nongnu.org; Sat, 19 Sep 2009 13:11:08 -0400 Received: from [199.232.76.173] (port=37184 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1Mp3Sh-0001Wx-6T for qemu-devel@nongnu.org; Sat, 19 Sep 2009 13:11:03 -0400 Received: from moutng.kundenserver.de ([212.227.17.8]:64146) by monty-python.gnu.org with esmtp (Exim 4.60) (envelope-from ) id 1Mp3Sg-0001K7-I3 for qemu-devel@nongnu.org; Sat, 19 Sep 2009 13:11:02 -0400 Message-ID: <4AB510A1.9010702@mail.berlios.de> Date: Sat, 19 Sep 2009 19:10:57 +0200 From: Stefan Weil MIME-Version: 1.0 References: <4AB4BD2B.4090809@web.de> <4AB4C761.5030909@mail.berlios.de> In-Reply-To: <4AB4C761.5030909@mail.berlios.de> Content-Type: multipart/mixed; boundary="------------060307040807060206010408" Subject: [Qemu-devel] Re: eepro100.c List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Michal Filka Cc: qemu-devel , "kvm@vger.kernel.org" This is a multi-part message in MIME format. --------------060307040807060206010408 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Stefan Weil schrieb: > Jan Kiszka schrieb: >> Michal Filka wrote: >>> Hi, >>> >>> I have question on i82557b emulation in kvm. >>> >>> I have run a proprietary operating system (Pharlap OS) in kvm. Part >>> of the job has been porting driver for i8255x. During the job I >>> discovered that kvm's emulation doesn't support a RNR interrupt (it >>> is disabled in the code). >>> >>> Why is it disabled? I'm running heavily loaded application and I have >>> to poll descriptor list instead of waiting for RNR interrupt. >> Best asked on qemu-devel, CC'ing the developer of this device model. >> >> Jan >> > > Hi, > > up to now, nobody complained about the missing RNR interrupts. > Linux obviously works without them, so I had no reason to > implement them. > > The disabled code parts (function eepro100_rnr_interrupt) > can be enabled, but you still need code which calls them > at the right places. > > This should not be very difficult if there is some test case > which proves the working RNR interrupt. > > Regards > Stefan > Hi Michal, could you please try the appended patch and tell me whether it works? Regards Stefan --------------060307040807060206010408 Content-Type: text/x-diff; name="0001-eepro100-Add-RNR-interrupt.patch" Content-Transfer-Encoding: 7bit Content-Disposition: inline; filename="0001-eepro100-Add-RNR-interrupt.patch" >>From cb12daa2f7ecbf041655825054a1661668ab7274 Mon Sep 17 00:00:00 2001 From: Stefan Weil Date: Sat, 19 Sep 2009 19:05:07 +0200 Subject: [PATCH] eepro100: Add RNR interrupt Set RNR (receive unit not ready) interrupt for RU abort command. Signed-off-by: Stefan Weil --- hw/eepro100.c | 15 +++++++++++---- 1 files changed, 11 insertions(+), 4 deletions(-) diff --git a/hw/eepro100.c b/hw/eepro100.c index a8f6b3f..6d49bb8 100644 --- a/hw/eepro100.c +++ b/hw/eepro100.c @@ -114,7 +114,7 @@ #define RU_NOP 0x0000 #define RX_START 0x0001 #define RX_RESUME 0x0002 -#define RX_ABORT 0x0004 +#define RU_ABORT 0x0004 #define RX_ADDR_LOAD 0x0006 #define RX_RESUMENR 0x0007 #define INT_MASK 0x0100 @@ -363,13 +363,11 @@ static void eepro100_fr_interrupt(EEPRO100State * s) eepro100_interrupt(s, 0x40); } -#if 0 static void eepro100_rnr_interrupt(EEPRO100State * s) { /* RU is not ready. */ eepro100_interrupt(s, 0x10); } -#endif static void eepro100_mdi_interrupt(EEPRO100State * s) { @@ -865,6 +863,13 @@ static void eepro100_ru_command(EEPRO100State * s, uint8_t val) } set_ru_state(s, ru_ready); break; + case RU_ABORT: + /* RU abort. */ + if (get_ru_state(s) == ru_ready) { + eepro100_rnr_interrupt(s); + } + set_ru_state(s, ru_idle); + break; case RX_ADDR_LOAD: /* Load RU base. */ TRACE(OTHER, logout("val=0x%02x (RU base address)\n", val)); @@ -872,7 +877,7 @@ static void eepro100_ru_command(EEPRO100State * s, uint8_t val) break; default: logout("val=0x%02x (undefined RU command)\n", val); - missing("Undefined SU command"); + missing("Undefined RU command"); } } @@ -1531,6 +1536,8 @@ static ssize_t nic_receive(VLANClientState *vc, const uint8_t * buf, size_t size if (get_ru_state(s) != ru_ready) { /* No resources available. */ logout("no resources, state=%u\n", get_ru_state(s)); + /* TODO: RNR interrupt only at first failed frame? */ + eepro100_rnr_interrupt(s); s->statistics.rx_resource_errors++; //~ assert(!"no resources"); return -1; -- 1.5.6.5 --------------060307040807060206010408--