From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1H835a-0004a0-35 for qemu-devel@nongnu.org; Fri, 19 Jan 2007 18:24:06 -0500 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1H835Y-0004ZU-6a for qemu-devel@nongnu.org; Fri, 19 Jan 2007 18:24:05 -0500 Received: from [199.232.76.173] (helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1H835Y-0004ZJ-45 for qemu-devel@nongnu.org; Fri, 19 Jan 2007 18:24:04 -0500 Received: from [213.165.64.20] (helo=mail.gmx.net) by monty-python.gnu.org with smtp (Exim 4.52) id 1H835X-0008CT-LN for qemu-devel@nongnu.org; Fri, 19 Jan 2007 18:24:03 -0500 Message-ID: <45B15338.5070608@gmx.com> Date: Sat, 20 Jan 2007 00:24:40 +0100 From: Martin Bochnig MIME-Version: 1.0 Subject: Re: [Qemu-devel] [PATCH] ATAPI CDROM DMA support is incomplete References: <200701192055.l0JKt5B6002844@imap.tools.intra> In-Reply-To: <200701192055.l0JKt5B6002844@imap.tools.intra> Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit Reply-To: qemu-devel@nongnu.org List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Hi Juergen Keil wrote: >Hi, > > >some time ago, ATAPI DMA support was added to hw/ide.c, but the DMA >support is incomplete: DMA data transfers are implemented >for ATAPI/SCSI READ commands only (GPCMD_READ_10, GPCMD_READ_12, >GPCMD_READ_CD), but all other ATAPI/SCSI commands still try to >transfer data using PIO mode. > >Because of this problem, neither Windows ME nor Windows 2000 guests >will use ATAPI CD DMA transfers and fall back to PIO mode. > >Or a OpenSolaris x86 guest will report errors like this >"WARNING: /pci@0,0/pci-ide@1,1/ide@1 (ata1): timeout: abort request, >target=0 lun=0" and is unable to read data from the CD (You had to >force the ata/atapi driver to pio mode to work around this problem). > > >The attached patch adds DMA support for non-READ ATAPI/SCSI commands. > > >------------------------------------------------------------------------ > >Index: hw/ide.c >=================================================================== > You got it fixed once again, thank you Mr. Keil! -MB