- Devel - libvirt List Archives

[libvirt] Writing a new driver for libvirt
by Orr, David-P64407 27 Aug '08

27 Aug '08

Hey all, I am new to libvirt and have been asked to look at the possibility of writing a libvirt driver for a hypervisor we are using. I am wondering if someone can give me the basics on what is involved in accomplishing this task, what are the basics of writing a new driver? I have looked at the documentation posted and been poking around the code and just wanted to ask someone to point me in the right direction. Thanks in advance. David Orr General Dynamics Software Engineer David.Orr(a)gdc4s.com

2 3

[libvirt] [PATCH] storage pool discovery
by David Lively 27 Aug '08

27 Aug '08

Hi Folks - Here's my second pass at storage pool discovery. I've taken Daniel's suggestion and made it return a single XML doc containing <source> elements rather than an array of <pool> docs (and also incorporated suggestions from Daniel V and Jim M). Note that the storage <source> <name> patch is closely related (without it, the <source> docs returned for logical pools aren't correct). Dave

3 15

Re: [libvirt] Error compiling libvirt 0.4.4
by Richard W.M. Jones 27 Aug '08

27 Aug '08

On Wed, Aug 27, 2008 at 10:42:38AM -0300, Everton P. Alexandre wrote: > > "Richard W.M. Jones" <rjones(a)redhat.com> escreveu: On Thu, Aug 21, 2008 at 06:57:40PM -0300, Everton P. Alexandre wrote: > > > In file included from /usr/include/xen/dom0_ops.h:31, > > > from xen_unified.c:29: > > > /usr/include/xen/xen.h:578: error: expected '=', ',', ';', 'asm' or '__attribute__' before 'char' > > > > Did you find out what was causing this? > > No, I don't. Did this problem happen with you? No. Does it still happen for you? Have you tried the latest version from CVS? Rich. -- Richard Jones, Emerging Technologies, Red Hat http://et.redhat.com/~rjones Read my OCaml programming blog: http://camltastic.blogspot.com/ Fedora now supports 64 OCaml packages (the OPEN alternative to F#) http://cocan.org/getting_started_with_ocaml_on_red_hat_and_fedora

1 0

[libvirt] PATCH: Handle case of no domains in openvz driver
by Daniel P. Berrange 27 Aug '08

27 Aug '08

If running the openvz driver on a machine with no containers currently defined, it throws an error # ./src/virsh --connect openvz:///system list libvir: OpenVZ error : internal error Failed to parse vzlist output Id Name State ---------------------------------- This is because of a failure to check for the EOF condition. This patch adds the missing check Daniel diff -r 1fc0d8258838 src/openvz_conf.c --- a/src/openvz_conf.c Tue Aug 19 13:58:56 2008 +0100 +++ b/src/openvz_conf.c Tue Aug 19 13:59:51 2008 +0100 @@ -534,6 +534,9 @@ openvzGetVPSInfo(virConnectPtr conn) { vm = *pnext; if (fscanf(fp, "%d %s\n", &veid, status) != 2) { + if (feof(fp)) + break; + openvzError(conn, VIR_ERR_INTERNAL_ERROR, _("Failed to parse vzlist output")); goto error; -- |: Red Hat, Engineering, London -o- http://people.redhat.com/berrange/ :| |: http://libvirt.org -o- http://virt-manager.org -o- http://ovirt.org :| |: http://autobuild.org -o- http://search.cpan.org/~danberr/ :| |: GnuPG: 7D3B9505 -o- F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 :|

2 2

[libvirt] PATCH: REmove bogus call to virStateInit in openvz driver
by Daniel P. Berrange 27 Aug '08

27 Aug '08

For some unknown reason there is a call to virStateInitialize() in the openvz driver's open method. This is absolutely forbidden - this API is only intended for use by the daemon. This patch removes this bogus call and makes it call the openvz setup APIs directly. Daniel diff -r c2e95cd51bc3 src/openvz_driver.c --- a/src/openvz_driver.c Wed Aug 13 12:50:55 2008 +0000 +++ b/src/openvz_driver.c Tue Aug 19 12:11:37 2008 +0100 @@ -81,10 +81,6 @@ static int openvzNumDomains(virConnectPt static int openvzNumDomains(virConnectPtr conn); static int openvzListDefinedDomains(virConnectPtr conn, char **const names, int nnames); static int openvzNumDefinedDomains(virConnectPtr conn); -static int openvzStartup(void); -static int openvzShutdown(void); -static int openvzReload(void); -static int openvzActive(void); static virDomainPtr openvzDomainDefineXML(virConnectPtr conn, const char *xml); static virDomainPtr openvzDomainCreateLinux(virConnectPtr conn, const char *xml, @@ -697,7 +693,8 @@ static virDrvOpenStatus openvzOpen(virCo conn->privateData = &ovz_driver; - virStateInitialize(); + openvzAssignUUIDs(); + vms = openvzGetVPSInfo(conn); ovz_driver.vms = vms; @@ -849,27 +846,6 @@ Version: 2.2 static int openvzNumDefinedDomains(virConnectPtr conn ATTRIBUTE_UNUSED) { return ovz_driver.num_inactive; -} - -static int openvzStartup(void) { - openvzAssignUUIDs(); - - return 0; -} - -static int openvzShutdown(void) { - - return 0; -} - -static int openvzReload(void) { - - return 0; -} - -static int openvzActive(void) { - - return 1; } static virDriver openvzDriver = { @@ -934,17 +910,8 @@ static virDriver openvzDriver = { NULL, /* nodeGetFreeMemory */ }; -static virStateDriver openvzStateDriver = { - openvzStartup, - openvzShutdown, - openvzReload, - openvzActive, - NULL, /* sigHandler */ -}; - int openvzRegister(void) { virRegisterDriver(&openvzDriver); - virRegisterStateDriver(&openvzStateDriver); return 0; } -- |: Red Hat, Engineering, London -o- http://people.redhat.com/berrange/ :| |: http://libvirt.org -o- http://virt-manager.org -o- http://ovirt.org :| |: http://autobuild.org -o- http://search.cpan.org/~danberr/ :| |: GnuPG: 7D3B9505 -o- F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 :|

