- Devel - libvirt List Archives

[libvirt] [PATCH 0/2] Minor doc/error message fixes
by Peter Krempa 28 Nov '12

28 Nov '12

Peter Krempa (2): qemu: Fix error messages when dispatching guest agent commands qemu: Drop word "either" from comments for agent monitor functions src/qemu/qemu_agent.c | 11 ++++++----- src/qemu/qemu_domain.c | 10 +++++----- 2 files changed, 11 insertions(+), 10 deletions(-) -- 1.8.0

3 6

[libvirt] [PATCH 0/5] Support Ephemeral passthrough hostdevs
by Shradha Shah 28 Nov '12

28 Nov '12

The ephemeral flag helps support migration with PCI-passthrough. An ephemeral hostdev is automatically unplugged before migration and replugged (if one is available on the destination) after migration. Shradha Shah (5): Added ephemeral flag for hostdev in domain conf. Adding ephemeral flag for hostdev in network conf. Ephemeral flag mofication within the network driver. Ephemeral flag modification within the qemu driver. Migration support for ephemeral hostdevs. docs/schemas/domaincommon.rng | 16 ++++ docs/schemas/network.rng | 8 ++ src/conf/domain_conf.c | 23 +++++- src/conf/domain_conf.h | 1 + src/conf/network_conf.c | 11 +++ src/conf/network_conf.h | 1 + src/network/bridge_driver.c | 1 + src/qemu/qemu_command.c | 63 +++++++++----- src/qemu/qemu_migration.c | 94 +++++++++++++++++++- tests/networkxml2xmlin/hostdev-pf.xml | 2 +- tests/networkxml2xmlin/hostdev.xml | 2 +- tests/networkxml2xmlout/hostdev-pf.xml | 2 +- tests/networkxml2xmlout/hostdev.xml | 2 +- .../qemuxml2argv-hostdev-pci-address.xml | 2 +- .../qemuxml2argv-hostdev-usb-address.xml | 2 +- .../qemuxml2argvdata/qemuxml2argv-net-hostdev.xml | 2 +- tests/qemuxml2argvdata/qemuxml2argv-pci-rom.xml | 4 +- 17 files changed, 200 insertions(+), 36 deletions(-) -- 1.7.4.4

1 5

[libvirt] [PATCH 00/10 v3] Unprivileged SG_IO support
by Osier Yang 28 Nov '12

28 Nov '12

Hi, As a result of RFC [1], this implements the unprivleged SG_IO support. Testing is not that enough, but I'd like see the reviewing earlier, and meanwhile I'm not going to give up the further testing. v2 - v3: * Change the XML tag name to "cdbfilter" * Maintain an internal list of shared disks for QEMU driver. Patches 1/10 ~ 4/10 are to introduce the internal list for shared disks. Osier Yang (10): qemu: Introduce a list to maintain the shared disks between domains qemu: Init/Free the list with the driver's lifecyle qemu: Add/remove the shared disk entry during domain's lifecyle qemu: Add/Remove the entry of sharedDisks when live attaching/detaching docs: Add docs and rng schema for new XML cdbfilter conf: Parse and format the new XML tag cdbfilter util: Prepare helpers for unpriv_sgio setting qemu: Manage disk's cdbfilter in domain's lifecycle qemu: Do not restore the sysfs unpriv_sgio if the disk is being shared qemu: Error out when domain starting if the cdbfilter setting conflicts docs/formatdomain.html.in | 13 ++- docs/schemas/domaincommon.rng | 52 +++++-- src/conf/domain_conf.c | 71 +++++++-- src/conf/domain_conf.h | 13 ++ src/libvirt_private.syms | 5 + src/qemu/qemu_conf.c | 166 ++++++++++++++++++++ src/qemu/qemu_conf.h | 30 ++++ src/qemu/qemu_driver.c | 28 ++++ src/qemu/qemu_process.c | 103 ++++++++++++- src/util/util.c | 145 +++++++++++++++++ src/util/util.h | 7 + ...ml2argv-disk-scsi-lun-passthrough-cdbfilter.xml | 32 ++++ tests/qemuxml2xmltest.c | 1 + 13 files changed, 634 insertions(+), 32 deletions(-) create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-disk-scsi-lun-passthrough-cdbfilter.xml Regards, Osier

1 10

