Re: [libvirt] [PATCH 1/4] Make drive unit attribute optional in the XML schema

2011-12-21 Thread Michael Ellerman
On Wed, 2011-12-21 at 17:27 +1100, Michael Ellerman wrote: The unit attribute of a drive address is optional in the code, so should also be in the XML schema. Sorry for all the confusion with testing the previous version of this series. With the first three patches here I am seeing only one

[libvirt] [libvirt-glib 1/2] Add python example for LibvirtGConfig

2011-12-21 Thread Christophe Fergeau
--- examples/config-test.py | 96 +++ 1 files changed, 96 insertions(+), 0 deletions(-) create mode 100644 examples/config-test.py diff --git a/examples/config-test.py b/examples/config-test.py new file mode 100644 index 000..92e8a89 ---

[libvirt] [libvirt-glib 2/2] Add GVirConfigDomainSound

2011-12-21 Thread Christophe Fergeau
--- libvirt-gconfig/Makefile.am|2 + libvirt-gconfig/libvirt-gconfig-domain-sound.c | 86 libvirt-gconfig/libvirt-gconfig-domain-sound.h | 77 + libvirt-gconfig/libvirt-gconfig.h |1 +

Re: [libvirt] [PATCH 0/4] RFC: grant KVM guests retain arbitrary capabilities

2011-12-21 Thread Taku Izumi
Thank you for your comment. We could do with a feature like this for LXC too. Though I'd prefer the XML to be a little more concise. Perhaps process cap_sys_rawio/ /process One potential concern is that the capability names are OS specific, so perhaps rather than allow

Re: [libvirt] [PATCH 2/2] Do not generate security_model when fs driver is anything but 'path'

2011-12-21 Thread Daniel P. Berrange
On Wed, Dec 21, 2011 at 11:17:17AM +0530, Deepak C Shetty wrote: QEMU does not support security_model for anything but 'path' fs driver type. Currently in libvirt, when security_model ( accessmode attribute) is not specified it auto-generates it irrespective of the fs driver type. Also when

Re: [libvirt] [libvirt-glib 2/2] Add GVirConfigDomainSound

2011-12-21 Thread Daniel P. Berrange
On Wed, Dec 21, 2011 at 10:59:11AM +0100, Christophe Fergeau wrote: --- libvirt-gconfig/Makefile.am|2 + libvirt-gconfig/libvirt-gconfig-domain-sound.c | 86 libvirt-gconfig/libvirt-gconfig-domain-sound.h | 77 +

Re: [libvirt] [libvirt-glib 1/2] Add python example for LibvirtGConfig

2011-12-21 Thread Daniel P. Berrange
On Wed, Dec 21, 2011 at 10:59:10AM +0100, Christophe Fergeau wrote: --- examples/config-test.py | 96 +++ 1 files changed, 96 insertions(+), 0 deletions(-) create mode 100644 examples/config-test.py ACK, but lets call it config-demo.py since

[libvirt] [PATCH] nwfilter: Do not require DHCP requests to be broadcast

2011-12-21 Thread Stefan Berger
Remove the requirement that DHCP messages have to be broadcasted. --- src/nwfilter/nwfilter_ebiptables_driver.c |3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) Index: libvirt-acl/src/nwfilter/nwfilter_ebiptables_driver.c

Re: [libvirt] [PATCH] nwfilter: Do not require DHCP requests to be broadcast

2011-12-21 Thread Eric Blake
On 12/21/2011 08:19 AM, Stefan Berger wrote: Remove the requirement that DHCP messages have to be broadcasted. It might be worth listing more rationale, such as a scenario that fails without this patch. --- src/nwfilter/nwfilter_ebiptables_driver.c |3 +-- 1 file changed, 1

Re: [libvirt] [PATCH 0/4] RFC: grant KVM guests retain arbitrary capabilities

2011-12-21 Thread Daniel P. Berrange
On Wed, Dec 21, 2011 at 07:19:52PM +0900, Taku Izumi wrote: Thank you for your comment. We could do with a feature like this for LXC too. Though I'd prefer the XML to be a little more concise. Perhaps process cap_sys_rawio/ /process One potential concern is

Re: [libvirt] [PATCH] nwfilter: Do not require DHCP requests to be broadcast

