On Mon, Apr 09, 2018 at 02:41:55PM +0200, Thilo Cestonaro wrote:
> A configured target.port is currently totaly ignored, while contsructing
> qemu commandline, for all types of serial devices. This patch adds a -device
> parameter "index" for the target model isa-serial.
> This enables the user to specify which serial device will end in which ttySX
> device.
> 
> Signed-off-by: Thilo Cestonaro <thilo.ceston...@ts.fujitsu.com>
> ---
> Forgot to sign off! Here with Signed-off-by.
> 
>  src/qemu/qemu_command.c | 16 ++++++++++++++++
>  1 file changed, 16 insertions(+)

We'd generally expect to see a change to tests/qemuxml2argvtest.c for
this. Either adding a new test data file, or updating existing data
files if they would cover it.

> 
> diff --git a/src/qemu/qemu_command.c b/src/qemu/qemu_command.c
> index 514c3ab2e..5f770404b 100644
> --- a/src/qemu/qemu_command.c
> +++ b/src/qemu/qemu_command.c
> @@ -10291,6 +10291,22 @@ qemuBuildSerialChrDeviceStr(char **deviceStr,
>                        
> virDomainChrSerialTargetModelTypeToString(serial->targetModel),
>                        serial->info.alias, serial->info.alias);
>  
> +    switch ((virDomainChrSerialTargetModel) serial->targetModel) {
> +    case VIR_DOMAIN_CHR_SERIAL_TARGET_MODEL_ISA_SERIAL:
> +        if (serial->target.port != -1)
> +            virBufferAsprintf(&cmd, ",index=%d", serial->target.port);
> +        break;
> +    case VIR_DOMAIN_CHR_SERIAL_TARGET_MODEL_NONE:
> +    case VIR_DOMAIN_CHR_SERIAL_TARGET_MODEL_USB_SERIAL:
> +    case VIR_DOMAIN_CHR_SERIAL_TARGET_MODEL_PCI_SERIAL:
> +    case VIR_DOMAIN_CHR_SERIAL_TARGET_MODEL_SPAPR_VTY:
> +    case VIR_DOMAIN_CHR_SERIAL_TARGET_MODEL_PL011:
> +    case VIR_DOMAIN_CHR_SERIAL_TARGET_MODEL_SCLPCONSOLE:
> +    case VIR_DOMAIN_CHR_SERIAL_TARGET_MODEL_SCLPLMCONSOLE:
> +    case VIR_DOMAIN_CHR_SERIAL_TARGET_MODEL_LAST:
> +        break;
> +    }
> +
>      if (qemuBuildDeviceAddressStr(&cmd, def, &serial->info, qemuCaps) < 0)
>          goto error;
>  
> -- 
> 2.15.1
> 
> --
> libvir-list mailing list
> libvir-list@redhat.com
> https://www.redhat.com/mailman/listinfo/libvir-list

Regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|

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

Reply via email to