[libvirt] [PATCH 0/4] Introduce support for FITRIM within guest OS
by Michal Privoznik 28 Nov '12

28 Nov '12

https://bugzilla.redhat.com/show_bug.cgi?id=831159 Michal Privoznik (4): Introduce virDomainFSTrim() public API remote: Implement virDomainFSTrim qemu: Implement virDomainFSTrim virsh: Expose new virDomainFSTrim API include/libvirt/libvirt.h.in | 4 ++ src/driver.h | 6 +++ src/libvirt.c | 50 +++++++++++++++++++++++++ src/libvirt_public.syms | 5 +++ src/qemu/qemu_agent.c | 25 +++++++++++++ src/qemu/qemu_agent.h | 2 + src/qemu/qemu_driver.c | 83 ++++++++++++++++++++++++++++++++++++++++++ src/remote/remote_driver.c | 1 + src/remote/remote_protocol.x | 10 +++++- src/remote_protocol-structs | 7 ++++ src/rpc/gendispatch.pl | 1 + tools/virsh-domain.c | 40 ++++++++++++++++++++ tools/virsh.pod | 12 ++++++ 13 files changed, 245 insertions(+), 1 deletions(-) -- 1.7.8.6

3 10

[libvirt] [PATCH v3] bitmap: fix typo to use UL type of integer constant in virBitmapIsAllSet
by Guannan Ren 28 Nov '12

28 Nov '12

This bug leads to getting incorrect vcpupin information via qemudDomainGetVcpuPinInfo() API when the number of maximum cpu on a host falls into a range such as 31 < ncpus < 64. gcc warning: left shift count >= width of type The following bug is such the case https://bugzilla.redhat.com/show_bug.cgi?id=876415 --- src/util/bitmap.c | 4 ++-- tests/virbitmaptest.c | 37 +++++++++++++++++++++++++++++++++++++ 2 files changed, 39 insertions(+), 2 deletions(-) diff --git a/src/util/bitmap.c b/src/util/bitmap.c index 5ec5440..c29f5f3 100644 --- a/src/util/bitmap.c +++ b/src/util/bitmap.c @@ -574,8 +574,8 @@ bool virBitmapIsAllSet(virBitmapPtr bitmap) return false; if (unusedBits > 0) { - if ((bitmap->map[sz] & ((1U << (VIR_BITMAP_BITS_PER_UNIT - unusedBits)) - 1)) - != ((1U << (VIR_BITMAP_BITS_PER_UNIT - unusedBits)) - 1)) + if ((bitmap->map[sz] & ((1UL << (VIR_BITMAP_BITS_PER_UNIT - unusedBits)) - 1)) + != ((1UL << (VIR_BITMAP_BITS_PER_UNIT - unusedBits)) - 1)) return false; } diff --git a/tests/virbitmaptest.c b/tests/virbitmaptest.c index f1eb9d5..af94dab 100644 --- a/tests/virbitmaptest.c +++ b/tests/virbitmaptest.c @@ -347,6 +347,41 @@ error: return ret; } +static int test7(const void *v ATTRIBUTE_UNUSED) +{ + virBitmapPtr bitmap; + size_t i; + size_t maxBit[] = { + 1, 8, 31, 32, 63, 64, 95, 96, 127, 128, 159, 160 + }; + size_t nmaxBit = 12; + + for (i = 0; i < nmaxBit; i++) { + bitmap = virBitmapNew(maxBit[i]); + if (!bitmap) + goto error; + + if (virBitmapIsAllSet(bitmap)) + goto error; + + ignore_value(virBitmapSetBit(bitmap, 1)); + if (virBitmapIsAllSet(bitmap)) + goto error; + + virBitmapSetAll(bitmap); + if (!virBitmapIsAllSet(bitmap)) + goto error; + + virBitmapFree(bitmap); + } + + return 0; + +error: + virBitmapFree(bitmap); + return -1; +} + static int mymain(void) { @@ -364,6 +399,8 @@ mymain(void) ret = -1; if (virtTestRun("test6", 1, test6, NULL) < 0) ret = -1; + if (virtTestRun("test7", 1, test7, NULL) < 0) + ret = -1; return ret; -- 1.7.11.2

3 3

[libvirt] [PATCH v3 00/13] parallels: implement operations with hard disks
by Dmitry Guryanov 28 Nov '12

