[libvirt] [PATCHv2 8/9] qemu: clean up qemuBuildCommandline loop that builds controller args
Ján Tomko
jtomko at redhat.com
Fri May 15 15:26:53 UTC 2015
On Thu, May 14, 2015 at 03:36:28PM -0400, Laine Stump wrote:
> Reorganize the loop that builds controller args to remove unnecessary
> duplicated code and superfluous else clauses. No functional change
> (this was split out from the following patch to make review easier).
This note can be dropped from git history.
> ---
>
> New in V2 - this was a part of patch 11/13 in V1. jtomko requested
> that I separate out the cleanup from the bugfix, so this is the
> cleanup of existing code, and the bugfix will be in the next patch.
Thanks, it's much easier to follow now.
>
> src/qemu/qemu_command.c | 81 ++++++++++++++++++++++++-------------------------
> 1 file changed, 39 insertions(+), 42 deletions(-)
>
>
> - /* Only recent QEMU implements a SATA (AHCI) controller */
> - if (cont->type == VIR_DOMAIN_CONTROLLER_TYPE_SATA) {
> - if (!virQEMUCapsGet(qemuCaps, QEMU_CAPS_ICH9_AHCI)) {
> - virReportError(VIR_ERR_CONFIG_UNSUPPORTED, "%s",
> - _("SATA is not supported with this "
> - "QEMU binary"));
> - goto error;
> - } else if (cont->idx == 0 && qemuDomainMachineIsQ35(def)) {
> - /* first SATA controller on Q35 machines is implicit */
> + /* first SATA controller on Q35 machines is implicit */
> + if (cont->type == VIR_DOMAIN_CONTROLLER_TYPE_SATA &&
> + cont->idx == 0 && qemuDomainMachineIsQ35(def))
> continue;
Unless intended, this is indented one level too many.
> - } else {
> - char *devstr;
> -
> - virCommandAddArg(cmd, "-device");
> - if (!(devstr = qemuBuildControllerDevStr(def, cont,
> - qemuCaps, NULL)))
> - goto error;
>
> - virCommandAddArg(cmd, devstr);
> - VIR_FREE(devstr);
> - }
Jan
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20150515/fbd88d95/attachment-0001.sig>
More information about the libvir-list
mailing list