[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] Re: [kvm-devel] [PATCH 0/4] Rework alarm timer infrastrucur
From: |
Luca Tettamanti |
Subject: |
[Qemu-devel] Re: [kvm-devel] [PATCH 0/4] Rework alarm timer infrastrucure - take2 |
Date: |
Tue, 21 Aug 2007 21:38:34 +0200 |
User-agent: |
Mutt/1.5.16 (2007-06-11) |
Avi Kivity ha scritto:
> Luca Tettamanti wrote:
>> At 1000Hz:
>>
>> QEMU
>> hpet 5.5%
>> dynticks 11.7%
>>
>> KVM
>> hpet 3.4%
>> dynticks 7.3%
>>
>> No surprises here, you can see the additional 1k syscalls per second.
>
> This is very surprising to me. The 6.2% difference for the qemu case
> translates to 62ms per second, or 62us per tick at 1000Hz. That's more
> than a hundred simple syscalls on modern processors. We shouldn't have to
> issue a hundred syscalls per guest clock tick.
APIC or PIT interrupts are delivered using the timer, which will be
re-armed after each tick, so I'd expect 1k timer_settime per second. But
according to strace it's not happening, maybe I'm misreading the code?
> The difference with kvm is smaller (just 3.9%), which is not easily
> explained as the time for the extra syscalls should be about the same. My
> guess is that guest behavior is different; with dynticks the guest does
> about twice as much work as with hpet.
Actually I'm having troubles with cyclesoak (probably it's calibration),
numbers are not very stable across multiple runs...
I've also tried APC which was suggested by malc[1] and:
- readings are far more stable
- the gap between dynticks and non-dynticks seems not significant
> Can you verify this by running
>
> strace -c -p `pgrep qemu` & sleep 10; pkill strace
>
> for all 4 cases, and posting the results?
Plain QEMU:
With dynticks:
% time seconds usecs/call calls errors syscall
------ ----------- ----------- --------- --------- ----------------
57.97 0.000469 0 13795 clock_gettime
32.88 0.000266 0 1350 gettimeofday
7.42 0.000060 0 1423 1072 sigreturn
1.73 0.000014 0 5049 timer_gettime
0.00 0.000000 0 1683 1072 select
0.00 0.000000 0 2978 timer_settime
------ ----------- ----------- --------- --------- ----------------
100.00 0.000809 26278 2144 total
HPET:
% time seconds usecs/call calls errors syscall
------ ----------- ----------- --------- --------- ----------------
87.48 0.010459 1 10381 10050 select
8.45 0.001010 0 40736 clock_gettime
2.73 0.000326 0 10049 gettimeofday
1.35 0.000161 0 10086 10064 sigreturn
------ ----------- ----------- --------- --------- ----------------
100.00 0.011956 71252 20114 total
Unix (SIGALRM):
% time seconds usecs/call calls errors syscall
------ ----------- ----------- --------- --------- ----------------
90.36 0.011663 1 10291 9959 select
7.38 0.000953 0 40355 clock_gettime
2.05 0.000264 0 9960 gettimeofday
0.21 0.000027 0 9985 9969 sigreturn
------ ----------- ----------- --------- --------- ----------------
100.00 0.012907 70591 19928 total
And KVM:
dynticks:
% time seconds usecs/call calls errors syscall
------ ----------- ----------- --------- --------- ----------------
78.90 0.004001 1 6681 5088 rt_sigtimedwait
10.87 0.000551 0 27901 clock_gettime
4.93 0.000250 0 7622 timer_settime
4.30 0.000218 0 10078 timer_gettime
0.39 0.000020 0 3863 gettimeofday
0.35 0.000018 0 6054 ioctl
0.26 0.000013 0 4196 select
0.00 0.000000 0 1593 rt_sigaction
------ ----------- ----------- --------- --------- ----------------
100.00 0.005071 67988 5088 total
HPET:
% time seconds usecs/call calls errors syscall
------ ----------- ----------- --------- --------- ----------------
90.20 0.011029 0 32437 22244 rt_sigtimedwait
4.46 0.000545 0 44164 clock_gettime
2.59 0.000317 0 12128 gettimeofday
1.50 0.000184 0 10193 rt_sigaction
1.10 0.000134 0 12461 select
0.15 0.000018 0 6060 ioctl
------ ----------- ----------- --------- --------- ----------------
100.00 0.012227 117443 22244 total
Unix:
% time seconds usecs/call calls errors syscall
------ ----------- ----------- --------- --------- ----------------
83.29 0.012522 0 31652 21709 rt_sigtimedwait
6.91 0.001039 0 43125 clock_gettime
3.50 0.000526 0 6042 ioctl
2.74 0.000412 0 9943 rt_sigaction
1.98 0.000298 0 12183 select
1.58 0.000238 0 11850 gettimeofday
------ ----------- ----------- --------- --------- ----------------
100.00 0.015035 114795 21709 total
The guest is an idle kernel with HZ=1000.
Luca
[1] copy_to_user inside spinlock is a big no-no ;)
--
La somma dell'intelligenza sulla terra e` una costante.
La popolazione e` in aumento.
- Re: [kvm-devel] [Qemu-devel] Re: [PATCH 0/4] Rework alarm timer infrastrucure - take2, (continued)
- [Qemu-devel] Re: [kvm-devel] [PATCH 0/4] Rework alarm timer infrastrucure - take2, Luca, 2007/08/19
- [Qemu-devel] Re: [kvm-devel] [PATCH 0/4] Rework alarm timer infrastrucure - take2, Avi Kivity, 2007/08/19
- [Qemu-devel] Re: [kvm-devel] [PATCH 0/4] Rework alarm timer infrastrucure - take2, Luca Tettamanti, 2007/08/20
- Re: [Qemu-devel] Re: [kvm-devel] [PATCH 0/4] Rework alarm timer infrastrucure - take2, malc, 2007/08/20
- Re: [kvm-devel] [Qemu-devel] Re: [PATCH 0/4] Rework alarm timer infrastrucure - take2, Luca, 2007/08/20
- [Qemu-devel] Re: [kvm-devel] [PATCH 0/4] Rework alarm timer infrastrucure - take2, Avi Kivity, 2007/08/21
- [Qemu-devel] Re: [kvm-devel] [PATCH 0/4] Rework alarm timer infrastrucure - take2,
Luca Tettamanti <=
- [Qemu-devel] Re: [kvm-devel] [PATCH 0/4] Rework alarm timer infrastrucure - take2, malc, 2007/08/21
- [Qemu-devel] Re: [kvm-devel] [PATCH 0/4] Rework alarm timer infrastrucure - take2, Avi Kivity, 2007/08/22
- [Qemu-devel] Re: [kvm-devel] [PATCH 0/4] Rework alarm timer infrastrucure - take2, Luca Tettamanti, 2007/08/22
- [Qemu-devel] Re: [kvm-devel] [PATCH 0/4] Rework alarm timer infrastrucure - take2, Avi Kivity, 2007/08/22
- [Qemu-devel] Re: [kvm-devel] [PATCH 0/4] Rework alarm timer infrastrucure - take2, Luca, 2007/08/22
- [Qemu-devel] Re: [kvm-devel] [PATCH 0/4] Rework alarm timer infrastrucure - take2, Avi Kivity, 2007/08/22
- [Qemu-devel] Re: [kvm-devel] [PATCH 0/4] Rework alarm timer infrastrucure - take2, Luca, 2007/08/22
- [Qemu-devel] Re: [kvm-devel] [PATCH 0/4] Rework alarm timer infrastrucure - take2, Luca, 2007/08/22
- [Qemu-devel] Re: [kvm-devel] [PATCH 0/4] Rework alarm timer infrastrucure - take2, Luca, 2007/08/22
- [Qemu-devel] RE: [kvm-devel] [PATCH 0/4] Rework alarm timer infrastrucure - take2, Dor Laor, 2007/08/22