Re: [PATCH 1/3] qemu: fix pit timer tick policy=delay

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Since this has been sitting unreviewed for a while...


On 12/09/2016 09:28 AM, Maxim Nestratov wrote:
> By a mistake, 'delay' libvirt xml parameter was converted to 'discard' QEMU
> command line string one.
> 
> Test "kvm-pit-delay" is fixed accordinly, so that redundant test cases removed
> as there is no need to specify both QEMU_CAPS_NO_KVM_PIT and
> QEMU_CAPS_KVM_PIT_TICK_POLICY simultaneusly in tests as they are mutually
> exclusive and "kvm-pit-device" becomes just the same as "kvm-pit-delay".

I'd like to alter the commit to be:

qemu: Fix pit timer tick policy=delay

By a mistake, for the VIR_DOMAIN_TIMER_TICKPOLICY_DELAY qemu
command line creation, 'discard' was used instead of 'delay'
in commit id '1569fa14'.

Test "kvm-pit-delay" is fixed accordingly to show the correct
option being generated.

Remove the (now) redundant kvm-pit-device tests. As it turns
out there is no need to specify both QEMU_CAPS_NO_KVM_PIT and
QEMU_CAPS_KVM_PIT_TICK_POLICY since they are mutually exclusive
and "kvm-pit-device" becomes just the same as "kvm-pit-delay".

...

ACK - (let me know if you agree with the adjust commit - I can push if
you would like, although I know you have that access.)

John
> 
> Signed-off-by: Maxim Nestratov <mnestratov@xxxxxxxxxxxxx>
> ---
>  src/qemu/qemu_command.c                                | 2 +-
>  tests/qemuxml2argvdata/qemuxml2argv-kvm-pit-delay.args | 2 +-
>  tests/qemuxml2argvtest.c                               | 5 +----
>  3 files changed, 3 insertions(+), 6 deletions(-)
> 
> diff --git a/src/qemu/qemu_command.c b/src/qemu/qemu_command.c
> index 815abff..cd243e4 100644
> --- a/src/qemu/qemu_command.c
> +++ b/src/qemu/qemu_command.c
> @@ -6139,7 +6139,7 @@ qemuBuildClockCommandLine(virCommandPtr cmd,
>                     (-no-kvm-pit), otherwise, the default is catchup. */
>                  if (virQEMUCapsGet(qemuCaps, QEMU_CAPS_KVM_PIT_TICK_POLICY))
>                      virCommandAddArgList(cmd, "-global",
> -                                         "kvm-pit.lost_tick_policy=discard", NULL);
> +                                         "kvm-pit.lost_tick_policy=delay", NULL);
>                  else if (virQEMUCapsGet(qemuCaps, QEMU_CAPS_NO_KVM_PIT))
>                      virCommandAddArg(cmd, "-no-kvm-pit-reinjection");
>                  break;
> diff --git a/tests/qemuxml2argvdata/qemuxml2argv-kvm-pit-delay.args b/tests/qemuxml2argvdata/qemuxml2argv-kvm-pit-delay.args
> index 1d69797..7a02d36 100644
> --- a/tests/qemuxml2argvdata/qemuxml2argv-kvm-pit-delay.args
> +++ b/tests/qemuxml2argvdata/qemuxml2argv-kvm-pit-delay.args
> @@ -14,7 +14,7 @@ QEMU_AUDIO_DRV=none \
>  -nographic \
>  -nodefaults \
>  -monitor unix:/tmp/lib/domain--1-QEMUGuest1/monitor.sock,server,nowait \
> --no-kvm-pit-reinjection \
> +-global kvm-pit.lost_tick_policy=delay \
>  -no-acpi \
>  -boot c \
>  -usb \
> diff --git a/tests/qemuxml2argvtest.c b/tests/qemuxml2argvtest.c
> index 90d6aaf..b8619dd 100644
> --- a/tests/qemuxml2argvtest.c
> +++ b/tests/qemuxml2argvtest.c
> @@ -2059,10 +2059,7 @@ mymain(void)
>              QEMU_CAPS_KVM);
>      qemuTestSetHostArch(driver.caps, VIR_ARCH_NONE);
>  
> -    DO_TEST("kvm-pit-device", QEMU_CAPS_KVM_PIT_TICK_POLICY);
> -    DO_TEST("kvm-pit-delay", QEMU_CAPS_NO_KVM_PIT);
> -    DO_TEST("kvm-pit-device", QEMU_CAPS_NO_KVM_PIT,
> -            QEMU_CAPS_KVM_PIT_TICK_POLICY);
> +    DO_TEST("kvm-pit-delay", QEMU_CAPS_KVM_PIT_TICK_POLICY);
>  
>      DO_TEST("panic", QEMU_CAPS_DEVICE_PANIC,
>              QEMU_CAPS_NODEFCONFIG);
> 

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]
  Powered by Linux