2011-12-21 Thread Stefan Berger
On 12/21/2011 10:29 AM, Eric Blake wrote: On 12/21/2011 08:19 AM, Stefan Berger wrote: Remove the requirement that DHCP messages have to be broadcasted. It might be worth listing more rationale, such as a scenario that fails without this patch. DHCP requests are most often sent via broadcast

[libvirt] [PATCH] qemu: Support readonly filesystem passthrough

2011-12-21 Thread Osier Yang
Upstream QEMU starts to support it from commit 2c74c2cb. --- docs/formatdomain.html.in|5 +++-- src/qemu/qemu_capabilities.c |7 ++- src/qemu/qemu_capabilities.h |1 + src/qemu/qemu_command.c | 11 +++ 4 files changed, 21 insertions(+), 3 deletions(-) diff

Re: [libvirt] [PATCH] qemu: Support readonly filesystem passthrough

2011-12-21 Thread Daniel P. Berrange
On Thu, Dec 22, 2011 at 12:26:06AM +0800, Osier Yang wrote: Upstream QEMU starts to support it from commit 2c74c2cb. --- docs/formatdomain.html.in|5 +++-- src/qemu/qemu_capabilities.c |7 ++- src/qemu/qemu_capabilities.h |1 + src/qemu/qemu_command.c | 11

Re: [libvirt] [PATCH 1/4] virsh: Move job watch code to a separate function

2011-12-21 Thread Eric Blake
On 12/20/2011 08:21 AM, Michal Privoznik wrote: called do_watch_job. This can be later used in other job oriented commands like dump, save, managedsave to report progress and allow user to cancel via ^C. --- tools/virsh.c | 187 ++-- 1

Re: [libvirt] [PATCH 3/4] virsh: Use do_job_watch in cmdSave

2011-12-21 Thread Eric Blake
On 12/20/2011 08:21 AM, Michal Privoznik wrote: This patch alters saving code, so we can report progress and allow cancel via ^C. --- tools/virsh.c | 86 ++- tools/virsh.pod |8 ++-- 2 files changed, 76 insertions(+), 18

Re: [libvirt] [PATCH 4/4] virsh: Use do_job_watch in cmdManagedSave

2011-12-21 Thread Eric Blake
On 12/20/2011 08:21 AM, Michal Privoznik wrote: This patch alters saving code, so we can report progress and allow cancel via ^C. --- tools/virsh.c | 73 --- tools/virsh.pod |8 +++--- 2 files changed, 68 insertions(+), 13

Re: [libvirt] [libvirt-glib] Correct namespace prefix for GVirConfig symbols

2011-12-21 Thread Zeeshan Ali (Khattak)
On Thu, Dec 22, 2011 at 1:43 AM, Zeeshan Ali (Khattak) zeesha...@gnome.org wrote: From: Zeeshan Ali (Khattak) zeesha...@gnome.org Breaks API and ABI on the fundamental level but lets fix this now while we don't guarantee any API/ABI stability. Forgot to mention that this patch is on top of

Re: [libvirt] [libvirt-glib] Correct namespace prefix for GVirConfig symbols

2011-12-21 Thread Zeeshan Ali (Khattak)
On Thu, Dec 22, 2011 at 1:46 AM, Zeeshan Ali (Khattak) zeesha...@gnome.org wrote: On Thu, Dec 22, 2011 at 1:43 AM, Zeeshan Ali (Khattak) zeesha...@gnome.org wrote: From: Zeeshan Ali (Khattak) zeesha...@gnome.org Breaks API and ABI on the fundamental level but lets fix this now while we don't

[libvirt] question on kvm

2011-12-21 Thread hero
on redhat5.6 and redhat 6.1, I can got the MAC address of the kvm VMs,but I don't know how to get the ip address of the VMs,and virsh has not got a command,can you help me to resolve it.-- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list

Re: [libvirt] question on kvm

2011-12-21 Thread Alex Jia
As usual, you can try the following methods: 1. log in the guest then ifconfig I guess it's not what you want 2. filter /var/lib/dnsmasq/dnsmasq.leases file according to actual guest mac address 3. using nmap to scan guest subnet paste the following shell codes into mac2ip.sh: #! /bin/bash

Re: [libvirt] [PATCH] qemu: Support readonly filesystem passthrough

