Re: [libvirt] [RFC] BlockJob: Support sync/async virDomainBlockJobAbort

2011-12-09 Thread Stefan Hajnoczi
On Thu, Dec 8, 2011 at 10:33 PM, Adam Litke a...@us.ibm.com wrote: Qemu has changed the semantics of the block_job_cancel API.  Originally, the operation was synchronous (ie. upon command completion, the operation was guaranteed to be completely stopped).  With the new semantics, a

Re: [libvirt] how to use macvtap in kvm guest

2011-12-09 Thread Amit Tewari
Hi, Can somebody help me in solving the dmesg error: macvtap0:no ipv6 routers present From: xhu [mailto:x...@redhat.com] Sent: Friday, December 09, 2011 10:53 AM To: Amit Tewari Cc: libvir-list@redhat.com Subject: Re: [libvirt] how to use macvtap in

[libvirt] Build issue with libvirt 0.9.8

2011-12-09 Thread Christophe Fergeau
Hi, When building libvirt 0.9.8 with jhbuild, I have now reproduced twice some issue during the build. Namely, after successfully building and installing libvirt 0.9.8, when I try to run virsh, I get: virsh: /home/teuf/jhbuild-boxes/opt/lib64/libvirt.so.0: version `LIBVIRT_PRIVATE_0.9.7' not

Re: [libvirt] Build issue with libvirt 0.9.8

2011-12-09 Thread Daniel P. Berrange
On Fri, Dec 09, 2011 at 11:43:21AM +0100, Christophe Fergeau wrote: Hi, When building libvirt 0.9.8 with jhbuild, I have now reproduced twice some issue during the build. Namely, after successfully building and installing libvirt 0.9.8, when I try to run virsh, I get: virsh:

Re: [libvirt] [PATCH] virsh: correct the cmdDomIfGetLink function return value error

2011-12-09 Thread Peter Krempa
On 12/08/2011 10:31 AM, Alex Jia wrote: On 12/08/2011 05:25 PM, Peter Krempa wrote: On 12/08/2011 07:30 AM, a...@redhat.com wrote: From: Alex Jiaa...@redhat.com * tools/virsh.c: Although finding interface with matching mac addr, forgot to set 'ret' boolean value to 'true'. * how to

Re: [libvirt] [PATCH] cpu: Add cpu flags supported by newest qemu

2011-12-09 Thread Peter Krempa
On 12/08/2011 08:01 PM, Eric Blake wrote: On 12/08/2011 07:18 AM, Peter Krempa wrote: Add support for newly supported Intel cpu features. Newly supported flags are: pclmuldq, dtes64, smx, fma, pdcm, movbe, xsave, osxsave and avx. This adds support for Intel's Sandy Bridge platform. --- I did

[libvirt] [PATCH] bridge: Fix forward delay APIs

2011-12-09 Thread Jiri Denemark
Due to copypaste error in c1df2c14b590b3d68b707aa4f3a570f95a6bc548, virNetDevBridge[SG]etSTPDelay APIs were accessing wrong file. --- src/util/virnetdevbridge.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/util/virnetdevbridge.c b/src/util/virnetdevbridge.c

Re: [libvirt] [PATCH] bridge: Fix forward delay APIs

2011-12-09 Thread Michal Privoznik
On 09.12.2011 13:08, Jiri Denemark wrote: Due to copypaste error in c1df2c14b590b3d68b707aa4f3a570f95a6bc548, virNetDevBridge[SG]etSTPDelay APIs were accessing wrong file. --- src/util/virnetdevbridge.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) ACK Michal --

[libvirt] FYI: SpecVirt results using libvirt + KVM

2011-12-09 Thread Daniel P. Berrange
Some folks may be aware of the SpecVirt benchmark, where KVM is giving the competition something to worry about :-) http://www.spec.org/virt_sc2010/results/ From the POV of libvirt uses though, there has been one annoyance with the previously submitted results though, in that they have been

Re: [libvirt] (no subject)

2011-12-09 Thread Osier Yang
On 2011年12月06日 22:38, Daniel P. Berrange wrote: On Fri, Nov 11, 2011 at 07:56:58PM +0800, Osier Yang wrote: Hi, all This is a basic implementation of libvirt Native Linux KVM Tool driver. Note that this is just made with my own interest and spare time, it's not an endorsement/effort by Red

Re: [libvirt] (no subject)

2011-12-09 Thread Osier Yang
On 2011年12月07日 14:21, Sasha Levin wrote: On Tue, 2011-12-06 at 14:38 +, Daniel P. Berrange wrote: On Fri, Nov 11, 2011 at 07:56:58PM +0800, Osier Yang wrote: * Lacks of options for user's configuration, such as -vnc, there is no option for user to configure the properties for the

Re: [libvirt] (no subject)

2011-12-09 Thread Osier Yang
On 2011年12月07日 17:16, Daniel P. Berrange wrote: On Wed, Dec 07, 2011 at 08:21:06AM +0200, Sasha Levin wrote: On Tue, 2011-12-06 at 14:38 +, Daniel P. Berrange wrote: On Fri, Nov 11, 2011 at 07:56:58PM +0800, Osier Yang wrote: * KVM tool manages the network completely itself (with DHCP

[libvirt] issues with migrating using copy-storage-all

2011-12-09 Thread Reinier Schoof
Hi, today I was trying to use the --copy-storage-all feature of virsh migrate, in an attempt to migrate KVM-instances to another storage backend. Doing so, I ran into some trouble: First of all, it turned out the disk image-file has to be present on the receiving end of the migration. When,

[libvirt] Virtio Ballon driver problem

2011-12-09 Thread Pankaj Rawat
Hi .well I am trying to use virtioballon driver on my KVM host . I have assigned 400 MB current memory to my guest The ballon size is 600MB but when I used all the 400MB of guest the RAM of guest is not increasing. I watch the ram usage by running following command #free -m But the guest is

Re: [libvirt] Build issue with libvirt 0.9.8

2011-12-09 Thread Christophe Fergeau
Hi, On Fri, Dec 09, 2011 at 11:18:48AM +, Daniel P. Berrange wrote: There is something odd about the way it is being built: The combination of these 3 sets of messages, makes me think you have confused libtool into linking to the old binary What arguments are you passing to

[libvirt] [libvirt-glib 11/10] Add gvir_config_storage_pool_set_pool_type

2011-12-09 Thread Christophe Fergeau
--- libvirt-gconfig/libvirt-gconfig-storage-pool.c | 12 libvirt-gconfig/libvirt-gconfig-storage-pool.h | 12 libvirt-gconfig/libvirt-gconfig.sym|2 ++ libvirt-gconfig/tests/test-domain-create.c |1 + 4 files changed, 27 insertions(+), 0

[libvirt] [libvirt-glib] Add annotations for gvir_stream_add_watch{_full}

2011-12-09 Thread Christophe Fergeau
This should silence this compile warning: GENLibvirtGObject-1.0.gir libvirt-gobject-stream.c:603: Warning: LibvirtGObject: gvir_stream_add_watch: argument func: Missing (scope) annotation for callback without GDestroyNotify (valid: call, async) This also adds API documentation for these

[libvirt] [PATCH] threadpool: Don't wait on condition if pool has no workers

2011-12-09 Thread Michal Privoznik
Pool creates new workers dynamically. However, it is possible for a pool to have no workers. If we want to free that pool, we don't want to wait on quit condition as it will never be signaled. --- Pushing under trivial rule src/util/threadpool.c |3 ++- 1 files changed, 2 insertions(+), 1

[libvirt] [PATCH] fix error when parsing ppc64 models on x86 host

2011-12-09 Thread Stefan Berger
When parsing ppc64 models on an x86 host an out-of-memory error message is displayed due to it checking for retcpus being NULL. Fix this by removing the check whether retcpus is NULL since we will realloc into this variable. Also in the X86 model parser display the OOM error at the location

Re: [libvirt] [PATCH] virsh: correct the cmdDomIfGetLink function return value error

2011-12-09 Thread Alex Jia
Peter, it's okay for me, thanks. Regards, Alex - Original Message - From: Peter Krempa pkre...@redhat.com To: Alex Jia a...@redhat.com Cc: libvir-list@redhat.com Sent: Friday, December 9, 2011 7:45:50 PM Subject: Re: [libvirt] [PATCH] virsh: correct the cmdDomIfGetLink function return

[libvirt] [PATCH] fix memory leak in src/nodeinfo.c

2011-12-09 Thread Stefan Berger
Fix memory leak: ==27534== 24 bytes in 1 blocks are definitely lost in loss record 207 of 530 ==27534==at 0x4A05E46: malloc (vg_replace_malloc.c:195) ==27534==by 0x38EC26EC37: vasprintf (in /lib64/libc-2.13.so) ==27534==by 0x4E998E6: virVasprintf (util.c:1677) ==27534==by

[libvirt] [PATCH 3/6] Add access to elements of variables via index

2011-12-09 Thread Stefan Berger
This patch adds access to single elements of variables via index. Example: rule action='accept' direction='in' priority='500' tcp srcipaddr='$ADDR[1]' srcportstart='$B[2]'/ /rule --- src/conf/nwfilter_params.c | 82 +++--

[libvirt] [PATCH 4/6] Address side effects of accessing vars via index

2011-12-09 Thread Stefan Berger
Address side effect of accessing a variable via an index: Filters accessing a variable where an element is accessed that is beyond the size of the list (for example $TEST[10] and only 2 elements are available) cannot instantiate that filter. Test for this and report proper error to user. ---

[libvirt] [PATCH 0/6] nwfilter: Enable access to variables via iterator or index

2011-12-09 Thread Stefan Berger
This patch enables access to variables in filters using indep. iterators ($TEST[$@2]) or via index ($TEST[1]). Three test cases are added that are also being used for libvirt-TCK to check that the instantiation of the filtering rules is correct. Regards, Stefan -- libvir-list mailing

[libvirt] [PATCH 2/6] Introduce possiblity to have an iterator per variable

2011-12-09 Thread Stefan Berger
This patch introduces the capability to use a different iterator per variable. The currently supported notation of variables in a filtering rule like rule action='accept' direction='out' tcp srcipaddr='$A' srcportstart='$B'/ /rule processes the two lists 'A' and 'B' in parallel. This

[libvirt] [PATCH 5/6] Add test cases for new ways to access variables in filters

2011-12-09 Thread Stefan Berger
This patch adds a couple of XML parser / schema validator test cases for the new 'ways' to access variables via index or iterator. --- tests/nwfilterxml2xmlin/iter-test1.xml |6 ++ tests/nwfilterxml2xmlin/iter-test2.xml | 23 +++

[libvirt] [PATCH 1/6] Optimize the elements the iterator visits.

2011-12-09 Thread Stefan Berger
In this patch we introduce testing whether the iterator points to a unique set of entries that have not been seen before at one of the previous iterations. The point is to eliminate duplicates and with that unnecessary filtering rules by preventing identical filtering rules from being

[libvirt] [PATCH 6/6] Add documentation new ways to access members of variables

2011-12-09 Thread Stefan Berger
This patch adds documentation about the new 'ways' that users can access the contents of variables in filters: - access via index: $TEST[2] - access via iterators $TEST[@1] --- docs/formatnwfilter.html.in | 32 +++- 1 file changed, 31 insertions(+), 1 deletion(-)

[libvirt] libvirt hyper-v

2011-12-09 Thread Andy Li
Dear Sir or Madam: I am engineer who want to connect hyper-v R2 via libvirt 0.9.3, but after create ceritificate file, and set the hyper-v winRM status to AllowUnencrypted=true Then I connect the hyper-v by the command: virsh -c hyperv://192.168.58.26:5985 it always reply: Error: unable to

[libvirt] make uninstall fails

2011-12-09 Thread Dave Allan
Hey all, I just noticed that make uninstall from the git HEAD fails: rmdir /etc/sasl2/ rmdir: failed to remove `/etc/sasl2/': Directory not empty it's correct that /etc/sasl2 should not be removed, as it has other files in it, so I think this removal should be more targeted, although I haven't

