Re: [libvirt] [PATCHv3] numatune: Fix parsing of empty nodeset (0, ^0)

2014-08-12 Thread Ján Tomko
On 08/11/2014 04:06 PM, Erik Skultety wrote:
 Resolves https://bugzilla.redhat.com/show_bug.cgi?id=1121837
 ---
  src/util/virbitmap.c   |  3 +++
  ...emuxml2argv-numatune-memory-invalid-nodeset.xml | 31 
 ++
  tests/qemuxml2argvtest.c   |  1 +
  3 files changed, 35 insertions(+)
  create mode 100644 
 tests/qemuxml2argvdata/qemuxml2argv-numatune-memory-invalid-nodeset.xml

ACK and pushed. Congratulations on your first libvirt patch!

Jan




signature.asc
Description: OpenPGP digital signature
--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list

[libvirt] [PATCHv3] numatune: Fix parsing of empty nodeset (0,^0)

2014-08-11 Thread Erik Skultety
Resolves https://bugzilla.redhat.com/show_bug.cgi?id=1121837
---
 src/util/virbitmap.c   |  3 +++
 ...emuxml2argv-numatune-memory-invalid-nodeset.xml | 31 ++
 tests/qemuxml2argvtest.c   |  1 +
 3 files changed, 35 insertions(+)
 create mode 100644 
tests/qemuxml2argvdata/qemuxml2argv-numatune-memory-invalid-nodeset.xml

diff --git a/src/util/virbitmap.c b/src/util/virbitmap.c
index 27282df..b6bd074 100644
--- a/src/util/virbitmap.c
+++ b/src/util/virbitmap.c
@@ -378,6 +378,9 @@ virBitmapParse(const char *str,
 }
 }
 
+if (virBitmapIsAllClear(*bitmap))
+goto error;
+
 return virBitmapCountBits(*bitmap);
 
  error:
diff --git 
a/tests/qemuxml2argvdata/qemuxml2argv-numatune-memory-invalid-nodeset.xml 
b/tests/qemuxml2argvdata/qemuxml2argv-numatune-memory-invalid-nodeset.xml
new file mode 100644
index 000..079ca9d
--- /dev/null
+++ b/tests/qemuxml2argvdata/qemuxml2argv-numatune-memory-invalid-nodeset.xml
@@ -0,0 +1,31 @@
+domain type='qemu'
+  nameQEMUGuest1/name
+  uuidc7a5fdbd-edaf-9455-926a-d65c16db1809/uuid
+  memory unit='KiB'219136/memory
+  currentMemory unit='KiB'219136/currentMemory
+  vcpu placement='static' cpuset='0-1'2/vcpu
+  numatune
+memory mode=strict nodeset=0,^0/
+  /numatune
+  os
+type arch='i686' machine='pc'hvm/type
+boot dev='hd'/
+  /os
+  cpu
+topology sockets='2' cores='1' threads='1'/
+  /cpu
+  clock offset='utc'/
+  on_poweroffdestroy/on_poweroff
+  on_rebootrestart/on_reboot
+  on_crashdestroy/on_crash
+  devices
+emulator/usr/bin/qemu/emulator
+disk type='block' device='disk'
+  source dev='/dev/HostVG/QEMUGuest1'/
+  target dev='hda' bus='ide'/
+  address type='drive' controller='0' bus='0' target='0' unit='0'/
+/disk
+controller type='ide' index='0'/
+memballoon model='virtio'/
+  /devices
+/domain
diff --git a/tests/qemuxml2argvtest.c b/tests/qemuxml2argvtest.c
index 1c121ff..62b969c 100644
--- a/tests/qemuxml2argvtest.c
+++ b/tests/qemuxml2argvtest.c
@@ -1210,6 +1210,7 @@ mymain(void)
 DO_TEST(cputune-zero-shares, QEMU_CAPS_NAME);
 
 DO_TEST(numatune-memory, NONE);
+DO_TEST_PARSE_ERROR(numatune-memory-invalid-nodeset, NONE);
 DO_TEST(numatune-memnode, QEMU_CAPS_NUMA, QEMU_CAPS_OBJECT_MEMORY_RAM);
 DO_TEST_FAILURE(numatune-memnode, NONE);
 
-- 
1.9.3

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list