On 04/22/10 20:47, Anthony Liguori wrote:
On 04/12/2010 07:23 AM, Jamie Lokier wrote:
Some simple but versatile hook ideas:
-emulator-append-option (no space splitting, one option,
appended)
-emulator-append-options (space splitting multiple options)
-emulator-prepend-option
On Thu, Apr 22, 2010 at 11:41:04PM +0900, Satoru SATOH wrote:
The following series of patches is a prototype implementation of dnsmasq
module.
It implements an idea to save dhcp hosts' macaddr vs. ipaddr mappings to
static file and make dnsmasq loading it with --dhcp-hostsfile option,
Satoru SATOH wrote:
...
+static int
+hostsfileWrite(const char *path,
+ dnsmasqDhcpHost *hosts,
+ unsigned int nhosts)
+{
+char *tmp;
+FILE *f;
+bool istmp = true;
+unsigned int i;
+int saved_errno;
+
+if (nhosts == 0 unlink(path) ==
On Thu, Apr 22, 2010 at 01:45:27PM -0500, Anthony Liguori wrote:
On 04/09/2010 09:27 AM, Daniel P. Berrange wrote:
On Fri, Apr 09, 2010 at 09:41:39AM -0400, Chris Lalancette wrote:
domain type='kvm'
namemyguest/name
...
debug
monitorpassthrough/
commandline
On Thu, Apr 22, 2010 at 01:47:55PM -0500, Anthony Liguori wrote:
On 04/12/2010 07:23 AM, Jamie Lokier wrote:
Daniel Veillard wrote:
On Sun, Apr 11, 2010 at 11:17:38PM +0100, Jamie Lokier wrote:
It's not that hard to write this for trivial extra options:
emulator/bin/sh -c
In cases where the security driver failed to restore a label after a
guest has saved, we mistakenly jumped to the error cleanup paths.
This is not good, because the operation has in fact completed and
cannot be rolled back completely. Label restore is non-critical, so
just log the problem instead.
On Fri, Apr 23, 2010 at 11:49:38AM +0100, Daniel P. Berrange wrote:
In cases where the security driver failed to restore a label after a
guest has saved, we mistakenly jumped to the error cleanup paths.
This is not good, because the operation has in fact completed and
cannot be rolled back
On 04/22/2010 09:49 PM, Anthony Liguori wrote:
real API. Say, adding a device libvirt doesn't know about or stopping
the VM
while libvirt thinks it's still running or anything like that.
Another problem is issuing Monitor commands that could confuse
libvirt's
We need to make libvirt and
On 04/23/2010 05:28 AM, Daniel P. Berrange wrote:
On Thu, Apr 22, 2010 at 01:45:27PM -0500, Anthony Liguori wrote:
On 04/09/2010 09:27 AM, Daniel P. Berrange wrote:
On Fri, Apr 09, 2010 at 09:41:39AM -0400, Chris Lalancette wrote:
domain type='kvm'
namemyguest/name
On 04/23/2010 07:48 AM, Avi Kivity wrote:
On 04/22/2010 09:49 PM, Anthony Liguori wrote:
real API. Say, adding a device libvirt doesn't know about or
stopping the VM
while libvirt thinks it's still running or anything like that.
Another problem is issuing Monitor commands that could confuse
On 04/22/2010 02:15 PM, Eric Blake wrote:
I'm also pushing a followup patch to update AUTHORS with seven recent
patch contributors (to avoid posting raw email addresses for spammers to
harvest, I won't post that patch to the list unless specifically
requested). I think that qualifies as
On Fri, Apr 23, 2010 at 08:40:49AM -0500, Anthony Liguori wrote:
On 04/23/2010 05:28 AM, Daniel P. Berrange wrote:
On Thu, Apr 22, 2010 at 01:45:27PM -0500, Anthony Liguori wrote:
On 04/09/2010 09:27 AM, Daniel P. Berrange wrote:
On Fri, Apr 09, 2010 at 09:41:39AM -0400, Chris
On 04/23/2010 04:48 PM, Anthony Liguori wrote:
On 04/23/2010 07:48 AM, Avi Kivity wrote:
On 04/22/2010 09:49 PM, Anthony Liguori wrote:
real API. Say, adding a device libvirt doesn't know about or
stopping the VM
while libvirt thinks it's still running or anything like that.
Another problem
Daniel, thanks for the comments .. most of them are integrated, but I
have a problem with changing MAC addresses, because the domain xml is
regenerated every time, so I cannot rely on the interface name being
eth0 as fedora during boot creates a new ethX for each new MAC
address .. any good
On Fri, Apr 23, 2010 at 08:48:51AM -0500, Anthony Liguori wrote:
On 04/23/2010 07:48 AM, Avi Kivity wrote:
On 04/22/2010 09:49 PM, Anthony Liguori wrote:
real API. Say, adding a device libvirt doesn't know about or
stopping the VM
while libvirt thinks it's still running or anything like
I am extending the schema with the recently added connlimit-above
attribute and adding a test case to the test suite.
Signed-off-by: Stefan Berger stef...@us.ibm.com
---
docs/schemas/nwfilter.rng |5 +
tests/nwfilterxml2xmlin/conntrack-test.xml | 14 ++
On Fri, Apr 23, 2010 at 05:24:34PM +0300, Avi Kivity wrote:
On 04/23/2010 04:48 PM, Anthony Liguori wrote:
On 04/23/2010 07:48 AM, Avi Kivity wrote:
On 04/22/2010 09:49 PM, Anthony Liguori wrote:
real API. Say, adding a device libvirt doesn't know about or
stopping the VM
while libvirt
On 04/23/2010 08:34 AM, Stefan Berger wrote:
I am extending the schema with the recently added connlimit-above
attribute and adding a test case to the test suite.
Signed-off-by: Stefan Berger stef...@us.ibm.com
---
docs/schemas/nwfilter.rng |5 +
On 04/22/2010 12:32 PM, David Allan wrote:
* Fix for the bug reported at:
https://bugzilla.redhat.com/show_bug.cgi?id=573908
---
src/conf/storage_encryption_conf.c |6 +++---
1 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/src/conf/storage_encryption_conf.c
On Fri, Apr 23, 2010 at 08:41:54AM -0600, Eric Blake wrote:
On 04/22/2010 12:32 PM, David Allan wrote:
* Fix for the bug reported at:
https://bugzilla.redhat.com/show_bug.cgi?id=573908
---
src/conf/storage_encryption_conf.c |6 +++---
1 files changed, 3 insertions(+), 3
On Fri, Apr 23, 2010 at 03:52:41PM +0100, Daniel P. Berrange wrote:
On Fri, Apr 23, 2010 at 08:41:54AM -0600, Eric Blake wrote:
On 04/22/2010 12:32 PM, David Allan wrote:
* Fix for the bug reported at:
https://bugzilla.redhat.com/show_bug.cgi?id=573908
---
Anthony Liguori anth...@codemonkey.ws writes:
On 04/23/2010 07:48 AM, Avi Kivity wrote:
On 04/22/2010 09:49 PM, Anthony Liguori wrote:
real API. Say, adding a device libvirt doesn't know about or
stopping the VM
while libvirt thinks it's still running or anything like that.
Another
On Fri, Apr 23, 2010 at 12:18:11PM -0300, Luiz Capitulino wrote:
It's not needed and is currently ignored, but this is a bug.
It will get fixed soon and QMP will return an error for keys
it doesn't know about, this will break libvirt.
Signed-off-by: Luiz Capitulino lcapitul...@redhat.com
Eric Blake ebl...@redhat.com wrote on 04/23/2010 10:40:29 AM:
On 04/23/2010 08:34 AM, Stefan Berger wrote:
I am extending the schema with the recently added connlimit-above
attribute and adding a test case to the test suite.
[...]
+attribute name=connlimit-above
+
On Fri, Apr 23, 2010 at 12:18:11PM -0300, Luiz Capitulino wrote:
It's not needed and is currently ignored, but this is a bug.
It will get fixed soon and QMP will return an error for keys
it doesn't know about, this will break libvirt.
Signed-off-by: Luiz Capitulino lcapitul...@redhat.com
Under cygwin, winsock2.h is intentionally incompatible with,
sys/socket.h, and checking for existence is wrong.
Under mingw, HAVE_WINSOCK2_H is defined on our behalf by
gnulib, in a way that does not interfere with cygwin.
* configure.ac: Drop unnecessary header check.
Reported by Matthias
Cygwin has mntent.h but lacks getmntent_r. Update preprocessor
checks to catch this combination.
---
src/qemu/qemu_conf.c |2 +-
src/util/cgroup.c| 10 +-
src/util/util.c |8
3 files changed, 10 insertions(+), 10 deletions(-)
diff --git a/src/qemu/qemu_conf.c
---
configure.ac |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/configure.ac b/configure.ac
index 99bc906..e34f417 100644
--- a/configure.ac
+++ b/configure.ac
@@ -528,7 +528,7 @@ dnl check for kernel headers required by src/bridge.c
dnl
if test $with_qemu = yes ||
Also move the equivalent checks for LXC and UML before their header
checks. This way configure doesn't check for the headers when the driver
gets disabled anyway.
---
configure.ac | 15 +--
1 files changed, 9 insertions(+), 6 deletions(-)
diff --git a/configure.ac b/configure.ac
---
src/Makefile.am |7 ++-
src/libvirt_nwfilter.syms | 14 ++
src/libvirt_private.syms | 13 +
3 files changed, 21 insertions(+), 13 deletions(-)
create mode 100644 src/libvirt_nwfilter.syms
diff --git a/src/Makefile.am b/src/Makefile.am
index
The defined __sun is there, because inside_daemon is used in xenUnifiedOpen
if __sun is defined. Also initialize it to 0.
---
src/xen/xen_driver.c |4 +++-
1 files changed, 3 insertions(+), 1 deletions(-)
diff --git a/src/xen/xen_driver.c b/src/xen/xen_driver.c
index 5ab169d..b16a16a 100644
2010/4/23 Eric Blake ebl...@redhat.com:
Under cygwin, winsock2.h is intentionally incompatible with,
sys/socket.h, and checking for existence is wrong.
Under mingw, HAVE_WINSOCK2_H is defined on our behalf by
gnulib, in a way that does not interfere with cygwin.
* configure.ac: Drop
On 04/23/2010 10:57 AM, Matthias Bolte wrote:
---
configure.ac |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/configure.ac b/configure.ac
index 99bc906..e34f417 100644
--- a/configure.ac
+++ b/configure.ac
@@ -528,7 +528,7 @@ dnl check for kernel headers
On 04/23/2010 11:02 AM, Matthias Bolte wrote:
* configure.ac: Drop unnecessary header check.
Reported by Matthias Bolte.
---
-AC_CHECK_HEADERS([pwd.h paths.h regex.h sys/syslimits.h sys/utsname.h
sys/wait.h winsock2.h sched.h termios.h sys/poll.h syslog.h mntent.h
net/ethernet.h])
On 04/23/2010 10:58 AM, Matthias Bolte wrote:
Also move the equivalent checks for LXC and UML before their header
checks. This way configure doesn't check for the headers when the driver
gets disabled anyway.
ACK.
--
Eric Blake ebl...@redhat.com+1-801-349-2682
Libvirt virtualization
On 04/23/2010 11:00 AM, Matthias Bolte wrote:
The defined __sun is there, because inside_daemon is used in xenUnifiedOpen
if __sun is defined. Also initialize it to 0.
---
src/xen/xen_driver.c |4 +++-
1 files changed, 3 insertions(+), 1 deletions(-)
diff --git a/src/xen/xen_driver.c
On 04/23/2010 11:00 AM, Matthias Bolte wrote:
---
src/Makefile.am |7 ++-
src/libvirt_nwfilter.syms | 14 ++
src/libvirt_private.syms | 13 +
3 files changed, 21 insertions(+), 13 deletions(-)
create mode 100644 src/libvirt_nwfilter.syms
On 04/23/2010 11:00 AM, Dave Allan wrote:
On Fri, Apr 23, 2010 at 03:52:41PM +0100, Daniel P. Berrange wrote:
On Fri, Apr 23, 2010 at 08:41:54AM -0600, Eric Blake wrote:
On 04/22/2010 12:32 PM, David Allan wrote:
* Fix for the bug reported at:
On 04/23/2010 11:12 AM, Eric Blake wrote:
On 04/23/2010 11:02 AM, Matthias Bolte wrote:
* configure.ac: Drop unnecessary header check.
Reported by Matthias Bolte.
---
-AC_CHECK_HEADERS([pwd.h paths.h regex.h sys/syslimits.h sys/utsname.h
sys/wait.h winsock2.h sched.h termios.h sys/poll.h
In particular I was forgetting to take the qemuMonitorPrivatePtr
lock (via qemuDomainObjBeginJob), which would cause problems
if two users tried to access the same domain at the same time.
This patch also fixes a problem where I was forgetting to remove
a transient domain from the list of domains.
Hello,
This series of patches fixes up a number of problems discovered
while testing the snapshot code. Descriptions of the fixes are
in the individual patches. Please review.
Thanks,
Chris Lalancette
--
libvir-list mailing list
libvir-list@redhat.com
Signed-off-by: Chris Lalancette clala...@redhat.com
---
src/qemu/qemu_driver.c |4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c
index 7314328..b0c1877 100644
--- a/src/qemu/qemu_driver.c
+++ b/src/qemu/qemu_driver.c
@@
While running libvirtd under valgrind and doing some
snapshot testing I noticed that we would always leak a
connection reference. The problem was actually that we
were leaking a domain reference in the libvirtd remote
snapshot code, which was in turn causing a leaked
connection reference. Fix
We were forgetting to release the memory allocated by
virDomainSnapshotListNames. Free the memory properly.
Signed-off-by: Chris Lalancette clala...@redhat.com
---
tools/virsh.c |4 +++-
1 files changed, 3 insertions(+), 1 deletions(-)
diff --git a/tools/virsh.c b/tools/virsh.c
index
Signed-off-by: Chris Lalancette clala...@redhat.com
---
src/conf/domain_conf.c |3 ++-
src/conf/domain_conf.h |1 -
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c
index 43d74cf..6dc26c3 100644
--- a/src/conf/domain_conf.c
+++
While doing some testing of the snapshot code I noticed that
if qemuDomainSnapshotLoad failed, it would print a NULL as
part of the error. That's not desirable, so leave the
full_path variable around until after we are done printing
errors.
Signed-off-by: Chris Lalancette clala...@redhat.com
---
We were freeing the virDomainSnapshotDefPtr, but not
the virDomainSnapshotObjPtr in virDomainSnapshotObjFree.
Signed-off-by: Chris Lalancette clala...@redhat.com
---
src/conf/domain_conf.c |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/src/conf/domain_conf.c
Signed-off-by: Chris Lalancette clala...@redhat.com
---
src/conf/domain_conf.c |3 +--
1 files changed, 1 insertions(+), 2 deletions(-)
diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c
index 6dc26c3..139712a 100644
--- a/src/conf/domain_conf.c
+++ b/src/conf/domain_conf.c
@@
2010/4/23 Chris Lalancette clala...@redhat.com:
While running libvirtd under valgrind and doing some
snapshot testing I noticed that we would always leak a
connection reference. The problem was actually that we
were leaking a domain reference in the libvirtd remote
snapshot code, which was
2010/4/23 Chris Lalancette clala...@redhat.com:
Signed-off-by: Chris Lalancette clala...@redhat.com
---
src/conf/domain_conf.c | 3 ++-
src/conf/domain_conf.h | 1 -
2 files changed, 2 insertions(+), 2 deletions(-)
ACK.
Matthias
--
libvir-list mailing list
libvir-list@redhat.com
2010/4/23 Chris Lalancette clala...@redhat.com:
Signed-off-by: Chris Lalancette clala...@redhat.com
---
src/conf/domain_conf.c | 3 +--
1 files changed, 1 insertions(+), 2 deletions(-)
diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c
index 6dc26c3..139712a 100644
---
2010/4/23 Chris Lalancette clala...@redhat.com:
We were freeing the virDomainSnapshotDefPtr, but not
the virDomainSnapshotObjPtr in virDomainSnapshotObjFree.
Signed-off-by: Chris Lalancette clala...@redhat.com
---
src/conf/domain_conf.c | 1 +
1 files changed, 1 insertions(+), 0
On 04/23/2010 10:57 AM, Matthias Bolte wrote:
Cygwin has mntent.h but lacks getmntent_r. Update preprocessor
checks to catch this combination.
Ultimately, cygwin might be taught to implement getmntent_r, so this may
have to be revisited in the future. But that said,
+++
2010/4/23 Chris Lalancette clala...@redhat.com:
While doing some testing of the snapshot code I noticed that
if qemuDomainSnapshotLoad failed, it would print a NULL as
part of the error. That's not desirable, so leave the
full_path variable around until after we are done printing
errors.
2010/4/23 Chris Lalancette clala...@redhat.com:
We were forgetting to release the memory allocated by
virDomainSnapshotListNames. Free the memory properly.
Signed-off-by: Chris Lalancette clala...@redhat.com
---
tools/virsh.c | 4 +++-
1 files changed, 3 insertions(+), 1 deletions(-)
2010/4/23 Eric Blake ebl...@redhat.com:
On 04/23/2010 10:58 AM, Matthias Bolte wrote:
Also move the equivalent checks for LXC and UML before their header
checks. This way configure doesn't check for the headers when the driver
gets disabled anyway.
ACK.
Thanks, pushed.
Matthias
--
2010/4/23 Eric Blake ebl...@redhat.com:
On 04/23/2010 11:00 AM, Matthias Bolte wrote:
The defined __sun is there, because inside_daemon is used in xenUnifiedOpen
if __sun is defined. Also initialize it to 0.
---
src/xen/xen_driver.c | 4 +++-
1 files changed, 3 insertions(+), 1
2010/4/23 Eric Blake ebl...@redhat.com:
On 04/23/2010 10:57 AM, Matthias Bolte wrote:
Cygwin has mntent.h but lacks getmntent_r. Update preprocessor
checks to catch this combination.
Ultimately, cygwin might be taught to implement getmntent_r, so this may
have to be revisited in the future.
On 04/23/2010 01:57 PM, Matthias Bolte wrote:
2010/4/23 Chris Lalancette clala...@redhat.com:
While running libvirtd under valgrind and doing some
snapshot testing I noticed that we would always leak a
connection reference. The problem was actually that we
were leaking a domain reference in
On 04/23/2010 02:08 PM, Matthias Bolte wrote:
2010/4/23 Chris Lalancette clala...@redhat.com:
We were freeing the virDomainSnapshotDefPtr, but not
the virDomainSnapshotObjPtr in virDomainSnapshotObjFree.
Signed-off-by: Chris Lalancette clala...@redhat.com
---
src/conf/domain_conf.c |1
On 04/23/2010 01:59 PM, Matthias Bolte wrote:
2010/4/23 Chris Lalancette clala...@redhat.com:
Signed-off-by: Chris Lalancette clala...@redhat.com
---
src/conf/domain_conf.c |3 ++-
src/conf/domain_conf.h |1 -
2 files changed, 2 insertions(+), 2 deletions(-)
ACK.
Thanks,
On 04/23/2010 02:10 PM, Matthias Bolte wrote:
2010/4/23 Chris Lalancette clala...@redhat.com:
While doing some testing of the snapshot code I noticed that
if qemuDomainSnapshotLoad failed, it would print a NULL as
part of the error. That's not desirable, so leave the
full_path variable
On 04/23/2010 09:24 AM, Avi Kivity wrote:
On 04/23/2010 04:48 PM, Anthony Liguori wrote:
On 04/23/2010 07:48 AM, Avi Kivity wrote:
On 04/22/2010 09:49 PM, Anthony Liguori wrote:
real API. Say, adding a device libvirt doesn't know about or
stopping the VM
while libvirt thinks it's still
On 04/23/2010 09:21 AM, Daniel P. Berrange wrote:
Say libvirt is running a 'offline core dump' operation. This consists of
us invoking
stop
migrate exec:cat foo.dump
cont
I don't want other debug commands accidentally being issued in between
these steps. These 3 commands are in
Thanks again for your advice!
On Fri, Apr 23, 2010 at 10:42:14AM +0200, Jim Meyering wrote:
Satoru SATOH wrote:
...
+static int
+hostsfileWrite(const char *path,
+ dnsmasqDhcpHost *hosts,
+ unsigned int nhosts)
+{
+char *tmp;
+FILE *f;
+
Hi there,
I have been playing with libvirt/KVM on saving the VM to a checkpoint file and
restoring it from this file. It has worked very well for me on my RHEL-5.5
setup, until I added hot-plugged volumes.
If I add a volume to a running VM using virsh attach-disk and then call
virsh save. The
Here is updated version of the patch adds the files implements dnsmasq
(hostsfile) module [1] based on advices from Jim-san.
[1] https://www.redhat.com/archives/libvir-list/2010-April/msg01003.html
[2] https://www.redhat.com/archives/libvir-list/2010-April/msg01046.html
Signed-off-by: Satoru
On Fri, Apr 23, 2010 at 09:33:20AM +0200, Daniel Veillard wrote:
On Thu, Apr 22, 2010 at 11:41:04PM +0900, Satoru SATOH wrote:
The following series of patches is a prototype implementation of dnsmasq
module.
It implements an idea to save dhcp hosts' macaddr vs. ipaddr mappings to
Satoru SATOH wrote:
Here is updated version of the patch adds the files implements dnsmasq
(hostsfile) module [1] based on advices from Jim-san.
[1] https://www.redhat.com/archives/libvir-list/2010-April/msg01003.html
[2] https://www.redhat.com/archives/libvir-list/2010-April/msg01046.html
This patch adds support for the RARP protocol. This may be needed due to
qemu sending out a RARP packet (at least that's what it seems to want to
do even though the protocol id is wrong) when migration finishes and
we'd need a rule to let the packets pass.
Unfortunately my installation of
This patch adds a test case for the RARP protocol.
Signed-off-by: Stefan Berger stef...@us.ibm.com
---
tests/nwfilterxml2xmlin/rarp-test.xml | 33 +
tests/nwfilterxml2xmlout/rarp-test.xml | 18 ++
tests/nwfilterxml2xmltest.c|1
The following patch fixes the storage XML indentation problem caused by my
earlier patch to fix domain XML indentation problems. I liked the style of
Laine's suggestion of passing in the number of spaces to indent, so I adopted
that approach. I also added a regression test for the domain XML.
On 04/22/2010 07:43 AM, Daniel P. Berrange wrote:
It is possible to use block devices with domain save/restore. Upon
failure QEMU unlinks the path being saved to. This isn't good when
it is a block device !
* src/qemu/qemu_driver.c: Don't unlink block devices if save fails
---
On 04/23/2010 06:49 AM, Daniel P. Berrange wrote:
In cases where the security driver failed to restore a label after a
guest has saved, we mistakenly jumped to the error cleanup paths.
This is not good, because the operation has in fact completed and
cannot be rolled back completely. Label
74 matches
Mail list logo