From mboxrd@z Thu Jan 1 00:00:00 1970 From: Rusty Russell Subject: Re: [PATCH 5/5] virtio-blk: implement ->make_request Date: Thu, 06 Oct 2011 12:22:14 +1030 Message-ID: <87obxuga41.fsf@rustcorp.com.au> References: <20111005195403.407628164@bombadil.infradead.org> <20111005195530.558142038@bombadil.infradead.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Chris Wright , Jens Axboe , Stefan Hajnoczi , kvm@vger.kernel.org, linux-kernel@vger.kernel.org To: Christoph Hellwig Return-path: Received: from ozlabs.org ([203.10.76.45]:60348 "EHLO ozlabs.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S935035Ab1JFF2X (ORCPT ); Thu, 6 Oct 2011 01:28:23 -0400 In-Reply-To: <20111005195530.558142038@bombadil.infradead.org> Sender: kvm-owner@vger.kernel.org List-ID: On Wed, 05 Oct 2011 15:54:08 -0400, Christoph Hellwig wrote: > Add an alternate I/O path that implements ->make_request for virtio-blk. > This is required for high IOPs devices which get slowed down to 1/5th of > the native speed by all the locking, memory allocation and other overhead > in the request based I/O path. Ouch. I'd be tempted to just switch across to this, though I'd be interested to see if the simple add_buf change I referred to before has some effect by itself (I doubt it). Also, though it's overkill I'd use standard list primitives rather than open-coding a single linked list. Thanks! Rusty.