2 2

[libvirt] PATCH: Allow specific FDs to be kept open in virExec()
by Daniel P. Berrange 27 Aug '08

27 Aug '08

With my recent patches to virExec(), all FDs except stdin/out/err are closed before the child is exec'd to prevent accidental leaks. Of course I forgot the one key place where we need to propagate FDs... The TAP devices passed to QEMU. This patch adds a 'keepfd' parameter to virExec() which allows the caller to specify a bitset of file descriptors to keep open, and updates the callers to use this as required. The QEMU driver specifies any TAP fds and the LXC driver specifies its PTY this way removing a previous hack. QEMU networking works again with fix.... lxc_driver.c | 17 ++++++++++++++--- openvz_driver.c | 6 ++++-- qemu_driver.c | 26 +++++++++++++++++++------- storage_backend.c | 6 ++++-- util.c | 8 ++++++-- util.h | 7 +++++++ 6 files changed, 54 insertions(+), 16 deletions(-) Daniel Index: src/lxc_driver.c =================================================================== RCS file: /data/cvs/libvirt/src/lxc_driver.c,v retrieving revision 1.23 diff -u -p -r1.23 lxc_driver.c --- src/lxc_driver.c 20 Aug 2008 20:55:32 -0000 1.23 +++ src/lxc_driver.c 21 Aug 2008 13:52:25 -0000 @@ -615,6 +615,8 @@ static int lxcControllerStart(virConnect const char **largv = NULL; pid_t child; int status; + char *keepfd = NULL; + char appPtyStr[30]; #define ADD_ARG_SPACE \ do { \ @@ -638,11 +640,13 @@ static int lxcControllerStart(virConnect goto no_memory; \ } while (0) + snprintf(appPtyStr, sizeof(appPtyStr), "%d", appPty); + ADD_ARG_LIT(vm->def->emulator); ADD_ARG_LIT("--name"); ADD_ARG_LIT(vm->def->name); ADD_ARG_LIT("--console"); - ADD_ARG_LIT("0"); /* Passing console master PTY as FD 0 */ + ADD_ARG_LIT(appPtyStr); ADD_ARG_LIT("--background"); for (i = 0 ; i < nveths ; i++) { @@ -652,10 +656,15 @@ static int lxcControllerStart(virConnect ADD_ARG(NULL); - vm->stdin_fd = appPty; /* Passing console master PTY as FD 0 */ + vm->stdin_fd = -1; vm->stdout_fd = vm->stderr_fd = logfd; - if (virExec(conn, largv, NULL, &child, + if (VIR_ALLOC_N(keepfd, VIR_EXEC_FDSET_SIZE()) < 0) + goto no_memory; + + VIR_EXEC_FDSET_ON(keepfd, appPty); + + if (virExec(conn, largv, NULL, keepfd, &child, vm->stdin_fd, &vm->stdout_fd, &vm->stderr_fd, VIR_EXEC_NONE) < 0) goto cleanup; @@ -686,6 +695,8 @@ static int lxcControllerStart(virConnect ret = 0; cleanup: + VIR_FREE(keepfd); + for (i = 0 ; i < largc ; i++) VIR_FREE(largv[i]); Index: src/openvz_driver.c =================================================================== RCS file: /data/cvs/libvirt/src/openvz_driver.c,v retrieving revision 1.43 diff -u -p -r1.43 openvz_driver.c --- src/openvz_driver.c 20 Aug 2008 20:48:36 -0000 1.43 +++ src/openvz_driver.c 21 Aug 2008 13:52:25 -0000 @@ -807,7 +807,8 @@ static int openvzListDomains(virConnectP char *endptr; const char *cmd[] = {VZLIST, "-ovpsid", "-H" , NULL}; - ret = virExec(conn, cmd, NULL, &pid, -1, &outfd, &errfd, VIR_EXEC_NONE); + ret = virExec(conn, cmd, NULL, NULL, + &pid, -1, &outfd, &errfd, VIR_EXEC_NONE); if(ret == -1) { openvzError(conn, VIR_ERR_INTERNAL_ERROR, _("Could not exec %s"), VZLIST); @@ -844,7 +845,8 @@ static int openvzListDefinedDomains(virC const char *cmd[] = {VZLIST, "-ovpsid", "-H", "-S", NULL}; /* the -S options lists only stopped domains */ - ret = virExec(conn, cmd, NULL, &pid, -1, &outfd, &errfd, VIR_EXEC_NONE); + ret = virExec(conn, cmd, NULL, NULL, + &pid, -1, &outfd, &errfd, VIR_EXEC_NONE); if(ret == -1) { openvzError(conn, VIR_ERR_INTERNAL_ERROR, _("Could not exec %s"), VZLIST); Index: src/qemu_driver.c =================================================================== RCS file: /data/cvs/libvirt/src/qemu_driver.c,v retrieving revision 1.111 diff -u -p -r1.111 qemu_driver.c --- src/qemu_driver.c 20 Aug 2008 20:48:36 -0000 1.111 +++ src/qemu_driver.c 21 Aug 2008 13:52:26 -0000 @@ -847,6 +847,7 @@ static int qemudStartVMDaemon(virConnect int *tapfds = NULL; int ntapfds = 0; int qemuCmdFlags; + char *keepfd = NULL; if (virDomainIsActive(vm)) { qemudReportError(conn, NULL, NULL, VIR_ERR_INTERNAL_ERROR, @@ -950,12 +951,22 @@ static int qemudStartVMDaemon(virConnect vm->stdout_fd = -1; vm->stderr_fd = -1; - ret = virExec(conn, argv, NULL, &vm->pid, - vm->stdin_fd, &vm->stdout_fd, &vm->stderr_fd, - VIR_EXEC_NONBLOCK); - if (ret == 0) { - vm->def->id = driver->nextvmid++; - vm->state = migrateFrom ? VIR_DOMAIN_PAUSED : VIR_DOMAIN_RUNNING; + if (VIR_ALLOC_N(keepfd, VIR_EXEC_FDSET_SIZE()) < 0) { + ret = -1; + qemudReportError(conn, NULL, NULL, VIR_ERR_NO_MEMORY, NULL); + } else { + for (i = 0 ; i < ntapfds ; i++) + VIR_EXEC_FDSET_ON(keepfd, tapfds[i]); + + ret = virExec(conn, argv, NULL, keepfd, &vm->pid, + vm->stdin_fd, &vm->stdout_fd, &vm->stderr_fd, + VIR_EXEC_NONBLOCK); + if (ret == 0) { + vm->def->id = driver->nextvmid++; + vm->state = migrateFrom ? VIR_DOMAIN_PAUSED : VIR_DOMAIN_RUNNING; + } + + VIR_FREE(keepfd); } for (i = 0 ; argv[i] ; i++) @@ -1219,7 +1230,8 @@ dhcpStartDhcpDaemon(virConnectPtr conn, if (qemudBuildDnsmasqArgv(conn, network, &argv) < 0) return -1; - ret = virExec(conn, argv, NULL, &network->dnsmasqPid, -1, NULL, NULL, VIR_EXEC_NONBLOCK); + ret = virExec(conn, argv, NULL, NULL, + &network->dnsmasqPid, -1, NULL, NULL, VIR_EXEC_NONBLOCK); for (i = 0; argv[i]; i++) VIR_FREE(argv[i]); Index: src/storage_backend.c =================================================================== RCS file: /data/cvs/libvirt/src/storage_backend.c,v retrieving revision 1.19 diff -u -p -r1.19 storage_backend.c --- src/storage_backend.c 20 Aug 2008 09:24:14 -0000 1.19 +++ src/storage_backend.c 21 Aug 2008 13:52:26 -0000 @@ -403,7 +403,8 @@ virStorageBackendRunProgRegex(virConnect /* Run the program and capture its output */ - if (virExec(conn, prog, NULL, &child, -1, &fd, NULL, VIR_EXEC_NONE) < 0) { + if (virExec(conn, prog, NULL, NULL, + &child, -1, &fd, NULL, VIR_EXEC_NONE) < 0) { goto cleanup; } @@ -537,7 +538,8 @@ virStorageBackendRunProgNul(virConnectPt v[i] = NULL; /* Run the program and capture its output */ - if (virExec(conn, prog, NULL, &child, -1, &fd, NULL, VIR_EXEC_NONE) < 0) { + if (virExec(conn, prog, NULL, NULL, + &child, -1, &fd, NULL, VIR_EXEC_NONE) < 0) { goto cleanup; } Index: src/util.c =================================================================== RCS file: /data/cvs/libvirt/src/util.c,v retrieving revision 1.53 diff -u -p -r1.53 util.c --- src/util.c 20 Aug 2008 19:42:36 -0000 1.53 +++ src/util.c 21 Aug 2008 13:52:26 -0000 @@ -132,6 +132,7 @@ int virExec(virConnectPtr conn, const char *const*argv, const char *const*envp, + const char *keepfd, int *retpid, int infd, int *outfd, int *errfd, int flags) { @@ -293,7 +294,9 @@ virExec(virConnectPtr conn, if (i != infd && i != null && i != childout && - i != childerr) + i != childerr && + (!keepfd || + !VIR_EXEC_FDSET_ISON(keepfd, i))) close(i); if (flags & VIR_EXEC_DAEMON) { @@ -403,7 +406,8 @@ virRun(virConnectPtr conn, int *status) { int childpid, exitstatus, ret; - if ((ret = virExec(conn, argv, NULL, &childpid, -1, NULL, NULL, VIR_EXEC_NONE)) < 0) + if ((ret = virExec(conn, argv, NULL, NULL, + &childpid, -1, NULL, NULL, VIR_EXEC_NONE)) < 0) return ret; while ((ret = waitpid(childpid, &exitstatus, 0) == -1) && errno == EINTR); Index: src/util.h =================================================================== RCS file: /data/cvs/libvirt/src/util.h,v retrieving revision 1.25 diff -u -p -r1.25 util.h --- src/util.h 20 Aug 2008 19:42:36 -0000 1.25 +++ src/util.h 21 Aug 2008 13:52:26 -0000 @@ -33,9 +33,16 @@ enum { VIR_EXEC_DAEMON = (1 << 1), }; +#define VIR_EXEC_FDSET_SIZE() (sysconf(_SC_OPEN_MAX)/8) +#define VIR_EXEC_FDSET_CLEAR(set) memset((set), 0, VIR_EXEC_FDSET_SIZE()) +#define VIR_EXEC_FDSET_ON(set, fd) (set[(fd)/8] |= (1 << ((fd)%8))) +#define VIR_EXEC_FDSET_OFF(set, fd) (set[(fd)/8] &= ~(1 << ((fd)%8))) +#define VIR_EXEC_FDSET_ISON(set, fd) (set[(fd)/8] & (1 << ((fd)%8))) + int virExec(virConnectPtr conn, const char *const*argv, const char *const*envp, + const char *keepfd, int *retpid, int infd, int *outfd, -- |: Red Hat, Engineering, London -o- http://people.redhat.com/berrange/ :| |: http://libvirt.org -o- http://virt-manager.org -o- http://ovirt.org :| |: http://autobuild.org -o- http://search.cpan.org/~danberr/ :| |: GnuPG: 7D3B9505 -o- F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 :|

