- Devel - libvirt List Archives

[libvirt] [PATCH 0/3] Multiple fixes related to lookup of gluster volumes
by Peter Krempa 25 Feb '14

25 Feb '14

Peter Krempa (3): storage: Don't lie about path used to look up in error message util: file: Don't sanitize URI protocol separator in virFileSanitizePath gluster: Fix "key" attribute for gluster volumes src/storage/storage_backend_gluster.c | 10 ++++++++-- src/storage/storage_driver.c | 12 +++++++++--- src/util/virfile.c | 12 ++++++++++++ 3 files changed, 29 insertions(+), 5 deletions(-) -- 1.8.5.5

3 11

[libvirt] [PATCH 0/3] Various RBD storage pool changes
by Wido den Hollander 25 Feb '14

25 Feb '14

This series of patches changes a couple of things in the RBD storage pool driver. The first change is that it includes the return status of librados and librbd in log and debug messages making it easier for users to track down the real cause of the problem. The second patch simplifies some code. The third patch leverages the new timeout options from librados. Should the backing Ceph cluster not respond for any reason librados will timeout. This prevents libvirt from handing on a rados_* or rbd_* call and locking up libvirt. If the librados version on the system does not support these timeout options nothing happens. libvirt compiles and runs just fine, but the timeout simply won't work. Wido den Hollander (3): rbd: Include return statusses from librados/librbd in logging rbd: Simplify opening RADOS IoCTX rbd: Set timeout options for librados src/storage/storage_backend_rbd.c | 137 ++++++++++++++++++++++--------------- 1 file changed, 80 insertions(+), 57 deletions(-) -- 1.7.9.5

2 8

[libvirt] [PATCH] virsh: initialize str to NULL for solving a build issue
by Chen Hanxiao 25 Feb '14

25 Feb '14

