From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:45355) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1SAnMg-000302-CE for qemu-devel@nongnu.org; Thu, 22 Mar 2012 15:08:04 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1SAnMe-0003ar-Jm for qemu-devel@nongnu.org; Thu, 22 Mar 2012 15:08:01 -0400 Received: from alpha.arachsys.com ([91.203.57.7]:44514) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1SAnMe-0003ae-EM for qemu-devel@nongnu.org; Thu, 22 Mar 2012 15:08:00 -0400 Date: Thu, 22 Mar 2012 19:07:52 +0000 From: Chris Webb Message-ID: <20120322190751.GA12145@arachsys.com> References: <1331603611-9103-1-git-send-email-zwu.kernel@gmail.com> <4F685230.9040209@redhat.com> <4F6854B2.8000209@redhat.com> <20120320114442.GA30819@stefanha-thinkpad.localdomain> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20120320114442.GA30819@stefanha-thinkpad.localdomain> Subject: Re: [Qemu-devel] [PATCH] block: add the support to drain throttled requests List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Stefan Hajnoczi Cc: Kevin Wolf , Zhi Yong Wu , Zhi Yong Wu , qemu-devel@nongnu.org, Paolo Bonzini Stefan Hajnoczi writes: > Yesterday I only posted an analysis of the bug but here are some > thoughts on how to move forward. Throttling itself is not the problem. > We've known that synchronous operations in the vcpu thread are a problem > long before throttling. This is just another reason to convert device > emulation to use asynchronous interfaces. > > Here is the list of device models that perform synchronous block I/O: > hw/fdc.c > hw/ide/atapi.c > hw/ide/core.c > hw/nand.c > hw/onenand.c > hw/pflash_cfi01.c > hw/pflash_cfi02.c > hw/sd.c > > Zhi Hui Li is working on hw/fdc.c and recently sent a patch. > > I think it's too close to QEMU 1.1 to convert all the remaining devices > and test them properly before the soft-freeze. But it's probably > possible to convert IDE before the soft-freeze. IDE is the only of these that would affect us as a typical user of throttling. The others aren't really the kind of devices which you'd be using in a hosting setting in any case. However, being able to throttle ide and virtio drives to limit the harm one VM can cause to another on the same host would make an enormous difference to us, and for what it's worth, we're desperately keen to see this problem and the assert() issue fixed so we can deploy this feature, back-porting it if necessary. Just to reiterate a previous offer: we'd be very happy to pay for someone's contracting time to work on this if that would help. The feature is great and works well apart from the risk of these ide deadlocks and asserts. Unfortunately, ide represents the bulk of our VMs at the moment. Best wishes, Chris.