From: Josh Durgin <jdurgin@redhat.com>
To: Sage Weil <sage@newdream.net>, Jason Dillaman <dillaman@redhat.com>
Cc: Milosz Tanski <milosz@adfin.com>,
Haomai Wang <haomaiwang@gmail.com>,
ceph-devel <ceph-devel@vger.kernel.org>
Subject: Re: About Adding eventfd support for LibRBD
Date: Mon, 13 Jul 2015 12:58:05 -0700 [thread overview]
Message-ID: <55A4184D.8020505@redhat.com> (raw)
In-Reply-To: <alpine.DEB.2.00.1507131140480.8878@cobra.newdream.net>
On 07/13/2015 11:42 AM, Sage Weil wrote:
> On Mon, 13 Jul 2015, Jason Dillaman wrote:
>>> But it doesn't provide an easily compassable way
>>> of integrating waiting on other events in the application. eventfd is
>>> easy to embed in your (e)pool loop or any kind of event library
>>> (libev).
>>
>> Agreed -- which is why I asked about the proposed design since it
>> appears (to me) that everything is hidden behind the librbd API and thus
>> not embeddable within a generic app event loop. It might just be a
>> misunderstanding on my part, which is why I asked for an example
>> integration.
>
> Bonus points if this helps out the qemu librbd usage, which (if memory
> serves) does some annoying stuff with a pipe in the callback to notify
> qemu of IO completion. Perhaps qemu can work with an eventfd more
> directly?
This was fixed a little while back when qemu was converted to using
coroutines and more than one thread, but it would make usage in other
applications with simpler threading models easier.
IIRC the xen blktap driver for rbd used the same pipe workaround to
deal with librbd's current callback api.
next prev parent reply other threads:[~2015-07-13 19:58 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-07-07 15:18 About Adding eventfd support for LibRBD Haomai Wang
2015-07-08 3:08 ` Josh Durgin
2015-07-08 3:46 ` Haomai Wang
2015-07-10 3:16 ` Haomai Wang
2015-07-13 13:52 ` Jason Dillaman
2015-07-13 17:14 ` Haomai Wang
2015-07-13 17:32 ` Jason Dillaman
2015-07-13 18:16 ` Milosz Tanski
2015-07-13 18:39 ` Jason Dillaman
2015-07-13 18:42 ` Sage Weil
2015-07-13 19:58 ` Josh Durgin [this message]
2015-07-20 5:03 ` Alexandre DERUMIER
2015-07-13 19:36 ` Milosz Tanski
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=55A4184D.8020505@redhat.com \
--to=jdurgin@redhat.com \
--cc=ceph-devel@vger.kernel.org \
--cc=dillaman@redhat.com \
--cc=haomaiwang@gmail.com \
--cc=milosz@adfin.com \
--cc=sage@newdream.net \
/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.