3 8

[libvirt] Proposed functionality: API to extract host description from saved state, restore with modified XML
by Charles Duffy 27 Aug '08

27 Aug '08

Per subject. Obviously only for use by folks who *really* know what they're doing. :) I'm looking for the ability to restore a snapshot saved with domain.save(), with disk images pointing at a different location (in my particular use case, pointing at a an empty copy-and-write image backending into a snapshot of the old disk state). Further, as I may run multiple copies of the same host at once (from the same initial state, against different copy-on-write disk images backending into the same read-only store, into different disconnected networks), the UUID as well as the name may need to be changed. I could just rewrite the header of the saved-state file, but this has a few disadvantages: - To mimic the copy-on-write behavior of the disk images, I would need to copy the entire saved-domain file. I'm trying to minimize the time and disk penalty of snapshot/restore, so this is suboptimal. - My code is dependent on the file format used by the particular backend, and may break when that backend is revved. Would a patch adding API calls to - return the XML description of a saved host, given the name - restore a saved host with a different description be considered welcome? I presume that to maintain backwards compatibility, the latter would be expected to be exposed to clients via a new call, rather than adding a parameter to virDomainRestore(); in communicating with the drivers, on the other hand, my first instinct would be to add an extra parameter to virDrvDomainRestore. Thanks!

