On Tue, Mar 08, 2016 at 02:03:16PM -0500, John Ferlan wrote: > Add new function to manage adding the network device options to the > command line removing that task from the mainline qemuBuildCommandLine. > > Signed-off-by: John Ferlan <jferlan@xxxxxxxxxx> > --- > src/qemu/qemu_command.c | 133 +++++++++++++++++++++++++++++------------------- > 1 file changed, 82 insertions(+), 51 deletions(-) > > @@ -9029,8 +9062,6 @@ qemuBuildCommandLine(virConnectPtr conn, > /* free up any resources in the network driver > * but don't overwrite the original error */ This saving of the error was done because of the virDomainConfNWFilterTeardown call. Removing the call made the *Error calls pointless. > originalError = virSaveLastError(); > - for (i = 0; last_good_net != -1 && i <= last_good_net; i++) > - virDomainConfNWFilterTeardown(def->nets[i]); Before, this tore down the filters if any part of the command line building failed. Now it's only done when qemuBuildNetCommandLine fails. Fortunately, it seems these calls were redundant because qemuProcessStop also calls virDomainConfVMNWFilterTeardown. Jan -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list