- Original Message -
> From: "Eric Blake"
> To: "Francesco Romani" , libvir-list@redhat.com
> Sent: Wednesday, September 3, 2014 12:14:58 AM
> Subject: Re: [libvirt] [PATCHv2 03/11] qemu: add helper to get the block
> stats
>
> On 09/02/2014 06:31 AM, Francesco Romani wrote:
> > Add
- Original Message -
> From: "Eric Blake"
> To: "Francesco Romani" , libvir-list@redhat.com
> Sent: Tuesday, September 2, 2014 11:42:09 PM
> Subject: Re: [libvirt] [PATCH 02/11] qemu: extract helper to gather vcpu data
>
> On 09/02/2014 06:31 AM, Francesco Romani wrote:
> > Extracts an he
- Original Message -
> From: "Eric Blake"
> To: "Francesco Romani" , libvir-list@redhat.com
> Sent: Tuesday, September 2, 2014 11:01:25 PM
> Subject: Re: [libvirt] [PATCH 01/11] qemu: extract helper to get the current
> balloon
Hi Eric, thanks for the review(s).
> On 09/02/2014 06:31 AM
Add umask to _virCommand, allow user to set umask to command.
Set umask(002) to qemu process to overwrite default umask(022) so
that unix sockets created for virtio-serial has expected permissions.
Fix problem reported here:
https://sourceware.org/bugzilla/show_bug.cgi?id=13078#c11
https://bugzill
On 09/03/2014 07:19 AM, Eric Blake wrote:
> On 08/29/2014 01:56 AM, Li Wei wrote:
>> This patch add the block group for bulk stats.
>> The following typed parameter used for each block stats:
>> block.count - number of block devices in this domain
>> block.0.name - name of the block devic
On 08/29/2014 01:56 AM, Li Wei wrote:
> Add "--block" option to "domstats" command for querying block stats.
>
> Signed-off-by: Li Wei
> ---
> tools/virsh-domain-monitor.c | 7 +++
> tools/virsh.pod | 4 ++--
> 2 files changed, 9 insertions(+), 2 deletions(-)
ACK; but depends o
On 08/29/2014 01:56 AM, Li Wei wrote:
> This patch add the block group for bulk stats.
> The following typed parameter used for each block stats:
> block.count - number of block devices in this domain
> block.0.name - name of the block device
> block.0.rd_bytes - number of read bytes
>
On 09/02/2014 06:31 AM, Francesco Romani wrote:
> This patchset enhances the QEMU support
> for the new bulk stats API to include
> equivalents of these APIs:
>
> virDomainBlockInfo
> virDomainGetInfo - for balloon stats
> virDomainGetCPUStats
> virDomainBlockStatsFlags
> virDomainInterfaceStats
>
On 09/02/2014 06:31 AM, Francesco Romani wrote:
> Management software, want to be able to allocate disk space on demand.
s/software, want/software wants/
> To support this, they need keep track of the space occupation
> of the block device.
(what's more, during a blockcopy or active blockcommit,
On 09/02/2014 06:31 AM, Francesco Romani wrote:
> This patch implements the VIR_DOMAIN_STATS_BLOCK
> group of statistics.
>
> Signed-off-by: Francesco Romani
> ---
> include/libvirt/libvirt.h.in | 1 +
> src/libvirt.c| 13 +
> src/qemu/qemu_driver.c | 65
> +++
On 09/02/2014 06:31 AM, Francesco Romani wrote:
> This patch implements the VIR_DOMAIN_STATS_INTERFACE
> group of statistics.
>
> Signed-off-by: Francesco Romani
> ---
> include/libvirt/libvirt.h.in | 1 +
> src/libvirt.c| 13 +++
> src/qemu/qemu_driver.c | 85
> +
On 09/02/2014 06:31 AM, Francesco Romani wrote:
> This patch implements the VIR_DOMAIN_STATS_VCPU
> group of statistics.
>
> Signed-off-by: Francesco Romani
> ---
> include/libvirt/libvirt.h.in | 1 +
> src/libvirt.c| 8 +
> src/qemu/qemu_driver.c | 72
>
On 09/02/2014 06:31 AM, Francesco Romani wrote:
> This patch implements the VIR_DOMAIN_STATS_BALLOON
> group of statistics.
>
> Signed-off-by: Francesco Romani
> ---
> include/libvirt/libvirt.h.in | 1 +
> src/libvirt.c| 4
> src/qemu/qemu_driver.c | 32 +
On 09/02/2014 06:31 AM, Francesco Romani wrote:
> This patch implements the VIR_DOMAIN_STATS_CPU_TOTAL
> group of statistics.
>
> Signed-off-by: Francesco Romani
> ---
> include/libvirt/libvirt.h.in | 1 +
> src/libvirt.c| 8 +++
> src/qemu/qemu_driver.c | 56
> +
On 09/02/2014 06:31 AM, Francesco Romani wrote:
> This patch implements the VIR_DOMAIN_STATS_CPU_TOTAL
> group of statistics.
>
> Signed-off-by: Francesco Romani
> ---
> include/libvirt/libvirt.h.in | 1 +
> src/libvirt.c| 8 +++
> src/qemu/qemu_driver.c | 56
> +
On 09/02/2014 06:31 AM, Francesco Romani wrote:
> Future patches which will implement more
> bulk stats groups for QEMU will need to access
> the connection object, so enrich the worker
> prototype.
>
> Signed-off-by: Francesco Romani
> ---
> src/qemu/qemu_driver.c | 9 +
> 1 file change
On 09/02/2014 06:31 AM, Francesco Romani wrote:
> This patch adds the reporting of the highest
> block stats written to the block stats.
>
> This information was already there since the
> beginning, and doing so we gain the information
> provided by GetBlockInfo without entering the
> monitor agai
On 09/02/2014 06:31 AM, Francesco Romani wrote:
> Add an helper function to get the block stats
> of a disk.
> This helper is meant to be used by the bulk stats API.
>
> Signed-off-by: Francesco Romani
> ---
> src/qemu/qemu_driver.c | 41 +++
> src/qemu/qemu_monitor.c |
On 09/02/2014 06:31 AM, Francesco Romani wrote:
> Extracts an helper to gether the VCpu
s/an/a/
s/gether/gather/
> information.
>
> Signed-off-by: Francesco Romani
> ---
> src/qemu/qemu_driver.c | 29 +
> 1 file changed, 25 insertions(+), 4 deletions(-)
>
> diff --
On 09/02/2014 06:31 AM, Francesco Romani wrote:
> Refactor the code to extract an helper method
> to get the current balloon settings.
>
> Signed-off-by: Francesco Romani
> ---
> src/qemu/qemu_driver.c | 98
> ++
> 1 file changed, 60 insertions(+)
Sorry about that. I am having some trouble posting to this mailing
list. Just ignore this message.
--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
On 09/02/2014 04:27 PM, Gene Czarcinski wrote:
Sorry about that. I am having some trouble posting to this mailing
list. Just ignore this message.
Gene
--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
Sorry about that. I am having some trouble posting to this mailing
list. Just ignore this message.
Gene
--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
On Thu, Jul 31, 2014 at 01:13:19PM +0200, Ján Tomko wrote:
> Hello developers!
>
> Currently, our default cgroup layout is:
> -top level cgroup
> \-machine (machine.slice with systemd)
> `-vm1.libvirt-qemu (machine-qemu\x2dvm1.scope with systemd)
> `-emulator
> `-vcpu0
> \-vcpu1
> \-vm2.li
There are two ways how to tell qemu to use huge pages. The first one
is suitable for domains with NUMA nodes: the path to hugetlbfs mount
is appended to NUMA node definition on the command line. The second
one is suitable for UMA domains: here there's this global '-mem-path'
argument that accepts p
As of 136ad4974 it is possible to specify different huge pages per
guest NUMA node. However, there's no check if nodeset specified in
./hugepages/page contains only those guest NUMA nodes that exist.
In other words with current code it is possible to define meaningless
combination:
*** BLURB HERE ***
Michal Privoznik (2):
conf: Disallow nonexistent NUMA nodes for hugepages
qemu: Honor hugepages for UMA domains
src/conf/domain_conf.c | 36 +
src/qemu/qemu_command.c| 25 ++--
.../qemuxml2argv
Bcc: Raghavendra K T
Subject: Re: [libvirt] [RFC] exclusive vcpu-cpu pinning
Reply-To: Raghavendra K T
In-Reply-To: <53da24cf.8030...@redhat.com>
* J?n Tomko [2014-07-31 13:13:19]:
> Hello developers!
>
> Currently, our default cgroup layout is:
> -top level cgroup
> \-machine (machine.slice
* J?n Tomko [2014-07-31 13:13:19]:
> Hello developers!
>
> Currently, our default cgroup layout is:
> -top level cgroup
> \-machine (machine.slice with systemd)
> `-vm1.libvirt-qemu (machine-qemu\x2dvm1.scope with systemd)
> `-emulator
> `-vcpu0
> \-vcpu1
> \-vm2.libvirt-qemu
> `-emula
On 09/02/14 15:27, Martin Kletzander wrote:
> On Tue, Sep 02, 2014 at 03:18:39PM +0200, Peter Krempa wrote:
>> The 'elem' variable along with the domain object would be leaked when
>> taking the error path.
>>
>> Found by coverity.
>> ---
>> src/remote/remote_driver.c | 8 ++--
>> 1 file changed
On 08/28/14 13:27, Jincheng Miao wrote:
> Currently it lacks synchronization to modify domain's snapshot object list,
> that race condition causes unsafe access to some freed snapshot objects.
> Therefore, this patch wraps all access of snapshot object list in vm job lock.
>
> Only the qemuDomainS
Benoît Canet writes:
> The Monday 01 Sep 2014 à 13:41:01 (+0200), Markus Armbruster wrote :
>> Benoît Canet writes:
>>
>> > The Monday 01 Sep 2014 à 11:52:00 (+0200), Markus Armbruster wrote :
[...]
>> >> A quick stab at tasks:
>> >>
>> >> * QMP interface, either a compatible extension of quer
On Tue, Sep 02, 2014 at 03:18:39PM +0200, Peter Krempa wrote:
The 'elem' variable along with the domain object would be leaked when
taking the error path.
Found by coverity.
---
src/remote/remote_driver.c | 8 ++--
1 file changed, 6 insertions(+), 2 deletions(-)
ACK,
Martin
diff --git a
The 'elem' variable along with the domain object would be leaked when
taking the error path.
Found by coverity.
---
src/remote/remote_driver.c | 8 ++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/src/remote/remote_driver.c b/src/remote/remote_driver.c
index fda27f7..8bc4baa
py_record_domain and py_record_stats would be accessed uninitialized if
an out-of-memory condition would happen in the first loop. Unlikely, but
coverity complained.
Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1136354
---
Pushed as trivial.
libvirt-override.c | 4 ++--
1 file changed,
Add an helper function to get the block stats
of a disk.
This helper is meant to be used by the bulk stats API.
Signed-off-by: Francesco Romani
---
src/qemu/qemu_driver.c | 41 +++
src/qemu/qemu_monitor.c | 23 +
src/qemu/qemu_monitor.h | 18 +++
src/qe
This patch implements the VIR_DOMAIN_STATS_INTERFACE
group of statistics.
Signed-off-by: Francesco Romani
---
include/libvirt/libvirt.h.in | 1 +
src/libvirt.c| 13 +++
src/qemu/qemu_driver.c | 85
3 files changed, 99 insert
This patch implements the VIR_DOMAIN_STATS_VCPU
group of statistics.
Signed-off-by: Francesco Romani
---
include/libvirt/libvirt.h.in | 1 +
src/libvirt.c| 8 +
src/qemu/qemu_driver.c | 72
3 files changed, 81 insertions(+)
This patch implements the VIR_DOMAIN_STATS_BALLOON
group of statistics.
Signed-off-by: Francesco Romani
---
include/libvirt/libvirt.h.in | 1 +
src/libvirt.c| 4
src/qemu/qemu_driver.c | 32
3 files changed, 37 insertions(+)
diff --g
Management software, want 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.
This patch extend the block information in the bulk stats with
the allocation infor
This patch implements the VIR_DOMAIN_STATS_BLOCK
group of statistics.
Signed-off-by: Francesco Romani
---
include/libvirt/libvirt.h.in | 1 +
src/libvirt.c| 13 +
src/qemu/qemu_driver.c | 65
3 files changed, 79 insertio
Extracts an helper to gether the VCpu
information.
Signed-off-by: Francesco Romani
---
src/qemu/qemu_driver.c | 29 +
1 file changed, 25 insertions(+), 4 deletions(-)
diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c
index bbd16ed..1842e60 100644
--- a/src
Future patches which will implement more
bulk stats groups for QEMU will need to access
the connection object, so enrich the worker
prototype.
Signed-off-by: Francesco Romani
---
src/qemu/qemu_driver.c | 9 +
1 file changed, 5 insertions(+), 4 deletions(-)
diff --git a/src/qemu/qemu_dri
This patch implements the VIR_DOMAIN_STATS_CPU_TOTAL
group of statistics.
Signed-off-by: Francesco Romani
---
include/libvirt/libvirt.h.in | 1 +
src/libvirt.c| 8 +++
src/qemu/qemu_driver.c | 56
3 files changed, 65 insert
Refactor the code to extract an helper method
to get the current balloon settings.
Signed-off-by: Francesco Romani
---
src/qemu/qemu_driver.c | 98 ++
1 file changed, 60 insertions(+), 38 deletions(-)
diff --git a/src/qemu/qemu_driver.c b/src/qemu
This patch adds the reporting of the highest
block stats written to the block stats.
This information was already there since the
beginning, and doing so we gain the information
provided by GetBlockInfo without entering the
monitor again.
Signed-off-by: Francesco Romani
---
src/qemu/qemu_monito
This patchset enhances the QEMU support
for the new bulk stats API to include
equivalents of these APIs:
virDomainBlockInfo
virDomainGetInfo - for balloon stats
virDomainGetCPUStats
virDomainBlockStatsFlags
virDomainInterfaceStats
virDomainGetVcpusFlags
virDomainGetVcpus
This subset of API is the
"Daniel P. Berrange" writes:
> On Tue, Sep 02, 2014 at 03:40:42PM +0800, Chunyan Liu wrote:
>> To use virtio-serial device, unix socket created for chardev with
>> default umask(022) has insufficient permissions.
>>
>> e.g. start kvm guest with:
>> -device virtio-serial \
>> -chardev socket,path
OK, hopefully this mailing list is more active and I can get some
response to my questions.
-
I have been "playing with" Secure Containers running a lighttpd server
and have it up
https://bugzilla.redhat.com/show_bug.cgi?id=1135388
---
Pushed as trivial.
src/test/test_driver.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/test/test_driver.c b/src/test/test_driver.c
index 3b22cf6..9918d2c 100644
--- a/src/test/test_driver.c
+++ b/src/test/test_driver.c
@@ -521,6
When virsh vol-clone is attempted on a raw file where capacity > allocation,
the resulting cloned volume has a size that matches the virtual-size of
the parent; in place of matching its actual, disk size.
For example : Given this image:
[root@localhost]# qemu-img info /var/lib/libvirt/images/f20.ra
>>> On 9/2/2014 at 05:16 PM, in message <20140902091655.gb21...@redhat.com>,
"Daniel P. Berrange" wrote:
> On Tue, Sep 02, 2014 at 03:08:51AM -0600, Chun Yan Liu wrote:
> >
> >
> > >>> On 9/2/2014 at 04:54 PM, in message
> > >>> <20140902085434.ga21...@redhat.com>,
> > "Daniel P. Berrang
On 01.09.2014 10:31, Philipp Hahn wrote:
Hello Tomas,
On 01.09.2014 01:51, Tomas Meszaros wrote:
I've recently worked with rather large number of virtual machines
and needed to rename all domains. I couldn't find better way how
to rename domain other than:
virsh dumpxml domain > domain.xm
Remove unused label 'cleanup' in 'libvirt_virConnectGetAllDomainStats'
function and remove unused variable 'conn' in function
'libvirt_virDomainListGetStats'.
Signed-off-by: Pavel Hrdina
---
Pushed as trivial
libvirt-override.c | 3 ---
1 file changed, 3 deletions(-)
diff --git a/libvirt-over
Am 29.08.2014 12:03, schrieb Daniel Veillard:
> On Wed, Aug 27, 2014 at 08:45:29PM +0200, Richard Weinberger wrote:
>> On Wed, Aug 27, 2014 at 9:18 AM, Daniel Veillard wrote:
>>> So I tagged 1.2.8-rc1 in git and made tarball and signed rpms
>>
>> Can you please sign the tarball too?
>
> Okay,
On Tue, Sep 02, 2014 at 03:08:51AM -0600, Chun Yan Liu wrote:
>
>
> >>> On 9/2/2014 at 04:54 PM, in message <20140902085434.ga21...@redhat.com>,
> "Daniel P. Berrange" wrote:
> > On Tue, Sep 02, 2014 at 03:40:42PM +0800, Chunyan Liu wrote:
> > > To use virtio-serial device, unix socket created
>>> On 9/2/2014 at 04:54 PM, in message <20140902085434.ga21...@redhat.com>,
"Daniel P. Berrange" wrote:
> On Tue, Sep 02, 2014 at 03:40:42PM +0800, Chunyan Liu wrote:
> > To use virtio-serial device, unix socket created for chardev with
> > default umask(022) has insufficient permissions.
>
Signed-off-by: Michal Privoznik
---
Pushed under trivial rule.
setup.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/setup.py b/setup.py
index f5c6542..167f568 100755
--- a/setup.py
+++ b/setup.py
@@ -309,7 +309,7 @@ class my_clean(clean):
_c_modules, _py_modules = get_m
Signed-off-by: Michal Privoznik
---
Pushed under trivial rule.
configure.ac | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/configure.ac b/configure.ac
index f93c6c2..b4fb99a 100644
--- a/configure.ac
+++ b/configure.ac
@@ -16,7 +16,7 @@ dnl You should have received a copy o
On Tue, Sep 02, 2014 at 03:40:42PM +0800, Chunyan Liu wrote:
> To use virtio-serial device, unix socket created for chardev with
> default umask(022) has insufficient permissions.
>
> e.g. start kvm guest with:
> -device virtio-serial \
> -chardev socket,path=/tmp/foo,server,nowait,id=foo \
> -dev
It's out ! Tagged and signed in git as well as tarball and rpms
are available from the usual place:
ftp://libvirt.org/libvirt/
I also made a release for the python bindings available from
ftp://libvirt.org/libvirt/python/
This is a rather large release (surprizingly for August !) with a l
To use virtio-serial device, unix socket created for chardev with
default umask(022) has insufficient permissions.
e.g. start kvm guest with:
-device virtio-serial \
-chardev socket,path=/tmp/foo,server,nowait,id=foo \
-device virtserialport,chardev=foo,name=org.fedoraproject.port.0
Check permiss
To use virtio-serial device, unix socket created for communication with
default umask(022) has insufficient permissions.
e.g.
1. Setup a virtual machine with a virtio-serial device:
# virsh edit myvm
(...)
(...)
2. Start this virtual machine:
# virsh
https://sourceware.org/bugzilla/show_bug.cgi?id=13078#c11
https://bugzilla.novell.com/show_bug.cgi?id=888166
To use virtio-serial device, unix socket created for chardev with
default umask(022) has insufficient permissions.
e.g.:
-device virtio-serial \
-chardev socket,path=/tmp/foo,server,nowait,
64 matches
Mail list logo