Re: [libvirt] make uninstall fails

2011-12-09 Thread Daniel P. Berrange
On Fri, Dec 09, 2011 at 10:24:06AM -0500, Dave Allan wrote: Hey all, I just noticed that make uninstall from the git HEAD fails: rmdir /etc/sasl2/ rmdir: failed to remove `/etc/sasl2/': Directory not empty it's correct that /etc/sasl2 should not be removed, as it has other files in it,

Re: [libvirt] Virtio Ballon driver problem

2011-12-09 Thread Eric Blake
On 12/09/2011 02:39 AM, Pankaj Rawat wrote: Hi .well I am trying to use virtioballon driver on my KVM host . I have assigned 400 MB current memory to my guest The ballon size is 600MB but when I used all the 400MB of guest the RAM of guest is not increasing. I watch the ram usage by

Re: [libvirt] [PATCH] threadpool: Don't wait on condition if pool has no workers

2011-12-09 Thread Eric Blake
On 12/09/2011 07:26 AM, Michal Privoznik wrote: Pool creates new workers dynamically. However, it is possible for a pool to have no workers. If we want to free that pool, we don't want to wait on quit condition as it will never be signaled. --- Pushing under trivial rule

Re: [libvirt] [PATCH] fix memory leak in src/nodeinfo.c

2011-12-09 Thread Eric Blake
On 12/09/2011 07:51 AM, Stefan Berger wrote: Fix memory leak: ==27534== 24 bytes in 1 blocks are definitely lost in loss record 207 of 530 ==27534==at 0x4A05E46: malloc (vg_replace_malloc.c:195) ==27534==by 0x38EC26EC37: vasprintf (in /lib64/libc-2.13.so) ==27534==by 0x4E998E6:

Re: [libvirt] [PATCH] fix error when parsing ppc64 models on x86 host

2011-12-09 Thread Eric Blake
On 12/09/2011 07:27 AM, Stefan Berger wrote: When parsing ppc64 models on an x86 host an out-of-memory error message is displayed due to it checking for retcpus being NULL. Fix this by removing the check whether retcpus is NULL since we will realloc into this variable. Also in the X86 model

[libvirt] [PATCH 2/2] threadpool: Use while loop on virCondWait

2011-12-09 Thread Michal Privoznik
instead of simple 'if' statement as virCondWait can return even if associated condition was not signaled. --- src/util/threadpool.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/src/util/threadpool.c b/src/util/threadpool.c index 883d1e5..e8689d9 100644 ---

[libvirt] [PATCH 1/2] threads: Document spurious wakeups on virCondWait

2011-12-09 Thread Michal Privoznik
--- src/util/threads.h |5 + 1 files changed, 5 insertions(+), 0 deletions(-) diff --git a/src/util/threads.h b/src/util/threads.h index b72610c..7d8a6ed 100644 --- a/src/util/threads.h +++ b/src/util/threads.h @@ -88,6 +88,11 @@ void virMutexUnlock(virMutexPtr m); int

Re: [libvirt] [PATCH 2/2] threadpool: Use while loop on virCondWait

2011-12-09 Thread Eric Blake
On 12/09/2011 09:29 AM, Michal Privoznik wrote: instead of simple 'if' statement as virCondWait can return even if associated condition was not signaled. --- src/util/threadpool.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/src/util/threadpool.c

Re: [libvirt] [PATCH 1/2] threads: Document spurious wakeups on virCondWait

2011-12-09 Thread Eric Blake
On 12/09/2011 09:29 AM, Michal Privoznik wrote: --- src/util/threads.h |5 + 1 files changed, 5 insertions(+), 0 deletions(-) diff --git a/src/util/threads.h b/src/util/threads.h index b72610c..7d8a6ed 100644 --- a/src/util/threads.h +++ b/src/util/threads.h @@ -88,6 +88,11 @@

[libvirt] [PATCH v2] fix error when parsing ppc64 models on x86 host

2011-12-09 Thread Stefan Berger
When parsing ppc64 models on an x86 host an out-of-memory error message is displayed due to it checking for retcpus being NULL. Fix this by removing the check whether retcpus is NULL since we will realloc into this variable. Also in the X86 model parser display the OOM error at the location

Re: [libvirt] [PATCH] virsh: plug memory leak on cmdDomblkstat

2011-12-09 Thread Alex Jia
BTW, This patch is missing your review. Thanks, Alex - Original Message - From: Alex Jia a...@redhat.com To: libvir-list@redhat.com Cc: a...@redhat.com Sent: Thursday, December 8, 2011 5:22:51 PM Subject: Re: [libvirt][PATCH] virsh: plug memory leak on cmdDomblkstat On 12/08/2011 05:11

Re: [libvirt] [PATCH v2] fix error when parsing ppc64 models on x86 host

2011-12-09 Thread Eric Blake
On 12/09/2011 09:45 AM, Stefan Berger wrote: When parsing ppc64 models on an x86 host an out-of-memory error message is displayed due to it checking for retcpus being NULL. Fix this by removing the check whether retcpus is NULL since we will realloc into this variable. Also in the X86 model

Re: [libvirt] [PATCH v2] fix error when parsing ppc64 models on x86 host

2011-12-09 Thread Stefan Berger
On 12/09/2011 12:01 PM, Eric Blake wrote: On 12/09/2011 09:45 AM, Stefan Berger wrote: When parsing ppc64 models on an x86 host an out-of-memory error message is displayed due to it checking for retcpus being NULL. Fix this by removing the check whether retcpus is NULL since we will realloc

Re: [libvirt] [PATCH] fix memory leak in src/nodeinfo.c

2011-12-09 Thread Stefan Berger
On 12/09/2011 10:56 AM, Eric Blake wrote: On 12/09/2011 07:51 AM, Stefan Berger wrote: Fix memory leak: ==27534== 24 bytes in 1 blocks are definitely lost in loss record 207 of 530 ==27534==at 0x4A05E46: malloc (vg_replace_malloc.c:195) ==27534==by 0x38EC26EC37: vasprintf (in

Re: [libvirt] how to use macvtap in kvm guest

2011-12-09 Thread Laine Stump
Please do not top post in replies - make your replies to questions inline, it makes following the conversation *much* easier. On 12/09/2011 02:00 AM, Amit Tewari wrote: Hi, No guest can not get ip. When I do ifup eth0 it give error device not found. Hostos=rhel 6.x86_64 Guest os= rhel

Re: [libvirt] [PATCH] virsh: plug memory leak on cmdDomblkstat

2011-12-09 Thread Eric Blake
On 12/08/2011 02:22 AM, Alex Jia wrote: On 12/08/2011 05:11 PM, a...@redhat.com wrote: From: Alex Jiaa...@redhat.com Detected by valgrind. Leak introduced in commit 88a993b: * tools/virsh.c: fix memory leak on cmdDomblkstat. * how to reproduce? % valgrind -v --leak-check=full virsh

[libvirt] [PATCH] build: follow directory install conventions

2011-12-09 Thread Eric Blake
make uninstall failed: rmdir: failed to remove `/etc/sasl2/': Directory not empty Fix this by following the conventions already present in src/Makefile.am of installing with MKDIR_P, and cleaning up any directory in a builtroot while allowing the directory to still exist if non-empty in an

