Re: [libvirt] virsh console giving Error

2011-12-12 Thread Peter Krempa
Dňa 12.12.2011 8:05, Amit Tewari wrote / napísal(a): Hi, I am using libvirt 0.9.7 This is a known regression in 0.9.7. It has been fixed by two commits upstream and is now included in 0.9.8 release. An upgrade should fix your problem. Peter -Original Message- From: Kashyap

[libvirt] macvtap support in libvirt 0.9.8

2011-12-12 Thread Amit Tewari
Hi all, My test environment Host os=rhel6.1 Guest os = rhel6.1 Libvirt=0.9.8 Kvm hypervisor I am creating a macvtap interface on host machine at eth0 interface #ip link show macvtap0 51: macvtap0@eth0: BROADCAST,MULTICAST,UP,LOWER_UP mtu 1500 qdisc mq state UNKNOWN qlen 500

Re: [libvirt] [PATCH] storage: Activate/deactivate logical volumes only on local node

2011-12-12 Thread Michal Privoznik
On 12.12.2011 08:40, Osier Yang wrote: From: Rommer rom...@active.by Current -ay | -an has problems on pool starting/refreshing if the volumes are clustered. Rommer has posted a patch to list 2 months ago. https://www.redhat.com/archives/libvir-list/2011-October/msg01116.html But IMO

Re: [libvirt] [test-API][PATCH] Add test case set_cpu_shares.py for setting cpu scheduler info

2011-12-12 Thread Nan Zhang
On 12/12/2011 03:46 PM, Osier Yang wrote: On 2011年12月12日 15:22, Nan Zhang wrote: * repos/domain/set_cpu_shares.py: set the value of cpu_shares property of the guest. --- repos/domain/set_cpu_shares.py | 111 1 files changed, 111 insertions(+), 0

Re: [libvirt] macvtap support in libvirt 0.9.8

2011-12-12 Thread Amit Tewari
Hi all, Please ignore my previous mail.. This is my test scenario My test environment Host os=rhel6.1 Guest os = rhel6.1 Libvirt=0.9.8 Kvm hypervisor I am creating a macvtap interface on host machine at eth0 interface #ip link show macvtap0 51: macvtap0@eth0:

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

2011-12-12 Thread Michal Privoznik
On 09.12.2011 21:58, Dave Allan wrote: 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 || : --- ACKed pushed. Michal --

Re: [libvirt] macvtap support in libvirt 0.9.8

2011-12-12 Thread Amit Tewari
Hi all, When I see dmesg on host machine it gives error- macvtap0: no IPv6 routers present Is this error causing saome problem.. Even if guest starts it gives error Bringing up interface eth0: Device eth0 does not seem to be present, delaying initialization. [FAILED] Is this dmesg = -

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

2011-12-12 Thread Michal Privoznik
On 11.12.2011 23:38, Daniel P. Berrange wrote: On Fri, Dec 09, 2011 at 07:49:40PM -0800, Josh Durgin wrote: 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

[libvirt] [PATCH v2 0/4] Interface pools and passthrough mode

2011-12-12 Thread Shradha Shah
Interface Pools and Passthrough mode: Current Method: The passthrough mode uses a macvtap direct connection to connect each guest to the network. The physical interface to be used is picked from among those listed in interface sub elements of the forward element. The current specification for

[libvirt] [PATCH v2 3/4] Adding the element pf to network xml.

2011-12-12 Thread Shradha Shah
This element will help the user to just specify the SR-IOV physical function in order to access all the Virtual functions attached to it. --- docs/schemas/network.rng |7 src/conf/network_conf.c | 69 +++-- src/conf/network_conf.h |3 ++ 3

[libvirt] [PATCH v2 4/4] Functionality to implicitly get interface pool from SR-IOV PF.

2011-12-12 Thread Shradha Shah
If a system has 64 or more VF's, it is quite tedious to mention each VF in the interface pool. The following modification will implicitly create an interface pool from the SR-IOV PF. --- src/network/bridge_driver.c | 93 +- 1 files changed, 73

[libvirt] [PATCH v2 2/4] Added Function virNetDevGetVirtualFunctions

