Re: [libvirt] [PATCH v7 0/8] Add basic driver for Parallels Virtuozzo Server

2012-06-26 Thread Daniel Veillard
On Mon, Jun 25, 2012 at 02:52:15PM +0100, Daniel P. Berrange wrote: > On Mon, Jun 25, 2012 at 09:49:03PM +0800, Daniel Veillard wrote: > > On Mon, Jun 25, 2012 at 12:57:55PM +0400, Dmitry Guryanov wrote: > > > Parallels Cloud Server is a virtualization solution > > > that allows users to simultaneo

Re: [libvirt] [PATCH v2] storage backend: Add sheepdog support

2012-06-26 Thread Sebastian Wiedenroth
Am 20.06.2012 um 01:59 schrieb Eric Blake: >> To work right this patch parses the output of collie, >> so it relies on the raw output option. There recently was a bug which caused >> size information to be reported wrong. This is fixed upstream already and >> will be in the next release. > > Sho

[libvirt] [PATCH libvirt-java 1/2] Return a byte[] array with secretGetValue

2012-06-26 Thread Wido den Hollander
We break the API with this, but Java does not support multiple method signatures with different return types. The old method returned a String, but since a secret can be binary data this type is not suited. Users who now that their secret is in fact a String, can use cast with: Secret secret =

Re: [libvirt] [PATCH libvirt-java 1/2] Return a byte[] array with secretGetValue

2012-06-26 Thread Wido den Hollander
On 06/26/2012 10:07 AM, Wido den Hollander wrote: We break the API with this, but Java does not support multiple method signatures with different return types. The old method returned a String, but since a secret can be binary data this type is not suited. Users who now that their secret is i

[libvirt] [PATCH libvirt-java 2/2] Add flags to StoragePoolRefresh

2012-06-26 Thread Wido den Hollander
Signed-off-by: Wido den Hollander --- src/main/java/org/libvirt/StoragePool.java |2 +- src/main/java/org/libvirt/jna/Libvirt.java |2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/main/java/org/libvirt/StoragePool.java b/src/main/java/org/libvirt/StoragePool.jav

Re: [libvirt] Stored secrets seem to get corrupted

2012-06-26 Thread Wido den Hollander
On 06/25/2012 04:54 PM, Daniel P. Berrange wrote: On Mon, Jun 25, 2012 at 04:37:48PM +0200, Wido den Hollander wrote: Hi, On one of my systems I'm having troubles with my RBD storage backend. At first I thought it was a problem with my code, but after trying the same code on a second machine I

[libvirt] [PATCH] Don't install systemd service files executable

2012-06-26 Thread Guido Günther
since they aren't. Detected by Debian's lintian. --- daemon/Makefile.am |2 +- tools/Makefile.am |2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/daemon/Makefile.am b/daemon/Makefile.am index fbb0ae1..473fa98 100644 --- a/daemon/Makefile.am +++ b/daemon/Makefile.am @@

Re: [libvirt] [PATCH v2 2/5] virsh: Expose virDomainGetInterfacesAddresses