3 2

[libvirt] virConnectOpenReadOnly?
by Jun Koi 27 Aug '08

27 Aug '08

Hi, According to the comment of virConnectOpenReadOnly, this function has some restrictions on available methods to control domains. Could you be more specific? What kind of restrictions we are having (compare to virConnectOpen())? And when should we use virConnectOpenReadOnly() instead of virConnectOpen() ? Thanks, Jun

2 1

[libvirt] Error compiling libvirt 0.4.4
by Everton P. Alexandre 27 Aug '08

27 Aug '08

hi I'm trying to compile the libvirt 0.4.4. I'm getting the error below: gcc -DHAVE_CONFIG_H -I. -I.. -I../gnulib/lib -I../gnulib/lib -I../include -I../include -I../qemud -I/usr/local/include/libxml2 -DBINDIR=\"/usr/local/libexec\" -DSBINDIR=\"/usr/local/sbin\" -DSYSCONF_DIR=\"/usr/local/etc\" -DLOCALEBASEDIR=\"/usr/local/share/locale\" -DLOCAL_STATE_DIR=\"/usr/local/var\" -DGETTEXT_PACKAGE=\"libvirt\" -Wall -Wformat -Wformat-security -Wmissing-prototypes -Wnested-externs -Wpointer-arith -Wextra -Wshadow -Wcast-align -Wwrite-strings -Waggregate-return -Wstrict-prototypes -Winline -Wredundant-decls -Wno-sign-compare -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fasynchronous-unwind-tables -DWITH_QEMU -DWITH_TEST -DWITH_REMOTE -DWITH_LIBVIRTD -DWITH_XEN -DIN_LIBVIRT -g -O2 -MT libvirt_la-xen_unified.lo -MD -MP -MF .deps/libvirt_la-xen_unified.Tpo -c xen_unified.c -fPIC -DPIC -o .libs/libvirt_la-xen_unified.o In file included from /usr/include/xen/dom0_ops.h:31, from xen_unified.c:29: /usr/include/xen/xen.h:578: error: expected '=', ',', ';', 'asm' or '__attribute__' before 'char' /usr/include/xen/xen.h:579: error: expected '=', ',', ';', 'asm' or '__attribute__' before 'short' /usr/include/xen/xen.h:580: error: expected '=', ',', ';', 'asm' or '__attribute__' before 'int' /usr/include/xen/xen.h:581: error: expected '=', ',', ';', 'asm' or '__attribute__' before 'long' make[2]: *** [libvirt_la-xen_unified.lo] Error 1 make[2]: Leaving directory `/home/everton/libvirt-0.4.4/src' make[1]: *** [all-recursive] Error 1 make[1]: Leaving directory `/home/everton/libvirt-0.4.4' make: *** [all] Error 2 Thank you Everton Alexandre --------------------------------- Novos endereços, o Yahoo! que você conhece. Crie um email novo com a sua cara @ymail.com ou @rocketmail.com.

2 1

[libvirt] LIBVIR_VERSION_NUMBER ?
by Jun Koi 27 Aug '08

27 Aug '08

Hi, According to libvirt.h: /** * LIBVIR_VERSION_NUMBER: * * Macro providing the version of the library as * version * 1,000,000 + minor * 1000 + micro */ #define LIBVIR_VERSION_NUMBER 4004 The comment is incorrect, as 4004 is not how it is supposed to be. So we should fix either the comment (better?), or the version. Thanks, Jun

2 2

[libvirt] libvir: Xen error : Domain not found: xenUnifiedDomainLookupByID
by mantra UNIX 27 Aug '08

