From: Avi Kivity <avi@redhat.com>
To: Michael Tokarev <mjt@tls.msk.ru>
Cc: KVM list <kvm@vger.kernel.org>
Subject: Re: high load with usb device
Date: Tue, 14 Sep 2010 18:37:29 +0200 [thread overview]
Message-ID: <4C8FA4C9.2080208@redhat.com> (raw)
In-Reply-To: <4C8FA2D6.1090604@msgid.tls.msk.ru>
On 09/14/2010 06:29 PM, Michael Tokarev wrote:
> 14.09.2010 20:00, Avi Kivity wrote:
> >> As I mentioned in other emails in this thread:
> >>
> >> o yes, I do have CONFIG_EVENTFD set, and it is being used
> >> too (fd#12 in the above strace).
> >
> > I thought that was the signalfd.
>
> Uh. Yes, it was, i confused the two. And yes, CONFIG_EVENTFD is
> set and used too.
>
> $ grep FD= config-host.mak
> CONFIG_EVENTFD=y
> CONFIG_SIGNALFD=y
>
> >> o 0.13.0-rc1 behaves the same way (that is, it also shows
> >> high load when idle -- the same 18% of host CPU), but it
> >> has no pipe on fd#5.
> >
> > I think it's host_alarm_handler()'s use of qemu_notify_event(). It's
> > telling the main loop to rescan pending events, even though it's called
> > from the main loop itself. Please drop it and rerun.
>
> Without qemu_notify_event() in host_alarm_handler():
>
> % time seconds usecs/call calls errors syscall
> ------ ----------- ----------- --------- --------- ----------------
> 98.96 48.184832 13747 3505 ioctl
> 0.39 0.191613 25 7745 28 futex
> 0.37 0.181032 1 173192 select
> 0.09 0.045379 0 980369 clock_gettime
> 0.05 0.024362 0 351024 173220 read
> 0.05 0.023247 0 487766 gettimeofday
> 0.04 0.017996 0 319428 timer_gettime
> 0.03 0.013837 0 198267 timer_settime
> 0.02 0.010036 0 177790 rt_sigaction
> 0.00 0.000000 0 1 writev
> 0.00 0.000000 0 2 poll
> 0.00 0.000000 0 1 rt_sigpending
> 0.00 0.000000 0 1 1 rt_sigtimedwait
> ------ ----------- ----------- --------- --------- ----------------
> 100.00 48.692334 2699091 173249 total
>
> The picture is pretty similar to the one before :)
> (And yes, I'm sure I've run the right binary).
No more writes, and read() is cut to twice select() (due to the need to
see a 0, we can probably elimate it if we know it's a real eventfd),
somewhat fewer select()s.
What's the cpu load?
> But it looks like we're barking the wrong tree. It's spending
> 99% time in ioctl. Yes, there's quite large amount of reads
> and gettimes, but they're taking very small time, even combined.
>
> It is still spending much more time in the ioctl (apparently in
> kvm_run).
That time includes guest sleep time, not just cpu time, so it isn't an
indicator.
> For comparison, here's the same strace stats without -usbdevice:
>
> % time seconds usecs/call calls errors syscall
> ------ ----------- ----------- --------- --------- ----------------
> 97.70 0.080237 22 3584 select
> 1.09 0.000895 0 6018 3584 read
> 0.33 0.000271 0 9670 clock_gettime
> 0.31 0.000254 0 6086 gettimeofday
> 0.26 0.000210 0 2432 rt_sigaction
> 0.17 0.000137 0 3653 timer_gettime
> 0.15 0.000122 0 2778 timer_settime
> 0.00 0.000000 0 1 ioctl
> 0.00 0.000000 0 1 rt_sigpending
> 0.00 0.000000 0 1 1 rt_sigtimedwait
> ------ ----------- ----------- --------- --------- ----------------
> 100.00 0.082126 34224 3585 total
>
> Yes, it is still doing lots of unnecessary stuff, but the load
> is<1%.
>
> (This is without host_alarm_handler() in qemu_notify_event())
>
>
I guess it only saw a single ioctl, so it didn't measure it. Ping the
guest and you'll see ioctl times surge back.
--
error compiling committee.c: too many arguments to function
next prev parent reply other threads:[~2010-09-14 16:37 UTC|newest]
Thread overview: 36+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-09-03 12:21 high load with win7 & usb tablet Michael Tokarev
2010-09-10 11:47 ` Jes Sorensen
2010-09-10 12:16 ` Michael Tokarev
2010-09-10 16:17 ` high load with usb device Michael Tokarev
2010-09-12 9:26 ` Avi Kivity
2010-09-14 6:51 ` Michael Tokarev
2010-09-14 9:06 ` Avi Kivity
2010-09-14 9:08 ` Avi Kivity
2010-09-14 10:00 ` Michael Tokarev
2010-09-14 10:07 ` Avi Kivity
2010-09-14 10:25 ` Michael Tokarev
2010-09-14 10:39 ` Avi Kivity
2010-09-14 11:02 ` Michael Tokarev
2010-09-14 13:15 ` Michael Tokarev
2010-09-14 13:25 ` Avi Kivity
2010-09-14 13:29 ` Michael Tokarev
2010-09-14 13:38 ` Michael Tokarev
2010-09-14 14:45 ` Avi Kivity
2010-09-14 14:53 ` Michael Tokarev
2010-09-14 16:00 ` Avi Kivity
2010-09-14 16:29 ` Michael Tokarev
2010-09-14 16:37 ` Avi Kivity [this message]
2010-09-14 16:50 ` Michael Tokarev
2010-09-14 16:45 ` David S. Ahern
2010-09-14 15:51 ` David S. Ahern
2010-09-14 16:00 ` Michael Tokarev
2010-09-14 16:09 ` David S. Ahern
2010-09-10 12:19 ` high load with win7 & usb tablet Avi Kivity
2010-09-10 12:44 ` Michael Tokarev
2010-09-10 12:50 ` Avi Kivity
[not found] ` <4C8A44E6.2060008@msgid.tls.msk.ru>
2010-09-10 14:56 ` Avi Kivity
2010-09-10 15:03 ` Michael Tokarev
2010-09-10 16:03 ` Avi Kivity
2010-09-14 8:26 ` Brad Campbell
2010-09-14 8:33 ` Michael Tokarev
2010-09-14 8:46 ` Brad Campbell
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=4C8FA4C9.2080208@redhat.com \
--to=avi@redhat.com \
--cc=kvm@vger.kernel.org \
--cc=mjt@tls.msk.ru \
/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.