All of lore.kernel.org
 help / color / mirror / Atom feed
From: Anthony Liguori <anthony@codemonkey.ws>
To: Gerd Hoffmann <kraxel@redhat.com>
Cc: Ryan Harper <ryanh@us.ibm.com>,
	Marcelo Tosatti <mtosatti@redhat.com>,
	qemu-devel@nongnu.org, kvm@vger.kernel.org
Subject: Re: [Qemu-devel] Re: [PATCH 2/3] Move aio implementation out of raw block driver
Date: Tue, 23 Sep 2008 13:28:50 -0500	[thread overview]
Message-ID: <48D93562.6020009@codemonkey.ws> (raw)
In-Reply-To: <48D92FC2.2000203@redhat.com>

Gerd Hoffmann wrote:
> Anthony Liguori wrote:
>   
>> Gerd Hoffmann wrote:
>>     
>>> How about providing a aio interface implementation which simply uses
>>> read/write syscalls (thereby not being really async obviously)?  Then
>>> use that as fallback instead of aio emulation?  And also drop CONFIG_AIO
>>> then?
>>>       
>> Yeah, this is basically what block-raw-posix does today.  I was thinking
>> the same thing.  I was also thinking that you could do an aio
>> implementation for win32 and possibly reunify block-raw-posix and
>> block-raw-linux.
>>     
>
> Sure, that the next logical steps.  Later we can also convert all
> block-* drivers to the new aio interface and subsequently drop alot of
> dead block layer code.
>   

Yup.

>> But before going down this route, I want to see if linux-aio is really
>> the right tool for the job.
>>     
>
> IMHO this all makes sense even in case linux-aio turns out to not be
> worth it.
>   

Yes, I agree.  For a bit of a spoiler, initial results are that 
cache=off + my fd_pool patch is getting equivalent performance to 
linux-aio so it's looking like we can avoid linux-aio for now.

Regards,

Anthony Liguori

> cheers,
>   Gerd
>
> --
> To unsubscribe from this list: send the line "unsubscribe kvm" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>   


  reply	other threads:[~2008-09-23 18:29 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-09-22 23:17 [PATCH 0/3] Refactor AIO to allow multiple AIO implementations Ryan Harper
2008-09-22 23:17 ` [PATCH 1/3] Only call aio flush handler if set Ryan Harper
2008-09-23  2:38   ` Anthony Liguori
2008-09-23 14:26     ` [Qemu-devel] " Ryan Harper
2008-09-23 14:34       ` Anthony Liguori
2008-09-23 14:41         ` Ryan Harper
2008-09-23 14:50           ` Anthony Liguori
2008-09-22 23:17 ` [PATCH 2/3] Move aio implementation out of raw block driver Ryan Harper
2008-09-23  1:16   ` Ryan Harper
2008-09-23  2:45   ` Anthony Liguori
2008-09-23 14:39     ` [Qemu-devel] " Ryan Harper
2008-09-23 14:40       ` Anthony Liguori
2008-09-23 14:53       ` Gerd Hoffmann
2008-09-23 16:06         ` Anthony Liguori
2008-09-23 18:04           ` Gerd Hoffmann
2008-09-23 18:28             ` Anthony Liguori [this message]
2008-09-24 22:31     ` Marcelo Tosatti
     [not found] ` <1222125454-21744-4-git-send-email-ryanh@us.ibm.com>
2008-09-23  1:22   ` [PATCH 3/3] Add linux aio implementation for raw block devices Ryan Harper
2008-09-23  3:32 ` [PATCH 0/3] Refactor AIO to allow multiple AIO implementations Anthony Liguori
2008-09-23 14:43   ` [Qemu-devel] " Ryan Harper
2008-09-23 14:47     ` Anthony Liguori
2008-09-23 16:09     ` Anthony Liguori
2008-09-23 10:27 ` [Qemu-devel] " Jamie Lokier
2008-10-02 22:41 ` john cooper
2008-10-03 13:33   ` Ryan Harper

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=48D93562.6020009@codemonkey.ws \
    --to=anthony@codemonkey.ws \
    --cc=kraxel@redhat.com \
    --cc=kvm@vger.kernel.org \
    --cc=mtosatti@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=ryanh@us.ibm.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.