Fix a -Werror=maybe-uninitialized issue. Signed-off-by: Chen Hanxiao <chenhanxiao(a)cn.fujitsu.com> --- tools/virsh-domain-monitor.c | 2 +- tools/virsh-domain.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/tools/virsh-domain-monitor.c b/tools/virsh-domain-monitor.c index c67b833..1192565 100644 --- a/tools/virsh-domain-monitor.c +++ b/tools/virsh-domain-monitor.c @@ -218,7 +218,7 @@ VIR_ENUM_IMPL(vshDomainPMSuspendedReason, static const char * vshDomainStateReasonToString(int state, int reason) { - const char *str; + const char *str = NULL; switch ((virDomainState) state) { case VIR_DOMAIN_NOSTATE: str = vshDomainNostateReasonTypeToString(reason); diff --git a/tools/virsh-domain.c b/tools/virsh-domain.c index 8e42300..3e73340 100644 --- a/tools/virsh-domain.c +++ b/tools/virsh-domain.c @@ -10399,7 +10399,7 @@ VIR_ENUM_IMPL(vshDomainEventCrashed, static const char * vshDomainEventDetailToString(int event, int detail) { - const char *str; + const char *str = NULL; switch ((virDomainEventType) event) { case VIR_DOMAIN_EVENT_DEFINED: str = vshDomainEventDefinedTypeToString(detail); -- 1.8.5.3

2 1

[libvirt] [PATCH] virsh: mark CPU usage field names as translatable
by Ján Tomko 25 Feb '14

25 Feb '14

My commit ac75801 removed the translation markers when moving the field names into an array. --- tools/virsh-host.c | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/tools/virsh-host.c b/tools/virsh-host.c index 502203b..6a04f9d 100644 --- a/tools/virsh-host.c +++ b/tools/virsh-host.c @@ -352,12 +352,12 @@ VIR_ENUM_IMPL(vshCPUStats, VSH_CPU_LAST, VIR_NODE_CPU_STATS_UTILIZATION); const char *vshCPUOutput[] = { - "user:", - "system:", - "idle:", - "iowait:", - "intr:", - "usage:" + N_("user:"), + N_("system:"), + N_("idle:"), + N_("iowait:"), + N_("intr:"), + N_("usage:") }; static bool -- 1.8.3.2

2 2

[libvirt] [PATCH v1 0/2] support dumping guest memory in compressed format
by Qiao Nuohan 25 Feb '14

25 Feb '14

dumping guest's memroy is introduced without compression supported, and this is a freature regression of 'virsh dump --memory-only'. This patchset is used to add support in libvirt side to make qemu dump guest's memory in kdump-compressed format and please refer the following address to see implementation of the qemu side, the lastest version of qemu side is v9(ready for being queued). http://lists.nongnu.org/archive/html/qemu-devel/2014-02/msg03016.html qiaonuohan (2): make qemu dump memory in kdump-compressed format add dump_memory_format in qemu.conf include/libvirt/libvirt.h.in | 19 ++++++++++---- src/qemu/libvirtd_qemu.aug | 1 + src/qemu/qemu.conf | 13 +++++++++- src/qemu/qemu_conf.c | 3 +++ src/qemu/qemu_conf.h | 2 ++ src/qemu/qemu_driver.c | 51 +++++++++++++++++++++++++++++++++++--- src/qemu/qemu_monitor.c | 6 ++--- src/qemu/qemu_monitor.h | 3 ++- src/qemu/qemu_monitor_json.c | 4 ++- src/qemu/qemu_monitor_json.h | 3 ++- src/qemu/test_libvirtd_qemu.aug.in | 1 + tests/qemumonitorjsontest.c | 2 +- tools/virsh-domain.c | 42 +++++++++++++++++++++++++++++++ 13 files changed, 133 insertions(+), 17 deletions(-)

2 5

[libvirt] QEMU accepted for Google Summer of Code 2014
by Stefan Hajnoczi 24 Feb '14

24 Feb '14

Great news: the organizations for Google Summer of Code 2014 have been announced and QEMU is participating again this year! If you are a student who is interested in a 12-week full-time project working on QEMU, KVM, or libvirt this summer, head over to our project ideas page: http://qemu-project.org/Google_Summer_of_Code_2014 Student applications are open from March 10th to March 21st. See the Google Summer of Code timeline for details: https://www.google-melange.com/gsoc/events/google/gsoc2014 Also be sure to read the FAQs on the Summer of Code website to find out about eligibility, time requirements, and how the process works: https://www.google-melange.com/gsoc/document/show/gsoc_program/google/gsoc2… Thanks to our mentors for posting their project ideas on time, thus helping us produce a good organization application. I will send a private email to mentors to kickstart the process for GSoC 2014. Stefan

1 0

[libvirt] [PATCH 0/2] virsh cleanups
by Eric Blake 24 Feb '14

24 Feb '14

Followups written based on review to my 'virsh event' addition. Eric Blake (2): virsh: use more compact VIR_ENUM_IMPL virsh: kill over-engineered asprintf failure recovery tools/virsh-domain-monitor.c | 280 +++++++++++++++------------------- tools/virsh-domain.c | 347 ++++++++++++++++++------------------------- tools/virsh-network.c | 36 ++--- tools/virsh-pool.c | 139 ++++++----------- tools/virsh-volume.c | 104 ++++--------- 5 files changed, 358 insertions(+), 548 deletions(-) -- 1.8.5.3

2 7

[libvirt] [PATCH 0/4] libxl: improve domain lifecycle event support
by Jim Fehlig 24 Feb '14

24 Feb '14

This series is the result of a report from Kim Larry [1] about missing shutdown events in the libxl driver. Kim's report revealed that the libxl driver was ignoring the <on_*> domain lifecycle event configuration! Patch1 adds support for the four standard actions (destroy, restart, preserve, and rename-restart) of <on_*>. Patch2 queues the shutdown event earlier in the shutdown handler, to avoid it being delivered after the a start event in the case of a restart. Patches 3 and 4 add support for the extra <on_crash> actions coredump-destroy and coredump-restart. [1] https://www.redhat.com/archives/libvir-list/2014-February/msg01121.html Jim Fehlig (4): libxl: honor domain lifecycle event configuration libxl: queue domain event earlier in shutdown handler libxl: add dump dir to libxlDriverConfig object libxl: handle on_crash coredump actions src/libxl/libxl_conf.c | 3 + src/libxl/libxl_conf.h | 2 + src/libxl/libxl_driver.c | 148 ++++++++++++++++++++++++++++++++++++++--------- 3 files changed, 125 insertions(+), 28 deletions(-) -- 1.8.1.4

3 14

[libvirt] [PATCH] Document the keyboard as a valid input type
by Ján Tomko 24 Feb '14

24 Feb '14

Commit bc18373 added a new input type, but didn't change the documentation. --- docs/formatdomain.html.in | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/docs/formatdomain.html.in b/docs/formatdomain.html.in index ea1a97b..400de07 100644 --- a/docs/formatdomain.html.in +++ b/docs/formatdomain.html.in @@ -3847,15 +3847,16 @@ qemu-kvm -net nic,model=? /dev/null ... <devices> <input type='mouse' bus='usb'/> + <input type='keyboard' bus='usb'/> </devices> ...</pre> <dl> <dt><code>input</code></dt> <dd>The <code>input</code> element has one mandatory attribute, - the <code>type</code> whose value can be either 'mouse' or - 'tablet'. The latter provides absolute - cursor movement, while the former uses relative movement. The optional + the <code>type</code> whose value can be 'mouse', 'tablet' or + 'keyboard'. The tablet provides absolute cursor movement, + while the mouse uses relative movement. The optional <code>bus</code> attribute can be used to refine the exact device type. It takes values "xen" (paravirtualized), "ps2" and "usb".</dd> </dl> -- 1.8.3.2

2 2

[libvirt] [PATCH] virsh: Don't leak buffer if GetFDs fails in cmdCreate
by Ján Tomko 24 Feb '14

24 Feb '14

Change the logic of the function to return false by default and move the freeing of the buffer to the cleanup section. https://bugzilla.redhat.com/show_bug.cgi?id=1067338 --- tools/virsh-domain.c | 29 ++++++++++++++++------------- 1 file changed, 16 insertions(+), 13 deletions(-) diff --git a/tools/virsh-domain.c b/tools/virsh-domain.c index 00ace11..59e843f 100644 --- a/tools/virsh-domain.c +++ b/tools/virsh-domain.c @@ -6518,7 +6518,7 @@ cmdCreate(vshControl *ctl, const vshCmd *cmd) { virDomainPtr dom; const char *from = NULL; - bool ret = true; + bool ret = false; char *buffer; #ifndef WIN32 bool console = vshCommandOptBool(cmd, "console"); @@ -6534,7 +6534,7 @@ cmdCreate(vshControl *ctl, const vshCmd *cmd) return false; if (cmdStartGetFDs(ctl, cmd, &nfds, &fds) < 0) - return false; + goto cleanup; if (vshCommandOptBool(cmd, "paused")) flags |= VIR_DOMAIN_START_PAUSED; @@ -6545,20 +6545,23 @@ cmdCreate(vshControl *ctl, const vshCmd *cmd) dom = virDomainCreateXMLWithFiles(ctl->conn, buffer, nfds, fds, flags); else dom = virDomainCreateXML(ctl->conn, buffer, flags); - VIR_FREE(buffer); - if (dom != NULL) { - vshPrint(ctl, _("Domain %s created from %s\n"), - virDomainGetName(dom), from); -#ifndef WIN32 - if (console) - cmdRunConsole(ctl, dom, NULL, 0); -#endif - virDomainFree(dom); - } else { + if (!dom) { vshError(ctl, _("Failed to create domain from %s"), from); - ret = false; + goto cleanup; } + + vshPrint(ctl, _("Domain %s created from %s\n"), + virDomainGetName(dom), from); +#ifndef WIN32 + if (console) + cmdRunConsole(ctl, dom, NULL, 0); +#endif + virDomainFree(dom); + ret = true; + +cleanup: + VIR_FREE(buffer); VIR_FREE(fds); return ret; } -- 1.8.3.2

2 2