27 Aug '08

Hi, After rebooting/shutdown/poweroff a domU any of the "virsh" commands display the above error message. Only way to get rid of it is via "reboot", this sucks ! I am using RHEL5.2 with all the updates applied. Whats the alternative ? -- Regards, mantra - Instrument of Thought

1 0

RE: [libvirt] DISREGARD Problems starting XEN HVM with libvirt
by Matthew Donovan 26 Aug '08

26 Aug '08

It turns out everything was booting, they just weren't configured to give a console or display a GUI window. -matthew > -----Original Message----- > From: Matthew Donovan [mailto:matthew@atc-nycorp.com] > Sent: Tuesday, August 26, 2008 9:25 AM > To: libvir-list(a)redhat.com > Subject: [libvirt] Problems starting XEN HVM with libvirt > > I've been having trouble getting a Xen HVM guest started > using libvirt. > I've tried Windows Vista, Fedora Core 8, and Fedora Core 6 > guests. Domain 0 > is FC 8. The only guest that starts correctly is the FC 6 guest. Has > anyone else encountered something like this? Is there a > master list of > guest OS's that libvirt supports? > > Thanks > -matthew > > > CODE: > > #include <stdio.h> > #include <libvirt/libvirt.h> > #include <libvirt/virterror.h> > > > const char* foo = > "<domain type='xen'>" > "<name>fc6</name>" > "<os>" > "<type>hvm</type>" > "<loader>/usr/lib/xen/boot/hvmloader</loader>" > "<boot dev='hd'/>" > "</os>" > "<memory>512000</memory>" > "<vcpu>1</vcpu>" > "<on_shutdown>shutdown</on_shutdown>" > "<on_reboot>restart</on_reboot>" > "<on_crash>restart</on_crash>" > "<features>" > "<pae/>" > "<acpi/>" > "<apic/>" > "</features>" > "<clock sync='localtime'/>" > "<devices>" > "<emulator>/usr/lib/xen/bin/qemu-dm</emulator>" > "<disk type='block'>" > "<source dev='/dev/vgvms/fc6_test'/>" > "<target dev='hda'/>" > "</disk>" > "<interface type='ioemu'>" > "<mac address='7A:AB:D0:01:82:98'/>" > "</interface>" > "<input type='tablet' bus='usb'/>" > "<graphics type='sdl' password=''/>" > "<serial type='pty'/>" > "</devices>" > "</domain>"; > > > int main (int argc, char** argv) > { > virConnectPtr con; /* connection to the Xen hypervisor */ > virDomainPtr domain; > > virInitialize(); > > con = virConnectOpen ("xen:///"); > if (!con) { > virErrorPtr err = virGetLastError (); > printf ("virConnectOpen failed: %s\n", > err->message); > return -1; > } > > domain = virDomainDefineXML (con, foo); > if (!domain) { > virErrorPtr err = virConnGetLastError (con); > printf ("virDomainDefineXML failed: %s\n", > err->message); > > } else { > > if (virDomainCreate (domain) < 0) { > virErrorPtr err = virConnGetLastError (con); > printf ("virDomainCreate failed: %s\n", > err->message); > } else { > virDomainInfo info = {0}; > virDomainGetInfo (domain, &info); > printf ("state = %d\n", info.state); > getchar (); > } > } > > virConnectClose (con); > return 0; > } > > -- > Libvir-list mailing list > Libvir-list(a)redhat.com > https://www.redhat.com/mailman/listinfo/libvir-list >

1 0

[libvirt] Problems starting XEN HVM with libvirt
by Matthew Donovan 26 Aug '08

26 Aug '08

I've been having trouble getting a Xen HVM guest started using libvirt. I've tried Windows Vista, Fedora Core 8, and Fedora Core 6 guests. Domain 0 is FC 8. The only guest that starts correctly is the FC 6 guest. Has anyone else encountered something like this? Is there a master list of guest OS's that libvirt supports? Thanks -matthew CODE: #include <stdio.h> #include <libvirt/libvirt.h> #include <libvirt/virterror.h> const char* foo = "<domain type='xen'>" "<name>fc6</name>" "<os>" "<type>hvm</type>" "<loader>/usr/lib/xen/boot/hvmloader</loader>" "<boot dev='hd'/>" "</os>" "<memory>512000</memory>" "<vcpu>1</vcpu>" "<on_shutdown>shutdown</on_shutdown>" "<on_reboot>restart</on_reboot>" "<on_crash>restart</on_crash>" "<features>" "<pae/>" "<acpi/>" "<apic/>" "</features>" "<clock sync='localtime'/>" "<devices>" "<emulator>/usr/lib/xen/bin/qemu-dm</emulator>" "<disk type='block'>" "<source dev='/dev/vgvms/fc6_test'/>" "<target dev='hda'/>" "</disk>" "<interface type='ioemu'>" "<mac address='7A:AB:D0:01:82:98'/>" "</interface>" "<input type='tablet' bus='usb'/>" "<graphics type='sdl' password=''/>" "<serial type='pty'/>" "</devices>" "</domain>"; int main (int argc, char** argv) { virConnectPtr con; /* connection to the Xen hypervisor */ virDomainPtr domain; virInitialize(); con = virConnectOpen ("xen:///"); if (!con) { virErrorPtr err = virGetLastError (); printf ("virConnectOpen failed: %s\n", err->message); return -1; } domain = virDomainDefineXML (con, foo); if (!domain) { virErrorPtr err = virConnGetLastError (con); printf ("virDomainDefineXML failed: %s\n", err->message); } else { if (virDomainCreate (domain) < 0) { virErrorPtr err = virConnGetLastError (con); printf ("virDomainCreate failed: %s\n", err->message); } else { virDomainInfo info = {0}; virDomainGetInfo (domain, &info); printf ("state = %d\n", info.state); getchar (); } } virConnectClose (con); return 0; }

