Re: [PATCH 3/3] qemu: add support for splash-timeout

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

 



On 08/22/2014 05:10 PM, Martin Kletzander wrote:
> Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1021703
> 
> Signed-off-by: Martin Kletzander <mkletzan@xxxxxxxxxx>
> ---
>  src/qemu/qemu_command.c                                   | 15 +++++++++++++++
>  .../qemuxml2argv-boot-menu-enable-with-timeout.args       | 15 +++++++++++++++
>  tests/qemuxml2argvtest.c                                  |  4 ++++
>  3 files changed, 34 insertions(+)
>  create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-boot-menu-enable-with-timeout.args
> 
> diff --git a/src/qemu/qemu_command.c b/src/qemu/qemu_command.c
> index 0d7b12d..bb1c423 100644
> --- a/src/qemu/qemu_command.c
> +++ b/src/qemu/qemu_command.c
> @@ -7927,6 +7927,21 @@ qemuBuildCommandLine(virConnectPtr conn,
>                                def->os.bios.rt_delay);
>          }
> 
> +        if (def->os.bm_timeout_set) {
> +            if (!virQEMUCapsGet(qemuCaps, QEMU_CAPS_SPLASH_TIMEOUT)) {
> +                virReportError(VIR_ERR_CONFIG_UNSUPPORTED, "%s",
> +                               _("splash timeout is not supported "
> +                                 "by this QEMU binary"));
> +                virBufferFreeAndReset(&boot_buf);
> +                goto error;
> +            }
> +
> +            if (boot_nparams++)
> +                virBufferAddChar(&boot_buf, ',');
> +
> +            virBufferAsprintf(&boot_buf, "splash-time=%d", def->os.bm_timeout);

This should be %u for usigned int.

Also, it would be nice to get rid of 'boot_nparams', but that's out of scope
of this patch.

ACK series

Jan

Attachment: signature.asc
Description: OpenPGP digital signature

--
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]