[libvirt] [PATCH 0/6] Allow minimal bandwidth guarantee

2011-12-09 Thread Michal Privoznik
Since 0.9.4 release libvirt supports setting some QoS attributes on virtual bridges and/or virtual interfaces. Average and peak rate, namely. It lacks minimal guaranteed bandwidth aka 'floor' attribute, though. This patch set tries to fill that hole. However, there are some things you want to know

[libvirt] [PATCH 6/6] domain: Keep assigned class_id in domstatus XML

2011-12-09 Thread Michal Privoznik
Interfaces keeps a class_id, which is an ID from which bridge part of QoS settings is derived. We need to store class_id in domain status file, so we can later pass it to virNetDevBandwidthUnplug. --- src/conf/domain_conf.c |4 +- src/qemu/qemu_domain.c | 66

[libvirt] [PATCH 1/6] bandwidth: add new 'floor' attribute

2011-12-09 Thread Michal Privoznik
This is however supported only on domain interfaces with type='network'. Moreover, target network needs to have at least inbound QoS set. This is required by hierarchical traffic shaping. --- docs/formatdomain.html.in| 21 -- docs/schemas/networkcommon.rng |5 ++

[libvirt] [PATCH 5/6] network: Create status files

2011-12-09 Thread Michal Privoznik
Like we keep domain status file to keep some internal data during daemon restart, now it is needed for network as well. Because network has now class_id attribute which is hidden from users and thus not part of network XML. --- daemon/libvirtd.c |3 + src/conf/network_conf.c |

