From: Yehuda Sadeh Weinraub <yehudasa@gmail.com>
To: Christian Brunner <chb@muc.de>
Cc: ceph-devel@vger.kernel.org
Subject: Re: qemu-kvm/rbd: first attempt to implement aio
Date: Thu, 15 Apr 2010 17:09:45 -0700 [thread overview]
Message-ID: <r2va37615151004151709wef84ae66hed2dc4d713c94c14@mail.gmail.com> (raw)
In-Reply-To: <20100415202439.GA8005@ncolin.muc.de>
On Thu, Apr 15, 2010 at 1:24 PM, Christian Brunner <chb@muc.de> wrote:
> To increase the performance of the qemu rbd driver I started to implement
> aio functions. Attached is an intial aio implementation. However it is
> missing the callback of rados_aio_read or _write from time to time.
> This leads to a lockup of the disk driver until the guest OS is
> perfomring a bus reset.
>
> If someone has an explanation for the cause of the lost callbacks. I
> would be happy.
>
Yeah, it looks like the rados aio interface is racy. You should be
able to first set aio completion and only then to do the read/write
and not the other way around as it is now. I've just pushed a fix to
the unstable branch that changes the interface a bit. Now you first
need to do a rados_aio_create_completion() first and use it in the
rados_aio_read()/write() call. Note that I haven't tested it yet, so
let me know if there's any problem.
Thanks,
Yehuda
next prev parent reply other threads:[~2010-04-16 0:09 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-04-14 20:49 [PATCH] qemu-kvm/rbd: small fixes and cosmetics Christian Brunner
2010-04-14 21:12 ` Yehuda Sadeh Weinraub
2010-04-15 20:24 ` qemu-kvm/rbd: first attempt to implement aio Christian Brunner
2010-04-16 0:09 ` Yehuda Sadeh Weinraub [this message]
2010-04-16 20:53 ` AW: " Christian Brunner
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=r2va37615151004151709wef84ae66hed2dc4d713c94c14@mail.gmail.com \
--to=yehudasa@gmail.com \
--cc=ceph-devel@vger.kernel.org \
--cc=chb@muc.de \
/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.