28 Nov '12

This patch series adds ability to list existing disk images, create a new ones and add or remove hard disk devices to the VM. Changes in v2: rebased to current sources Changes in v3: fixed errors in make syntax-check Dmitry Guryanov (13): parallels: add info about hard disk devices parallels: handle disk devices in parallelsDomainDefineXML parallels: split parallelsStorageOpen function parallels: remove unused code from storage driver parallels: create storage pools by VM list parallels: fix leaks in parallelsFindVolumes parallels: add info about volumes parallels: fill volumes capacity parameter parallels: split parallelsStorageVolumeDelete function parallels: add function parallelsGetDiskBusName parallels: add support of disks creation parallels: apply config after VM creation parallels: add support of removing disks src/parallels/parallels_driver.c | 346 +++++++++++++++++++++++++-- src/parallels/parallels_storage.c | 479 +++++++++++++++++++++++++++++++------ src/parallels/parallels_utils.h | 3 + 3 files changed, 731 insertions(+), 97 deletions(-) -- 1.7.7.6

1 13

[libvirt] [test-API][PATCH] balloon_memory: add time break before dump xml
by Wayne Sun 28 Nov '12

28 Nov '12

the xml dumped after setMemory is not accurate due to time delay, so take 10 sec sleep before dump to show the right xml info Signed-off-by: Wayne Sun <gsun(a)redhat.com> --- repos/domain/balloon_memory.py | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/repos/domain/balloon_memory.py b/repos/domain/balloon_memory.py index 7051a0a..5bf023a 100644 --- a/repos/domain/balloon_memory.py +++ b/repos/domain/balloon_memory.py @@ -209,6 +209,7 @@ def balloon_memory(params): logger.debug("dump the xml description of guest virtual machine %s" % domname) + time.sleep(10) dom_xml = domobj.XMLDesc(0) logger.debug("the xml definination is %s" % dom_xml) @@ -240,6 +241,7 @@ def balloon_memory(params): logger.debug("dump the xml description of \ guest virtual machine %s" % domname) + time.sleep(10) dom_xml = domobj.XMLDesc(0) logger.debug("the xml definination is %s" % dom_xml) -- 1.7.1

2 3

[libvirt] [java PATCH] Fix build with jna >= 3.5.0
by Jiri Denemark 28 Nov '12

28 Nov '12