[libvirt] [PATCH 2/6] bandwidth: Create hierarchical shaping classes

2011-12-09 Thread Michal Privoznik
These classes can borrow unused bandwidth. Basically, only egress qdsics can have classes, therefore we can do this kind of traffic shaping only on host's outgoing, that is domain's incoming traffic. --- src/lxc/lxc_driver.c |3 ++- src/network/bridge_driver.c |4 +++-

[libvirt] [PATCH 3/6] bandwidth: Create (un)plug functions

2011-12-09 Thread Michal Privoznik
These set bridge part of QoS when bringing domain's interface up. --- po/POTFILES.in|1 + src/util/virnetdevbandwidth.c | 177 + src/util/virnetdevbandwidth.h | 14 +++ 3 files changed, 192 insertions(+), 0 deletions(-) diff --git

[libvirt] [PATCH 4/6] bandwidth: Create network (un)plug functions

2011-12-09 Thread Michal Privoznik
Network should be notified if we plug in or unplug an interface, so it can perform some action, e.g. set/unset network part of QoS. --- src/conf/domain_conf.h |1 + src/conf/network_conf.c |1 + src/conf/network_conf.h |2 + src/libvirt_network.syms|2 +

Re: [libvirt] [PATCH 1/6] Allow network model to contain -

2011-12-09 Thread Eric Blake
On 12/08/2011 05:38 PM, Eric Blake wrote: int i; for (i = 0 ; i strlen(model) ; i++) { Hmm - an O(n) function call on an O(n) loop, for a quadratic action (of course, it's in the noise, since the user's model name is likely short). But we can do better with a more efficient

Re: [libvirt] [PATCH 2/6] qemu: Use spapr-vscsi on pseries machine type

2011-12-09 Thread Eric Blake
On 12/07/2011 11:41 PM, Michael Ellerman wrote: From: Michael Ellerman mich...@ellerman.id.au On the PPC64 pseries machine type we need to use the spapr-vscsi device rather than an lsi. Signed-off-by: Michael Ellerman mich...@ellerman.id.au --- src/qemu/qemu_command.c | 13

Re: [libvirt] [PATCH 3/6] qemu: Add address in qemuBuildChrDeviceStr() on pseries

2011-12-09 Thread Eric Blake
On 12/07/2011 11:41 PM, Michael Ellerman wrote: From: Michael Ellerman mich...@ellerman.id.au For the PPC64 pseries machine type we need to add address information for the spapr-vty device. Signed-off-by: Michael Ellerman mich...@ellerman.id.au --- src/qemu/qemu_command.c | 10

[libvirt] [PATCH 1/1] Fix make uninstall

2011-12-09 Thread Dave Allan
Make uninstall currently fails with the following message: rmdir /etc/sasl2/ rmdir: failed to remove `/etc/sasl2/': Directory not empty That's fine (correct in fact) so force the command to return success with || : --- daemon/Makefile.am |2 +- 1 files changed, 1 insertions(+), 1

[libvirt] [PATCH] test: replace deprecated fedora-13 machine with pc-0.13

2011-12-09 Thread Laine Stump
One of the xml tests in the test suite was created using a now-deprecated qemu machine type (fedora-13, which was only ever valid for Fedora builds of qemu). Although strictly speaking it's not necessary to replace it with an actual supported qemu machine type (since the xml in question is never

Re: [libvirt] [PATCH] test: replace deprecated fedora-13 machine with pc-0.13

2011-12-09 Thread Eric Blake
On 12/09/2011 01:59 PM, Laine Stump wrote: One of the xml tests in the test suite was created using a now-deprecated qemu machine type (fedora-13, which was only ever valid for Fedora builds of qemu). Although strictly speaking it's not necessary to replace it with an actual supported qemu

Re: [libvirt] make uninstall fails

2011-12-09 Thread Dave Allan
On Fri, Dec 09, 2011 at 10:24:06AM -0500, Dave Allan wrote: Hey all, I just noticed that make uninstall from the git HEAD fails: rmdir /etc/sasl2/ rmdir: failed to remove `/etc/sasl2/': Directory not empty it's correct that /etc/sasl2 should not be removed, as it has other files in it,

Re: [libvirt] [PATCH 4/6] Add address type for SPAPR VIO devices

2011-12-09 Thread Eric Blake
On 12/07/2011 11:41 PM, Michael Ellerman wrote: From: Michael Ellerman micha...@au1.ibm.com This is a different address than you used for patch 2 and 3 (and yet a third address compared to the email where you sent this patch). We can cope with that, but it means picking a favorite address for

[libvirt] [PATCH] network: don't add iptables rules for externally managed networks

2011-12-09 Thread Laine Stump
From: Laine Stump la...@redhat.com This patch addresses https://bugzilla.redhat.com/show_bug.cgi?id=760442 When a network has any forward type other than route, nat or none, the network configuration should be done completely external to libvirt - libvirt only uses these types to allow

Re: [libvirt] [PATCH] network: don't add iptables rules for externally managed networks

2011-12-09 Thread Eric Blake
On 12/09/2011 02:59 PM, Laine Stump wrote: From: Laine Stump la...@redhat.com This patch addresses https://bugzilla.redhat.com/show_bug.cgi?id=760442 When a network has any forward type other than route, nat or none, the network configuration should be done completely external to libvirt -

Re: [libvirt] [PATCH 5/6] qemu: Prepare to cater for more general address assignment

2011-12-09 Thread Eric Blake
On 12/07/2011 11:41 PM, Michael Ellerman wrote: From: Michael Ellerman mich...@ellerman.id.au Currently qemuDomainAssignPCIAddresses() is called to assign addresses to PCI devices. We need to do something similar for devices with spapr-vio addresses. So create one place where address

Re: [libvirt] [PATCH 6/6] qemu: Add spapr-vio address assignment

2011-12-09 Thread Eric Blake
On 12/07/2011 11:41 PM, Michael Ellerman wrote: From: Michael Ellerman mich...@ellerman.id.au Add logic to assign addresses for devices with spapr-vio addresses. We also do validation of addresses specified by the user, ie. ensuring that there are not duplicate addresses on the bus.

[libvirt] [PATCH] docs: document address elements in one place

2011-12-09 Thread Eric Blake
Improve the documentation of what forms a valid address element, since these elements appear in numerous devices. * docs/formatdomain.html.in (elementsAddress): New section. (elementsControllers, elementsUSB, elementsNICS, elementsInput) (elementsHub, elementsCharChannel, elementsSound): Refer to

[libvirt] [PATCH] security: don't try to label network disks

2011-12-09 Thread Josh Durgin
Network disks don't have paths to be resolved or files to be checked for ownership. ee3efc41e6233e625aa03003bf3127319ccd546f checked this for some image label functions, but was partially reverted in a refactor. This finishes adding the check to each security driver's set and restore label

Re: [libvirt] [PATCH] test: replace deprecated fedora-13 machine with pc-0.13

2011-12-09 Thread Laine Stump
On 12/09/2011 04:08 PM, Eric Blake wrote: On 12/09/2011 01:59 PM, Laine Stump wrote: One of the xml tests in the test suite was created using a now-deprecated qemu machine type (fedora-13, which was only ever valid for Fedora builds of qemu). Although strictly speaking it's not necessary to

Re: [libvirt] [PATCH] test: fix potential lock corruption in test driver

2011-12-09 Thread Laine Stump
On 12/08/2011 02:39 PM, Eric Blake wrote: On 12/08/2011 12:31 PM, Laine Stump wrote: From: Laine Stumpla...@redhat.com In some error situations, the function testDomainRestoreFlags() could unlock the test driver mutex without first locking it. This patch moves the lock operation earlier, so

Re: [libvirt] [PATCH] network: don't add iptables rules for externally managed networks

2011-12-09 Thread Laine Stump
On 12/09/2011 05:07 PM, Eric Blake wrote: On 12/09/2011 02:59 PM, Laine Stump wrote: From: Laine Stumpla...@redhat.com This patch addresses https://bugzilla.redhat.com/show_bug.cgi?id=760442 When a network has any forward type other than route, nat or none, the network configuration should be