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-.scope systemd won't remove the cgroup and sub
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
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
> ---
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
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 (setns): Limit
---
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 b/src/
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.
---
src/
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 +
src/vbox/vbox_t
---
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 b/src/v
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 | 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 b/src/vbox
---
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 b/src/vb
---
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 b/src/vbox/vbo
---
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 b/src/vbox/vbox_n
---
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
--- a/src/vbox/vbox_
---
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 b/src/vbox/vbox
---
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 deletio
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 vbo
---
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(+), 2
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 vboxNetwor
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" wrote:
>V2:
>Addressed comments raised in review of V1.
>Consolidate calls to vi
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 i
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 f
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.1
On Wed, Oct 1, 2014 at 6:05 PM, Michal Privoznik 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
>> ---
>> sr
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
../npt
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
---
include
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
---
src/qemu/qemu_capabilities.c | 2 ++
src/qemu/qemu_capabili
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 the
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
---
src/qemu/qemu_monitor_json.c | 59 +++-
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
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 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 t
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 sepe
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 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
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 +-
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
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
s/pre
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 20f173b..88e431
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 behav
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 syntax
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
--- a/src/qem
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
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
+++ b/
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 fea
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
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-http:/
On 09/25/2014 08:06 AM, Peter Krempa wrote:
> From: Francesco Romani
>
> 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 block
> stats.
>
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 seperat
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 n
On 09/29/14 20:32, Eric Blake wrote:
> On 09/25/2014 06:06 AM, Peter Krempa wrote:
>> From: Francesco Romani
>>
>> 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 rep
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
> >>> ---
> >>>
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 U
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
> >>>---
> >>> in
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
> int
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
---
include/libvirt/libvirt.h.in | 1 +
src/libvirt.c| 7 +++
2 file
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 i
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 ../nptl/sysdeps/unix/sysv/linux/x86_64/pthre
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 a
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
ab
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.
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.
The
62 matches
Mail list logo