All of lore.kernel.org
 help / color / mirror / Atom feed
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

  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.