Since jna-3.5.0, all subclasses of com.sun.jna.Structure are required to implement getFieldOrder method which is supposed to return a list of fields the wrapped structure contains. --- src/main/java/org/libvirt/jna/virConnectAuth.java | 5 +++++ src/main/java/org/libvirt/jna/virConnectCredential.java | 6 ++++++ src/main/java/org/libvirt/jna/virDomainBlockInfo.java | 4 ++++ src/main/java/org/libvirt/jna/virDomainBlockStats.java | 5 +++++ src/main/java/org/libvirt/jna/virDomainInfo.java | 5 +++++ src/main/java/org/libvirt/jna/virDomainInterfaceStats.java | 5 +++++ src/main/java/org/libvirt/jna/virDomainJobInfo.java | 7 +++++++ src/main/java/org/libvirt/jna/virDomainMemoryStats.java | 4 ++++ src/main/java/org/libvirt/jna/virError.java | 6 ++++++ src/main/java/org/libvirt/jna/virNodeInfo.java | 8 +++++++- src/main/java/org/libvirt/jna/virSchedParameter.java | 5 +++++ src/main/java/org/libvirt/jna/virStoragePoolInfo.java | 5 +++++ src/main/java/org/libvirt/jna/virStorageVolInfo.java | 6 +++++- src/main/java/org/libvirt/jna/virVcpuInfo.java | 4 ++++ 14 files changed, 73 insertions(+), 2 deletions(-) diff --git a/src/main/java/org/libvirt/jna/virConnectAuth.java b/src/main/java/org/libvirt/jna/virConnectAuth.java index 74dd65c..369d2e5 100644 --- a/src/main/java/org/libvirt/jna/virConnectAuth.java +++ b/src/main/java/org/libvirt/jna/virConnectAuth.java @@ -1,14 +1,19 @@ package org.libvirt.jna; import com.sun.jna.Pointer; import com.sun.jna.Structure; /** * JNA mapping for the virConnectAuth structure */ public class virConnectAuth extends Structure { public Pointer credtype; public int ncredtype; public Libvirt.VirConnectAuthCallback cb; public Pointer cbdata; + + protected java.util.List getFieldOrder() { + return java.util.Arrays.asList(new String[] { + "credtype", "ncredtype", "cb", "cbdata" }); + } } diff --git a/src/main/java/org/libvirt/jna/virConnectCredential.java b/src/main/java/org/libvirt/jna/virConnectCredential.java index ba47482..dbfc38e 100644 --- a/src/main/java/org/libvirt/jna/virConnectCredential.java +++ b/src/main/java/org/libvirt/jna/virConnectCredential.java @@ -1,16 +1,22 @@ package org.libvirt.jna; import com.sun.jna.Structure; /** * JNA mapping for the virConnectCredential structure */ public class virConnectCredential extends Structure implements Structure.ByReference { public int type; public String prompt; public String challenge; public String defresult; // public Pointer result; public String result; public int resultlen; + + protected java.util.List getFieldOrder() { + return java.util.Arrays.asList(new String[] { + "type", "prompt", "challenge", "defresult", + "result", "resultlen" }); + } } diff --git a/src/main/java/org/libvirt/jna/virDomainBlockInfo.java b/src/main/java/org/libvirt/jna/virDomainBlockInfo.java index 234a381..cbd7385 100644 --- a/src/main/java/org/libvirt/jna/virDomainBlockInfo.java +++ b/src/main/java/org/libvirt/jna/virDomainBlockInfo.java @@ -1,10 +1,14 @@ package org.libvirt.jna; import com.sun.jna.Structure; public class virDomainBlockInfo extends Structure { public long capacity; public long allocation; public long physical; + protected java.util.List getFieldOrder() { + return java.util.Arrays.asList(new String[] { + "capacity", "allocation", "physical" }); + } } diff --git a/src/main/java/org/libvirt/jna/virDomainBlockStats.java b/src/main/java/org/libvirt/jna/virDomainBlockStats.java index f29dba0..71041cc 100644 --- a/src/main/java/org/libvirt/jna/virDomainBlockStats.java +++ b/src/main/java/org/libvirt/jna/virDomainBlockStats.java @@ -1,19 +1,24 @@ package org.libvirt.jna; import com.sun.jna.Structure; /** * JNA mapping for the virDomainBlockStats structure */ public class virDomainBlockStats extends Structure { public long rd_req; // this is a long long in the code, so a long mapping is // correct public long rd_bytes;// this is a long long in the code, so a long mapping // is correct public long wr_req; // this is a long long in the code, so a long mapping is // correct public long wr_bytes;// this is a long long in the code, so a long mapping // is correct public long errs; // this is a long long in the code, so a long mapping is // correct + + protected java.util.List getFieldOrder() { + return java.util.Arrays.asList(new String[] { + "rd_req", "rd_bytes", "wr_req", "wr_bytes", "errs" }); + } } diff --git a/src/main/java/org/libvirt/jna/virDomainInfo.java b/src/main/java/org/libvirt/jna/virDomainInfo.java index a93c0a5..994042a 100644 --- a/src/main/java/org/libvirt/jna/virDomainInfo.java +++ b/src/main/java/org/libvirt/jna/virDomainInfo.java @@ -1,15 +1,20 @@ package org.libvirt.jna; import com.sun.jna.NativeLong; import com.sun.jna.Structure; /** * JNA mapping for the virDomainInfo structure */ public class virDomainInfo extends Structure { public int state; public NativeLong maxMem; public NativeLong memory; public short nrVirtCpu; public long cpuTime; + + protected java.util.List getFieldOrder() { + return java.util.Arrays.asList(new String[] { + "state", "maxMem", "memory", "nrVirtCpu", "cpuTime" }); + } } diff --git a/src/main/java/org/libvirt/jna/virDomainInterfaceStats.java b/src/main/java/org/libvirt/jna/virDomainInterfaceStats.java index 1cb0e66..dcdd529 100644 --- a/src/main/java/org/libvirt/jna/virDomainInterfaceStats.java +++ b/src/main/java/org/libvirt/jna/virDomainInterfaceStats.java @@ -8,19 +8,24 @@ import com.sun.jna.Structure; public class virDomainInterfaceStats extends Structure { public long rx_bytes; // this is a long long in the code, so a long mapping // is correct public long rx_packets; // this is a long long in the code, so a long // mapping is correct public long rx_errs; // this is a long long in the code, so a long mapping // is correct public long rx_drop; // this is a long long in the code, so a long mapping // is correct public long tx_bytes; // this is a long long in the code, so a long mapping // is correct public long tx_packets; // this is a long long in the code, so a long // mapping is correct public long tx_errs; // this is a long long in the code, so a long mapping // is correct public long tx_drop; // this is a long long in the code, so a long mapping // is correct + protected java.util.List getFieldOrder() { + return java.util.Arrays.asList(new String[] { + "rx_bytes", "rx_packets", "rx_errs", "rx_drop", + "tx_bytes", "tx_packets", "tx_errs", "tx_drop" }); + } } diff --git a/src/main/java/org/libvirt/jna/virDomainJobInfo.java b/src/main/java/org/libvirt/jna/virDomainJobInfo.java index 98b4f59..af934ad 100644 --- a/src/main/java/org/libvirt/jna/virDomainJobInfo.java +++ b/src/main/java/org/libvirt/jna/virDomainJobInfo.java @@ -1,18 +1,25 @@ package org.libvirt.jna; import com.sun.jna.Structure; public class virDomainJobInfo extends Structure { public int type; public long timeElapsed; public long timeRemaining; public long dataTotal; public long dataProcessed; public long dataRemaining; public long memTotal; public long memProcessed; public long memRemaining; public long fileTotal; public long fileProcessed; public long fileRemaining; + + protected java.util.List getFieldOrder() { + return java.util.Arrays.asList(new String[] { + "type", "timeElapsed", "timeRemaining", "dataTotal", + "dataProcessed", "dataRemaining", "memTotal", "memProcessed", + "memRemaining", "fileTotal", "fileProcessed", "fileRemaining" }); + } } diff --git a/src/main/java/org/libvirt/jna/virDomainMemoryStats.java b/src/main/java/org/libvirt/jna/virDomainMemoryStats.java index 7a9f5d2..ec7b29e 100644 --- a/src/main/java/org/libvirt/jna/virDomainMemoryStats.java +++ b/src/main/java/org/libvirt/jna/virDomainMemoryStats.java @@ -1,8 +1,12 @@ package org.libvirt.jna; import com.sun.jna.Structure; public class virDomainMemoryStats extends Structure { public int tag ; public long val ; + + protected java.util.List getFieldOrder() { + return java.util.Arrays.asList(new String[] { "tag", "val" }); + } } diff --git a/src/main/java/org/libvirt/jna/virError.java b/src/main/java/org/libvirt/jna/virError.java index e2580dd..9009280 100644 --- a/src/main/java/org/libvirt/jna/virError.java +++ b/src/main/java/org/libvirt/jna/virError.java @@ -4,19 +4,25 @@ import com.sun.jna.Pointer; import com.sun.jna.Structure; /** * JNA mapping for the virError structure */ public class virError extends Structure { public int code; public int domain; public String message; public int level; public ConnectionPointer conn; public DomainPointer dom; public String str1; public String str2; public String str3; public int int1; public int int2; public NetworkPointer net; + + protected java.util.List getFieldOrder() { + return java.util.Arrays.asList(new String[] { + "code", "domain", "message", "level", "conn", "dom", + "str1", "str2", "str3", "int1", "int2", "net" }); + } } diff --git a/src/main/java/org/libvirt/jna/virNodeInfo.java b/src/main/java/org/libvirt/jna/virNodeInfo.java index 532aad9..f13eddc 100644 --- a/src/main/java/org/libvirt/jna/virNodeInfo.java +++ b/src/main/java/org/libvirt/jna/virNodeInfo.java @@ -6,19 +6,25 @@ import com.sun.jna.Structure; /** * JNA mapping for the virNodeInfo structure */ public class virNodeInfo extends Structure { public class ByReference extends virNodeInfo implements Structure.ByReference { }; public class ByValue extends virNodeInfo implements Structure.ByValue { }; public byte model[] = new byte[32]; public NativeLong memory; public int cpus; public int mhz; public int nodes; public int sockets; public int cores; public int threads; -} \ No newline at end of file + + protected java.util.List getFieldOrder() { + return java.util.Arrays.asList(new String[] { + "model", "memory", "cpus", "mhz", "nodes", + "sockets", "cores", "threads" }); + } +} diff --git a/src/main/java/org/libvirt/jna/virSchedParameter.java b/src/main/java/org/libvirt/jna/virSchedParameter.java index 0013fff..9496121 100644 --- a/src/main/java/org/libvirt/jna/virSchedParameter.java +++ b/src/main/java/org/libvirt/jna/virSchedParameter.java @@ -1,12 +1,17 @@ package org.libvirt.jna; import com.sun.jna.Structure; /** * JNA mapping for the virSchedParameter structure */ public class virSchedParameter extends Structure { public byte field[] = new byte[Libvirt.VIR_DOMAIN_SCHED_FIELD_LENGTH]; public int type; public virSchedParameterValue value; + + protected java.util.List getFieldOrder() { + return java.util.Arrays.asList(new String[] { + "field", "type", "value" }); + } } diff --git a/src/main/java/org/libvirt/jna/virStoragePoolInfo.java b/src/main/java/org/libvirt/jna/virStoragePoolInfo.java index 5a98e08..7f370cd 100644 --- a/src/main/java/org/libvirt/jna/virStoragePoolInfo.java +++ b/src/main/java/org/libvirt/jna/virStoragePoolInfo.java @@ -1,16 +1,21 @@ package org.libvirt.jna; import com.sun.jna.Structure; /** * JNA mapping for the virStoragePoolInfo structure */ public class virStoragePoolInfo extends Structure { public int state; public long capacity; // this is a long long in the code, so a long mapping // is correct public long allocation; // this is a long long in the code, so a long // mapping is correct public long available; // this is a long long in the code, so a long mapping // is correct + + protected java.util.List getFieldOrder() { + return java.util.Arrays.asList(new String[] { + "state", "capacity", "allocation", "available" }); + } } diff --git a/src/main/java/org/libvirt/jna/virStorageVolInfo.java b/src/main/java/org/libvirt/jna/virStorageVolInfo.java index db8a5bd..a02219e 100644 --- a/src/main/java/org/libvirt/jna/virStorageVolInfo.java +++ b/src/main/java/org/libvirt/jna/virStorageVolInfo.java @@ -1,15 +1,19 @@ package org.libvirt.jna; import com.sun.jna.Structure; /** * JNA mapping for the virStorageVolInfo structure */ public class virStorageVolInfo extends Structure { public int type; public long capacity; // this is a long long in the code, so a long mapping // is correct public long allocation; // this is a long long in the code, so a long // mapping is correct -} \ No newline at end of file + protected java.util.List getFieldOrder() { + return java.util.Arrays.asList(new String[] { + "type", "capacity", "allocation" }); + } +} diff --git a/src/main/java/org/libvirt/jna/virVcpuInfo.java b/src/main/java/org/libvirt/jna/virVcpuInfo.java index 16f237d..e5f4363 100644 --- a/src/main/java/org/libvirt/jna/virVcpuInfo.java +++ b/src/main/java/org/libvirt/jna/virVcpuInfo.java @@ -1,15 +1,19 @@ package org.libvirt.jna; import com.sun.jna.Structure; /** * JNA mapping for the virVcpuInfo structure */ public class virVcpuInfo extends Structure { public int number; public int state; public long cpuTime; // this is a long long in the code, so a long mapping // is correct public int cpu; + protected java.util.List getFieldOrder() { + return java.util.Arrays.asList(new String[] { + "number", "state", "cpuTime", "cpu" }); + } } -- 1.8.0

3 3

[libvirt] [PATCH v2] bitmap: fix typo to use UL type of integer constant in virBitmapIsAllSet
by Guannan Ren 28 Nov '12

28 Nov '12

This bug leads to get incorrect vcpupin information via qemudDomainGetVcpuPinInfo() API when the number of maximum cpu on a host falls into a range such as 31 < ncpus < 64. gcc warning: left shift count >= width of type The following bug is such the case https://bugzilla.redhat.com/show_bug.cgi?id=876415 --- src/util/bitmap.c | 4 ++-- tests/virbitmaptest.c | 33 +++++++++++++++++++++++++++++++++ 2 files changed, 35 insertions(+), 2 deletions(-) diff --git a/src/util/bitmap.c b/src/util/bitmap.c index 5ec5440..c29f5f3 100644 --- a/src/util/bitmap.c +++ b/src/util/bitmap.c @@ -574,8 +574,8 @@ bool virBitmapIsAllSet(virBitmapPtr bitmap) return false; if (unusedBits > 0) { - if ((bitmap->map[sz] & ((1U << (VIR_BITMAP_BITS_PER_UNIT - unusedBits)) - 1)) - != ((1U << (VIR_BITMAP_BITS_PER_UNIT - unusedBits)) - 1)) + if ((bitmap->map[sz] & ((1UL << (VIR_BITMAP_BITS_PER_UNIT - unusedBits)) - 1)) + != ((1UL << (VIR_BITMAP_BITS_PER_UNIT - unusedBits)) - 1)) return false; } diff --git a/tests/virbitmaptest.c b/tests/virbitmaptest.c index f1eb9d5..e21bb02 100644 --- a/tests/virbitmaptest.c +++ b/tests/virbitmaptest.c @@ -347,6 +347,37 @@ error: return ret; } +static int test7(const void *v ATTRIBUTE_UNUSED) +{ + virBitmapPtr bitmap; + size_t i; + size_t maxBit[] = { + 1, 8, 31, 32, 63, 64, 95, 96, 127, 128, 159, 160 + }; + size_t nmaxBit = 12; + + for (i = 0; i < nmaxBit; i++) { + bitmap = virBitmapNew(maxBit[i]); + if (!bitmap) + goto error; + + if (virBitmapIsAllSet(bitmap)) + goto error; + + virBitmapSetAll(bitmap); + if (!virBitmapIsAllSet(bitmap)) + goto error; + + virBitmapFree(bitmap); + } + + return 0; + +error: + virBitmapFree(bitmap); + return -1; +} + static int mymain(void) { @@ -364,6 +395,8 @@ mymain(void) ret = -1; if (virtTestRun("test6", 1, test6, NULL) < 0) ret = -1; + if (virtTestRun("test7", 1, test7, NULL) < 0) + ret = -1; return ret; -- 1.7.1

2 2

[libvirt] RFC: adding configuration for standard dhcp options to <network>
by Laine Stump 28 Nov '12

28 Nov '12

There have been multiple requests over the last couple years to make the DHCP server of libvirt's virutal networks more configurable (for example, see the following BZ: https://bugzilla.redhat.com/show_bug.cgi?id=824573 ). You can see a full list of the options supported by dnsmasq with this command: dnsmasq --help dhcp The biggest question here is in exactly how to represent the options in the network XML. I can see a few different options, among them: 1) make each option its own element within <dhcp>, e.g.: <dnsServer family='ipv4' address='x.x.x.x'/> <dnsServer family='ipv6' address='xxxx:xxxx::1'/> <ntpServer address='y.y.y.y'/> <smtpServer address='z.z.z.z'/> 2) A variation of (1) where each option could appear only once, but with two attributes, one for ipv6 address and one for ipv4: <dnsServer address4='x.x.x.x'address6='xxxx:xxxx::1'/> <ntpServer address4='y.y.y.y'/> 3) have a repeatable "option" element (again, a subelement of <dhcp> that can contain any number of the options as attributes: <option family='ipv4' dnsServer='x.x.x.x' ntpServer='y.y.y.y' smtpServer='z.z.z.z'/> <option family='ipv6' dnsServer='xxxx:xxxx:1'/> (in this case, we may or may not support repeating <option> to add more options for the same family). 4) yet another variation, where each option is a subelement of <option family='xxxx'>: <option family='ipv4'> <dnsServer>x.x.x.x</dnsServer> <ntpServer>y.y.y.y</ntpServer> <smtpServer>z.z.z.z</smtpServer> </option> <option family='ipv6'> <dnsServer>xxxx:xxxx::1</dnsServer> </option> I'm not sure which of these would be more straightforward for a user. One thought I've had is that dnsmasq at least allows specifying different groups of guest addresses (with a "tag") and applying different options to addresses with different tags. I don't know if we'll ever use that capability, but if we do it *seems* like any of these methods could be adapted to specifying multiple groups. Does anyone have a preference? (I have a narrow preference for (1), simply because it was the first to come to my mind, but could easily be swayed to one of the others, or something else, given a logical reason).

3 4