2011-12-21 Thread Osier Yang
On 2011年12月22日 00:03, Daniel P. Berrange wrote: On Thu, Dec 22, 2011 at 12:26:06AM +0800, Osier Yang wrote: Upstream QEMU starts to support it from commit 2c74c2cb. --- docs/formatdomain.html.in|5 +++-- src/qemu/qemu_capabilities.c |7 ++- src/qemu/qemu_capabilities.h |

Re: [libvirt] [PATCH 2/2] Do not generate security_model when fs driver is anything but 'path'

2011-12-21 Thread Deepak C Shetty
On 12/21/2011 08:11 PM, Daniel P. Berrange wrote: On Wed, Dec 21, 2011 at 11:17:17AM +0530, Deepak C Shetty wrote: QEMU does not support security_model for anything but 'path' fs driver type. Currently in libvirt, when security_model ( accessmode attribute) is not specified it auto-generates it

[libvirt] question of kvm

2011-12-21 Thread hero
I want to back up my VM including my own data, but the VM must be 'PASUED' or 'SHUT OFF' if I use 'virt-clone', I want to know how to back up my VM when it's running like make a ghost of windows. Thansk! SunYongGang-- libvir-list mailing list libvir-list@redhat.com

[libvirt] [PATCH v2 0/5] RFC: grant KVM guests retain arbitrary capabilities

2011-12-21 Thread Taku Izumi
Hi Daniel-san and all, This patchset adds an option for KVM guests to retain arbitrary capabilities. The first version is here: http://www.redhat.com/archives/libvir-list/2011-December/msg00857.html According to Daniel-san's comment, I changed my patch like the following: v1 - v2 -

[libvirt] [PATCH] Add new attribute writeout to filesystem element

2011-12-21 Thread Deepak C Shetty
This introduces new attribute writeout with only supported value as immediate. This will be an optional attribute with no defaults. This helps specify whether to skip the host page cache. When writeout is specified, meaning when writeout=immediate a writeback is explicitly initiated for the dirty

[libvirt] [PATCH v2 1/5] conf: add XML schema for capability XML

2011-12-21 Thread Taku Izumi
This patch introduces XML schema for capability XML. process and cap element are added. The list of cap elements represents process capabilities host supports. capabilities host ... process cap name='chown'/ cap name='dac_override'/ ... /process /host ...

[libvirt] [PATCH v2 2/5] conf: add XML schema for domain XML

2011-12-21 Thread Taku Izumi
This patch introduces XML schema for domains to retain arbitrary capabilities. For example, by adding the following XML to domain configuration, its domain can retain cap_sys_rawio capability. process cap name='sys_rawio'/ /process Signed-off-by: Taku Izumi izumi.t...@jp.fujitsu.com

[libvirt] [PATCH v2 3/5] util: add functions to keep capabilities

2011-12-21 Thread Taku Izumi
This patch introduces virKeepCapabilities() function and implements virCommandAllowCap() function. Existing virClearCapabilities() is function to clear all capabilities. Instead virKeepCapabilities() is function to keep arbitrary capabilities. Signed-off-by: Taku Izumi

[libvirt] [PATCH v2 4/5] util: extend virExecWithHook()

2011-12-21 Thread Taku Izumi
This patch extends virExecWithHook() to receive capability information. Signed-off-by: Taku Izumi izumi.t...@jp.fujitsu.com Signed-off-by: Shota Hirae m11g1...@hibikino.ne.jp --- src/util/command.c | 16 ++-- 1 file changed, 10 insertions(+), 6 deletions(-) Index:

[libvirt] [PATCH v2 5/5] qemu: make qemu processes to retain capabilities

2011-12-21 Thread Taku Izumi
This patch revises qemuProcessStart() function for qemu processes to retain arbitrary capabilities. Signed-off-by: Taku Izumi izumi.t...@jp.fujitsu.com Signed-off-by: Shota Hirae m11g1...@hibikino.ne.jp --- src/qemu/qemu_process.c |1 + 1 file changed, 1 insertion(+) Index:

Re: [libvirt] [PATCH] Add new attribute writeout to filesystem element

2011-12-21 Thread Osier Yang
On 2011年12月22日 14:54, Deepak C Shetty wrote: This introduces new attribute writeout with only supported value as immediate. This will be an optional attribute with no defaults. This helps specify whether to skip the host page cache. When writeout is specified, meaning when writeout=immediate a