From: Stefan Hajnoczi <stefanha@redhat.com>
To: Fam Zheng <famz@redhat.com>
Cc: Kevin Wolf <kwolf@redhat.com>,
Paolo Bonzini <pbonzini@redhat.com>,
qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH v4 0/7] aio: Support epoll by introducing qemu_poll abstraction
Date: Wed, 29 Apr 2015 11:06:17 +0100 [thread overview]
Message-ID: <20150429100617.GB15096@stefanha-thinkpad.redhat.com> (raw)
In-Reply-To: <1430277871-26761-1-git-send-email-famz@redhat.com>
[-- Attachment #1: Type: text/plain, Size: 1123 bytes --]
On Wed, Apr 29, 2015 at 11:24:24AM +0800, Fam Zheng wrote:
> v4: Rebase to 2.3 and rerun tests. timerfd+epoll shows clear improvement over
> current master when the virtio-console device attaches more fds to the main
> loop:
>
> syscall high # of fd low # of fd
> -------------------------------------------------
> qemu.git(ppoll) 24 83
> ppoll+epoll 49 82
> timerfd+epoll 49 91
> glib 20 82
>
> (In high # of fd case, 3 activated but idle virtio-console devices are
> attached, which will add us hundereds of fds to poll)
Thanks for posting these numbers. Nice job on speeding up the Linux
case, I'm looking forward to seeing the results with your new epoll
syscall patches.
It looks like there will be a 15-20% regression for non-Linux hosts with
high # of fds. But since the low # of fd regression between
qemu.git(ppoll) and glib is small, I think we can merge this now and
optimize later, if necessary.
Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
[-- Attachment #2: Type: application/pgp-signature, Size: 473 bytes --]
next prev parent reply other threads:[~2015-04-29 10:06 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-04-29 3:24 [Qemu-devel] [PATCH v4 0/7] aio: Support epoll by introducing qemu_poll abstraction Fam Zheng
2015-04-29 3:24 ` [Qemu-devel] [PATCH v4 1/7] poll: Introduce QEMU Poll API Fam Zheng
2015-04-29 3:24 ` [Qemu-devel] [PATCH v4 2/7] posix-aio: Use QEMU poll interface Fam Zheng
2015-04-29 3:24 ` [Qemu-devel] [PATCH v4 3/7] poll: Add epoll implementation for qemu_poll Fam Zheng
2015-04-29 3:24 ` [Qemu-devel] [PATCH v4 4/7] main-loop: Replace qemu_poll_ns with qemu_poll Fam Zheng
2015-04-29 3:24 ` [Qemu-devel] [PATCH v4 5/7] tests: Add test case for qemu_poll Fam Zheng
2015-04-29 3:24 ` [Qemu-devel] [PATCH v4 6/7] poll-glib: Support ppoll Fam Zheng
2015-04-29 3:24 ` [Qemu-devel] [PATCH v4 7/7] poll-linux: Add timerfd support Fam Zheng
2015-04-29 10:06 ` Stefan Hajnoczi [this message]
2015-04-30 14:27 ` [Qemu-devel] [PATCH v4 0/7] aio: Support epoll by introducing qemu_poll abstraction Kevin Wolf
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=20150429100617.GB15096@stefanha-thinkpad.redhat.com \
--to=stefanha@redhat.com \
--cc=famz@redhat.com \
--cc=kwolf@redhat.com \
--cc=pbonzini@redhat.com \
--cc=qemu-devel@nongnu.org \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).