* Srivatsa S. Bhat srivatsa.b...@linux.vnet.ibm.com [2011-09-12 14:55:26]:
Hi,
In order to exploit the host power management capabilities like S3/S4,
it was proposed to add additional tags in the libvirt capabilities XML
to export the power management features supported by the host. The
On Sun, Sep 18, 2011 at 09:37:22AM -0500, Adam Litke wrote:
I am getting SIGABRT and SIGSEGV in libvirtd when trying to catch blockJob
events.
When running under valgrind I get the following:
==19819== Thread 1:
==19819== Invalid free() / delete / delete[]
==19819==at 0x4C282ED: free
From: Daniel P. Berrange berra...@redhat.com
Inexpicably the sanlock code all got placed under the GPLv2-only,
so libvirt's use of sanlock introduces a license incompatibility.
The sanlock developers have now rearranged the code such that there
is a 'sanlock_client.so' which is LGPLv2+ while
Users of virsh complain that output of the domblkstat command
is not intuitive enough. This patch adds explanation of fields
returned by this command to the help section for domblkstat and
the man page of virsh. Also a switch --human is added for
domblkstat that prints the fields with more
I accidentaly posted an older version of this patch :(
Please disregard this v6, as it does not compile.
Peter
--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
Users of virsh complain that output of the domblkstat command
is not intuitive enough. This patch adds explanation of fields
returned by this command to the help section for domblkstat and
the man page of virsh. Also a switch --human is added for
domblkstat that prints the fields with more
Virsh man page lists driver types to be used with attach-device
command, but does not specify that those are usable only with the XEN
Hypervisor.
This patch adds statement, that those options specified are applicable
only on the Xen hypervisor and adds option usable with qemu emulator.
This
Silently setting timeout as -1 if the specified value is invalid
is a bit confused.
---
daemon/libvirtd.c |6 --
1 files changed, 4 insertions(+), 2 deletions(-)
diff --git a/daemon/libvirtd.c b/daemon/libvirtd.c
index c708ff7..4dd68d5 100644
--- a/daemon/libvirtd.c
+++
Patch 79cf07a missed one instance of devname in source for RPC code
generator.
---
src/remote/remote_protocol.x |2 +-
src/remote_protocol-structs |2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/remote/remote_protocol.x b/src/remote/remote_protocol.x
index
On 09/18/2011 07:02 PM, Wen Congyang wrote:
At 09/15/2011 11:10 PM, Michal Privoznik Write:
Even though we BuildRequire augeas in some cases, we need to require
it even after if we build with sanlock. virt-sanlock-cleanup use it.
---
libvirt.spec.in |2 ++
1 files changed, 2
Hi hi
On Fri, Sep 16, 2011 at 1:38 PM, Marc-André Lureau mlur...@redhat.com wrote:
How do we allow other devices to share the slot? It seems to me that
qemuDomainPCIAddressSetNextAddr() only allocate whole slot, while
making sure there is no conflicts on the same slot.
So, if the user
On 09/19/2011 10:22 AM, Eric Blake wrote:
On RHEL6.1GA, there is no package named augeas.
It only has augeas-libs.
On F15, it also only has augeas-libs.
What is augeas?
Augeas is the package that provides the standalone executable 'augtool'
(this is the command line access to the augeas-libs
On Mon, Sep 19, 2011 at 07:16:22PM +0200, Marc-André Lureau wrote:
Hi hi
On Fri, Sep 16, 2011 at 1:38 PM, Marc-André Lureau mlur...@redhat.com wrote:
How do we allow other devices to share the slot? It seems to me that
qemuDomainPCIAddressSetNextAddr() only allocate whole slot, while
On 09/19/2011 04:04 AM, Daniel P. Berrange wrote:
From: Daniel P. Berrangeberra...@redhat.com
Inexpicably the sanlock code all got placed under the GPLv2-only,
s/Inexpicably/Inexplicably/
so libvirt's use of sanlock introduces a license incompatibility.
The sanlock developers have now
On 09/18/2011 10:36 AM, a...@redhat.com wrote:
* daemon/remote.c: fix memory leak.
Your commit message didn't mention which function you fixed,...
Signed-off-by: Alex Jiaa...@redhat.com
---
daemon/remote.c |2 ++
1 files changed, 2 insertions(+), 0 deletions(-)
diff --git
On 09/18/2011 10:36 AM, a...@redhat.com wrote:
* src/qemu/qemu_command.c: fix memory leak.
Signed-off-by: Alex Jiaa...@redhat.com
---
src/qemu/qemu_command.c |1 -
1 files changed, 0 insertions(+), 1 deletions(-)
diff --git a/src/qemu/qemu_command.c b/src/qemu/qemu_command.c
index
On 09/18/2011 10:36 AM, a...@redhat.com wrote:
* src/remote/remote_driver.c: fix memory leak.
Signed-off-by: Alex Jiaa...@redhat.com
---
src/remote/remote_driver.c |2 ++
1 files changed, 2 insertions(+), 0 deletions(-)
diff --git a/src/remote/remote_driver.c
On 09/18/2011 10:36 AM, a...@redhat.com wrote:
* src/util/pci.c: fix memory leak.
Signed-off-by: Alex Jiaa...@redhat.com
---
src/util/pci.c |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/src/util/pci.c b/src/util/pci.c
index 9873f33..8d8e157 100644
---
On 09/18/2011 08:34 PM, a...@redhat.com wrote:
* src/vmx/vmx.c: fix memory leak, 'def' has a initial value 'NULL', so
'goto cleanup' is perfected instead of adding a virConfFree before
'return NULL'.
Signed-off-by: Alex Jiaa...@redhat.com
---
src/vmx/vmx.c |2 +-
1 files changed, 1
On 09/19/2011 06:23 AM, Peter Krempa wrote:
Users of virsh complain that output of the domblkstat command
is not intuitive enough. This patch adds explanation of fields
returned by this command to the help section for domblkstat and
the man page of virsh. Also a switch --human is added for
Dňa 19.9.2011 22:24, Eric Blake wrote / napísal(a):
ACK with my nits fixed, so I've pushed this. Here's what I squashed in:
Uff, thanks for the many suggestions and for fixing it right away.
Well, line length is one of my problems. I try to keep them short,
but on a wide angle monitor,
QEMU 0.13 introduced cache=unsafe for -drive, this patch exposes
it in the libvirt layer.
* Introduced a new QEMU capability flag ($prefix_CACHE_UNSAFE),
as even if $prefix_CACHE_V2 is set, we can't known if unsafe
is supported.
* qemuhelptest prints test case name on failure.
---
On 09/19/2011 02:24 PM, Eric Blake wrote:
virsh # domblkstat 1 vda --human
Device: vda
number of read operations: 3726
number of read bytes: 82815488
number of write operations: 478
number of bytes written: 4965376
For consistency, I think bytes read matches better with bytes written.
And
On 09/01/2011 07:44 AM, Osier Yang wrote:
Libvirt loads the domain conf from status XML if it's running when
starting up. The problem is there is no record of the original conf.
(dom-newDef is NULL here).
So libvirt won't be able to restore the domain conf to original one
when
Dňa 19.9.2011 22:49, Eric Blake wrote / napísal(a):
On 09/19/2011 02:24 PM, Eric Blake wrote:
virsh # domblkstat 1 vda --human
Device: vda
number of read operations: 3726
number of read bytes: 82815488
number of write operations: 478
number of bytes written: 4965376
For consistency, I think
On 09/19/2011 03:04 PM, Peter Krempa wrote:
- vshPrint(ctl, %s %s %s\n, device, field, value);
+ vshPrint(ctl, %-*s %s %s\n, human ? 31 : 0,
I think this is not completely correct
D'oh - you're right. I justified the wrong field. That's what I get
for writing the email with only a compile,
On 09/19/2011 06:57 AM, Peter Krempa wrote:
Virsh man page lists driver types to be used with attach-device
command, but does not specify that those are usable only with the XEN
Hypervisor.
This patch adds statement, that those options specified are applicable
only on the Xen hypervisor and
On 09/19/2011 08:14 AM, Peter Krempa wrote:
Patch 79cf07a missed one instance of devname in source for RPC code
generator.
---
src/remote/remote_protocol.x |2 +-
src/remote_protocol-structs |2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git
On 09/19/2011 08:28 AM, Osier Yang wrote:
Silently setting timeout as -1 if the specified value is invalid
is a bit confused.
---
daemon/libvirtd.c |6 --
1 files changed, 4 insertions(+), 2 deletions(-)
diff --git a/daemon/libvirtd.c b/daemon/libvirtd.c
index c708ff7..4dd68d5 100644
On 09/19/2011 08:28 AM, Osier Yang wrote:
Doing libvirt_parthelper on an not existed device path will get
an unfriendly error message. This patch is to prohibit it.
---
src/storage/storage_backend_disk.c |8 +++-
1 files changed, 7 insertions(+), 1 deletions(-)
diff --git
At 09/20/2011 01:16 AM, Marc-André Lureau Write:
Hi hi
On Fri, Sep 16, 2011 at 1:38 PM, Marc-André Lureau mlur...@redhat.com wrote:
How do we allow other devices to share the slot? It seems to me that
qemuDomainPCIAddressSetNextAddr() only allocate whole slot, while
making sure there is no
On 09/20/2011 03:03 AM, Eric Blake wrote:
On 09/18/2011 10:36 AM, a...@redhat.com wrote:
* daemon/remote.c: fix memory leak.
Your commit message didn't mention which function you fixed,...
Oh, I think 'git' will do this , and sorry about any inconvenient for you.
Thanks,
Alex
于 2011年09月20日 08:17, Eric Blake 写道:
On 09/19/2011 08:28 AM, Osier Yang wrote:
Doing libvirt_parthelper on an not existed device path will get
an unfriendly error message. This patch is to prohibit it.
---
src/storage/storage_backend_disk.c | 8 +++-
1 files changed, 7 insertions(+), 1
于 2011年09月20日 08:16, Eric Blake 写道:
On 09/19/2011 08:28 AM, Osier Yang wrote:
Silently setting timeout as -1 if the specified value is invalid
is a bit confused.
---
daemon/libvirtd.c | 6 --
1 files changed, 4 insertions(+), 2 deletions(-)
diff --git a/daemon/libvirtd.c b/daemon/libvirtd.c
At 09/15/2011 06:07 PM, Jiri Denemark Write:
Ever since we introduced fake reboot, we call qemuProcessKill as a
reaction to SHUTDOWN event. Unfortunately, qemu doesn't guarantee it
flushed all internal buffers before sending SHUTDOWN, in which case
killing the process forcibly may result in
于 2011年09月20日 04:47, Oskari Saarenmaa 写道:
QEMU 0.13 introduced cache=unsafe for -drive, this patch exposes
it in the libvirt layer.
* Introduced a new QEMU capability flag ($prefix_CACHE_UNSAFE),
as even if $prefix_CACHE_V2 is set, we can't known if unsafe
is supported.
*
于 2011年09月20日 04:58, Eric Blake 写道:
On 09/01/2011 07:44 AM, Osier Yang wrote:
Libvirt loads the domain conf from status XML if it's running when
starting up. The problem is there is no record of the original conf.
(dom-newDef is NULL here).
So libvirt won't be able to restore the domain conf
On Mon, Sep 19, 2011 at 08:15:18AM -0500, Adam Litke wrote:
On Mon, Sep 19, 2011 at 04:04:04PM +0800, Daniel Veillard wrote:
On Sun, Sep 18, 2011 at 09:37:22AM -0500, Adam Litke wrote:
Hum, I wonder if remoteRelayDomainEventBlockJob shouldn't strdup the
path string instead of using it
On Fri, 16 Sep 2011, Daniel Veillard wrote:
I don't think there's anything controversial here. I suspect the main
concern is that these fields are named in a sufficiently general fashion
so that they can be used for other backend storage types.
I guess the best is to start with the
The current support for qemu and Ceph RBD (rados block device) has two
main deficiencies: authentication doesn't work, and it relies on
environment variables (which don't work with latest upstream). This
patch set addresses both those problems.
The first two patches update the xml schemas and
The qemu RBD driver needs access to the conn in order to get the secret
needed for connecting to the ceph cluster.
Signed-off-by: Sage Weil s...@newdream.net
---
src/qemu/qemu_command.c |5 +++--
src/qemu/qemu_command.h |3 ++-
src/qemu/qemu_driver.c | 17 -
Add a new secret type to store a Ceph authentication key. The ceph_id
field contains the name of the key (e.g. 'admin' for the ceph superuser).
Signed-off-by: Sage Weil s...@newdream.net
---
docs/schemas/secret.rng | 17 +++
include/libvirt/libvirt.h.in |3 ++
Add additional fields to let you specify the how to authenticate with a
network disk type. The authId is the name to authenticate as, and the
authDomain optionally describes the domain that user exists in. The latter
allows us to locate a secret in using the libvirt secrets API, as the user
is
This improves the support for qemu rbd devices by adding support for a few
key features (e.g., authentication) and cleaning up the way in which
rbd configuration options are passed to qemu.
And auth member of the disk source xml specifies how librbd should
authenticate. The id property is the
Implement a generic helper to escape a given set of characters with a
leading '\'. Generalizes virBufferEscapeSexpr().
Signed-off-by: Sage Weil s...@newdream.net
---
src/libvirt_private.syms |1 +
src/util/buf.c | 33 ++---
src/util/buf.h |
On Mon, Sep 19, 2011 at 09:23:02AM -0700, Jason Helfman thus spake:
On Mon, Sep 19, 2011 at 11:56:52AM +0800, Daniel Veillard thus spake:
On Sun, Sep 18, 2011 at 08:48:23PM -0700, Jason Helfman wrote:
On Sun, Sep 18, 2011 at 06:55:06PM +0800, Daniel Veillard thus spake:
I have made a third
于 2011年09月20日 12:43, Jason Helfman 写道:
On Mon, Sep 19, 2011 at 09:23:02AM -0700, Jason Helfman thus spake:
On Mon, Sep 19, 2011 at 11:56:52AM +0800, Daniel Veillard thus spake:
On Sun, Sep 18, 2011 at 08:48:23PM -0700, Jason Helfman wrote:
On Sun, Sep 18, 2011 at 06:55:06PM +0800, Daniel
47 matches
Mail list logo