Re: [PATCH v3 3/4] qemu_command: Drop IDE validation code in qemuBuildControllerDevStr

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

 



On Fri, Dec 01, 2017 at 07:31:31PM +0800, Lin Ma wrote:
> The corresponding code will be moved to qemuDomainControllerDefValidate
> by next patch.
> 
> Signed-off-by: Lin Ma <lma@xxxxxxxx>
> ---
>  src/qemu/qemu_command.c | 10 +---------
>  1 file changed, 1 insertion(+), 9 deletions(-)
> 
> diff --git a/src/qemu/qemu_command.c b/src/qemu/qemu_command.c
> index 6a8da1d..2074ff6 100644
> --- a/src/qemu/qemu_command.c
> +++ b/src/qemu/qemu_command.c
> @@ -3127,17 +3127,9 @@ qemuBuildControllerDevStr(const virDomainDef *domainDef,
>          /* Since we currently only support the integrated IDE
>           * controller on various boards, if we ever get to here, it's
>           * because some other machinetype had an IDE controller
> -         * specified, or one with a single IDE contraller had multiple
> +         * specified, or one with a single IDE controller had multiple
>           * ide controllers specified.
>           */
> -        if (qemuDomainHasBuiltinIDE(domainDef))
> -            virReportError(VIR_ERR_CONFIG_UNSUPPORTED, "%s",
> -                           _("Only a single IDE controller is supported "
> -                             "for this machine type"));
> -        else
> -            virReportError(VIR_ERR_CONFIG_UNSUPPORTED, "%s",
> -                           _("IDE controllers are unsupported for "
> -                             "this QEMU binary or machine type"));

This is not correct, we shouldn't error out without setting any error
message.  This whole case should be removed and IDE controller should
be handled by the "default" path if it ever happen to reach this code.

Pavel

>          goto error;
>  
>      default:
> -- 
> 2.9.2
> 
> --
> libvir-list mailing list
> libvir-list@xxxxxxxxxx
> https://www.redhat.com/mailman/listinfo/libvir-list

Attachment: signature.asc
Description: PGP 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]
  Powered by Linux