|
 |
51d9a2 |
From 8a029bcd0227b8c9829858254f9129a9406a167f Mon Sep 17 00:00:00 2001
|
|
|
51d9a2 |
Message-Id: <8a029bcd0227b8c9829858254f9129a9406a167f@dist-git>
|
|
|
51d9a2 |
From: =?UTF-8?q?Daniel=20P=2E=20Berrang=C3=A9?= <berrange@redhat.com>
|
|
|
51d9a2 |
Date: Fri, 6 Jul 2018 15:46:59 +0100
|
|
|
51d9a2 |
Subject: [PATCH] qemu: remove chardevStdioLogd param from vhostuser code path
|
|
|
51d9a2 |
MIME-Version: 1.0
|
|
|
51d9a2 |
Content-Type: text/plain; charset=UTF-8
|
|
|
51d9a2 |
Content-Transfer-Encoding: 8bit
|
|
|
51d9a2 |
|
|
|
51d9a2 |
https://bugzilla.redhat.com/show_bug.cgi?id=1597940
|
|
|
51d9a2 |
|
|
|
51d9a2 |
The vhostuser network backend is only supported with the UNIX domain
|
|
|
51d9a2 |
socket chardev backend, so passing around chardevStdioLogd is not
|
|
|
51d9a2 |
required.
|
|
|
51d9a2 |
|
|
|
51d9a2 |
Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
|
|
|
51d9a2 |
(cherry picked from commit 4c76266baca407ae4981a180aed45193e50a8ade)
|
|
|
51d9a2 |
Reviewed-by: Jiri Denemark <jdenemar@redhat.com>
|
|
|
51d9a2 |
---
|
|
|
51d9a2 |
src/qemu/qemu_command.c | 20 +++++++-------------
|
|
|
51d9a2 |
1 file changed, 7 insertions(+), 13 deletions(-)
|
|
|
51d9a2 |
|
|
|
51d9a2 |
diff --git a/src/qemu/qemu_command.c b/src/qemu/qemu_command.c
|
|
|
51d9a2 |
index 4120e4f983..41050bfea3 100644
|
|
|
51d9a2 |
--- a/src/qemu/qemu_command.c
|
|
|
51d9a2 |
+++ b/src/qemu/qemu_command.c
|
|
|
51d9a2 |
@@ -8223,8 +8223,7 @@ qemuBuildVhostuserCommandLine(virQEMUDriverPtr driver,
|
|
|
51d9a2 |
virDomainDefPtr def,
|
|
|
51d9a2 |
virDomainNetDefPtr net,
|
|
|
51d9a2 |
virQEMUCapsPtr qemuCaps,
|
|
|
51d9a2 |
- unsigned int bootindex,
|
|
|
51d9a2 |
- bool chardevStdioLogd)
|
|
|
51d9a2 |
+ unsigned int bootindex)
|
|
|
51d9a2 |
{
|
|
|
51d9a2 |
virQEMUDriverConfigPtr cfg = virQEMUDriverGetConfig(driver);
|
|
|
51d9a2 |
char *chardev = NULL;
|
|
|
51d9a2 |
@@ -8245,7 +8244,7 @@ qemuBuildVhostuserCommandLine(virQEMUDriverPtr driver,
|
|
|
51d9a2 |
cmd, cfg, def,
|
|
|
51d9a2 |
net->data.vhostuser,
|
|
|
51d9a2 |
net->info.alias, qemuCaps, false,
|
|
|
51d9a2 |
- chardevStdioLogd)))
|
|
|
51d9a2 |
+ false)))
|
|
|
51d9a2 |
goto cleanup;
|
|
|
51d9a2 |
break;
|
|
|
51d9a2 |
|
|
|
51d9a2 |
@@ -8319,8 +8318,7 @@ qemuBuildInterfaceCommandLine(virQEMUDriverPtr driver,
|
|
|
51d9a2 |
virNetDevVPortProfileOp vmop,
|
|
|
51d9a2 |
bool standalone,
|
|
|
51d9a2 |
size_t *nnicindexes,
|
|
|
51d9a2 |
- int **nicindexes,
|
|
|
51d9a2 |
- bool chardevStdioLogd)
|
|
|
51d9a2 |
+ int **nicindexes)
|
|
|
51d9a2 |
{
|
|
|
51d9a2 |
int ret = -1;
|
|
|
51d9a2 |
char *nic = NULL, *host = NULL;
|
|
|
51d9a2 |
@@ -8443,8 +8441,7 @@ qemuBuildInterfaceCommandLine(virQEMUDriverPtr driver,
|
|
|
51d9a2 |
|
|
|
51d9a2 |
case VIR_DOMAIN_NET_TYPE_VHOSTUSER:
|
|
|
51d9a2 |
ret = qemuBuildVhostuserCommandLine(driver, logManager, secManager, cmd, def,
|
|
|
51d9a2 |
- net, qemuCaps, bootindex,
|
|
|
51d9a2 |
- chardevStdioLogd);
|
|
|
51d9a2 |
+ net, qemuCaps, bootindex);
|
|
|
51d9a2 |
goto cleanup;
|
|
|
51d9a2 |
break;
|
|
|
51d9a2 |
|
|
|
51d9a2 |
@@ -8628,8 +8625,7 @@ qemuBuildNetCommandLine(virQEMUDriverPtr driver,
|
|
|
51d9a2 |
bool standalone,
|
|
|
51d9a2 |
size_t *nnicindexes,
|
|
|
51d9a2 |
int **nicindexes,
|
|
|
51d9a2 |
- unsigned int *bootHostdevNet,
|
|
|
51d9a2 |
- bool chardevStdioLogd)
|
|
|
51d9a2 |
+ unsigned int *bootHostdevNet)
|
|
|
51d9a2 |
{
|
|
|
51d9a2 |
size_t i;
|
|
|
51d9a2 |
int last_good_net = -1;
|
|
|
51d9a2 |
@@ -8656,8 +8652,7 @@ qemuBuildNetCommandLine(virQEMUDriverPtr driver,
|
|
|
51d9a2 |
if (qemuBuildInterfaceCommandLine(driver, logManager, secManager, cmd, def, net,
|
|
|
51d9a2 |
qemuCaps, bootNet, vmop,
|
|
|
51d9a2 |
standalone, nnicindexes,
|
|
|
51d9a2 |
- nicindexes,
|
|
|
51d9a2 |
- chardevStdioLogd) < 0)
|
|
|
51d9a2 |
+ nicindexes) < 0)
|
|
|
51d9a2 |
goto error;
|
|
|
51d9a2 |
|
|
|
51d9a2 |
last_good_net = i;
|
|
|
51d9a2 |
@@ -10318,8 +10313,7 @@ qemuBuildCommandLine(virQEMUDriverPtr driver,
|
|
|
51d9a2 |
|
|
|
51d9a2 |
if (qemuBuildNetCommandLine(driver, logManager, secManager, cmd, def,
|
|
|
51d9a2 |
qemuCaps, vmop, standalone,
|
|
|
51d9a2 |
- nnicindexes, nicindexes, &bootHostdevNet,
|
|
|
51d9a2 |
- chardevStdioLogd) < 0)
|
|
|
51d9a2 |
+ nnicindexes, nicindexes, &bootHostdevNet) < 0)
|
|
|
51d9a2 |
goto error;
|
|
|
51d9a2 |
|
|
|
51d9a2 |
if (qemuBuildSmartcardCommandLine(logManager, secManager, cmd, cfg, def, qemuCaps,
|
|
|
51d9a2 |
--
|
|
|
51d9a2 |
2.18.0
|
|
|
51d9a2 |
|