From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1MDqSM-0001b6-B7 for qemu-devel@nongnu.org; Mon, 08 Jun 2009 21:48:54 -0400 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1MDqSH-0001aN-MS for qemu-devel@nongnu.org; Mon, 08 Jun 2009 21:48:53 -0400 Received: from [199.232.76.173] (port=55258 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1MDqSH-0001aK-Jw for qemu-devel@nongnu.org; Mon, 08 Jun 2009 21:48:49 -0400 Received: from mail.windriver.com ([147.11.1.11]:59114 helo=mail.wrs.com) by monty-python.gnu.org with esmtp (Exim 4.60) (envelope-from ) id 1MDqSH-000885-0M for qemu-devel@nongnu.org; Mon, 08 Jun 2009 21:48:49 -0400 Received: from ALA-MAIL03.corp.ad.wrs.com (ala-mail03 [147.11.57.144]) by mail.wrs.com (8.13.6/8.13.6) with ESMTP id n591mfJg021289 for ; Mon, 8 Jun 2009 18:48:41 -0700 (PDT) Received: from unknown-46-211 (IDENT:U2FsdGVkX186Fr//txBzHDipcn/GcbDdrXe3LCKWHEk@[147.11.46.211]) by huisne.wrs.com (8.9.1/8.9.1) with ESMTP id SAA20018 for ; Mon, 8 Jun 2009 18:48:41 -0700 (PDT) From: Bill Paul Date: Mon, 8 Jun 2009 18:48:39 -0700 MIME-Version: 1.0 Content-Disposition: inline Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: 7bit Message-Id: <200906081848.39468.wpaul@windriver.com> Subject: [Qemu-devel] bug report + fix: e1000.c in 0.10.5 does not properly emulate real hardware List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Hi, I hope this is the right forum for this. Apologies if it's not. I downloaded QEMU 0.10.5 and tested it against VxWorks 6.7 using the e1000 emulated network interface, and ran into a couple of problems. The VxWorks Intel PRO/1000 driver has been tested against a real Intel 82540EM adapter, and it works fine, however it does not work with the emulated 82540 in QEMU, because it doesn't quite duplicate the behavior of real hardware. There are two issues: 1) The ICS register is not emulated correctly. It's not easy to discern from the Intel documentation, but the ICS register can be used in place of the ICR register in order to read the currently pending interrupt sources without automatically clearing them. The VxWorks driver needs to check interrupt events twice: once in its ISR, and again in task context. The auto-clear behavior of ICR makes it undesirable to use in the interrupt service routine, since it will clear the interrupt events, preventing the task level code from seeing them too (unless you preserve the values in software, which is tricky to do correcly). Consequently, VxWorks reads the ICS register in its interrupt service routine instead. This doesn't work in QEMU because: - There is no entry in the readops table for reading the ICS register, so reading it always returns 0. - The ICS register contents are not updated to reflect pending events in the set_interrupt_cause() routine. 2) The EERD register is not emulated correctly, which breaks VxWorks' EEPROM access code. The commonly available Intel drivers for Linux and *BSD don't use this register, and neither does the e1000 PXE ROM that comes with QEMU, so it probably hasn't been tested extensively. In real hardware, the register should only be updated when both an EEPROM offset and the START bit are written -- setting the START bit is what triggers an actual EEPROM read transaction. When the transaction is complete, the START bit is cleared, and the DONE bit is set. In QEMU, writing just the EEPROM offset is enough to cause the read transaction to occur: the simulated EEPROM contents appear and the DONE bit is set whether the START bit was set or not. I was able to fix both of these issues in my local copy of e1000.c, and now the VxWorks PRO/1000 driver works correctly. I put the original code, patched version, and a context diff at the following URL: http://www.freebsd.org/~wpaul/qemu -Bill -- ============================================================================= -Bill Paul (510) 749-2329 | Senior Engineer, Master of Unix-Fu wpaul@windriver.com | Wind River Systems ============================================================================= "I put a dollar in a change machine. Nothing changed." - George Carlin =============================================================================