From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1IMFpP-00072o-12 for qemu-devel@nongnu.org; Sat, 18 Aug 2007 00:22:23 -0400 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1IMFpO-00072Y-DO for qemu-devel@nongnu.org; Sat, 18 Aug 2007 00:22:22 -0400 Received: from [199.232.76.173] (helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1IMFpO-00072T-8M for qemu-devel@nongnu.org; Sat, 18 Aug 2007 00:22:22 -0400 Received: from ns.suse.de ([195.135.220.2] helo=mx1.suse.de) by monty-python.gnu.org with esmtps (TLS-1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.60) (envelope-from ) id 1IMFpO-0002lQ-5T for qemu-devel@nongnu.org; Sat, 18 Aug 2007 00:22:22 -0400 Received: from Relay2.suse.de (mail2.suse.de [195.135.221.8]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.suse.de (Postfix) with ESMTP id 1B26A1224E for ; Sat, 18 Aug 2007 06:22:17 +0200 (CEST) Date: Fri, 17 Aug 2007 21:22:13 -0700 From: Brandon Philips Subject: Re: [Qemu-devel] [PATCH][RFC] Fix bugs in the ATAPI cdrom driver Message-ID: <20070818042213.GA4603@ifup.org> References: <20070817234304.GB10490@ifup.org> <1187397752.5168.3.camel@fuego> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1187397752.5168.3.camel@fuego> 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 On 17:42 Fri 17 Aug 2007, Matthew Kent wrote: > On Fri, 2007-17-08 at 16:43 -0700, Brandon Philips wrote: > > The new libata-eh in the Linux kernel is throwing a fit over the QEMU > > cdrom device for two reasons: > > Nice thanks, was suffering from this issue as well. > > Tested with some linux 2.6.23-rc3 guests, looks great now. Great. I would like to see someone from the QEMU project sign off on the correctness of the code too before we carry the patch in our package. Thanks for testing! Brandon