2011-12-12 Thread Shradha Shah
This functions enables us to get the Virtual Functions attached to a Physical function given the name of a SR-IOV physical functio. In order to accomplish the task, added a getter function pciGetDeviceAddrString to get the BDF of the Virtual Function in a char array. --- src/util/pci.c |

[libvirt] [PATCH v2 1/4] Added function pciSysfsFile to enable access to the PCI SYSFS files.

2011-12-12 Thread Shradha Shah
--- src/util/pci.c | 16 src/util/pci.h |2 ++ 2 files changed, 18 insertions(+), 0 deletions(-) diff --git a/src/util/pci.c b/src/util/pci.c index cd82b43..d0ba4c5 100644 --- a/src/util/pci.c +++ b/src/util/pci.c @@ -2027,6 +2027,22 @@ out: } /* + * Returns a path to

Re: [libvirt] [PATCH] storage: Activate/deactivate logical volumes only on local node

2011-12-12 Thread Osier Yang
On 2011年12月12日 17:54, Michal Privoznik wrote: On 12.12.2011 08:40, Osier Yang wrote: From: Rommerrom...@active.by Current -ay | -an has problems on pool starting/refreshing if the volumes are clustered. Rommer has posted a patch to list 2 months ago.

Re: [libvirt] virtio console: name=foo is not supported

2011-12-12 Thread Daniel P. Berrange
On Mon, Dec 12, 2011 at 10:18:23AM +0900, MATSUDA, Daiki wrote: In addition, the QEMU guest agent requires specified socket. Virt IO Console, too. But unfortunately libvirt does not support to xml tags to give socket name options to QEMU for the QEMU guest agent and Virt IO Console.

Re: [libvirt] [PATCH] storage: Activate/deactivate logical volumes only on local node

2011-12-12 Thread Osier Yang
Hi, Rommer, I add you in AUTHORs as below Rommer rom...@active.by Please let me known if you have other prefered name. Thanks Osier On 2011年12月12日 21:53, Osier Yang wrote: On 2011年12月12日 17:54, Michal Privoznik wrote: On 12.12.2011 08:40, Osier Yang wrote: From: Rommerrom...@active.by

[libvirt] [PATCH] virsh: Free returned MIME type string

2011-12-12 Thread Michal Privoznik
In terms of documentation to virDomainScreenshot, caller MUST free returned value. But virsh was not. --- Pushing under trivial rule. tools/virsh.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/tools/virsh.c b/tools/virsh.c index d859b3a..d58b827 100644 ---

[libvirt] [PATCH] qemu: Disable EOF processing during qemuDomainDestroy

2011-12-12 Thread Jiri Denemark
When destroying a domain qemuDomainDestroy kills its qemu process and starts a new job, which means it unlocks the domain object and locks it again after some time. Although the object is usually unlocked for a pretty short time, chances are another thread processing an EOF event on qemu monitor

Re: [libvirt] Expand GVirConfigDomainInterface a bit

2011-12-12 Thread Daniel P. Berrange
On Wed, Dec 07, 2011 at 11:02:32AM +0100, Christophe Fergeau wrote: Hi, This patch series add setters for the GVirConfigDomainInterface properties which are common to all classes deriving from GVirConfigDomainInterface. It also adds GVirConfigDomainInterfaceUser so that it's possible to

Re: [libvirt] [libvirt-glib 2/2] Add missing items in GVirConfigDomainVirtType

2011-12-12 Thread Daniel P. Berrange
On Wed, Dec 07, 2011 at 12:14:28PM +0100, Christophe Fergeau wrote: Put all the items available in virDomainVirt --- libvirt-gconfig/libvirt-gconfig-domain.h | 14 -- 1 files changed, 12 insertions(+), 2 deletions(-) diff --git a/libvirt-gconfig/libvirt-gconfig-domain.h

Re: [libvirt] [libvirt-glib 1/2] Fix gvir_config_domain_disk_set_target_bus API

2011-12-12 Thread Daniel P. Berrange
On Wed, Dec 07, 2011 at 12:14:27PM +0100, Christophe Fergeau wrote: The bus type is actually an enum, let's reflect that in libvirt-gconfig API --- libvirt-gconfig/libvirt-gconfig-domain-disk.c |8 ++-- libvirt-gconfig/libvirt-gconfig-domain-disk.h | 13 -

[libvirt] [PATCH libvirt-glib 2/2] Add binding for virDomainOpenGraphics

2011-12-12 Thread Daniel P. Berrange
From: Daniel P. Berrange berra...@redhat.com --- libvirt-gobject/libvirt-gobject-domain.c | 41 ++ libvirt-gobject/libvirt-gobject-domain.h |6 2 files changed, 47 insertions(+), 0 deletions(-) diff --git a/libvirt-gobject/libvirt-gobject-domain.c

[libvirt] [PATCH libvirt-glib 1/2] Add binding for virDomainOpenConsole

2011-12-12 Thread Daniel P. Berrange
From: Daniel P. Berrange berra...@redhat.com --- libvirt-gobject/libvirt-gobject-domain.c | 48 ++ libvirt-gobject/libvirt-gobject-domain.h |6 libvirt-gobject/libvirt-gobject.sym |1 + 3 files changed, 55 insertions(+), 0 deletions(-) diff --git

Re: [libvirt] [PATCH] qemu: Disable EOF processing during qemuDomainDestroy

2011-12-12 Thread Eric Blake
On 12/12/2011 08:31 AM, Jiri Denemark wrote: When destroying a domain qemuDomainDestroy kills its qemu process and starts a new job, which means it unlocks the domain object and locks it again after some time. Although the object is usually unlocked for a pretty short time, chances are another

Re: [libvirt] [PATCH] qemu: Disable EOF processing during qemuDomainDestroy

2011-12-12 Thread Jiri Denemark
On Mon, Dec 12, 2011 at 09:12:13 -0700, Eric Blake wrote: On 12/12/2011 08:31 AM, Jiri Denemark wrote: When destroying a domain qemuDomainDestroy kills its qemu process and starts a new job, which means it unlocks the domain object and locks it again after some time. Although the object is

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

2011-12-12 Thread Michal Privoznik
On 09.12.2011 19:08, Eric Blake wrote: 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

Re: [libvirt] Virtio Ballon driver problem

2011-12-12 Thread Eric Blake
On 12/11/2011 08:42 PM, Pankaj Rawat wrote: Well the process which get killed is the process running on guest trying to use memory , It is a simple c program having malloc function to do the memory requirement thing(Tested and working) The memory given to the guest is 400MB which is less then

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

2011-12-12 Thread Eric Blake
On 12/12/2011 09:49 AM, Michal Privoznik wrote: On 09.12.2011 19:08, Eric Blake wrote: make uninstall failed: rmdir: failed to remove `/etc/sasl2/': Directory not empty Well, this part of the commit message is no longer valid, so I tweaked it a bit... Fix this by following the conventions

Re: [libvirt] [PATCH] storage: Fix a potential crash when creating vol object

2011-12-12 Thread Eric Blake
On 12/12/2011 12:26 AM, Osier Yang wrote: If the vol object is newly created, it increases the volumes count, but doesn't decrease the volumes count when do cleanup. It can cause libvirtd crash when one trying to free the volume objects s/crash/to crash/ like: for (i = 0; i

Re: [libvirt] Release of libvirt-0.9.8

2011-12-12 Thread Jason Helfman
On Thu, Dec 08, 2011 at 05:46:17PM +0800, Daniel Veillard thus spake: One month exactly after the previous release, 0.9.8 is available at ftp://libvirt.org/libvirt/ This release is a balanced mix of new features, improvements and various bug fixes, and with a higher than usual amount of

Re: [libvirt] Release of libvirt-0.9.8

2011-12-12 Thread Eric Blake
On 12/12/2011 11:08 AM, Jason Helfman wrote: On Thu, Dec 08, 2011 at 05:46:17PM +0800, Daniel Veillard thus spake: One month exactly after the previous release, 0.9.8 is available at ftp://libvirt.org/libvirt/ This release is a balanced mix of new features, improvements and various bug

Re: [libvirt] virsh console giving Error

2011-12-12 Thread Eric Blake
On 12/12/2011 01:03 AM, Kashyap Chamarthy wrote: On 12/12/2011 01:25 PM, Peter Krempa wrote: Dňa 12.12.2011 8:05, Amit Tewari wrote / napísal(a): Hi, I am using libvirt 0.9.7 This is a known regression in 0.9.7. It has been fixed by two commits upstream and is now included in 0.9.8

[libvirt] [PATCH] docs: tweak 'virsh edit' wording

2011-12-12 Thread Eric Blake
I was wondering why 'virsh edit' didn't support the same '--inactive' option as 'virsh dumpxml'; reading the source code showed that --inactive was already implied, and that the only way to alter a running guest rather than affecting next boot is by hot-plugging individual devices, or by something

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

2011-12-12 Thread Michael Ellerman
On Mon, 2011-12-12 at 12:04 -0700, Eric Blake wrote: On 12/09/2011 11:58 PM, Laine Stump wrote: On 12/09/2011 06:35 PM, Eric Blake wrote: I would say something like: On PowerPC pseries guests, devices can be assigned to the SPAPR VIO bus. It has a flat 64-bit address space, by

Re: [libvirt] macvtap support in libvirt 0.9.8

2011-12-12 Thread Stefan Berger
On 12/12/2011 05:20 AM, Amit Tewari wrote: Hi all, Please ignore my previous mail.. This is my test scenario My test environment Host os=rhel6.1 Guest os = rhel6.1 Libvirt=0.9.8 Kvm hypervisor I am creating a macvtap interface on host machine at eth0 interface #ip link show macvtap0

[libvirt] [PATCH 4/6] tests: Add fake PPC64 emulator for QEMU testing

2011-12-12 Thread Michael Ellerman
Create a fake PPC64 QEMU so that we can run PPC64 QEMU tests when we don't have a real version of the emulator available. Signed-off-by: Michael Ellerman mich...@ellerman.id.au --- tests/testutilsqemu.c | 30 ++ 1 files changed, 30 insertions(+), 0 deletions(-)

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

2011-12-12 Thread Michael Ellerman
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. Signed-off-by: Michael Ellerman mich...@ellerman.id.au --- src/qemu/qemu_command.c | 89

[libvirt] [PATCH 1/6] Add address type for SPAPR VIO devices

2011-12-12 Thread Michael Ellerman
For QEMU PPC64 we have a machine type (pseries) which has a virtual bus called spapr-vio. We need to be able to create devices on this bus, and as such need a way to specify the address for those devices. This patch adds a new address type spapr-vio, which achieves this. The addressing is

[libvirt] [PATCH 6/6] tests: Add qemuxml2argv tests for PPC64 pseries machine

2011-12-12 Thread Michael Ellerman
Add four tests of the XML - argv handling for the PPC64 pseries machine. The first is just a basic test of a bare bones machine. The three others test various aspects of the spapr-vio address handling. It seems that currently we can't include network devices, doing so leads to a segfault

[libvirt] [PATCH 2/6] Add New address type spapr-vio to domain.rng

2011-12-12 Thread Bharata B Rao
Original patch by Bharata. Updated to use {1,16} in spaprvioReg based on example from Eric Blake. Signed-off-by: Bharata B Rao bhar...@linux.vnet.ibm.com Signed-off-by: Prerna Saxena pre...@linux.vnet.ibm.com Signed-off-by: Michael Ellerman mich...@ellerman.id.au ---

[libvirt] [PATCH 5/6] tests: Teach qemuxml2argvtest about spapr-vio addresses

2011-12-12 Thread Michael Ellerman
We can't call qemuCapsExtractVersionInfo() from test code, because it expects to be able to call the emulator, and for testing we have fake emulators that can't be executed. For that reason qemuxml2argvtest.c doesn't call qemuDomainAssignPCIAddresses(), instead it open codes its own version. That

Re: [libvirt] Virtio Ballon driver problem

2011-12-12 Thread John Magee
On Mon, Dec 12, 2011 at 11:53 AM, Eric Blake ebl...@redhat.com wrote: On 12/11/2011 08:42 PM, Pankaj Rawat wrote: Well the process which get killed is the process running on guest trying to use memory , It is a simple c program having malloc function to do the memory requirement

Re: [libvirt] [PATCH] storage: Fix a potential crash when creating vol object

2011-12-12 Thread Osier Yang
On 2011年12月13日 01:37, Eric Blake wrote: On 12/12/2011 12:26 AM, Osier Yang wrote: If the vol object is newly created, it increases the volumes count, but doesn't decrease the volumes count when do cleanup. It can cause libvirtd crash when one trying to free the volume objects s/crash/to

[libvirt] [test-API][PATCH 2/2] Add test case set_cpu_shares.py for setting cpu scheduler info

2011-12-12 Thread Nan Zhang
* repos/domain/set_cpu_shares.py: set the value of cpu_shares property of the guest. --- repos/domain/set_cpu_shares.py | 121 1 files changed, 121 insertions(+), 0 deletions(-) create mode 100644 repos/domain/set_cpu_shares.py diff --git

[libvirt] [test-API][PATCH 1/2] Add test case get_cpu_shares.py for cpu scheduler info testing

2011-12-12 Thread Nan Zhang
* repos/domain/get_cpu_shares.py: get the value of cpu_shares property of the guest. --- lib/domainAPI.py |2 +- repos/domain/get_cpu_shares.py | 117 2 files changed, 118 insertions(+), 1 deletions(-) create mode 100644

Re: [libvirt] virtio console: name=foo is not supported

2011-12-12 Thread MATSUDA, Daiki
(2011/12/12 22:58), Daniel P. Berrange wrote: On Mon, Dec 12, 2011 at 10:18:23AM +0900, MATSUDA, Daiki wrote: In addition, the QEMU guest agent requires specified socket. Virt IO Console, too. But unfortunately libvirt does not support to xml tags to give socket name options to QEMU for the

[libvirt] [PATCH] virsh: support multifunction in attach-disk

2011-12-12 Thread KAMEZAWA Hiroyuki
From: KAMEZAWA Hiroyuki kamezawa.hir...@jp.fujitsu.com PCI address... can be specified by attach-disk but multifunction cannot be specified. add --multifunction support. --- tools/virsh.c |7 ++- tools/virsh.pod |3 ++- 2 files changed, 8 insertions(+), 2 deletions(-) diff --git

[libvirt] [PATCH] tests: plug memory leak on linuxTestNodeInfo

2011-12-12 Thread ajia
From: Alex Jia a...@redhat.com Detected by valgrind. Leak introduced in commit 82ff25e. * tests/nodeinfotest.c: avoid memory leak on nodeinfo test case. * how to reproduce? % cd tests valgrind -v --leak-check=full ./nodeinfotest * actual valgrind result: ==22147== 65 bytes in 1 blocks are

Re: [libvirt] [PATCH] virsh: support multifunction in attach-disk

2011-12-12 Thread Wen Congyang
At 12/13/2011 02:12 PM, KAMEZAWA Hiroyuki Write: From: KAMEZAWA Hiroyuki kamezawa.hir...@jp.fujitsu.com PCI address... can be specified by attach-disk but multifunction cannot be specified. add --multifunction support. IIRC, libvirt does not support hot plugging multifunction PCI device. Why

[libvirt] macvtap not working on kvm

2011-12-12 Thread Amit Tewari
Hi all, My test environment Host os=rhel6.1 Guest os = rhel6.1 Libvirt=0.9.8 Kvm hypervisor I have made this entry in guest xml file interface type='direct' source dev='eth0' mode='bridge'/ model type='virtio'/ address type='pci' domain='0x' bus='0x00'

[libvirt] macvtap not working on kvm

2011-12-12 Thread Amit Tewari
Hi all, My test environment Host os=rhel6.1 Guest os = rhel6.1 Libvirt=0.9.8 Kvm hypervisor I have made this entry in guest xml file interface type='direct' source dev='eth0' mode='bridge'/ model type='virtio'/ address type='pci' domain='0x' bus='0x00'

Re: [libvirt] [PATCH] virsh: support multifunction in attach-disk

2011-12-12 Thread KAMEZAWA Hiroyuki
On Tue, 13 Dec 2011 14:51:58 +0800 Wen Congyang we...@cn.fujitsu.com wrote: At 12/13/2011 02:12 PM, KAMEZAWA Hiroyuki Write: From: KAMEZAWA Hiroyuki kamezawa.hir...@jp.fujitsu.com PCI address... can be specified by attach-disk but multifunction cannot be specified. add --multifunction