2012-06-26 Thread Peter Krempa
On 06/21/12 15:55, Michal Privoznik wrote: --- tools/virsh.c | 41 + tools/virsh.pod |9 + 2 files changed, 50 insertions(+), 0 deletions(-) /* "domifstat" command */ static const vshCmdInfo info_domifstat[] = { @@ -17841,6 +1788

Re: [libvirt] [PATCH] Don't install systemd service files executable

2012-06-26 Thread Peter Krempa
On 06/26/12 10:26, Guido Günther wrote: since they aren't. Detected by Debian's lintian. --- daemon/Makefile.am |2 +- tools/Makefile.am |2 +- 2 files changed, 2 insertions(+), 2 deletions(-) ACK Peter -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/ma

Re: [libvirt] [PATCH v4 0/7] file descriptor passing using pass-fd

2012-06-26 Thread Daniel P. Berrange
On Fri, Jun 22, 2012 at 02:36:07PM -0400, Corey Bryant wrote: > libvirt's sVirt security driver provides SELinux MAC isolation for > Qemu guest processes and their corresponding image files. In other > words, sVirt uses SELinux to prevent a QEMU process from opening > files that do not belong to i

Re: [libvirt] [libvirt-glib v2 1/8] Setters/adders for GVirConfigCapabilitiesCpu props

2012-06-26 Thread Christophe Fergeau
On Tue, Jun 26, 2012 at 05:56:13AM +0300, Zeeshan Ali (Khattak) wrote: > From: "Zeeshan Ali (Khattak)" > > - gvir_config_capabilities_cpu_add_feature > - gvir_config_capabilities_cpu_set_topology > --- > libvirt-gconfig/libvirt-gconfig-capabilities-cpu.c | 32 > > libvirt

Re: [libvirt] [libvirt-glib v2 2/8] Add gvir_config_capabilities_cpu_topology_new*

2012-06-26 Thread Christophe Fergeau
ACK On Tue, Jun 26, 2012 at 05:56:14AM +0300, Zeeshan Ali (Khattak) wrote: > From: "Zeeshan Ali (Khattak)" > > Provide constructor methods for GVirConfigCapabilitiesCpuTopology. > --- > .../libvirt-gconfig-capabilities-cpu-topology.c| 25 > > .../libvirt-gconfig-capa

Re: [libvirt] [libvirt-glib v2 3/8] Setters for CapabilitiesCpuTopology props

2012-06-26 Thread Christophe Fergeau
On Tue, Jun 26, 2012 at 05:56:15AM +0300, Zeeshan Ali (Khattak) wrote: > From: "Zeeshan Ali (Khattak)" > > --- > .../libvirt-gconfig-capabilities-cpu-topology.c| 27 > > .../libvirt-gconfig-capabilities-cpu-topology.h|9 +++ > libvirt-gconfig/libvirt-gconf

Re: [libvirt] [libvirt-glib v2 4/8] Add gvir_config_capabilities_cpu_feature_set_name()

2012-06-26 Thread Christophe Fergeau
ACK On Tue, Jun 26, 2012 at 05:56:16AM +0300, Zeeshan Ali (Khattak) wrote: > From: "Zeeshan Ali (Khattak)" > > Setter for CapabilitiesCpuFeature.name. > --- > libvirt-gconfig/libvirt-gconfig-capabilities-cpu-feature.c | 12 > > libvirt-gconfig/libvirt-gconfig-capabilities-cpu-fea

Re: [libvirt] [libvirt-glib v2 5/8] Add GVirConfigDomainCpuFeature class

2012-06-26 Thread Christophe Fergeau
On Tue, Jun 26, 2012 at 05:56:17AM +0300, Zeeshan Ali (Khattak) wrote: > From: "Zeeshan Ali (Khattak)" > > API to handle 'domain/cpu/feature' nodes. > --- > libvirt-gconfig/Makefile.am|2 + > .../libvirt-gconfig-domain-cpu-feature.c | 100 > +++

Re: [libvirt] [PATCH] qemu_agent: support guest-info command

2012-06-26 Thread Daniel P. Berrange
On Tue, Jun 26, 2012 at 12:42:03PM +0900, MATSUDA, Daiki wrote: > Currently, libvirt qemu agent supports some QEMU Guest Agent's commands > or use them. But they are not adapted to communication test to the > Domain OS. > > So, QEMU Guest Agent provide 'guest-info' command to display its version >

Re: [libvirt] [PATCH v2 3/5] qemu_agent: Implement 'guest-network-get-interfaces' command handling

2012-06-26 Thread Peter Krempa
On 06/21/12 15:56, Michal Privoznik wrote: This command returns an array of all guest interfaces among with their IP and HW addresses. --- src/qemu/qemu_agent.c | 135 + src/qemu/qemu_agent.h |2 + 2 files changed, 137 insertions(+), 0 dele

Re: [libvirt] [PATCH v2 4/5] qemu: Implement virDomainInterfacesAddresses

2012-06-26 Thread Peter Krempa
On 06/21/12 15:56, Michal Privoznik wrote: > --- > src/qemu/qemu_driver.c | 68 > > 1 files changed, 68 insertions(+), 0 deletions(-) > > + > static virDriver qemuDriver = { > .no = VIR_DRV_QEMU, > .name = QEMU_DRIVER_NAME, > @@

Re: [libvirt] [PATCH v2 0/5] Introduce API for dumping domain IP addresses

2012-06-26 Thread Daniel P. Berrange
On Thu, Jun 21, 2012 at 03:55:57PM +0200, Michal Privoznik wrote: > This feature has been requested for a very long time. However, > we had to wait for guest agent to obtain reliable results as > user might create totally different structure of interfaces than > seen from outside (e.g. bonding, vir

[libvirt] [PATCHv2 3/9] Introduce the function virCgroupForHypervisor

2012-06-26 Thread Hu Tao
From: Wen Congyang Introduce the function virCgroupForHypervisor() to create sub directory for hypervisor thread(include I/O thread, vhost-net thread) --- src/libvirt_private.syms |1 + src/util/cgroup.c| 42 ++ src/util/cgroup.h|

[libvirt] [PATCHv2 0/9] support to set cpu bandwidth for hypervisor threads

2012-06-26 Thread Hu Tao
Currently, we only can set cpu bandwidth for vcpu. If the hypervisor threads consume too much cpu time, it may affect the vcpu. This patchset allows the user to control the cpu bandwidth for hypervisor threads. Changes: v2: - rename qemuSetupCgroupVcpuBW to qemuSetupCgroupCpuBandwidth - limi

[libvirt] [PATCHv2 2/9] limit cpu bandwidth only for vcpus

2012-06-26 Thread Hu Tao
This patch changes the behaviour of xml element cputune.period and cputune.quota to limit cpu bandwidth only for vcpus, and no longer limit cpu bandwidth for the whole guest. The reasons to do this are: - This matches docs of cputune.period and cputune.quota. - The other parts excepting vcpus

[libvirt] [PATCHv2 1/9] rename qemuSetupCgroupVcpuBW to qemuSetupCgroupCpuBandwidth

2012-06-26 Thread Hu Tao
This function sets cpu bandwidth, not only for vcpu. so rename it. --- src/qemu/qemu_cgroup.c |9 + src/qemu/qemu_cgroup.h |6 +++--- src/qemu/qemu_driver.c |6 +++--- 3 files changed, 11 insertions(+), 10 deletions(-) diff --git a/src/qemu/qemu_cgroup.c b/src/qemu/qemu_cgroup

[libvirt] [PATCHv2 4/9] introduce the function virCgroupMoveTask()

2012-06-26 Thread Hu Tao
From: Wen Congyang introduce a new API to move all tasks from a cgroup to another cgroup --- src/libvirt_private.syms |1 + src/util/cgroup.c| 54 ++ src/util/cgroup.h|2 + 3 files changed, 57 insertions(+), 0 deletions(-) di

[libvirt] [PATCHv2 9/9] update doc about hypervisor_period/hypervisor_quota

2012-06-26 Thread Hu Tao
--- tools/virsh.pod | 11 ++- 1 files changed, 6 insertions(+), 5 deletions(-) diff --git a/tools/virsh.pod b/tools/virsh.pod index f83a29d..8ae3177 100644 --- a/tools/virsh.pod +++ b/tools/virsh.pod @@ -1189,7 +1189,8 @@ available for each hypervisor are: LXC (posix scheduler) : cpu

[libvirt] [PATCHv2 8/9] qemu: Implement hypervisor_period and hypervisor_quota's modification

2012-06-26 Thread Hu Tao
From: Wen Congyang allow the user change/get hypervisor's period and quota when the vm is running. --- include/libvirt/libvirt.h.in | 16 + src/qemu/qemu_driver.c | 133 +- 2 files changed, 148 insertions(+), 1 deletions(-) diff --git a/inclu

[libvirt] [PATCHv2 7/9] qemu: Implement hypervisor's period and quota tunable XML configuration and parsing

2012-06-26 Thread Hu Tao
From: Wen Congyang set hypervisor's period and quota to limit cpu bandwidth when the vm starts. --- src/conf/domain_conf.c | 25 +++-- src/conf/domain_conf.h |2 ++ src/qemu/qemu_cgroup.c | 13 +++-- 3 files changed, 36 insertions(+), 4 deletions(-) diff --gi

[libvirt] [PATCHv2 5/9] create a new cgroup and move all hypervisor threads to the new cgroup

2012-06-26 Thread Hu Tao
From: Wen Congyang create a new cgroup and move all hypervisor threads to the new cgroup. And then we can limit cpu bandwidth for hypervisor threads (include vhost-net threads). --- src/qemu/qemu_cgroup.c | 57 +++ src/qemu/qemu_cgroup.h |2 +

[libvirt] [PATCHv2 6/9] Update XML Schema for new entries

2012-06-26 Thread Hu Tao
From: Wen Congyang --- docs/schemas/domaincommon.rng | 10 ++ 1 files changed, 10 insertions(+), 0 deletions(-) diff --git a/docs/schemas/domaincommon.rng b/docs/schemas/domaincommon.rng index 8630625..ee2a314 100644 --- a/docs/schemas/domaincommon.rng +++ b/docs/schemas/domaincommon.

Re: [libvirt] [libvirt-glib v2 7/8] Add GVirConfigDomainCpu class

2012-06-26 Thread Christophe Fergeau
On Tue, Jun 26, 2012 at 05:56:19AM +0300, Zeeshan Ali (Khattak) wrote: > From: "Zeeshan Ali (Khattak)" > > API to handle 'domain/cpu' nodes. > --- > libvirt-gconfig/Makefile.am |2 + > libvirt-gconfig/libvirt-gconfig-domain-cpu.c | 179 > ++ > libvi

Re: [libvirt] [libvirt-glib v2 6/8] Make GVirConfigCapabilitiesCpu.get_features virtual

2012-06-26 Thread Christophe Fergeau
ACK On Tue, Jun 26, 2012 at 05:56:18AM +0300, Zeeshan Ali (Khattak) wrote: > From: "Zeeshan Ali (Khattak)" > > --- > libvirt-gconfig/libvirt-gconfig-capabilities-cpu.c | 30 > +--- > libvirt-gconfig/libvirt-gconfig-capabilities-cpu.h |4 ++- > 2 files changed, 23 insertio

Re: [libvirt] [libvirt-glib v2 8/8] Test cases for new domain CPU APIs

2012-06-26 Thread Christophe Fergeau
ACK Christophe On Tue, Jun 26, 2012 at 05:56:20AM +0300, Zeeshan Ali (Khattak) wrote: > From: "Zeeshan Ali (Khattak)" > > --- > libvirt-gconfig/tests/test-domain-create.c | 55 > > 1 file changed, 55 insertions(+) > > diff --git a/libvirt-gconfig/tests/test-dom

[libvirt] [PATCH v3] storage backend: Add sheepdog support

2012-06-26 Thread Sebastian Wiedenroth
This patch brings support to manage sheepdog pools and volumes to libvirt. It uses the "collie" command-line utility that comes with sheepdog for that. A sheepdog pool in libvirt maps to a sheepdog cluster. It needs a host and port to connect to, which in most cases is just going to be the defaul

Re: [libvirt] [Qemu-devel] [PATCH v4 0/7] file descriptor passing using pass-fd

2012-06-26 Thread Kevin Wolf
Am 26.06.2012 11:10, schrieb Daniel P. Berrange: > I was thinking about some of the sources complexity when using > FD passing from libvirt and wanted to raise one idea for discussion > before we continue. > > With this proposed series, we have usage akin to: > > 1. pass_fd FDSET={M} -> returns

Re: [libvirt] [PATCH 2/2] nodeinfo: Fix CPU core counting

2012-06-26 Thread Peter Krempa
On 06/25/12 16:43, Peter Krempa wrote: Parsing of the processor information from /sys/ didn't take into account that cores on different physical processors may have different core ID's. This resulted in reporting more cores than actually were present. This patch changes the process of counting c

Re: [libvirt] [PATCH] Don't install systemd service files executable

2012-06-26 Thread Eric Blake
On 06/26/2012 02:26 AM, Guido Günther wrote: > since they aren't. Detected by Debian's lintian. > --- > daemon/Makefile.am |2 +- > tools/Makefile.am |2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/daemon/Makefile.am b/daemon/Makefile.am > index fbb0ae1..473fa9

Re: [libvirt] [PATCHv2] Don't install systemd service files executable

2012-06-26 Thread Daniel P. Berrange
On Tue, Jun 26, 2012 at 03:34:47PM +0200, Guido Günther wrote: > since they aren't. Detected by Debian's lintian. > --- > Changes since last version: use INSTALL_DATA instead of -m0644 > > daemon/Makefile.am |2 +- > tools/Makefile.am |2 +- > 2 files changed, 2 insertions(+), 2 deletion

[libvirt] [PATCHv2] Don't install systemd service files executable

2012-06-26 Thread Guido Günther
since they aren't. Detected by Debian's lintian. --- Changes since last version: use INSTALL_DATA instead of -m0644 daemon/Makefile.am |2 +- tools/Makefile.am |2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/daemon/Makefile.am b/daemon/Makefile.am index fbb0ae1..71e

Re: [libvirt] [PATCH 00/02] libvirt-java add domain flags and error constants up to 0.9, 2

2012-06-26 Thread Stefan Majer
Hi Daniel, thanks a lot, i will try my very best for the next patch submission. Greetings Stefan On Tue, Jun 26, 2012 at 5:30 AM, Daniel Veillard wrote: > On Tue, Jun 19, 2012 at 11:06:57AM +0200, Stefan Majer wrote: >> Hi, >> >> in order to bring libvirt-java closer to head i will start send a

Re: [libvirt] [Qemu-devel] [PATCH v4 0/7] file descriptor passing using pass-fd

2012-06-26 Thread Corey Bryant
On 06/26/2012 05:10 AM, Daniel P. Berrange wrote: On Fri, Jun 22, 2012 at 02:36:07PM -0400, Corey Bryant wrote: libvirt's sVirt security driver provides SELinux MAC isolation for Qemu guest processes and their corresponding image files. In other words, sVirt uses SELinux to prevent a QEMU pro

Re: [libvirt] [Qemu-devel] [PATCH v4 0/7] file descriptor passing using pass-fd

2012-06-26 Thread Daniel P. Berrange
On Tue, Jun 26, 2012 at 09:52:51AM -0400, Corey Bryant wrote: > >So now from a client's POV you'd have a flow like > > > >* drive_add "file=/dev/fd/N" FDSET={N} > > IIUC then drive_add would loop and pass each fd in the set via SCM_RIGHTS? Yes, you'd probably use the JSON to tell QEMU exactl

Re: [libvirt] [Qemu-devel] [PATCH v4 0/7] file descriptor passing using pass-fd

2012-06-26 Thread Daniel P. Berrange
On Tue, Jun 26, 2012 at 03:11:40PM +0100, Daniel P. Berrange wrote: > On Tue, Jun 26, 2012 at 09:52:51AM -0400, Corey Bryant wrote: > > >So now from a client's POV you'd have a flow like > > > > > >* drive_add "file=/dev/fd/N" FDSET={N} > > > > IIUC then drive_add would loop and pass each fd

Re: [libvirt] [libvirt-glib v2 1/8] Setters/adders for GVirConfigCapabilitiesCpu props

2012-06-26 Thread Zeeshan Ali (Khattak)
On Tue, Jun 26, 2012 at 12:16 PM, Christophe Fergeau wrote: > On Tue, Jun 26, 2012 at 05:56:13AM +0300, Zeeshan Ali (Khattak) wrote: >> From: "Zeeshan Ali (Khattak)" >> >> - gvir_config_capabilities_cpu_add_feature >> - gvir_config_capabilities_cpu_set_topology >> --- >>  libvirt-gconfig/libvirt-

Re: [libvirt] [libvirt-glib v2 1/8] Setters/adders for GVirConfigCapabilitiesCpu props

2012-06-26 Thread Christophe Fergeau
On Tue, Jun 26, 2012 at 06:05:41PM +0300, Zeeshan Ali (Khattak) wrote: > On Tue, Jun 26, 2012 at 12:16 PM, Christophe Fergeau > wrote: > > On Tue, Jun 26, 2012 at 05:56:13AM +0300, Zeeshan Ali (Khattak) wrote: > >> From: "Zeeshan Ali (Khattak)" > >> > >> - gvir_config_capabilities_cpu_add_feature

Re: [libvirt] [PATCH v7 0/8] Add basic driver for Parallels Virtuozzo Server

2012-06-26 Thread Dmitry Guryanov
On 06/26/2012 10:54 AM, Daniel Veillard wrote: On Mon, Jun 25, 2012 at 02:52:15PM +0100, Daniel P. Berrange wrote: On Mon, Jun 25, 2012 at 09:49:03PM +0800, Daniel Veillard wrote: On Mon, Jun 25, 2012 at 12:57:55PM +0400, Dmitry Guryanov wrote: Parallels Cloud Server is a virtualization soluti

Re: [libvirt] [PATCH v2] virsh: Add domdisplay cmd for VNC, SPICE and RDP

2012-06-26 Thread Doug Goldstein
On Sun, Jun 24, 2012 at 4:14 PM, Doug Goldstein wrote: > v2: > - Refactored to use virBuffer > - Refactored to use virXPath wrappers > - Added support for tls-port and password for SPICE > - Added optional flag to disable SPICE password to the URI > - Added support for RDP > - Fixed code reviews >

[libvirt] [PATCH] Add /tools/libvirt-guests.service to .gitignore

2012-06-26 Thread Guido Günther
since it's an autogenerated file --- .gitignore |1 + 1 file changed, 1 insertion(+) diff --git a/.gitignore b/.gitignore index cd978d2..e5d5db9 100644 --- a/.gitignore +++ b/.gitignore @@ -160,6 +160,7 @@ /tests/xmconfigtest /tools/*.[18] /tools/libvirt-guests.init +/tools/libvirt-guests.

Re: [libvirt] [PATCHv2] Don't install systemd service files executable

2012-06-26 Thread Guido Günther
On Tue, Jun 26, 2012 at 02:40:09PM +0100, Daniel P. Berrange wrote: > On Tue, Jun 26, 2012 at 03:34:47PM +0200, Guido Günther wrote: > > since they aren't. Detected by Debian's lintian. > > --- > > Changes since last version: use INSTALL_DATA instead of -m0644 > > > > daemon/Makefile.am |2 +-

Re: [libvirt] [Qemu-devel] [PATCH v4 0/7] file descriptor passing using pass-fd

2012-06-26 Thread Corey Bryant
On 06/26/2012 11:03 AM, Daniel P. Berrange wrote: On Tue, Jun 26, 2012 at 03:11:40PM +0100, Daniel P. Berrange wrote: On Tue, Jun 26, 2012 at 09:52:51AM -0400, Corey Bryant wrote: So now from a client's POV you'd have a flow like * drive_add "file=/dev/fd/N" FDSET={N} IIUC then drive_

Re: [libvirt] [PATCH] Add /tools/libvirt-guests.service to .gitignore

2012-06-26 Thread Daniel P. Berrange
On Tue, Jun 26, 2012 at 06:15:31PM +0200, Guido Günther wrote: > since it's an autogenerated file > --- > .gitignore |1 + > 1 file changed, 1 insertion(+) > > diff --git a/.gitignore b/.gitignore > index cd978d2..e5d5db9 100644 > --- a/.gitignore > +++ b/.gitignore > @@ -160,6 +160,7 @@ > /

Re: [libvirt] [Qemu-devel] [PATCH v4 0/7] file descriptor passing using pass-fd

2012-06-26 Thread Corey Bryant
On 06/26/2012 11:37 AM, Corey Bryant wrote: On 06/26/2012 11:03 AM, Daniel P. Berrange wrote: On Tue, Jun 26, 2012 at 03:11:40PM +0100, Daniel P. Berrange wrote: On Tue, Jun 26, 2012 at 09:52:51AM -0400, Corey Bryant wrote: So now from a client's POV you'd have a flow like * drive_add

Re: [libvirt] [PATCH] Add /tools/libvirt-guests.service to .gitignore

2012-06-26 Thread Guido Günther
On Tue, Jun 26, 2012 at 05:29:33PM +0100, Daniel P. Berrange wrote: > On Tue, Jun 26, 2012 at 06:15:31PM +0200, Guido Günther wrote: > > since it's an autogenerated file > > --- > > .gitignore |1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/.gitignore b/.gitignore > > index cd97

[libvirt] Libvirt on Ohloh

2012-06-26 Thread Guido Günther
Hi, I just noticed that ohloh tracks both libvirt-glib and libvirt under libvirt: https://www.ohloh.net/p/libvirt/enlistments Any objections to removing libvirt-glib from there (it's a separate project on ohloh as well). Cheers, -- Guido -- libvir-list mailing list libvir-list@redhat.co

[libvirt] pci bus slot in libvirt

2012-06-26 Thread Fong Vang
What's the best way to configure multiple disks in libvirt? How many slots can you have? This is from the xml dump: In some of my configurations, I have this: while others are created as such: /usr/bin/qemu-kvm

Re: [libvirt] Libvirt on Ohloh

2012-06-26 Thread Daniel P. Berrange
On Tue, Jun 26, 2012 at 09:53:42PM +0200, Guido Günther wrote: > Hi, > I just noticed that ohloh tracks both libvirt-glib and libvirt under > libvirt: > > https://www.ohloh.net/p/libvirt/enlistments > > Any objections to removing libvirt-glib from there (it's a separate > project on ohloh a

Re: [libvirt] [PATCH v4 2/7] qapi: Convert getfd and closefd

2012-06-26 Thread Luiz Capitulino
On Fri, 22 Jun 2012 14:36:09 -0400 Corey Bryant wrote: > Signed-off-by: Corey Bryant If this patch doesn't change anymore you can add: Reviewed-by: Luiz Capitulino > --- > v2: > -Convert getfd and closefd to QAPI (lcapitul...@redhat.com) > -Remove changes that returned fd from getfd (lcap

Re: [libvirt] [Qemu-devel] [PATCH v4 0/7] file descriptor passing using pass-fd

2012-06-26 Thread Daniel P. Berrange
On Tue, Jun 26, 2012 at 02:40:03PM -0400, Corey Bryant wrote: > > > On 06/26/2012 11:37 AM, Corey Bryant wrote: > > > > > >On 06/26/2012 11:03 AM, Daniel P. Berrange wrote: > >>On Tue, Jun 26, 2012 at 03:11:40PM +0100, Daniel P. Berrange wrote: > >>>On Tue, Jun 26, 2012 at 09:52:51AM -0400, Corey

Re: [libvirt] [PATCH v4 4/7] qapi: Re-arrange monitor.c functions

2012-06-26 Thread Luiz Capitulino
On Fri, 22 Jun 2012 14:36:11 -0400 Corey Bryant wrote: > Signed-off-by: Corey Bryant > --- > v4: > -This patch is new in v4. Can you explain why it's needed? > > monitor.c | 40 > 1 file changed, 20 insertions(+), 20 deletions(-) > > diff --git a/

Re: [libvirt] pci bus slot in libvirt

2012-06-26 Thread Daniel P. Berrange
On Tue, Jun 26, 2012 at 01:09:24PM -0700, Fong Vang wrote: > What's the best way to configure multiple disks in libvirt? How many > slots can you have? This is from the xml dump: > > In some of my configurations, I have this: > > > > > > >function='0x0'/

Re: [libvirt] [Qemu-devel] [PATCH v4 0/7] file descriptor passing using pass-fd

2012-06-26 Thread Luiz Capitulino
On Tue, 26 Jun 2012 13:45:52 +0200 Kevin Wolf wrote: > Am 26.06.2012 11:10, schrieb Daniel P. Berrange: > > I was thinking about some of the sources complexity when using > > FD passing from libvirt and wanted to raise one idea for discussion > > before we continue. > > > > With this proposed se

Re: [libvirt] pci bus slot in libvirt

2012-06-26 Thread Fong Vang
Very informational. Thank you very much. On Tue, Jun 26, 2012 at 1:51 PM, Daniel P. Berrange wrote: > On Tue, Jun 26, 2012 at 01:09:24PM -0700, Fong Vang wrote: >> What's the best way to configure multiple disks in libvirt?  How many >> slots can you have?  This is from the xml dump: >> >> In so

Re: [libvirt] [Qemu-devel] [PATCH v4 4/7] qapi: Re-arrange monitor.c functions

2012-06-26 Thread Corey Bryant
On 06/26/2012 04:45 PM, Luiz Capitulino wrote: On Fri, 22 Jun 2012 14:36:11 -0400 Corey Bryant wrote: Signed-off-by: Corey Bryant --- v4: -This patch is new in v4. Can you explain why it's needed? This just made patch 3/7 easier to review. monitor.c | 40 --

Re: [libvirt] [PATCH v4 2/7] qapi: Convert getfd and closefd

2012-06-26 Thread Corey Bryant
On 06/26/2012 04:44 PM, Luiz Capitulino wrote: On Fri, 22 Jun 2012 14:36:09 -0400 Corey Bryant wrote: Signed-off-by: Corey Bryant If this patch doesn't change anymore you can add: Reviewed-by: Luiz Capitulino Thanks! I'll do that. I'll assume we still want this patch even if pas

Re: [libvirt] pci bus slot in libvirt

2012-06-26 Thread Fong Vang
On Tue, Jun 26, 2012 at 2:00 PM, Fong Vang wrote: > Very informational.  Thank you very much. > > On Tue, Jun 26, 2012 at 1:51 PM, Daniel P. Berrange > wrote: >> On Tue, Jun 26, 2012 at 01:09:24PM -0700, Fong Vang wrote: >>> What's the best way to configure multiple disks in libvirt?  How many >

Re: [libvirt] [Qemu-devel] [PATCH v4 4/7] qapi: Re-arrange monitor.c functions

2012-06-26 Thread Luiz Capitulino
On Tue, 26 Jun 2012 17:16:28 -0400 Corey Bryant wrote: > > > On 06/26/2012 04:45 PM, Luiz Capitulino wrote: > > On Fri, 22 Jun 2012 14:36:11 -0400 > > Corey Bryant wrote: > > > >> Signed-off-by: Corey Bryant > >> --- > >> v4: > >> -This patch is new in v4. > > > > Can you explain why it's n

Re: [libvirt] [PATCH v4 2/7] qapi: Convert getfd and closefd

2012-06-26 Thread Luiz Capitulino
On Tue, 26 Jun 2012 17:15:05 -0400 Corey Bryant wrote: > > > On 06/26/2012 04:44 PM, Luiz Capitulino wrote: > > On Fri, 22 Jun 2012 14:36:09 -0400 > > Corey Bryant wrote: > > > >> Signed-off-by: Corey Bryant > > > > If this patch doesn't change anymore you can add: > > > > Reviewed-by: Luiz

Re: [libvirt] [Qemu-devel] [PATCH v4 4/7] qapi: Re-arrange monitor.c functions

2012-06-26 Thread Corey Bryant
On 06/26/2012 05:28 PM, Luiz Capitulino wrote: On Tue, 26 Jun 2012 17:16:28 -0400 Corey Bryant wrote: On 06/26/2012 04:45 PM, Luiz Capitulino wrote: On Fri, 22 Jun 2012 14:36:11 -0400 Corey Bryant wrote: Signed-off-by: Corey Bryant --- v4: -This patch is new in v4. Can you explai

Re: [libvirt] [PATCH v4 2/7] qapi: Convert getfd and closefd

2012-06-26 Thread Corey Bryant
On 06/26/2012 05:28 PM, Luiz Capitulino wrote: On Tue, 26 Jun 2012 17:15:05 -0400 Corey Bryant wrote: On 06/26/2012 04:44 PM, Luiz Capitulino wrote: On Fri, 22 Jun 2012 14:36:09 -0400 Corey Bryant wrote: Signed-off-by: Corey Bryant If this patch doesn't change anymore you can add:

Re: [libvirt] [Qemu-devel] [PATCH v4 0/7] file descriptor passing using pass-fd

2012-06-26 Thread Corey Bryant
On 06/26/2012 04:50 PM, Luiz Capitulino wrote: On Tue, 26 Jun 2012 13:45:52 +0200 Kevin Wolf wrote: Am 26.06.2012 11:10, schrieb Daniel P. Berrange: I was thinking about some of the sources complexity when using FD passing from libvirt and wanted to raise one idea for discussion before we c

Re: [libvirt] [Qemu-devel] [PATCH v4 0/7] file descriptor passing using pass-fd

2012-06-26 Thread Corey Bryant
On 06/26/2012 04:42 PM, Daniel P. Berrange wrote: On Tue, Jun 26, 2012 at 02:40:03PM -0400, Corey Bryant wrote: On 06/26/2012 11:37 AM, Corey Bryant wrote: On 06/26/2012 11:03 AM, Daniel P. Berrange wrote: On Tue, Jun 26, 2012 at 03:11:40PM +0100, Daniel P. Berrange wrote: On Tue, Jun 2

Re: [libvirt] [Qemu-devel] [PATCH v4 0/7] file descriptor passing using pass-fd

2012-06-26 Thread Eric Blake
On 06/26/2012 04:28 PM, Corey Bryant wrote: With this proposed series, we have usage akin to: 1. pass_fd FDSET={M} -> returns a string "/dev/fd/N" showing QEMU's view of the FD 2. drive_add file=/dev/fd/N 3. if failure: close_fd "/dev/fd/N"

[libvirt] [PATCH] virsh: Cleanup virsh -V output

2012-06-26 Thread Doug Goldstein
Fixed up virsh -V output by removing invalid WITH_PROXY & WITH_ONE checks, adding Hyper-V check and fixing the DTrace check. Signed-off-by: Doug Goldstein --- tools/virsh.c |9 +++-- 1 files changed, 3 insertions(+), 6 deletions(-) diff --git a/tools/virsh.c b/tools/virsh.c index 2b4cb2

Re: [libvirt] [PATCHv2 0/5] Basic Enablement of s390

2012-06-26 Thread Daniel Veillard
On Mon, Jun 25, 2012 at 06:54:12PM +0200, Viktor Mihajlovski wrote: > On 06/25/2012 04:43 PM, Michal Privoznik wrote: > >On 25.06.2012 15:54, Daniel Veillard wrote: > >>On Mon, Jun 25, 2012 at 09:43:41PM +0800, Daniel Veillard wrote: [...] > thanks for reviewing the patches, we will send out a

Re: [libvirt] [PATCH] virsh: Cleanup virsh -V output

2012-06-26 Thread Osier Yang
On 2012年06月27日 02:31, Doug Goldstein wrote: Fixed up virsh -V output by removing invalid WITH_PROXY& WITH_ONE checks, adding Hyper-V check and fixing the DTrace check. Signed-off-by: Doug Goldstein --- tools/virsh.c |9 +++-- 1 files changed, 3 insertions(+), 6 deletions(-) diff --g

Re: [libvirt] pci bus slot in libvirt

2012-06-26 Thread Osier Yang
On 2012年06月27日 05:25, Fong Vang wrote: On Tue, Jun 26, 2012 at 2:00 PM, Fong Vang wrote: Very informational. Thank you very much. On Tue, Jun 26, 2012 at 1:51 PM, Daniel P. Berrange wrote: On Tue, Jun 26, 2012 at 01:09:24PM -0700, Fong Vang wrote: What's the best way to configure multiple

Re: [libvirt] [PATCH 1/2] nodeinfo: Fix code style and some minor bugs.

2012-06-26 Thread Osier Yang
On 2012年06月25日 22:43, Peter Krempa wrote: This patch cleans up some line breaks and fixes two minor bugs: - buffer pointer increment to the actual length that should be skipped - jump to cleanup section instead of returning -1 --- src/nodeinfo.c | 31 +++ 1 files

[libvirt] IPs via DHCP for guest domains

2012-06-26 Thread Fong Vang
I'm curious to know how you are providing IPs for your Windows VMs when connected via a bridge. Since broadcasts from the VMs are going out the bridged interface, I'm thinking about implementing ebtables to block dhcp broadcasts from going in/out the interface so that dnsmasq would only respond to

Re: [libvirt] Libvirt on Ohloh

2012-06-26 Thread Guido Günther
On Tue, Jun 26, 2012 at 09:44:33PM +0100, Daniel P. Berrange wrote: > On Tue, Jun 26, 2012 at 09:53:42PM +0200, Guido Günther wrote: > > Hi, > > I just noticed that ohloh tracks both libvirt-glib and libvirt under > > libvirt: > > > > https://www.ohloh.net/p/libvirt/enlistments > > > > Any ob

Re: [libvirt] [libvirt-php PATCH] Add ability to managed save libvirt domains

2012-06-26 Thread Daniel Veillard
On Sat, Jun 23, 2012 at 07:01:51AM +0400, y...@billing.fastvps.ru wrote: > From: Pavel Odintsov > > --- > src/libvirt-php.c | 22 ++ > src/libvirt-php.h |1 + > 2 files changed, 23 insertions(+), 0 deletions(-) I see Michal actually commited your patch, thanks ! I'm