On 06/14/2017 03:27 PM, Erik Skultety wrote: > On Wed, Jun 14, 2017 at 10:48:33AM +0200, Michal Privoznik wrote: >> https://bugzilla.redhat.com/show_bug.cgi?id=1214369 >> >> My fix 671d18594f4 was incomplete. If domain doesn't have >> hugepages enalbed, because of missing condition we would still be > > s/enalbed/enabled > >> putting hugepages path onto qemu cmd line. Clean up the >> conditions so that it's more visible next time. >> >> Signed-off-by: Michal Privoznik <mprivozn@xxxxxxxxxx> >> --- >> src/qemu/qemu_command.c | 12 +++++----- >> .../qemuxml2argv-cpu-numa-memshared.args | 26 ++++++++++++++++++++++ >> tests/qemuxml2argvtest.c | 1 + >> 3 files changed, 33 insertions(+), 6 deletions(-) >> create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-cpu-numa-memshared.args >> > Fixes the issue described by > https://bugzilla.redhat.com/show_bug.cgi?id=1214369#c16. > ACK Pushed, thanks. Michal -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list