1 0

[libvirt] "static" ip address
by Olivier Deckmyn 26 Aug '08

26 Aug '08

Hi, I'm using KVM with the wonderful help of libvir (virsh), on a to-be-production-ready server of mine. I would like to be able to choose on a way or another the ip of each of my VM. As far as I can understand, the only way to get an IP (from a vm point of view) if the dhcp it. But I have no garantee vm1 will always get x.x.x.1 Ip adress... Is there a solution please ? (googling this list did'nt answer :() Thanx

7 20

[libvirt] libvir: Xen error
by mantra UNIX 25 Aug '08

25 Aug '08

Hello everyone, I get the following error when i try to redtart a domain; # virsh list libvir: Xen error : Domain not found: xenUnifiedDomainLookupByID I am using RHEL5.2 on i386, I have searched for the error on web and on RedHat but could not find any. -- Regards, mantra - Instrument of Thought

1 0

[libvirt] <bootloader /> BUG
by Stefan de Konink 25 Aug '08

25 Aug '08

With the tip of DanB I am currently using the inactive XML storage quite satisfied with the concept of first undefining than starting. But there is still a bug. The inactive XML adds a <bootloader /> tag. Now that is (in my perspective) not a bug, but it it is that libvirt expects a config. In my personal believes the tag shouldn't be there if there is no config; or libvirt should not libvir: Xen Daemon error : POST operation failed: xend_post: error from xen daemon: (xend.err 'Error creating domain: Had a bootloader specified, but no disks are bootable') error: Failed to create domain from /mnt/netapp/users/klant2/mono/index.xml I just checked with the last git version and the bug still exists... I hope someone can look into this asap. It is a blocker for development. Stefan

3 3

[libvirt] inactive domain list in virsh
by Jun Koi 25 Aug '08

25 Aug '08

Hi, I am wondering if there is a way to list all the inactive domain inside virsh? That would be useful to have such a command, so we can see which domain can be started. Thanks, J

1 1

[libvirt] ruby-libvirt bindings
by Donald Plummer 25 Aug '08

25 Aug '08

I'm trying to get the ruby-libvirt bindings working. I've installed them from the gem (ruby-libvirt-0.0.7.gem), but I'm having trouble. My setup is: Client: Debian Lenny, libvirt 0.4.4 compilied with Xen API 3.0.1 support Server: CentOS 5.2, libvirt 0.3.3 compilied with Xen API 3.0.1 (running Xen 3.1.0) The virsh command will connect from the client to the server using xen+ssh, but not the default TLS. The error I get when trying to use TLS is: dplummer@vps-manager:~$ virsh -c xen://lvh2/ libvir: Remote error : No route to host error: failed to connect to the hypervisor I've setup certs on both the client and the server, there are no firewalls between them, and I can ping between them just fine. Anyway, what I'm really trying to do is get the ruby bindings working, and since the xen+ssh virsh connection works, I'm trying that. But getting the error "socket closed unexpectedly". -------------------- dplummer@vps-manager:~$ irb irb(main):001:0> require 'libvirt' => true irb(main):002:0> system('virsh -c xen+ssh://root@lvh2/ version') Compiled against library: libvir 0.4.4 Using library: libvir 0.4.4 Using API: Xen 3.0.1 Running hypervisor: Xen 3.1.0 => true irb(main):003:0> conn = Libvirt::open('xen+ssh://root@lvh2/') libvir: Remote error : socket closed unexpectedly Libvirt::ConnectionError: Failed to open xen+ssh://root@lvh2 from (irb):3:in `open' from (irb):3 from :0 irb(main):004:0> -------------------- Anyone have any suggestions? Anyone get this to work? Thanks!

2 1

[libvirt] [PATCH] Fix cd eject segfault
by Cole Robinson 23 Aug '08

23 Aug '08

