At 04/11/2012 06:47 AM, Eric Blake Wrote:
> Probably in the noise, but this will let us scale more efficiently
> as we recognize ever more qemu events.
>
> * src/qemu/qemu_monitor_json.c (eventHandlers): Sort.
> (eventSearch): New helper function.
> (qemuMonitorJSONIOProcessEvent): Optimize event
Probably in the noise, but this will let us scale more efficiently
as we recognize ever more qemu events.
* src/qemu/qemu_monitor_json.c (eventHandlers): Sort.
(eventSearch): New helper function.
(qemuMonitorJSONIOProcessEvent): Optimize event lookup.
---
In reply to:
https://www.redhat.com/archi
On 11/04/2012, at 8:36 AM, Eric Blake wrote:
>> Thanks for the research. Would you care to help write up a
>> documentation patch that mentions this as yet another client project?
>> If you don't have time, I will probably get around to it later in the week.
>
> Turns out it's an internal use, r
On 04/09/2012 10:25 AM, Eric Blake wrote:
> On 04/09/2012 12:28 AM, Justin Clift wrote:
>> Hey everyone,
>>
>> Just noticed that Etsy uses libvirt:
>>
>>
>> http://codeascraft.etsy.com/2012/03/13/making-it-virtually-easy-to-deploy-on-day-one/
>>
>> Of specific note, there's this bit:
>>
>> "Li
On 04/10/2012 03:01 PM, Laine Stump wrote:
> On 04/09/2012 11:52 PM, Eric Blake wrote:
>> From: Adam Litke
>>
>> Qemu has changed the semantics of the "block_job_cancel" API. The original
>> qed implementation (pretty much only backported to RHEL 6.2 qemu) was
>> synchronous (ie. upon command com
On 04/09/2012 11:52 PM, Eric Blake wrote:
> From: Adam Litke
>
> Qemu has changed the semantics of the "block_job_cancel" API. The original
> qed implementation (pretty much only backported to RHEL 6.2 qemu) was
> synchronous (ie. upon command completion, the operation was guaranteed to
> be comp
On Tue, Apr 10, 2012 at 10:29:59PM +0200, Guido Günther wrote:
> otherwise the build fails with:
>
> $ CCLD conn-test
> ../libvirt-gobject/.libs/libvirt-gobject-1.0.so: undefined reference to
> `gvir_config_domain_new_from_xml@LIBVIRT_GCONFIG_0.0.4'
> ../libvirt-gobject/.libs/libvirt-gobject-1.
On Tue, Apr 10, 2012 at 10:31:55PM +0200, Guido Günther wrote:
> Triggered by http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=667683
> ---
> If this looks correct I'll have a look at libvirt-glib and
> libvirt-gobject too.
Doing this would break ABI. We should just make sure future symbols
are a
Triggered by http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=667683
---
If this looks correct I'll have a look at libvirt-glib and
libvirt-gobject too.
-- Guido
libvirt-gconfig/libvirt-gconfig.sym | 50 +++
1 files changed, 33 insertions(+), 17 deletions(-)
d
otherwise the build fails with:
$ CCLD conn-test
../libvirt-gobject/.libs/libvirt-gobject-1.0.so: undefined reference to
`gvir_config_domain_new_from_xml@LIBVIRT_GCONFIG_0.0.4'
../libvirt-gobject/.libs/libvirt-gobject-1.0.so: undefined reference to
`gvir_set_error@LIBVIRT_GLIB_0.0.4'
../libvir
On 04/10/2012 11:24 AM, Daniel P. Berrange wrote:
>>> As of this[1] qemu email, both commands have been proposed but not yet
>>> incorporated into the tree; in fact, the implementation I tested with
>>> has changed to match this[2] email that suggested a mandatory
>>> 'full':'bool' argument rather
On Tue, Apr 10, 2012 at 01:17:44PM -0400, Laine Stump wrote:
> On 04/09/2012 11:52 PM, Eric Blake wrote:
> > The new block copy storage migration sequence requires both the
> > 'drive-mirror' action in 'transaction' (present if the 'drive-mirror'
> > standalone monitor command also exists) and the
On 04/09/2012 11:52 PM, Eric Blake wrote:
> The new block copy storage migration sequence requires both the
> 'drive-mirror' action in 'transaction' (present if the 'drive-mirror'
> standalone monitor command also exists) and the 'drive-reopen' monitor
> command (it would be nice if that were also
On 04/10/2012 09:05 AM, Jiri Denemark wrote:
> qemuDomainObjEnterMonitor{,WithDriver} should not be called from async
> jobs, only EnterMonitorAsync variant is allowed.
> ---
> src/qemu/qemu_domain.c |6 ++
> 1 files changed, 6 insertions(+), 0 deletions(-)
>
> diff --git a/src/qemu/qemu_
On 04/10/2012 07:45 AM, Jiri Denemark wrote:
> As reported by Daniel Berrangé, we have a huge performance regression
> for virDomainGetInfo() due to the change which makes virDomainEndJob()
> save the XML status file every time it is called. Previous to that
> change, 2000 calls to virDomainGetInfo
I noticed compiler warnings when building for the s390 architecture.
* src/node_device/node_device_udev.c (udevDeviceMonitorStartup):
Mark unused variable.
* src/nodeinfo.c (linuxNodeInfoCPUPopulate): Avoid unused variable.
---
Pushing under the trivial rule.
src/node_device/node_device_udev.c
On 04/10/2012 07:29 AM, Peter Krempa wrote:
> The new safe console handling introduced a possibility to deadlock the
> qemu driver when a new console connection forcibly disconnects a
> previous console stream that belongs to an already closed connection.
>
> The virStreamFree function calls subse
On 04/10/2012 07:42 AM, Jiri Denemark wrote:
> ---
> po/POTFILES.in |1 +
> 1 files changed, 1 insertions(+), 0 deletions(-)
>
> diff --git a/po/POTFILES.in b/po/POTFILES.in
> index 5d5739a..4c49200 100644
> --- a/po/POTFILES.in
> +++ b/po/POTFILES.in
> @@ -1,3 +1,4 @@
> +daemon/libvirtd-conf
qemuDomainObjEnterMonitor{,WithDriver} should not be called from async
jobs, only EnterMonitorAsync variant is allowed.
---
src/qemu/qemu_domain.c |6 ++
1 files changed, 6 insertions(+), 0 deletions(-)
diff --git a/src/qemu/qemu_domain.c b/src/qemu/qemu_domain.c
index 7f1f8ee..4dda2e0 10
In case an API fails with "cannot acquire state change lock", searching
for the API that possibly forgot to end its job is not always easy.
Let's keep track of the job owner and print it out for easier
identification.
---
src/qemu/qemu_domain.c| 53 +++
From: "Daniel P. Berrange"
* src/qemu/qemu_command.c: Wire up -bios with
* tests/qemuxml2argvdata/qemuxml2argv-bios.args,
tests/qemuxml2argvdata/qemuxml2argv-bios.xml: Expand
existing BIOS test case to cover
---
src/qemu/qemu_command.c |9 +
tests/qemuxml2
On 04/10/2012 08:03 AM, Daniel P. Berrange wrote:
> From: "Daniel P. Berrange"
>
> * src/qemu/qemu_command.c: Wire up -bios with
> * tests/qemuxml2argvdata/qemuxml2argv-bios.args,
> tests/qemuxml2argvdata/qemuxml2argv-bios.xml: Expand
> existing BIOS test case to cover
> ---
> src/qemu/qem
On 04/10/2012 02:55 PM, Eric Blake wrote:
On 04/10/2012 06:44 AM, Peter Krempa wrote:
This patch cleans up variables used to store boolean command flags that
are inquired by vshCommandOptBool to use the bool data type instead of
an integer.
Additionaly this patch cleans up flag variables that a
dist/redhat/env_inspect.py: initialize shared connection object
for use in all testcases
generator.py: make use of sharedmod
---
dist/redhat/env_inspect.py | 192
generator.py | 15 +---
2 files c
On 04/10/2012 02:43 PM, Eric Blake wrote:
On 04/10/2012 04:35 AM, Peter Krempa wrote:
The documentation for the flag doesn't clearly state that the flag only
enhances the output and the user needs to specify other flags to list
inactive domains, that are enhanced by this flag.
---
tools/virsh.
sharedmod.py
---
sharedmod.py | 16
1 files changed, 16 insertions(+), 0 deletions(-)
create mode 100644 sharedmod.py
diff --git a/sharedmod.py b/sharedmod.py
new file mode 100644
index 000..8af26d8
--- /dev/null
+++ b/sharedmod.py
@@ -0,0 +1,16 @@
+# This is a module
On 10.04.2012 15:33, Cole Robinson wrote:
> On 04/10/2012 09:32 AM, Stefan Bader wrote:
>> On 10.04.2012 15:22, Cole Robinson wrote:
>>> On 04/10/2012 04:46 AM, Stefan Bader wrote:
On 04/08/2012 03:08 PM, Cole Robinson wrote:
> On 04/02/2012 10:38 AM, Stefan Bader wrote:
>> xend_intern
As reported by Daniel Berrangé, we have a huge performance regression
for virDomainGetInfo() due to the change which makes virDomainEndJob()
save the XML status file every time it is called. Previous to that
change, 2000 calls to virDomainGetInfo() took ~2.5 seconds. After that
change, 2000 calls t
---
po/POTFILES.in |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/po/POTFILES.in b/po/POTFILES.in
index 5d5739a..4c49200 100644
--- a/po/POTFILES.in
+++ b/po/POTFILES.in
@@ -1,3 +1,4 @@
+daemon/libvirtd-config.c
daemon/libvirtd.c
daemon/qemu_dispatch.h
daemon/remote.c
dist/redhat/env_inspect.py: initialize shared connection object
for use in all testcases
generator.py: make use of sharedmod
---
dist/redhat/env_inspect.py | 192
generator.py | 15 +---
2 files c
sharedmod.py
---
sharedmod.py | 16
1 files changed, 16 insertions(+), 0 deletions(-)
create mode 100644 sharedmod.py
diff --git a/sharedmod.py b/sharedmod.py
new file mode 100644
index 000..8af26d8
--- /dev/null
+++ b/sharedmod.py
@@ -0,0 +1,16 @@
+# This is a module
On 04/10/2012 09:32 AM, Stefan Bader wrote:
> On 10.04.2012 15:22, Cole Robinson wrote:
>> On 04/10/2012 04:46 AM, Stefan Bader wrote:
>>> On 04/08/2012 03:08 PM, Cole Robinson wrote:
On 04/02/2012 10:38 AM, Stefan Bader wrote:
> xend_internal: Use domain/status for shutdown check
>
>>
On 10.04.2012 15:22, Cole Robinson wrote:
> On 04/10/2012 04:46 AM, Stefan Bader wrote:
>> On 04/08/2012 03:08 PM, Cole Robinson wrote:
>>> On 04/02/2012 10:38 AM, Stefan Bader wrote:
xend_internal: Use domain/status for shutdown check
On newer xend (v3.x and after) there is no state
The new safe console handling introduced a possibility to deadlock the
qemu driver when a new console connection forcibly disconnects a
previous console stream that belongs to an already closed connection.
The virStreamFree function calls subsequently a the virReleaseConnect
function that tries to
On 04/10/2012 04:46 AM, Stefan Bader wrote:
> On 04/08/2012 03:08 PM, Cole Robinson wrote:
>> On 04/02/2012 10:38 AM, Stefan Bader wrote:
>>> xend_internal: Use domain/status for shutdown check
>>>
>>> On newer xend (v3.x and after) there is no state and domid reported
>>> for inactive domains. Whe
On 04/10/2012 06:44 AM, Peter Krempa wrote:
> This patch cleans up variables used to store boolean command flags that
> are inquired by vshCommandOptBool to use the bool data type instead of
> an integer.
>
> Additionaly this patch cleans up flag variables that are inferred from
s/Additionaly/Add
This patch cleans up variables used to store boolean command flags that
are inquired by vshCommandOptBool to use the bool data type instead of
an integer.
Additionaly this patch cleans up flag variables that are inferred from
existing flags.
---
tools/virsh.c | 120 --
On 04/10/2012 04:35 AM, Peter Krempa wrote:
> The documentation for the flag doesn't clearly state that the flag only
> enhances the output and the user needs to specify other flags to list
> inactive domains, that are enhanced by this flag.
> ---
> tools/virsh.c |2 +-
> tools/virsh.pod |
On 04/10/2012 07:28 PM, Martin Kletzander wrote:
On 04/06/2012 11:14 AM, Guannan Ren wrote:
sharedmod.py: in root directory
---
sharedmod.py | 13 +
1 files changed, 13 insertions(+), 0 deletions(-)
create mode 100644 sharedmod.py
diff --git a/sharedmod.py b/sharedmod.py
On Tue, Apr 10, 2012 at 02:12:01PM +0200, Jiri Denemark wrote:
> Without this, libvirtdconftest fails to build with "fatal error:
> daemon/libvirtd-config.h: No such file or directory"
> ---
> tests/Makefile.am |1 +
> 1 files changed, 1 insertions(+), 0 deletions(-)
>
> diff --git a/tests/Ma
Hey,
we're running a VM-pool based on libvirt and QEMU/KVM. The host machines
run debian 6 (squeeze). All VM's have a similar libvirt XML-definition.
Since we've moved to the combination of Linux kernel 3.2.0-1 and libvirt
0.9.9 (installed from testing/wheezy repo's), we periodically notice
p
Hi Alex,
I am stuck here too. Where you able to resolve the issue? I tried installing
libdevmapper-devel it gave dependency error for libdevmapper and then that gave
dependency error of libc.so.6.
--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/lib
On Tue, Apr 10, 2012 at 12:16:19PM +0100, Daniel P. Berrange wrote:
> From: "Daniel P. Berrange"
>
> ---
> src/rpc/virnettlscontext.c | 10 +++---
> 1 files changed, 7 insertions(+), 3 deletions(-)
>
> diff --git a/src/rpc/virnettlscontext.c b/src/rpc/virnettlscontext.c
> index 74a61e0..7
Without this, libvirtdconftest fails to build with "fatal error:
daemon/libvirtd-config.h: No such file or directory"
---
tests/Makefile.am |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/tests/Makefile.am b/tests/Makefile.am
index 57358e9..c4d550f 100644
--- a/tests/Makef
On 04/06/2012 11:14 AM, Guannan Ren wrote:
> sharedmod.py: in root directory
> ---
> sharedmod.py | 13 +
> 1 files changed, 13 insertions(+), 0 deletions(-)
> create mode 100644 sharedmod.py
>
> diff --git a/sharedmod.py b/sharedmod.py
> new file mode 100644
> index 000..f
From: "Daniel P. Berrange"
---
src/rpc/virnettlscontext.c | 10 +++---
1 files changed, 7 insertions(+), 3 deletions(-)
diff --git a/src/rpc/virnettlscontext.c b/src/rpc/virnettlscontext.c
index 74a61e0..7440c7a 100644
--- a/src/rpc/virnettlscontext.c
+++ b/src/rpc/virnettlscontext.c
@@ -
On Tue, Apr 10, 2012 at 12:08:14PM +0100, Richard W.M. Jones wrote:
> On Mon, Apr 09, 2012 at 10:19:38AM -0600, Eric Blake wrote:
> > On 04/07/2012 03:33 AM, Richard W.M. Jones wrote:
> >
> > > However the above commit is later amended by this commit:
> > >
> > > -
On Mon, Apr 09, 2012 at 10:19:38AM -0600, Eric Blake wrote:
> On 04/07/2012 03:33 AM, Richard W.M. Jones wrote:
>
> > However the above commit is later amended by this commit:
> >
> > --
> > commit eaddec976ef06457fee4a4ce86b8c
The documentation for the flag doesn't clearly state that the flag only
enhances the output and the user needs to specify other flags to list
inactive domains, that are enhanced by this flag.
---
tools/virsh.c |2 +-
tools/virsh.pod |5 +++--
2 files changed, 4 insertions(+), 3 deletions
On 04/09/2012 12:23 PM, Eric Blake wrote:
On 04/06/2012 01:13 PM, Stefan Berger wrote:
Below code failed to compile on a 32 bit machine with error
typewrappers.c: In function 'libvirt_intUnwrap':
typewrappers.c:135:5: error: logical 'and' of mutually exclusive tests
is always false [-Werror=log
On Fri, Apr 06, 2012 at 11:05:02AM -0600, Eric Blake wrote:
> On 04/04/2012 08:07 AM, Daniel P. Berrange wrote:
> > +
> > +//VIR_DEBUG("New config [%s]", newdata);
>
> Did you mean to leave this commented?
Yes, it makes the debug output far too verbose.
Daniel
--
|: http://berrange.com
On Tue, Apr 10, 2012 at 11:58:52AM +0800, Daniel Veillard wrote:
>
> I realize that we have that behaviour for quite some times but I
> wonder about it, basically we always dump an usb controller on the
> XML domain format:
>
>
>
> even if there is no USB device connected to the machine. Th
---
src/qemu/qemu_monitor.c | 36
src/qemu/qemu_monitor.h | 13 +
src/qemu/qemu_monitor_json.c | 42 ++
src/qemu/qemu_monitor_json.h |7 +++
4 files changed, 98 insertions(+), 0 deletio
On 04/08/2012 03:08 PM, Cole Robinson wrote:
> On 04/02/2012 10:38 AM, Stefan Bader wrote:
>> xend_internal: Use domain/status for shutdown check
>>
>> On newer xend (v3.x and after) there is no state and domid reported
>> for inactive domains. When initially creating connections this is
>> handled
On 04/08/2012 03:19 PM, Cole Robinson wrote:
> On 04/02/2012 10:38 AM, Stefan Bader wrote:
>> xen-hypervisor: GetVcpus may not find a certain domain
>>
>> Observed on connections that have been running and then shut
>> down. The hypervisor subdriver fills the log with internal
>> errors while the x
---
src/qemu/qemu_domain.c |1 +
src/qemu/qemu_domain.h |1 +
src/qemu/qemu_driver.c | 42 --
3 files changed, 42 insertions(+), 2 deletions(-)
diff --git a/src/qemu/qemu_domain.c b/src/qemu/qemu_domain.c
index 69d9e6e..e3a668a 100644
--- a/src/qe
Currently, we use migrate to dump guest's memory. There is one
restriction in migrate command: the device's status should be
stored in qemu because the device's status should be passed to
target machine.
If we passthrough a host device to guest, the device's status
is stored in the real device. So
IMHO there is not any benifit to use class in a utils script,
except you have to construct the object again and again in
scripts. :-)
Incidental cleanups:
* s/parser_uri/parse_uri/
* s/#this/# This/
* Useless comments on the top are removed.
---
utils/utils.py | 769 +++
These five patches are to change utils.py into a collection of functions.
And cleanup all of existing testcase to use the functions in it directly.
(1) Destroy Utils class definition
(2) Remove "util = utils.Utils()"
(3) Substitue "util." with "utils."
(4) Remove useless utils import "from utils
$ for i in $(find . -type f -name "*.py"); do \
sed -i -e '/util *= *utils\.Utils()/d' $i; \
done
---
repos/domain/attach_disk.py|1 -
repos/domain/attach_interface.py |1 -
repos/domain/autostart.py |1 -
repos
---
repos/domain/autostart.py|1 -
repos/domain/blkstats.py |1 -
repos/domain/console_mutex.py|1 -
repos/domain/domain_blkinfo.py |1 -
repos/domain/domain_id.py|1 -
repos/domain/domain_uuid.py
$ for i in $(find . -type f -name "*.py"); do \
sed -i -e 's/utils\.Utils()/utils/g' $i; \
done
---
repos/domain/cpu_affinity.py |2 +-
utils/xmlgenerator.py| 12 ++--
2 files changed, 7 insertions(+), 7 deletions(-)
diff --git a/repos/domain/cpu_affinity.py b/repos/
62 matches
Mail list logo