No answer ?
De : Yves Vinter
Envoyé : vendredi 26 septembre 2014 10:31
À : 'Eric Blake'; libvir-list@redhat.com
Objet : RE: [libvirt] [PATCH] New features implemented in hyperv libvirt driver
Hi Eric,
As you know, the is no more development in hyperv libvirt driver since version
0.10.2.
Hi Jim,
i use libvirt 1.1.3.5 on fedora core 20, the vm´s i startet are xen-vm´s.
i also take a look on libvirt 1.1.2 on an openSuSE 13.1 und also on libvirt
1.0.2 on an OpenSuSE 12.3 and i didn´t see threads that are created and not
closed . But on the openSuSE-Systems the vm´s are KVM/qemu.
Hi Yves,
On Wed, Oct 01, 2014 at 06:59:52AM +, Yves Vinter wrote:
No answer ?
If you split the patch in several smaller chunks as Eric requested and
send them to the mailing list for review, the next steps will be that
people on this list will review pour patch, potentially make comments
This had been on the Debian package list before but its time to take
this onwards. So the goal would be to have one set to rule them all
(when using apparmor) and drop the seperate set of definitions which
exist at least in the Ubuntu packaging.
Right now the patch would be at a state which adds
Hello,
sorry for my later answer.
so, after i started libvirtd (no vm´s running) and attach gdb i get the
following threads
(gdb) info thread
Id Target Id Frame
11 Thread 0x7f18fef4e700 (LWP 20695) libvirtd pthread_cond_wait@@GLIBC_2.3.2
() at
Hi Christophe,
I understand clearly the process.
I was just asking if there are some other requirements I should be aware of
juts to save time.
BTW.
Splitting the developments into several patchs (by features) will introduce a
dependency between them; they will have to be reviewed and applied
On 2014-09-30 17:16, Daniel P. Berrange wrote:
On Tue, Sep 30, 2014 at 05:11:03PM +0200, Jiri Denemark wrote:
On Tue, Sep 30, 2014 at 16:39:22 +0200, Cristian Klein wrote:
Signed-off-by: Cristian Klein cristian.kl...@cs.umu.se
---
include/libvirt/libvirt.h.in | 1 +
src/libvirt.c
On Wed, Oct 01, 2014 at 08:37:43AM +, Yves Vinter wrote:
Hi Christophe,
I understand clearly the process.
I was just asking if there are some other requirements I should
be aware of juts to save time.
BTW.
Splitting the developments into several patchs (by features) will
introduce a
On Wed, Oct 01, 2014 at 10:45:33AM +0200, Cristian KLEIN wrote:
On 2014-09-30 17:16, Daniel P. Berrange wrote:
On Tue, Sep 30, 2014 at 05:11:03PM +0200, Jiri Denemark wrote:
On Tue, Sep 30, 2014 at 16:39:22 +0200, Cristian Klein wrote:
Signed-off-by: Cristian Klein cristian.kl...@cs.umu.se
On Wed, Oct 01, 2014 at 10:30:58AM +0200, Stefan Bader wrote:
This had been on the Debian package list before but its time to take
this onwards. So the goal would be to have one set to rule them all
(when using apparmor) and drop the seperate set of definitions which
exist at least in the
On Wed, Oct 01, 2014 at 10:45:33 +0200, Cristian KLEIN wrote:
On 2014-09-30 17:16, Daniel P. Berrange wrote:
On Tue, Sep 30, 2014 at 05:11:03PM +0200, Jiri Denemark wrote:
On Tue, Sep 30, 2014 at 16:39:22 +0200, Cristian Klein wrote:
Signed-off-by: Cristian Klein cristian.kl...@cs.umu.se
On 09/29/14 20:32, Eric Blake wrote:
On 09/25/2014 06:06 AM, Peter Krempa wrote:
From: Francesco Romani from...@redhat.com
Management software wants to be able to allocate disk space on demand.
To support this they need keep track of the space occupation of the
block device. This
As planned the release of libvirt 1.2.9 is available, tagged in
git, signed sources and rpms are available at the usual place:
ftp://libvirt.org/libvirt/
I also pushed the libvirt-python release 1.2.9 to its own location:
ftp://libvirt.org/libvirt/python/
This release introduce some
On 01.10.2014 11:04, Daniel P. Berrange wrote:
On Wed, Oct 01, 2014 at 10:30:58AM +0200, Stefan Bader wrote:
This had been on the Debian package list before but its time to take
this onwards. So the goal would be to have one set to rule them all
(when using apparmor) and drop the seperate set
On 09/25/2014 08:06 AM, Peter Krempa wrote:
From: Francesco Romani from...@redhat.com
Management software wants to be able to allocate disk space on demand.
To support this they need keep track of the space occupation of the
block device. This information is reported by qemu as part of
On Wed, Oct 01, 2014 at 11:03:40AM +, Yves Vinter wrote:
Do I need credentials and/or specific rights on the libvirt git to do that?
No, you do everything in your local GIT checkout - there's no need for your
branch to be public
Regards,
Daniel
--
|: http://berrange.com -o-
On 09/24/2014 09:11 AM, John Ferlan wrote:
https://bugzilla.redhat.com/show_bug.cgi?id=1141732
Attempting to hot unplug a scsi_host device fails. The first patch
resolves the issue (and has some details in the commit message). The
second patch removes the now unnecessary virConnectPtr
Do I need credentials and/or specific rights on the libvirt git to do that?
-Message d'origine-
De : Daniel P. Berrange [mailto:berra...@redhat.com]
Envoyé : mercredi 1 octobre 2014 10:58
À : Yves Vinter
Cc : Christophe Fergeau; libvir-list@redhat.com
Objet : Re: [libvirt] [PATCH] New
Otherwise we'd report a different error.
Reported by John Ferlan's coverity run.
---
src/qemu/qemu_monitor_json.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/qemu/qemu_monitor_json.c b/src/qemu/qemu_monitor_json.c
index 5f00101..7fcf245 100644
--- a/src/qemu/qemu_monitor_json.c
+++
Patch 1/2 will be pushed soon under the trivial rule.
Patch 2/2 will be pushed soon under the build breaker rule.
Peter Krempa (2):
qemu: json: Fix missing break in error reporting function
qemu: monitor: Avoid shadowing variable devname on FreeBSD. Again.
src/qemu/qemu_monitor_json.c | 11
FreeBSD's compiler complains that we shadow the symbol. Sigh.
s/devname/dev_name/
---
src/qemu/qemu_monitor_json.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/src/qemu/qemu_monitor_json.c b/src/qemu/qemu_monitor_json.c
index 7fcf245..90a125f 100644
---
On Wed, Oct 01, 2014 at 02:43:13PM +0200, Peter Krempa wrote:
FreeBSD's compiler complains that we shadow the symbol. Sigh.
s/devname/dev_name/
---
src/qemu/qemu_monitor_json.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
ACK
but Seems like a candidate for
Tuning NUMA or network interface parameters require root
privileges to manage cgroups, thus an attempt to set some of these
parameters in session mode on a running domain should be invalid
followed by an error.
As an example might be memory tuning which raises an error in such case.
Following
This lets me build against an uninstalled source tree without
libtool.
---
src/libvirt-lxc.pc.in | 2 +-
src/libvirt-qemu.pc.in | 2 +-
src/libvirt.pc.in | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/src/libvirt-lxc.pc.in b/src/libvirt-lxc.pc.in
index
Thank you.
On 09/29/2014 01:53 PM, John Ferlan wrote:
On 09/26/2014 06:56 AM, John Ferlan wrote:
On 09/25/2014 10:26 AM, Erik Skultety wrote:
According to our documentation logical pool supports formats 'auto' and
'lvm2'. However, in storage_conf.c we prevously defined storage pool
Quoting Stefan Bader (stefan.ba...@canonical.com):
On 01.10.2014 11:04, Daniel P. Berrange wrote:
On Wed, Oct 01, 2014 at 10:30:58AM +0200, Stefan Bader wrote:
This had been on the Debian package list before but its time to take
this onwards. So the goal would be to have one set to rule
and tweak the code to avoid using it.
---
cfg.mk | 12
include/libvirt/libvirt.h.in | 2 +-
src/conf/interface_conf.c | 10 +-
src/conf/interface_conf.h | 2 +-
src/interface/interface_backend_udev.c | 2
On 10/01/2014 08:14 AM, Peter Krempa wrote:
and tweak the code to avoid using it.
---
cfg.mk | 12
include/libvirt/libvirt.h.in | 2 +-
src/conf/interface_conf.c | 10 +-
src/conf/interface_conf.h | 2
On 10/01/14 16:23, Eric Blake wrote:
On 10/01/2014 08:14 AM, Peter Krempa wrote:
and tweak the code to avoid using it.
---
cfg.mk | 12
include/libvirt/libvirt.h.in | 2 +-
src/conf/interface_conf.c | 10 +-
On 10/01/2014 04:04 AM, Daniel P. Berrange wrote:
On Wed, Oct 01, 2014 at 10:30:58AM +0200, Stefan Bader wrote:
This had been on the Debian package list before but its time to take
this onwards. So the goal would be to have one set to rule them all
(when using apparmor) and drop the seperate
On Wed, Oct 01, 2014 at 09:46:08AM -0500, Jamie Strandboge wrote:
On 10/01/2014 04:04 AM, Daniel P. Berrange wrote:
On Wed, Oct 01, 2014 at 10:30:58AM +0200, Stefan Bader wrote:
This had been on the Debian package list before but its time to take
this onwards. So the goal would be to have
On 10/01/2014 01:55 AM, John Ferlan wrote:
On 09/22/2014 06:41 AM, Erik Skultety wrote:
When trying to update bandwidth limits on a running domain, limits get
updated in our internal structures, however XML parser reads
bandwidth limits from network 'actual' definition. Commiting this patch
On 10/01/2014 11:17 AM, Erik Skultety wrote:
On 10/01/2014 01:55 AM, John Ferlan wrote:
On 09/22/2014 06:41 AM, Erik Skultety wrote:
When trying to update bandwidth limits on a running domain, limits get
updated in our internal structures, however XML parser reads
bandwidth limits from
On 30.09.2014 16:09, Matthias Gatto wrote:
Detect if the the qemu binary currently in use suport the bps_max option,
If yes add it to the command, if not, just ignore the options.
Signed-off-by: Matthias Gatto matthias.ga...@outscale.com
---
src/qemu/qemu_monitor_json.c | 59
On 30.09.2014 16:08, Matthias Gatto wrote:
Add defines for the news options total_bytes_sec_max, write_bytes_sec_max,
read_bytes_sec_max
total_iops_sec_max, write_iops_sec_max, read_iops_sec_max, size_iops_sec.
Modify the structure _virDomainBlockIoTuneInfo to support these options.
Change
On 30.09.2014 16:09, Matthias Gatto wrote:
Add support for bps_max and friends in the driver part.
In the part checking if a qemu is running, check if the running binary support
bps_max,
if not print an error message, if yes add it to info variable
Signed-off-by: Matthias Gatto
On 30.09.2014 16:09, Matthias Gatto wrote:
Add a value in the enum virQEMUCapsFlags for the qemu capability.
Set it with virQEMUCapsSet if the binary suport bps_max and they friends.
Signed-off-by: Matthias Gatto matthias.ga...@outscale.com
---
src/qemu/qemu_capabilities.c | 2
On 01.10.2014 10:31, web2 wrote:
Hello,
sorry for my later answer.
so, after i started libvirtd (no vm´s running) and attach gdb i get the
following threads
(gdb) info thread
Id Target Id Frame
11 Thread 0x7f18fef4e700 (LWP 20695) libvirtd
pthread_cond_wait@@GLIBC_2.3.2 () at
On Wed, Oct 1, 2014 at 6:05 PM, Michal Privoznik mpriv...@redhat.com wrote:
On 30.09.2014 16:09, Matthias Gatto wrote:
Detect if the the qemu binary currently in use suport the bps_max option,
If yes add it to the command, if not, just ignore the options.
Signed-off-by: Matthias Gatto
Hi
ursprüngliche Nachricht-
Von: Michal Privoznik mpriv...@redhat.com
An: web2 usterman...@web.de , libvirt-users redhat.com
, libvirt-list redhat.com
Datum: Wed, 01 Oct 2014 18:12:45 +0200
-
On 01.10.2014
On 09/30/2014 03:47 PM, Laine Stump wrote:
On 09/30/2014 02:28 PM, Tony Krowiak wrote:
On 09/24/2014 05:50 AM, Laine Stump wrote:
These patches set up an event handler for qemu's NIC_RX_FILTER_CHANGED
event, which is sent whenever a guest makes a change to a network
device's unicast/multicast
Hi,
On Fri, Sep 05, 2014 at 03:26:09PM +0200, Wouter Verhelst wrote:
Tunneling the entire protocol inside an SSL connection doesn't fix that;
if an attacker is able to hijack your TCP connections and change flags,
then this attacker is also able to hijack your TCP connection and
redirect it
Should I recreate this patch on the top of the latest tree and resubmit?
Or, is there anything that I missed out? Any feedback will be highly
appreciated. Thanks.
Anirban
On 9/26/14, 10:52 AM, Anirban Chakraborty abc...@juniper.net wrote:
V2:
Addressed comments raised in review of V1.
---
po/POTFILES.in|1 +
src/Makefile.am |1 +
src/vbox/vbox_network.c | 58 +
src/vbox/vbox_tmpl.c | 24 -
src/vbox/vbox_uniformed_api.h |3 +++
5 files changed, 63 insertions(+),
This seriers of patches rewrite the vbox network driver.
Diff from v2:
*Seperate the network driver from the domain driver.
*Prevent vbox code from being compiled too many times
when building network driver.
Taowei (14):
vbox: Interfaces for register per parties
vbox: Rewrite
The patch dbb4cbf532fa by Michal has splited the vbox driver into
three parties. This modification brings a more suitable interface
to the previous patch.
The new function vboxGetDriver is introduced to get the
corresponding vbox domain driver directly thought the vbox version.
Functions like
---
src/vbox/vbox_common.h| 15 ++
src/vbox/vbox_network.c | 55 ++
src/vbox/vbox_tmpl.c | 65 -
src/vbox/vbox_uniformed_api.h | 10 +++
4 files changed, 111 insertions(+), 34
---
src/vbox/vbox_network.c | 45 +
src/vbox/vbox_tmpl.c | 34 ---
src/vbox/vbox_uniformed_api.h |1 +
3 files changed, 46 insertions(+), 34 deletions(-)
diff --git a/src/vbox/vbox_network.c
---
src/vbox/vbox_network.c | 57 +
src/vbox/vbox_tmpl.c | 45
src/vbox/vbox_uniformed_api.h |1 +
3 files changed, 58 insertions(+), 45 deletions(-)
diff --git a/src/vbox/vbox_network.c
---
src/vbox/vbox_network.c | 77 +
src/vbox/vbox_tmpl.c | 52
src/vbox/vbox_uniformed_api.h |2 ++
3 files changed, 86 insertions(+), 45 deletions(-)
diff --git a/src/vbox/vbox_network.c
---
src/vbox/vbox_network.c |7 +++
src/vbox/vbox_tmpl.c |6 --
src/vbox/vbox_uniformed_api.h |1 +
3 files changed, 8 insertions(+), 6 deletions(-)
diff --git a/src/vbox/vbox_network.c b/src/vbox/vbox_network.c
index 2e09b4b..af38562 100644
---
---
src/vbox/vbox_network.c | 72 +
src/vbox/vbox_tmpl.c | 68 --
src/vbox/vbox_uniformed_api.h |1 +
3 files changed, 73 insertions(+), 68 deletions(-)
diff --git a/src/vbox/vbox_network.c
This patch rewrites two public APIs. They are vboxNetworkUndefine
and vboxNetworkDestroy. They use the same core function
vboxNetworkUndefineDestroy. I merged it in one patch.
---
src/vbox/vbox_network.c | 109 +
src/vbox/vbox_tmpl.c | 136
---
src/vbox/vbox_network.c | 49 +
src/vbox/vbox_tmpl.c | 54 -
src/vbox/vbox_uniformed_api.h |4 +++
3 files changed, 69 insertions(+), 38 deletions(-)
diff --git a/src/vbox/vbox_network.c
---
src/vbox/vbox_network.c | 191 +
src/vbox/vbox_tmpl.c | 235 +
src/vbox/vbox_uniformed_api.h |8 ++
3 files changed, 248 insertions(+), 186 deletions(-)
diff --git a/src/vbox/vbox_network.c
---
src/vbox/vbox_network.c | 65 +
src/vbox/vbox_tmpl.c | 57 +---
src/vbox/vbox_uniformed_api.h |8 +
3 files changed, 86 insertions(+), 44 deletions(-)
diff --git a/src/vbox/vbox_network.c
This patch seperate the domain driver and the network driver.
libvirt_driver_vbox_impl.la has been linked in the network driver.
So that the version specified codes in vbox_V*.c would only be
compiled once.
The vboxGetNetworkDriver provides a simple interface to get vbox
network driver.
---
This patch actually contains two public API, virNetworkDefineXML
and virNetworkCreateXML. They use the same core function
vboxNetworkDefineCreateXML. So I merged it together.
---
src/vbox/vbox_common.h|1 +
src/vbox/vbox_network.c | 214 +
A cygwin build of 1.2.9 fails with:
util/virprocess.c:87:27: fatal error: sys/syscall.h: No such file or directory
# include sys/syscall.h
But in reality, the ONLY user of setns() is lxc, which is Linux-only.
It's easiest to just limit the setns workarounds to Linux.
* src/util/virprocess.c
web2 wrote:
Hi
ursprüngliche Nachricht-
Von: Michal Privoznik mpriv...@redhat.com
An: web2 usterman...@web.de , libvirt-users redhat.com
, libvirt-list redhat.com
Datum: Wed, 01 Oct 2014 18:12:45 +0200
-
On Mon, Sep 29, 2014 at 09:47:08AM +0200, Martin Kletzander wrote:
On Sun, Sep 28, 2014 at 10:35:52AM +0200, Guido Günther wrote:
Debian wants to use 'sensible-editor' instead of vi other distros might
want to use other defaults. This avoids distro specific patches.
---
configure.ac | 9
On Tue, Sep 30, 2014 at 03:22:41PM +0100, Daniel P. Berrange wrote:
On Tue, Sep 30, 2014 at 10:10:51AM -0400, Cole Robinson wrote:
On 09/25/2014 08:30 AM, Guido Günther wrote:
If we don't properly clean up all processes in the
machine-vmname.scope systemd won't remove the cgroup and
62 matches
Mail list logo