The cdrom eject code was trying to dereference the NULL source of an empty cdrom. Attached patch fixes this. Thanks, Cole commit 5925689b5b94b29a520dcfbc7f4f1cfa0a0a0183 Author: Cole Robinson <crobinso(a)dhcp-100-19-219.bos.redhat.com> Date: Thu Aug 21 17:56:25 2008 -0400 Prevent cdrom eject from segfaulting when setting new disk source. diff --git a/src/qemu_driver.c b/src/qemu_driver.c index 06fbe55..769f34f 100644 --- a/src/qemu_driver.c +++ b/src/qemu_driver.c @@ -2953,6 +2953,7 @@ static int qemudDomainChangeCDROM(virDomainPtr dom, virDomainDiskDefPtr newdisk) { struct qemud_driver *driver = (struct qemud_driver *)dom->conn->privateData; char *cmd, *reply, *safe_path; + char *newsrc = NULL; if (newdisk->src) { safe_path = qemudEscapeMonitorArg(newdisk->src); @@ -2972,6 +2973,13 @@ static int qemudDomainChangeCDROM(virDomainPtr dom, } VIR_FREE(safe_path); + newsrc = strdup(newdisk->src); + if (!newsrc) { + qemudReportError(dom->conn, dom, NULL, VIR_ERR_OPERATION_FAILED, + "%s", _("out of memory")); + return -1; + } + } else if (asprintf(&cmd, "eject cdrom") == -1) { qemudReportError(dom->conn, dom, NULL, VIR_ERR_OPERATION_FAILED, "%s", _("out of memory")); @@ -2982,11 +2990,17 @@ static int qemudDomainChangeCDROM(virDomainPtr dom, qemudReportError(dom->conn, dom, NULL, VIR_ERR_OPERATION_FAILED, "%s", _("cannot change cdrom media")); VIR_FREE(cmd); + VIR_FREE(newsrc); return -1; } VIR_FREE(reply); VIR_FREE(cmd); - strcpy(olddisk->src, newdisk->src); + + VIR_FREE(olddisk->src); + if (newsrc) { + olddisk->src = newsrc; + newsrc = NULL; + } olddisk->type = newdisk->type; return 0; }

3 3

[libvirt] [PATCH] Delete veth devices during lxcVMCleanup
by Dan Smith 22 Aug '08

22 Aug '08

This makes sure to delete veth devices after we've killed the container. In the case of a destroy or crash, this ensures the system is cleaned up properly. To be applied atop Daniel Berrange's current LXC patch stack. diff -r b18ae864eb98 -r cd0bd9a6288a src/lxc_driver.c --- a/src/lxc_driver.c Wed Aug 13 09:40:08 2008 -0700 +++ b/src/lxc_driver.c Thu Aug 14 08:50:49 2008 -0700 @@ -375,6 +375,7 @@ int rc = -1; int waitRc; int childStatus = -1; + virDomainNetDefPtr net; while (((waitRc = waitpid(vm->pid, &childStatus, 0)) == -1) && errno == EINTR) @@ -403,6 +404,11 @@ vm->pid = -1; vm->def->id = -1; vm->monitor = -1; + + for (net = vm->def->nets; net; net = net->next) { + vethInterfaceUpOrDown(net->ifname, 0); + vethDelete(net->ifname); + } return rc; }

3 3

[libvirt] [PATCH] Check disk target collision when parsing domain xml
by Cole Robinson 22 Aug '08

22 Aug '08

The current domain xml parsing doesn't check if disks are specified with duplicate targets ("hda" etc.). The attached patch adds a check for this. Thanks, Cole commit 27df1653474738a2ce83c89e7bdb2c4f7327f9b6 Author: Cole Robinson <crobinso(a)dhcp-100-19-219.bos.redhat.com> Date: Thu Aug 21 14:58:04 2008 -0400 Check for duplicate disk targets when parsing domain xml. diff --git a/src/domain_conf.c b/src/domain_conf.c index 6b23474..ed6cc8b 100644 --- a/src/domain_conf.c +++ b/src/domain_conf.c @@ -1955,6 +1955,12 @@ static virDomainDefPtr virDomainDefParseXML(virConnectPtr conn, } else { virDomainDiskDefPtr ptr = def->disks; while (ptr) { + if (ptr->next && STREQ(disk->dst, ptr->next->dst)) { + virDomainReportError(conn, VIR_ERR_INTERNAL_ERROR, + _("duplicate disk target '%s'"), + disk->dst); + goto error; + } if (!ptr->next || virDomainDiskCompare(disk, ptr->next) < 0) { disk->next = ptr->next; ptr->next = disk;

3 3

RE: [libvirt] Using Xen config files
by Matthew Donovan 22 Aug '08

22 Aug '08

Xen and Xen-devel are 3.1.2. I'm using version 0.4.4 of libvirt. -matthew > -----Original Message----- > From: Richard W.M. Jones [mailto:rjones@redhat.com] > Sent: Friday, August 22, 2008 4:19 AM > To: Matthew Donovan > Cc: Daniel P. Berrange; libvir-list(a)redhat.com > Subject: Re: [libvirt] Using Xen config files > > On Thu, Aug 21, 2008 at 02:16:55PM -0400, Matthew Donovan wrote: > > I'm still having problems with this. I've dug around and > found errors in > > the qemu-dm log. > > > > When I try to start the VM with libvirt, the error I'm getting is: > > > "xs_read(/vm/414e73de-cf1e-487c-87e0-d4ebf7a23576/rtc/timeoffs > et): read > > error" > > > > I did a xenstore-ls on the /vm/414e73de-cf1e-487c-87e0-d4ebf7a23576 > > directory and sure enough, rtc/timeoffset is not there. > When I start the VM > > with the Xen config file I used to generate the XML the > rtc/timeoffset is > > there (timeoffset is set to "0"). > > Which version of xen, xen-devel, etc? > > Rich. > > -- > Richard Jones, Emerging Technologies, Red Hat > http://et.redhat.com/~rjones > virt-df lists disk usage of guests without needing to install any > software inside the virtual machine. Supports Linux and Windows. > http://et.redhat.com/~rjones/virt-df/ >

1 0

[libvirt] PATCH: Improve python exception messages
by Daniel P. Berrange 22 Aug '08

22 Aug '08

