[libvirt] [PATCH 0/4] Fix up the mess in commandtest
by Jiri Denemark
commandtest is quite messy with lots of memory leaks, some filedescriptor leaks
and few other small issues. This patches fixes them.
Jiri Denemark (4):
tests: Fix code formating in commandtest
tests: Fix leaks in commandtest
tests: Fix commandtest in VPATH build
tests: Fix detection of expected error
tests/commandtest.c | 282 +++++++++++++++++++++++++++++++++------------------
1 files changed, 184 insertions(+), 98 deletions(-)
--
1.7.3.2
13 years, 11 months
[libvirt] [PATCH] virsh: Remove redundant commands in group domain
by Osier Yang
"echo" is already in group "virsh", "freecell" and "hostname" are
already in group "host", so remove them from group "domain"
* tools/virsh.c
---
tools/virsh.c | 3 ---
1 files changed, 0 insertions(+), 3 deletions(-)
diff --git a/tools/virsh.c b/tools/virsh.c
index 441fd77..9ce3202 100644
--- a/tools/virsh.c
+++ b/tools/virsh.c
@@ -9956,10 +9956,7 @@ static const vshCmdDef domManagementCmds[] = {
{"domxml-to-native", cmdDomXMLToNative, opts_domxmltonative, info_domxmltonative},
{"dump", cmdDump, opts_dump, info_dump},
{"dumpxml", cmdDumpXML, opts_dumpxml, info_dumpxml},
- {"echo", cmdEcho, opts_echo, info_echo},
{"edit", cmdEdit, opts_edit, info_edit},
- {"freecell", cmdFreecell, opts_freecell, info_freecell},
- {"hostname", cmdHostname, NULL, info_hostname},
{"managedsave", cmdManagedSave, opts_managedsave, info_managedsave},
{"managedsave-remove", cmdManagedSaveRemove, opts_managedsaveremove, info_managedsaveremove},
{"maxvcpus", cmdMaxvcpus, opts_maxvcpus, info_maxvcpus},
--
1.7.3.2
13 years, 11 months
[libvirt] RFC: Setting a permanent MAC address for virtual network bridges
by Laine Stump
This is in reference to the following bugs that are filed against RHEL,
but applicable to all Linux distros:
https://bugzilla.redhat.com/show_bug.cgi?id=560994
https://bugzilla.redhat.com/show_bug.cgi?id=609463
"MAC address of virtual network bridge changes when TAP devices are
added/removed"
The problem is that the MAC address of a bridge that hasn't had its
address specifically set, will be set to the MAC address of the first
interface attached to it. If that interface later goes away, it will be
set to the MAC address of another attached interface.
Daniel's comments in the bug 1) state that the MAC address of a bridge
device must match the MAC address of one of the attached interfaces, or
no traffic will pass, and 2) suggest creating a dummy interface that is
created and used purely to set a fixed MAC address that won't go away
when any guest is destroyed (because it's not associated with any guest.
I did some quick experimentation and found that:
1) It is possible to set a bridge's MAC address to something that isn't
the address of an attached interface (although the range of numbers
seems to be limited, at least when setting it using "ip link set virbr0
address xx:xx:xx:xx:xx:xx - anything starting with less than "B2" was
rejected).
2) When a bridge's MAC address is set in that way, as Daniel says, no
traffic passes. (Why is that, anyway?)
3) If I create a tap device, set a MAC address on it, attach it to the
bridge, and make sure the bridge is assigned that MAC address, traffic
does pass, and when guests are destroyed and restarted, the MAC address
of the bridge doesn't change.
So it looks like this is a viable way to solve the problem. Before
digging into it, though, I thought I should ask if there are any
alternate ideas, or problems that anyone sees with this approach.
Also, I'm wondering where the name and MAC address of the dummy
interfaces should come from, and whether I need to worry about storing
them in the network XML. So far, this is what I think on those topics:
1) the MAC address can be autogenerated if it's not in the XML, but
needs to be saved in the XML so that it's persistent across libvirtd
restarts.
2) the dummy interface name can just be some derivation of the bridge
device name ("virbr0-mac"?), and probably doesn't need to be saved in
the XML.
(Also, I just noticed the code that sets the first byte of the MAC
address of all tap interfaces being attached to virtual network bridges
to 0xFE. What's the story behind that? Does the bridge always take the
numerically lowest MAC that's currently connected?)
13 years, 11 months
[libvirt] [Patch 0/2] Add new feature into live migration
by Wen Congyang
If the memory of guest OS is changed constantly, the live migration
can not be ended ever for ever.
We can use the command 'virsh migrate-setmaxdowntime' to control the
live migration. But the value of maxdowntime is diffcult to calculate
because it depends on the transfer speed of network and constantly
changing memroy size. We need a easy way to control the live migration.
This patch set add the support of auto cold migration fallback on
timeout. With this patch set, when we migrate the guest OS, we can
specify a timeout. If the live migration timeouts, the migration will
fallback to cold migration.
TODO:
1. The timer for Windows has not been implemented.
Test of this patchset:
Env:
a. The size of guest OS's memory: 1GB
b. The transfer speed of network: about 100Mb/s
c. The size of constantly changing memory: more than 900MB
1. migrate without timeout
# virsh migrate --live RHEL6RC qemu+ssh://<dest IP>/system tcp://<dest IP>:49152
The migration does not end after 12 hours.
2. migrate with timeout(30 minutes):
# date
Tue Dec 7 10:00:13 CST 2010
# virsh migrate --live --timeout 1800 RHEL6RC qemu+ssh://10.167.225.67/system tcp:10.167.225.67:49152
# date
Tue Dec 7 10:43:28 CST 2010
Wen Congyang (2):
timer impl
auto cold migration fallback at timeout
configure.ac | 4 ++
src/Makefile.am | 6 ++-
src/libvirt.c | 2 +
src/libvirt_private.syms | 6 ++
src/util/timer.c | 116 ++++++++++++++++++++++++++++++++++++++++++++
src/util/timer.h | 37 ++++++++++++++
src/util/timer_linux.c | 121 ++++++++++++++++++++++++++++++++++++++++++++++
tools/virsh.c | 53 ++++++++++++++++++++
8 files changed, 343 insertions(+), 2 deletions(-)
create mode 100644 src/util/timer.c
create mode 100644 src/util/timer.h
create mode 100644 src/util/timer_linux.c
13 years, 11 months
[libvirt] Virsh commands in two help groups?
by Justin Clift
Hi Osier,
Just noticed that some of the virsh commands are now listed in two groups:
freecell: listed in both domain and host groups
hostname: listed in both domain and host groups
echo: listed in both domain and virsh groups
Wondering if that's on purpose, or if it needs to be changed? The "echo"
one seems a bit wrong in the domain group, but not sure about the others. :)
13 years, 11 months
[libvirt] [PATCH v4 RESEND 0/4] Support of auto-dump on watchdog event in libvirtd
by Hu Tao
This patch series adds a new watchdog action `dump' which lets libvirtd
can do auto-dump when receiving a watchdog event from qemu guest.
In order to make the function work, there must be a watchdog device
added to guest, and guest must have a watchdog daemon running, for
example, /etc/init.d/watchdog start or auto-started on boot.
Changes:
v4:
- updated threadpool to follow libvirt naming style, use appropriate
internals APIs, and hide the struct definitions from the header
(by Daniel)
- fix an error that qemuDomainObjBeginJobWithDriver() get lost in
qemuDomainCoreDump()
- use thread pool in libvirtd (qemud worker)
v3:
- let default auto-dump dir be /var/lib/libvirt/qemu/dump
Hu Tao (4):
threadpool impl
Add a new function doCoreDump
Add a watchdog action `dump'
Using threadpool API to manage qemud worker
cfg.mk | 1 +
daemon/libvirtd.c | 172 ++++++----------------------------
daemon/libvirtd.h | 4 +
src/Makefile.am | 1 +
src/conf/domain_conf.c | 1 +
src/conf/domain_conf.h | 1 +
src/libvirt_private.syms | 6 +
src/qemu/qemu.conf | 5 +
src/qemu/qemu_conf.c | 16 +++-
src/qemu/qemu_conf.h | 5 +
src/qemu/qemu_driver.c | 227 +++++++++++++++++++++++++++++++++-----------
src/util/threadpool.c | 235 ++++++++++++++++++++++++++++++++++++++++++++++
src/util/threadpool.h | 49 ++++++++++
13 files changed, 521 insertions(+), 202 deletions(-)
create mode 100644 src/util/threadpool.c
create mode 100644 src/util/threadpool.h
--
1.7.3
--
Thanks,
Hu Tao
13 years, 11 months
[libvirt] [PATCH] Fix funny off-by-one error in clock-variable
by Dan Kenigsberg
Humans consider January as month #1, while gmtime_r(3) calls it month #0.
While fixing it, render qemu's rtc parameter with leading zeros, as is more
commonplace.
Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=660194
---
src/qemu/qemu_conf.c | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/qemu/qemu_conf.c b/src/qemu/qemu_conf.c
index 83c0f83..973e95b 100644
--- a/src/qemu/qemu_conf.c
+++ b/src/qemu/qemu_conf.c
@@ -3510,9 +3510,9 @@ qemuBuildClockArgStr(virDomainClockDefPtr def)
now += def->data.adjustment;
gmtime_r(&now, &nowbits);
- virBufferVSprintf(&buf, "base=%d-%d-%dT%d:%d:%d",
+ virBufferVSprintf(&buf, "base=%d-%d-%dT%02d:%02d:%02d",
nowbits.tm_year + 1900,
- nowbits.tm_mon,
+ nowbits.tm_mon + 1,
nowbits.tm_mday,
nowbits.tm_hour,
nowbits.tm_min,
--
1.7.3.2
13 years, 11 months
[libvirt] Syntax error "Bad fd number" when stopping libvirt-guests
by Laurent Léonard
Hi,
When libvirt-guests is being stopped, I get the following message:
$Running guests on default URI: test-vm
$Suspending guests on default URI...
$Suspending test-vm: /etc/init.d/libvirt-guests: 340: Syntax error: Bad fd
number
The size of RAM already dumped should be displayed instead.
The attached patch fixes that issue.
--
Laurent Léonard
13 years, 11 months
[libvirt] [PATCH] util: Fix bug which will cause libvirtd crash
by Osier Yang
"virCommandRun": if "cmd->outbuf" or "cmd->errbuf" is NULL,
libvirtd will be crashed when trying to start a qemu domain
(which invokes "virCommandRun"), it caused by we try to use
"*cmd->outbuf" and "*cmd->errbuf" regardless of cmd->outbuf
or cmd->errbuf is NULL.
* src/util/command.c (virCommandRun)
---
src/util/command.c | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/util/command.c b/src/util/command.c
index aa43f76..38d462b 100644
--- a/src/util/command.c
+++ b/src/util/command.c
@@ -931,8 +931,8 @@ virCommandRun(virCommandPtr cmd, int *exitstatus)
ret = -1;
VIR_DEBUG("Result stdout: '%s' stderr: '%s'",
- NULLSTR(*cmd->outbuf),
- NULLSTR(*cmd->errbuf));
+ cmd->outbuf ? NULLSTR(*cmd->outbuf) : "(null)",
+ cmd->errbuf ? NULLSTR(*cmd->errbuf) : "(null)");
/* Reset any capturing, in case caller runs
* this identical command again */
--
1.7.3.2
13 years, 11 months