Most of the libvirt python API bindings use code snippet like this when raising an exception: if ret is None:raise libvirtError('virConnectOpen() failed') THis sets the message associated with the exception to "virConnectOpen() failed" This contains essentially zero useful information - you can see that it was virConnectOpen which failed from the stack trace. Now the libvirt error object has a real message, such as "authentication failed" Or "unable to connect to '/var/run/libvirt/libvirt-sock': Connection refused" This patch makes sure we extract this real error message and use it to set the message associated with the exception object. This is one step in getting better error reporting for virt-install/virt-manager, which is particularly needed for remote connections Daniel diff -u -r1.9 libvir.py --- python/libvir.py 11 Jun 2008 07:49:01 -0000 1.9 +++ python/libvir.py 18 Aug 2008 12:07:03 -0000 @@ -13,10 +13,22 @@ import types +def _getErrorMessage(conn, defmsg): + err = None + if conn is None: + err = virGetLastError() + else: + err = conn.virConnGetLastError() + if err is None: + return defmsg + else: + return err[2] + + # The root of all libvirt errors. class libvirtError(Exception): def __init__(self, msg, conn=None, dom=None, net=None, pool=None, vol=None): - Exception.__init__(self, msg) + Exception.__init__(self, _getErrorMessage(conn, msg)) if dom is not None: conn = dom._conn @@ -77,12 +89,6 @@ return None return self.err[8] - def __str__(self): - if self.get_error_message() is None: - return Exception.__str__(self) - else: - return Exception.__str__(self) + " " + self.get_error_message() - # # register the libvirt global error handler # -- |: Red Hat, Engineering, London -o- http://people.redhat.com/berrange/ :| |: http://libvirt.org -o- http://virt-manager.org -o- http://ovirt.org :| |: http://autobuild.org -o- http://search.cpan.org/~danberr/ :| |: GnuPG: 7D3B9505 -o- F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 :|

3 5

[libvirt] pthread_sigmask undefined reference error
by Jun Koi 22 Aug '08

22 Aug '08

Hi, The latest cvs version has an error when compiling: make[3]: Entering directory `/home/jun/projects/libvirt-0821/src' /bin/bash ../libtool --tag=CC --mode=link gcc -g -O2 -Wall -Wformat -Wformat-security -Wmissing-prototypes -Wnested-externs -Wpointer-arith -Wextra -Wshadow -Wcast-align -Wwrite-strings -Waggregate-return -Wstrict-prototypes -Winline -Wredundant-decls -Wno-sign-compare -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fasynchronous-unwind-tables -o virsh virsh-console.o virsh-util-lib.o virsh-virsh.o -Wall -Wformat -Wformat-security -Wmissing-prototypes -Wnested-externs -Wpointer-arith -Wextra -Wshadow -Wcast-align -Wwrite-strings -Waggregate-return -Wstrict-prototypes -Winline -Wredundant-decls -Wno-sign-compare -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fasynchronous-unwind-tables libvirt.la ../gnulib/lib/libgnu.la gcc -g -O2 -Wall -Wformat -Wformat-security -Wmissing-prototypes -Wnested-externs -Wpointer-arith -Wextra -Wshadow -Wcast-align -Wwrite-strings -Waggregate-return -Wstrict-prototypes -Winline -Wredundant-decls -Wno-sign-compare -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fasynchronous-unwind-tables -o .libs/virsh virsh-console.o virsh-util-lib.o virsh-virsh.o -Wall -Wformat -Wformat-security -Wmissing-prototypes -Wnested-externs -Wpointer-arith -Wextra -Wshadow -Wcast-align -Wwrite-strings -Waggregate-return -Wstrict-prototypes -Winline -Wredundant-decls -Wno-sign-compare -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fasynchronous-unwind-tables ./.libs/libvirt.so ../gnulib/lib/.libs/libgnu.a ./.libs/libvirt.so: undefined reference to `pthread_sigmask' collect2: ld returned 1 exit status make[3]: *** [virsh] Error 1 make[3]: Leaving directory `/home/jun/projects/libvirt-0821/src' make[2]: *** [all] Error 2 make[2]: Leaving directory `/home/jun/projects/libvirt-0821/src' make[1]: *** [all-recursive] Error 1 make[1]: Leaving directory `/home/jun/projects/libvirt-0821' make: *** [all] Error 2 It seems I lack a particular library?? Thanks, Jun

3 4

[libvirt] [PATCH] autobuild.sh: Fix minor shell-quoting bugs.
by Jim Meyering 22 Aug '08

22 Aug '08

I noticed some minor quoting problems in ovirt's autobuild.sh, and since part of that code came from here, ... Here's a patch: >From 50c37706729b7748dbf92290c6ecf6bc3982f681 Mon Sep 17 00:00:00 2001 From: Jim Meyering <meyering(a)redhat.com> Date: Thu, 21 Aug 2008 17:55:19 +0200 Subject: [PATCH] autobuild.sh: Fix minor shell-quoting bugs. * autobuild.sh: Remove unnecessary quotes. Don't choke on a file name argument containing a space. Don't misbehave for $AUTOBUILD_INSTALL_ROOT containing a shell meta-character. --- autobuild.sh | 8 ++++---- 1 files changed, 4 insertions(+), 4 deletions(-) diff --git a/autobuild.sh b/autobuild.sh index ce12692..7ae5d1e 100755 --- a/autobuild.sh +++ b/autobuild.sh @@ -5,7 +5,7 @@ set -v # Make things clean. -test -n "$1" && RESULTS="$1" || RESULTS="results.log" +test -n "$1" && RESULTS=$1 || RESULTS=results.log test -f Makefile && make -k distclean || : rm -rf coverage @@ -14,7 +14,7 @@ rm -rf coverage #mkdir build #cd build -./autogen.sh --prefix=$AUTOBUILD_INSTALL_ROOT \ +./autogen.sh --prefix="$AUTOBUILD_INSTALL_ROOT" \ --enable-test-coverage \ --enable-compile-warnings=error \ --with-openvz \ @@ -25,8 +25,8 @@ make make install set -o pipefail -make check 2>&1 | tee $RESULTS -make syntax-check 2>&1 | tee -a $RESULTS +make check 2>&1 | tee "$RESULTS" +make syntax-check 2>&1 | tee -a "$RESULTS" test -x /usr/bin/lcov && make cov rm -f *.tar.gz -- 1.6.0.90.g436ed

2 1