]> xenbits.xensource.com Git - libvirt.git/log
libvirt.git
14 years agoAdd Aurelien to AUTHORS
Daniel Veillard [Mon, 2 Aug 2010 19:55:33 +0000 (21:55 +0200)]
Add Aurelien to AUTHORS

14 years agoFix a couple of typo in iSCSI backend
Aurelien ROUGEMONT [Mon, 2 Aug 2010 19:52:02 +0000 (21:52 +0200)]
Fix a couple of typo in iSCSI backend

- a pure typo error and a wrong command referenced in an error message.

14 years agoMake virsh -d check its input
Daniel Veillard [Mon, 2 Aug 2010 19:20:26 +0000 (21:20 +0200)]
Make virsh -d check its input

it was using atoi direct without checking leading to confusion
in case of flag error for example with -c

* tools/virsh.c: vshParseArgv() use virStrToLong_i and remove the
  unchecked atoi used to parse teh parameter

14 years agoAdd iptables rule to fixup DHCP response checksum.
Laine Stump [Tue, 13 Jul 2010 02:59:58 +0000 (22:59 -0400)]
Add iptables rule to fixup DHCP response checksum.

This patch attempts to take advantage of a newly added netfilter
module to correct for a problem with some guest DHCP client
implementations when used in conjunction with a DHCP server run on the
host systems with packet checksum offloading enabled.

The problem is that, when the guest uses a RAW socket to read the DHCP
response packets, the checksum hasn't yet been fixed by the IP stack,
so it is incorrect.

The fix implemented here is to add a rule to the POSTROUTING chain of
the mangle table in iptables that fixes up the checksum for packets on
the virtual network's bridge that are destined for the bootpc port (ie
"dhcpc", ie port 68) port on the guest.

Only very new versions of iptables will have this support (it will be
in the next upstream release), so a failure to add this rule only
results in a warning message. The iptables patch is here:

  http://patchwork.ozlabs.org/patch/58525/

A corresponding kernel module patch is also required (the backend of
the iptables patch) and that will be in the next release of the
kernel.

14 years agoFix the ACS checking in the PCI code.
Chris Lalancette [Wed, 28 Jul 2010 20:53:00 +0000 (16:53 -0400)]
Fix the ACS checking in the PCI code.

When trying to assign a PCI device to a guest, we have
to check that all bridges upstream of that device support
ACS.  That means that we have to find the parent bridge of
the current device, check for ACS, then find the parent bridge
of that device, check for ACS, etc.  As it currently stands,
the code to do this iterates through all PCI devices on the
system, looking for a device that has a range of busses that
included the current device's bus.

That check is not restrictive enough, though.  Depending on
how we iterated through the list of PCI devices, we could first
find the *topmost* bridge in the system; since it necessarily had
a range of busses including the current device's bus, we
would only ever check the topmost bridge, and not check
any of the intermediate bridges.

Note that this also caused a fairly serious bug in the
secondary bus reset code, where we could erroneously
find and reset the topmost bus instead of the inner bus.

This patch changes pciGetParentDevice() so that it first
checks if a bridge device's secondary bus exactly matches
the bus of the device we are looking for.  If it does, we've
found the correct parent bridge and we are done.  If it does not,
then we check to see if this bridge device's busses *include* the
bus of the device we care about.  If so, we mark this bridge device
as best, and go on.  If we later find another bridge device whose
busses include this device, but is more restrictive, then we
free up the previous best and mark the new one as best.  This
algorithm ensures that in the normal case we find the direct
parent, but in the case that the parent bridge secondary bus
is not exactly the same as the device, we still find the
correct bridge.

This patch was tested by me on a 4-port NIC with a
bridge without ACS (where assignment failed), a 4-port
NIC with a bridge with ACS (where assignment succeeded),
and a 2-port NIC with no bridges (where assignment
succeeded).

Signed-off-by: Chris Lalancette <clalance@redhat.com>
14 years agoFree up memballoon def.
Chris Lalancette [Fri, 30 Jul 2010 15:40:59 +0000 (11:40 -0400)]
Free up memballoon def.

Forgetting to do this was causing a memory leak.

Signed-off-by: Chris Lalancette <clalance@redhat.com>
14 years agoDon't put a semicolon on the end of a VIR_ENUM_IMPL.
Chris Lalancette [Fri, 30 Jul 2010 15:40:42 +0000 (11:40 -0400)]
Don't put a semicolon on the end of a VIR_ENUM_IMPL.

Signed-off-by: Chris Lalancette <clalance@redhat.com>
14 years agoFix a bogus warning when parsing <hostdev>
Chris Lalancette [Fri, 30 Jul 2010 15:28:55 +0000 (11:28 -0400)]
Fix a bogus warning when parsing <hostdev>

When parsing hostdev, the following message would be emitted:

10:17:19.052: error : virDomainHostdevDefParseXML:3748 : internal error unknown node alias

However, alias is appropriately parsed in
virDomainDeviceInfoParseXML anyway.  Disable the error message
in the initial XML parsing loop.

Signed-off-by: Chris Lalancette <clalance@redhat.com>
14 years agoRemove duplicate </p> from downloads.html.in
Matthias Bolte [Sat, 31 Jul 2010 23:54:40 +0000 (01:54 +0200)]
Remove duplicate </p> from downloads.html.in

14 years agoUpdate ID after stopping a domain
Matthias Bolte [Fri, 30 Jul 2010 17:50:12 +0000 (19:50 +0200)]
Update ID after stopping a domain

14 years agoopenvzDomainCreateWithFlags: set domain id to the correct value
Jean-Baptiste Rouault [Fri, 30 Jul 2010 08:36:06 +0000 (10:36 +0200)]
openvzDomainCreateWithFlags: set domain id to the correct value

When an openvz domain is defined with virDomainDefineXML,
domain id is set to -1. A call to virDomainGetInfo after
starting the domain would then fail because this invalid
id is passed to openvzGetProcessInfo.

14 years agostorage: kill dead stores
Eric Blake [Fri, 30 Jul 2010 16:24:17 +0000 (10:24 -0600)]
storage: kill dead stores

Found by clang.  Clang complained that virStorageBackendProbeTarget
could dereference NULL if backingStoreFormat was NULL, but since all
callers passed a valid pointer, I added attributes instead of null
checks.

* src/storage/storage_backend.c
(virStorageBackendQEMUImgBackingFormat): Kill dead store.
* src/storage/storage_backend_fs.c (virStorageBackendProbeTarget):
Likewise.  Skip null checks, by adding attributes.

14 years agoqemu: kill some dead stores
Eric Blake [Fri, 30 Jul 2010 17:33:26 +0000 (11:33 -0600)]
qemu: kill some dead stores

Spotted by clang.

* src/qemu/qemu_monitor.c (qemuMonitorClose): Kill dead store.
* src/qemu/qemu_driver.c (qemudDomainSaveImageStartVM): Likewise.

14 years agonetwork: kill dead store
Eric Blake [Fri, 30 Jul 2010 15:59:17 +0000 (09:59 -0600)]
network: kill dead store

* src/network/bridge_driver.c (networkDefine): Kill dead store.

14 years agoxenapi: Update ID after starting a domain
Matthias Bolte [Fri, 30 Jul 2010 15:13:38 +0000 (17:13 +0200)]
xenapi: Update ID after starting a domain

14 years agoesx: Update ID after starting a domain
Matthias Bolte [Fri, 30 Jul 2010 15:06:20 +0000 (17:06 +0200)]
esx: Update ID after starting a domain

14 years agoFix DMI uuid parsing.
Chris Lalancette [Fri, 30 Jul 2010 14:19:51 +0000 (10:19 -0400)]
Fix DMI uuid parsing.

valgrind was complaining that virUUIDParse was depending on
an uninitialized value.  Indeed it was; virSetHostUUIDStr()
didn't initialize the dmiuuid buffer to 0's, meaning that
anything after the string read from /sys was uninitialized.
Clear out the dmiuuid buffer before use, and make sure to
always leave a \0 at the end.

Signed-off-by: Chris Lalancette <clalance@redhat.com>
14 years agoDo not activate boot=on on devices when not using KVM
Daniel Veillard [Fri, 30 Jul 2010 14:38:48 +0000 (16:38 +0200)]
Do not activate boot=on on devices when not using KVM

  Basically the 'boot=on' boot selection device is something present in
KVM but not in upstream QEmu, as a result if we boot a QEmu domain
without KVM acceleration we must disable boot=on ... even if the front
end kvm binary expose that capability in the help page.

* src/qemu/qemu_conf.c: in qemudBuildCommandLine if -no-kvm
  is passed, then deactivate QEMUD_CMD_FLAG_DRIVE_BOOT

14 years agoFix a memory leak in the qemudBuildCommandLine.
Chris Lalancette [Fri, 30 Jul 2010 13:39:48 +0000 (09:39 -0400)]
Fix a memory leak in the qemudBuildCommandLine.

ADD_ARG_LIT should only be used for literal arguments,
since it duplicates the memory.  Since virBufferContentAndReset
is already allocating memory, we should only use ADD_ARG.

Signed-off-by: Chris Lalancette <clalance@redhat.com>
14 years agoesx: Fix freeing of heterogeneous lists
Matthias Bolte [Wed, 28 Jul 2010 13:42:10 +0000 (15:42 +0200)]
esx: Fix freeing of heterogeneous lists

Always call the free function of the base type. The base type
function then dynamically dispatches the call to the free function
for the actual type.

14 years agoesx: Switch from name to number checks in the subdrivers
Matthias Bolte [Sun, 25 Jul 2010 17:04:22 +0000 (19:04 +0200)]
esx: Switch from name to number checks in the subdrivers

14 years agoesx: Improve blocked task detection and fix race condition
Matthias Bolte [Sun, 25 Jul 2010 15:45:19 +0000 (17:45 +0200)]
esx: Improve blocked task detection and fix race condition

esxVI_WaitForTaskCompletion can take a UUID to lookup the
corresponding domain and check if the current task for it
is blocked by a question. It calls another function to do
this: esxVI_LookupAndHandleVirtualMachineQuestion looks up
the VirtualMachine and checks for a question. If there is
a question it calls esxVI_HandleVirtualMachineQuestion to
handle it.

If there was no question or it has been answered the call
to esxVI_LookupAndHandleVirtualMachineQuestion returns 0.
If any error occurred during the lookup and answering
process -1 is returned. The problem with this is, that -1
is also returned when there was no error but the question
could not be answered. So esxVI_WaitForTaskCompletion cannot
distinguish between this two situations and reports that a
question is blocking the task even when there was actually
another problem.

This inherent problem didn't surface until vSphere 4.1 when
you try to define a new domain. The driver tries to lookup
the domain that is just in the process of being registered.
There seems to be some kind of race condition and the driver
manages to issue a lookup command before the ESX server was
able to register the domain. This used to work before.

Due to the return value problem described above the driver
reported a false error message in that case.

To solve this esxVI_WaitForTaskCompletion now takes an
additional occurrence parameter that describes whether or
not to expect the domain to be existent. Also add a new
parameter to esxVI_LookupAndHandleVirtualMachineQuestion
that allows to distinguish if the call returned -1 because
of an actual error or because the question could not be
answered.

14 years agoesx: silence spurious compiler warning
Eric Blake [Wed, 28 Jul 2010 23:28:44 +0000 (17:28 -0600)]
esx: silence spurious compiler warning

* src/esx/esx_vi_types.c (_DESERIALIZE_NUMBER)
(ESX_VI__TEMPLATE__DESERIALIZE_NUMBER): Add range check to shut up
gcc 4.5.0 regarding long long.

14 years agobuild: distribute libvirt_qemu.syms
Eric Blake [Thu, 29 Jul 2010 20:19:40 +0000 (14:19 -0600)]
build: distribute libvirt_qemu.syms

* src/Makefile.am (EXTRA_DIST): Ensure 'make distcheck' and
'rpmbuild' can reproduce a build.
* daemon/Makefile.am (DAEMON_SOURCES): Likewise.

14 years agobuild: restore operation of bit-rotted 'make cov'
Eric Blake [Wed, 9 Jun 2010 18:35:49 +0000 (12:35 -0600)]
build: restore operation of bit-rotted 'make cov'

'./autobuild.sh' with lcov installed discovered that our
coverage support has been bit-rotting for a while.  This
restores it back to a successful state, although I have
not yet spent any time looking through the resulting files to
look for low-hanging fruit in the unit test coverage front.

* configure.ac: Clear COMPILER_FLAGS at right place.
* Makefile.am (cov): Newer genhtml no longer likes plain -s.
* m4/compiler-flags.m4 (gl_COMPILER_FLAGS): Don't AC_SUBST
COMPILER_FLAGS; it is a shell variable for use in configure only.
* src/Makefile.am (AM_CFLAGS, AM_LDFLAGS): New variables, to make
it easier to provide global flag additions.  Use throughout, to
uniformly apply coverage flags.
* .gitignore: Globally ignore gcov output.
* daemon/.gitignore: Simplify.
* src/.gitignore: Likewise.
* tests/.gitignore: Likewise.

14 years agoxen: fix logic bug
Eric Blake [Wed, 28 Jul 2010 23:12:16 +0000 (17:12 -0600)]
xen: fix logic bug

The recent switch to enable -Wlogical-op paid off again.
gcc 4.5.0 (rawhide) is smarter than 4.4.4 (Fedora 13).

* src/xen/xend_internal.c (xenDaemonAttachDeviceFlags)
(xenDaemonUpdateDeviceFlags, xenDaemonDetachDeviceFlags): Use
correct operator.

Signed-off-by: Eric Blake <eblake@redhat.com>
14 years agobuild: fix 'make syntax-check' failure
Eric Blake [Thu, 29 Jul 2010 19:27:58 +0000 (13:27 -0600)]
build: fix 'make syntax-check' failure

src/lxc/veth.c:150:        VIR_DEBUG(_("Failed to delete '%s' (%d)"),
src/lxc/veth.c:188:            VIR_DEBUG(_("Failed to disable '%s' (%d)"),
maint.mk: do not mark these strings for translation

* src/lxc/veth.c (vethDelete, vethInterfaceUpOrDown): Don't
translate VIR_DEBUG.

14 years agolxc: Fix return values of veth.c functions
Ryota Ozaki [Fri, 23 Jul 2010 17:25:56 +0000 (02:25 +0900)]
lxc: Fix return values of veth.c functions

Previously, the functions in src/lxc/veth.c could sometimes return
positive values on failure rather than -1. This made accurate error
reporting difficult, and led to one failure to catch an error in a
calling function.

This patch makes all the functions in veth.c consistently return 0 on
success, and -1 on failure. It also fixes up the callers to the veth.c
functions where necessary.

Note that this patch may be related to the bug:

  https://bugzilla.redhat.com/show_bug.cgi?id=607496.

It will not fix the bug, but should unveil what happens.

* po/POTFILES.in - add veth.c, which previously had no translatable strings
* src/lxc/lxc_controller.c
* src/lxc/lxc_container.c
* src/lxc/lxc_driver.c    - fixup callers to veth.c, and remove error logs,
                            as they are now done in veth.c
* src/lxc/veth.c - make all functions consistently return -1 on error.
* src/lxc/veth.h - use ATTRIBUTE_NONNULL to protect against NULL args.

14 years agoEliminate memory leak in xenUnifiedDomainInfoListFree
Laine Stump [Thu, 29 Jul 2010 13:41:33 +0000 (09:41 -0400)]
Eliminate memory leak in xenUnifiedDomainInfoListFree

This fixes a leak described in

   https://bugzilla.redhat.com/show_bug.cgi?id=590073

xenUnifiedDomainInfoList has a pointer to a list of pointers to
xenUnifiedDomain. We were freeing up all the domains, but neglecting
to free the list.

This was found by Paolo Bonzini <pbonzini@redhat.com>.

14 years agolxc: Fix 'autostart' doesn't take effect actually
Ryota Ozaki [Thu, 29 Jul 2010 14:21:17 +0000 (23:21 +0900)]
lxc: Fix 'autostart' doesn't take effect actually

lxcStartup forgot to call lxcAutostartConfigs. Fix it.

This patch should fix https://bugzilla.redhat.com/show_bug.cgi?id=589863 .

14 years agoFix --with-xen-proxy related compile error
Matthias Bolte [Thu, 29 Jul 2010 14:03:48 +0000 (16:03 +0200)]
Fix --with-xen-proxy related compile error

Move virDomainChrTargetTypeToString out of the #ifndef PROXY
block, because it's used outside of it.

14 years agoFix a potential race in pciInitDevice.
Chris Lalancette [Wed, 28 Jul 2010 18:07:08 +0000 (14:07 -0400)]
Fix a potential race in pciInitDevice.

If detecting the FLR flag of a pci device fails, then we
could run into the situation of trying to close a file
descriptor twice, once in pciInitDevice() and once in pciFreeDevice().
Fix that by removing the pciCloseConfig() in pciInitDevice() and
just letting pciFreeDevice() handle it.

Thanks to Chris Wright for pointing out this problem.

While we are at it, fix an error check.  While it would actually
work as-is (since success returns 0), it's still more clear to
check for < 0 (as the rest of the code does).

Signed-off-by: Chris Lalancette <clalance@redhat.com>
14 years agoqemu: virtio console support
Cole Robinson [Wed, 14 Jul 2010 17:02:04 +0000 (13:02 -0400)]
qemu: virtio console support

Enable specifying a virtio console device with:

<console type='pty'>
  <target type='virtio'/>
</console>

14 years agodomain conf: Track <console> target type
Cole Robinson [Thu, 22 Jul 2010 17:56:21 +0000 (13:56 -0400)]
domain conf: Track <console> target type

All <console> devices now export a <target> type attribute. QEMU defaults
to 'serial', UML defaults to 'uml, xen can be either 'serial' or 'xen'
depending on fullvirt. Understandably there is lots of test fallout.

This will be used to differentiate between a serial vs. virtio console for
QEMU.

Signed-off-by: Cole Robinson <crobinso@redhat.com>
14 years agodomain conf: char: Add an explicit targetType field
Cole Robinson [Tue, 13 Jul 2010 20:06:38 +0000 (16:06 -0400)]
domain conf: char: Add an explicit targetType field

targetType only tracks the actual <target> format we are parsing. Currently
we only fill abide this value for channel devices.

Signed-off-by: Cole Robinson <crobinso@redhat.com>
14 years agodomain conf: Rename character prop targetType -> deviceType
Cole Robinson [Tue, 13 Jul 2010 19:05:34 +0000 (15:05 -0400)]
domain conf: Rename character prop targetType -> deviceType

There is actually a difference between the character device type (serial,
parallel, channel, ...) and the target type (virtio, guestfwd). Currently
they are awkwardly conflated.

Start to pull them apart by renaming targetType -> deviceType. This is
an entirely mechanical change.

Signed-off-by: Cole Robinson <crobinso@redhat.com>
14 years agodocs: domain: Document virtio <channel>
Cole Robinson [Wed, 14 Jul 2010 19:07:59 +0000 (15:07 -0400)]
docs: domain: Document virtio <channel>

Signed-off-by: Cole Robinson <crobinso@redhat.com>
14 years agotests: Test qemuxml2xml when expected xml changes
Cole Robinson [Sat, 24 Jul 2010 22:18:18 +0000 (18:18 -0400)]
tests: Test qemuxml2xml when expected xml changes

Add tests for auto memballon, implicit IDE, SCSI, virtio channel
controllers, and console/serial back compat.

Additionally, an explicit qemuxml2argvtest for scsi disks is added.

Signed-off-by: Cole Robinson <crobinso@redhat.com>
14 years agofix handling of PORT_PROFILE_RESPONSE_INPROGRESS netlink message
Gerhard Stenzel [Wed, 28 Jul 2010 14:17:21 +0000 (16:17 +0200)]
fix handling of PORT_PROFILE_RESPONSE_INPROGRESS netlink message

During function test of the 802.1Qbg implementation in lldpad we came
across a small problem in the handling of the netlink message
corresponding to PORT_PROFILE_RESPONSE_INPROGRESS. This should not
result in returning the default rc=1.

- src/util/macvtap.c: fix getPortProfileStatus() to return 0 in that
  case and also fix an indentation problem

14 years agomaint: fix comment typos
Eric Blake [Wed, 28 Jul 2010 14:05:55 +0000 (08:05 -0600)]
maint: fix comment typos

* src/network/bridge_driver.c
(networkAddMasqueradingIptablesRules): Fix spelling and grammar.

14 years agoFix up confusing indentation in qemudDomainAttachHostPciDevice.
Chris Lalancette [Tue, 27 Jul 2010 16:58:48 +0000 (12:58 -0400)]
Fix up confusing indentation in qemudDomainAttachHostPciDevice.

Signed-off-by: Chris Lalancette <clalance@redhat.com>
14 years agomaint: turn on gcc logical-op checking
Eric Blake [Fri, 23 Jul 2010 20:28:31 +0000 (14:28 -0600)]
maint: turn on gcc logical-op checking

This would have detected the bug in commit 38ad33931 (Aug 09), which
we missed until commit f828ca35 (Jul 10); over 11 months later.

However, on Fedora 13, it also triggers LOTS of warnings from
the libcurl-devel header for two files:

esx/esx_vi.c: In function 'esxVI_CURL_Perform':
esx/esx_vi.c:232: error: logical '&&' with non-zero constant will always evaluate as true [-Wlogical-op]
esx/esx_vi.c:232: error: logical '&&' with non-zero constant will always evaluate as true [-Wlogical-op]
esx/esx_vi.c:232: error: logical '&&' with non-zero constant will always evaluate as true [-Wlogical-op]
...
xenapi/xenapi_driver.c: In function 'call_func':
xenapi/xenapi_driver.c:1872: error: logical '&&' with non-zero constant will always evaluate as true [-Wlogical-op]
xenapi/xenapi_driver.c:1872: error: logical '&&' with non-zero constant will always evaluate as true [-Wlogical-op]
xenapi/xenapi_driver.c:1872: error: logical '&&' with non-zero constant will always evaluate as true [-Wlogical-op]
...

libcurl allows to disable the type-checking code that triggers those
warnings, along with the reduction in type-safety of calls to some
libcurl functions. I figure this is worth the improved compiler
checking throughout the rest of libvirt.

* acinclude.m4 (--enable-compile-warnings=error): Add -Wlogical-op.
* configure.ac: Add -DCURL_DISABLE_TYPECHECK to LIBCURL_CFLAGS to
avoid compilation warning.

Suggested by Daniel P. Berrange.
Tweaked by Matthias Bolte.

14 years agolibvirt-guests: add reload, condrestart
Eric Blake [Tue, 27 Jul 2010 18:21:31 +0000 (12:21 -0600)]
libvirt-guests: add reload, condrestart

Optional per LSB, but required by Fedora:
https://fedoraproject.org/wiki/Packaging/SysVInitScript

* daemon/libvirt-guests.init.in (main): Add more required
commands.

14 years agolibvirt-guests: enhance status
Eric Blake [Tue, 27 Jul 2010 19:50:35 +0000 (13:50 -0600)]
libvirt-guests: enhance status

LSB and https://fedoraproject.org/wiki/Packaging/SysVInitScript
require status to output something useful, rather than just use
the exit code.

* daemon/libvirt-guests.init.in (rh_status): Break into new routine,
and provide output.
(usage): Document status.

14 years agolibvirt-guests: detect invalid arguments
Eric Blake [Tue, 27 Jul 2010 18:05:27 +0000 (12:05 -0600)]
libvirt-guests: detect invalid arguments

Reject extra arguments.
Return the correct status for unknown arguments, as mandated by
https://fedoraproject.org/wiki/Packaging/SysVInitScript
Add --help, as a permitted extension.

* daemon/libvirt-guests.init.in (usage): New function.  Use it in
more places, and return correct value.

14 years agoInvert logic for checking for QEMU disk cache options
Daniel P. Berrange [Tue, 27 Jul 2010 10:21:10 +0000 (11:21 +0100)]
Invert logic for checking for QEMU disk cache options

QEMU has had two different syntax for disk cache options

 Old: on|off
 New: writeback|writethrough|none

QEMU recently added another 'unsafe' option which broke the
libvirt check. We can avoid this & future breakage, if we
do a negative check for the old syntax, instead of a positive
check for the new syntax

* src/qemu/qemu_conf.c: Invert cache option check

14 years agoqemu: Allow setting boot menu on/off
Cole Robinson [Mon, 26 Jul 2010 14:28:58 +0000 (10:28 -0400)]
qemu: Allow setting boot menu on/off

Add a new element to the <os> block:

  <bootmenu enable="yes|no"/>

Which maps to -boot,menu=on|off on the QEMU command line.

I decided to use an explicit 'enable' attribute rather than just make the
bootmenu element boolean. This allows us to treat lack of a bootmenu element
as 'use hypervisor default'.

14 years agodocs: Link wiki FAQ to main page
Cole Robinson [Mon, 26 Jul 2010 14:32:42 +0000 (10:32 -0400)]
docs: Link wiki FAQ to main page

Since DV recommended keeping the build instructions distributed with the
source, move them from the old FAQ to the downloads page.

14 years agoqemu: Error on unsupported graphics config
Cole Robinson [Mon, 26 Jul 2010 14:30:01 +0000 (10:30 -0400)]
qemu: Error on unsupported graphics config

Throw an explicit error if multiple graphics devices are specified, or
an unsupported type is specified (rdp).

14 years agolibvirt-guests: Don't throw errors if libvirtd is not installed
Jiri Denemark [Tue, 27 Jul 2010 12:21:51 +0000 (14:21 +0200)]
libvirt-guests: Don't throw errors if libvirtd is not installed

When only client parts of libvirt are installed (i.e., no libvirtd
daemon), libvirt-guests init script in its default configuration would
throw seriously looking errors during host shutdown:

Running guests on default URI: error: unable to connect to
'/var/run/libvirt/libvirt-sock', libvirtd may need to be started: No
such file or directory
error: failed to connect to the hypervisor

This patch changes the script to print rather harmless message in that
situation:

Running guests on default URI: libvirtd not installed; skipping this
URI.

14 years agoForce FLR on for buggy SR-IOV devices.
Chris Lalancette [Fri, 23 Jul 2010 19:03:29 +0000 (15:03 -0400)]
Force FLR on for buggy SR-IOV devices.

Some buggy PCI devices actually support FLR, but
forget to advertise that fact in their PCI config space.
However, Virtual Functions on SR-IOV devices are
*required* to support FLR by the spec, so force has_flr
on if this is a virtual function.

Signed-off-by: Chris Lalancette <clalance@redhat.com>
14 years agobuild: fix VPATH builds
Eric Blake [Mon, 26 Jul 2010 16:38:30 +0000 (10:38 -0600)]
build: fix VPATH builds

After the recent libvirt-qemu library addition, VPATH builds fail with:

  CC     libvirt_qemu_la-libvirt-qemu.lo
In file included from ../../src/libvirt-qemu.c:29:
../../include/libvirt/libvirt-qemu.h:17:22: error: libvirt.h: No such file or directory
...
  CCLD   libvirt-qmeu.la
/usr/bin/ld: cannot open linker script file libvirt_qemu.syms: No such file or directory

This fixes both issues (there are still some documentation VPATH issues,
but those don't show up with 'make check').

* configure.ac (LIBVIRT_QEMU_SYMBOL_FILE): While libvirt.syms is
generated and lives in $(builddir), libvirt_qemu.syms is static
and lives in $(srcdir).
* include/libvirt/libvirt-qemu.h (includes): Pull in libvirt.h via
the public location, since this is a public header.

14 years agopciResetDevice: use inactive devices to determine safe reset
Chris Wright [Mon, 26 Jul 2010 16:43:04 +0000 (18:43 +0200)]
pciResetDevice: use inactive devices to determine safe reset

When doing a PCI secondary bus reset, we must be sure that there are no
active devices on the same bus segment.  The active device tracking is
designed to only track host devices that are active in use by guests.
This ignores host devices that are actively in use by the host.  So the
current logic will reset host devices.

Switch this logic around and allow sbus reset when we are assigning all
devices behind a bridge to the same guest at guest startup or as a result
of a single attach-device command.

* src/util/pci.h: change signature of pciResetDevice to add an
  inactive devices list
* src/qemu/qemu_driver.c src/xen/xen_driver.c: use (or not) the new
  functionality of pciResetDevice() depending on the place of use
* src/util/pci.c: implement the interface and logic changes

14 years agoqemudDomainAttachHostPciDevice refactor to use new helpers
Chris Wright [Mon, 26 Jul 2010 16:34:24 +0000 (18:34 +0200)]
qemudDomainAttachHostPciDevice refactor to use new helpers

- src/qemu/qemu_driver.c: Eliminate code duplication by using the new
  helpers qemuPrepareHostdevPCIDevices and qemuDomainReAttachHostdevDevices.
  This reduces the number of open coded calls to pciResetDevice.

14 years agoAdd helpers qemuPrepareHostdevPCIDevice and qemuDomainReAttachHostdevDevices
Chris Wright [Mon, 26 Jul 2010 16:23:17 +0000 (18:23 +0200)]
Add helpers qemuPrepareHostdevPCIDevice and qemuDomainReAttachHostdevDevices

- src/qemu/qemu_driver.c: These new helpers take hostdev list and count
  directly rather than getting them indirectly from domain definition.
  This will allow reuse for the attach-device case.

14 years agoqemuGetPciHostDeviceList take hostdev list directly
Chris Wright [Mon, 26 Jul 2010 16:17:20 +0000 (18:17 +0200)]
qemuGetPciHostDeviceList take hostdev list directly

- src/qemu/qemu_driver.c: Update qemuGetPciHostDeviceList to take a
  hostdev list and count directly, rather than getting this indirectly
  from domain definition. This will allow reuse for the attach-device case.

14 years agoesx: Support vSphere 4.1
Matthias Bolte [Wed, 21 Jul 2010 23:19:04 +0000 (01:19 +0200)]
esx: Support vSphere 4.1

Also accept version > 4.1, but output a warning.

14 years agoesx: Add vpx:// scheme to allow direct connection to a vCenter
Matthias Bolte [Sun, 18 Jul 2010 15:27:05 +0000 (17:27 +0200)]
esx: Add vpx:// scheme to allow direct connection to a vCenter

Add a pointer to the primary context of a connection and use it in all
driver functions that don't dependent on the context type. This includes
almost all functions that deal with a virDomianPtr. Therefore, using
a vpx:// connection allows you to perform all the usual domain related
actions like start, destroy, suspend, resume, dumpxml etc.

Some functions that require an explicitly specified ESX server don't work
yet. This includes the host UUID, the hostname, the general node info, the
max vCPU count and the free memory. Also not working yet are migration and
defining new domains.

14 years agoesx: Don't ignore the vcenter query parameter
Matthias Bolte [Tue, 13 Jul 2010 20:44:07 +0000 (22:44 +0200)]
esx: Don't ignore the vcenter query parameter

Since 070f61002f47b602c15d1e4950a122ac9edefe1b the vcenter query
parameter has been ignored, because the refactoring to use
esxUtil_ParseQuery was incomplete. This effectively broke migration,
because the vcenter query parameter is essential for a migration.

14 years agoesx: Add autodetection for the SCSI controller model
Matthias Bolte [Tue, 6 Jul 2010 17:02:48 +0000 (19:02 +0200)]
esx: Add autodetection for the SCSI controller model

This works for file-backed SCSI disk device with a datastore
related source path.

14 years agoesx: Allow 'vmpvscsi' as SCSI controller model
Matthias Bolte [Thu, 24 Jun 2010 14:58:54 +0000 (16:58 +0200)]
esx: Allow 'vmpvscsi' as SCSI controller model

14 years agosecaatest: Fix compilation
Matthias Bolte [Fri, 23 Jul 2010 16:49:48 +0000 (18:49 +0200)]
secaatest: Fix compilation

Since 68719c4bddb85fbcc931a5b7d99ac7c8a0af09b0 virSecurityDriverStartup
takes and additional parameter to control disk format probing.

Pass false as third parameter.

14 years agovirt-aa-helper-test: Fix failure due to the new disk format probing option
Matthias Bolte [Fri, 23 Jul 2010 16:43:37 +0000 (18:43 +0200)]
virt-aa-helper-test: Fix failure due to the new disk format probing option

Commit 68719c4bddb85fbcc931a5b7d99ac7c8a0af09b0 added the disk format
probing option. This makes virt-aa-helper-test fail because the domain
config didn't specifiy the disk format and it didn't pass '-p 1' to
virt-aa-helper to allow disk format probing.

Specify the disk format in the domain config. Pass the '-p 1' option
to virt-aa-helper for the test case with two disks. This way this test
also covers this new option.

14 years agovirt-aa-helper: Make getopt accept the p option
Matthias Bolte [Fri, 23 Jul 2010 16:41:42 +0000 (18:41 +0200)]
virt-aa-helper: Make getopt accept the p option

Commit 68719c4bddb85fbcc931a5b7d99ac7c8a0af09b0 added the
p option to control disk format probing, but it wasn't added
to the getopt_long optstring parameter.

Add the p option to the getopt_long optstring parameter.

14 years agovirt-aa-helper: Fix return value of add_file_path
Matthias Bolte [Fri, 23 Jul 2010 16:27:43 +0000 (18:27 +0200)]
virt-aa-helper: Fix return value of add_file_path

Commit a8853344994a7c6aaca882a5e949ab5536821ab5 added this
function and wrapped vah_add_file in it. vah_add_file may
return -1, 0, 1. It returns 1 in case the call to valid_path
detects a restricted file. The original code treated a return
value != 0 as error. The refactored code treats a return
value < 0 as error. This triggers segfault in virt-aa-helper
and breaks virt-aa-helper-test for the restricted file tests.

Make sure that add_file_path returns -1 on error.

14 years agovirt-aa-helper: Ignore open errors again
Matthias Bolte [Fri, 23 Jul 2010 16:09:46 +0000 (18:09 +0200)]
virt-aa-helper: Ignore open errors again

virt-aa-helper used to ignore errors when opening files.
Commit a8853344994a7c6aaca882a5e949ab5536821ab5 refactored
the related code and changed this behavior. virt-aa-helper
didn't ignore open errors anymore and virt-aa-helper-test
fails.

Make sure that virt-aa-helper ignores open errors again.

14 years agoqemu-api: avoid build failure
Eric Blake [Fri, 23 Jul 2010 23:01:10 +0000 (17:01 -0600)]
qemu-api: avoid build failure

* src/remote_protocol-structs: Tweak to match intentional type
change (with no ABI change) in remote protocol.

14 years agoAdd tests for the new Qemu namespace XML.
Chris Lalancette [Tue, 20 Apr 2010 21:22:49 +0000 (17:22 -0400)]
Add tests for the new Qemu namespace XML.

Thanks to DV for knocking together the Relax-NG changes
quickly for me.

Changes since v1:
 - Change the domain.rng to correspond to the new schema
 - Don't allocate caps->ns in testQemuCapsInit since it is a static table

Changes since v2:
 - Change domain.rng to add restrictions on allowed environment names

Changes since v3:
 - Remove a bogus comment in the tests

Signed-off-by: Chris Lalancette <clalance@redhat.com>
14 years agoQemu remote protocol.
Chris Lalancette [Sat, 17 Apr 2010 02:09:25 +0000 (22:09 -0400)]
Qemu remote protocol.

Since we are adding a new "per-hypervisor" protocol, we
make it so that the qemu remote protocol uses a new
PROTOCOL and PROGRAM number.  This allows us to easily
distinguish it from the normal REMOTE protocol.

This necessitates changing the proc in remote_message_header
from a "remote_procedure" to an "unsigned", which should
be the same size (and thus preserve the on-wire protocol).

Changes since v1:
 - Fixed up a couple of script problems in remote_generate_stubs.pl
 - Switch an int flag to a bool in dispatch.c

Changes since v2:
 - None

Changes since v3:
 - Change unsigned proc to signed proc, to conform to spec

Signed-off-by: Chris Lalancette <clalance@redhat.com>
14 years agoQemu arbitrary monitor commands.
Chris Lalancette [Sat, 17 Apr 2010 02:12:45 +0000 (22:12 -0400)]
Qemu arbitrary monitor commands.

Implement the qemu driver's virDomainQemuMonitorCommand
and hook it into the API entry point.

Changes since v1:
 - Rename the (external) qemuMonitorCommand to qemuDomainMonitorCommand
 - Add virCheckFlags to qemuDomainMonitorCommand

Changes since v2:
 - Drop ATTRIBUTE_UNUSED from the flags

Changes since v3:
 - Add a flag to priv so we only print out monitor command warning once.  Note
   that this has not been plumbed into qemuDomainObjPrivateXMLFormat or
   qemuDomainObjPrivateXMLParse, which means that if you run a monitor command,
   restart libvirtd, and then run another monitor command, you may get an
   an erroneous VIR_INFO.  It's a pretty minor matter, and I didn't think it
   warranted the additional code.
 - Add BeginJob/EndJob calls around EnterMonitor/ExitMonitor

Signed-off-by: Chris Lalancette <clalance@redhat.com>
14 years agoQemu Monitor API entry point.
Chris Lalancette [Tue, 13 Apr 2010 18:02:46 +0000 (14:02 -0400)]
Qemu Monitor API entry point.

Add the library entry point for the new virDomainQemuMonitorCommand()
entry point.  Because this is not part of the "normal" libvirt API,
it gets its own header file, library file, and will eventually
get its own over-the-wire protocol later in the series.

Changes since v1:
 - Go back to using the virDriver table for qemuDomainMonitorCommand, due to
   linking issues
 - Added versioning information to the libvirt-qemu.so

Changes since v2:
 - None

Changes since v3:
 - Add LGPL header to libvirt-qemu.c
 - Make virLibConnError and virLibDomainError macros instead of function calls

Changes since v4:
 - Move exported symbols to libvirt_qemu.syms

Signed-off-by: Chris Lalancette <clalance@redhat.com>
14 years agoHandle arbitrary qemu command-lines in qemuParseCommandLine.
Chris Lalancette [Mon, 19 Apr 2010 20:34:00 +0000 (16:34 -0400)]
Handle arbitrary qemu command-lines in qemuParseCommandLine.

Now that we have the ability to specify arbitrary qemu
command-line parameters in the XML, use it to handle unknown
command-line parameters when doing a native-to-xml conversion.

Changes since v1:
 - Rename num_extra to num_args
 - Fix up a memory leak on an error path

Changes since v2:
 - Add a VIR_WARN when adding the argument via qemu:arg

Changes since v3:
 - None

Signed-off-by: Chris Lalancette <clalance@redhat.com>
14 years agoQemu arbitrary command-line arguments.
Chris Lalancette [Sat, 17 Apr 2010 01:49:31 +0000 (21:49 -0400)]
Qemu arbitrary command-line arguments.

Implement the qemu hooks for XML namespace data.  This
allows us to specify a qemu XML namespace, and then
specify:

<qemu:commandline>
 <qemu:arg value='arg'/>
 <qemu:env name='name' value='value'/>
</qemu:commandline>

In the domain XML.

Changes since v1:
 - Change the <qemu:arg>arg</qemu:arg> XML to <qemu:arg value='arg'/> XML
 - Fix up some memory leaks in qemuDomainDefNamespaceParse
 - Rename num_extra and extra to num_args and args, respectively
 - Fixed up some error messages
 - Make sure to escape user-provided data in qemuDomainDefNamespaceFormatXML

Changes since v2:
 - Add checking to ensure environment variable names are valid
 - Invert the logic in qemuDomainDefNamespaceFormatXML to return early

Changes since v3:
 - Change strspn() to c_isalpha() check of first letter of environment variable

Signed-off-by: Chris Lalancette <clalance@redhat.com>
14 years agoAdd namespace callback hooks to domain_conf.
Chris Lalancette [Sat, 17 Apr 2010 03:08:29 +0000 (23:08 -0400)]
Add namespace callback hooks to domain_conf.

This patch adds namespace XML parsers to be hooked into
the main domain parser.  This allows for individual hypervisor
drivers to add per-namespace XML into the main domain XML.

Changes since v1:
 - Use a statically declared table for caps->ns, removing the need to
   allocate/free it.

Changes since v2:
 - None

Changes since v3:
 - None

Signed-off-by: Chris Lalancette <clalance@redhat.com>
14 years agoFix SEGV on exit after domainEventDeregister()
Philipp Hahn [Tue, 13 Jul 2010 08:54:26 +0000 (10:54 +0200)]
Fix SEGV on exit after domainEventDeregister()

When the last callback is removed using domainEventDeregister(), the
events dispatcher is deregistered from the C-library, but
domainEventsCallbacks is still an empty list.
On shutdown __del__() deregisters the dispatacher again, which SEGVs

# You need the event-loop implementation from the Python examples;
# give the file a name which is importable by Python.
ln examples/domain-events/events-python/event-test.py eloop.py
python -c 'from eloop import *
import sys

def dump(*args): print " ".join(map(str, args))

virEventLoopPureStart()
c = libvirt.open("xen:///")
c.domainEventRegister(dump, None)
c.domainEventDeregister(dump)
sys.exit(0)'

domainEventDeregister() needs to delete domainEventCallbacks so subsequent
calls to __del__() and domainEventRegister() choose the right code paths.
Setting it to None is not enough, since calling domainEventRegiser() again
would trigger an TypeError.

Signed-off-by: Philipp Hahn <hahn@univention.de>
14 years agoFix .mailmap after accidental wrong committer address
Daniel P. Berrange [Fri, 23 Jul 2010 09:43:45 +0000 (10:43 +0100)]
Fix .mailmap after accidental wrong committer address

14 years agopciSharesBusWithActive fails to find multiple devices on bus
Chris Wright [Fri, 23 Jul 2010 09:25:24 +0000 (11:25 +0200)]
pciSharesBusWithActive fails to find multiple devices on bus

The first conditional is always true which means the iterator will
never find another device on the same bus.

    if (dev->domain != check->domain ||
        dev->bus != check->bus ||
  ----> (check->slot == check->slot &&
         check->function == check->function)) <-----

The goal of that check is to verify that the device is either:

  in a different pci domain
  on a different bus
  is the same identical device

This means libvirt may issue a secondary bus reset when there are
devices
on that bus that actively in use by the host or another guest.

* src/util/pci.c: fix a bogus test in pciSharesBusWithActive()

14 years agoFix incorrect use of private data in remote driver
Daniel P. Berrange [Thu, 22 Jul 2010 09:34:21 +0000 (05:34 -0400)]
Fix incorrect use of private data in remote driver

The remote driver is using the wrong privateData field in
a couple of functions. THis is harmless for stateful
drivers like QEMU/UML/LXC, but will crash with Xen

* src/remote/remote_driver.c: Fix use of privateData field

14 years agoSet a stable & high MAC addr for guest TAP devices on host
Daniel P. Berrange [Wed, 21 Jul 2010 10:08:52 +0000 (11:08 +0100)]
Set a stable & high MAC addr for guest TAP devices on host

A Linux software bridge will assume the MAC address of the enslaved
interface with the numerically lowest MAC addr. When the bridge
changes MAC address there is a period of network blackout, so a
change should be avoided. The kernel gives TAP devices a completely
random MAC address. Occassionally the random TAP device MAC is lower
than that of the physical interface (eth0, eth1etc) that is enslaved,
causing the bridge to change its MAC.

This change sets an explicit MAC address for all TAP devices created
using the configured MAC from the XML, but with the high byte set
to 0xFE. This should ensure TAP device MACs are higher than any
physical interface MAC.

* src/qemu/qemu_conf.c, src/uml/uml_conf.c: Pass in a MAC addr
  for the TAP device with high byte set to 0xFE
* src/util/bridge.c, src/util/bridge.h: Set a MAC when creating
  the TAP device to override random MAC

14 years agoFix PCI address assignment if no IDE controller is present
Daniel P. Berrange [Wed, 21 Jul 2010 11:40:21 +0000 (12:40 +0100)]
Fix PCI address assignment if no IDE controller is present

The PCI slot 1 must be reserved at all times, since PIIX3 is
always present, even if no IDE device is in use for guest disks

* src/qemu/qemu_conf.c: Always reserve slot 1 for PIIX3

14 years agolxc: force kill of init process by sending SIGKILL if needed
Ryota Ozaki [Thu, 22 Jul 2010 16:57:43 +0000 (18:57 +0200)]
lxc: force kill of init process by sending SIGKILL if needed

Init process may remain after sending SIGTERM for some reason.
For example, if original init program is used, it is definitely
not killed by SIGTERM.

* src/lxc/lxc_controller.c: kill with SIGKILL if SIGTERM wasn't
  sufficient

14 years agoRemove erroneous setting of return value to errno.
Laine Stump [Mon, 19 Jul 2010 23:52:57 +0000 (19:52 -0400)]
Remove erroneous setting of return value to errno.

One error exit in virStorageBackendCreateBlockFrom was setting the
return value to errno. The convention for volume build functions is to
return 0 on success or -1 on failure. Not only was it not necessary to
set the return value (it defaults to -1, and is set to 0 when
everything has been successfully completed), in the case that some
caller were checking for < 0 rather than != 0, they would incorrectly
believe that it completed successfully.

14 years agoChange virDirCreate to return -errno on failure.
Laine Stump [Mon, 19 Jul 2010 23:48:59 +0000 (19:48 -0400)]
Change virDirCreate to return -errno on failure.

virDirCreate also previously returned 0 on success and errno on
failure. This makes it fit the recommended convention of returning 0
on success, -errno (ie a negative number) on failure.

14 years agoMake virStorageBackendCopyToFD return -errno.
Laine Stump [Mon, 19 Jul 2010 23:35:30 +0000 (19:35 -0400)]
Make virStorageBackendCopyToFD return -errno.

Previously virStorageBackendCopyToFD would simply return -1 on
error. This made the error return from one of its callers inconsistent
(createRawFileOpHook is supposed to return -errno, but if
virStorageBackendCopyToFD failed, createRawFileOpHook would just
return -1). Since there is a useful errno in every case of error
return from virStorageBackendCopyToFD, and since the other uses of
that function ignore the return code (beyond simply checking to see if
it is < 0), this is a safe change.

14 years agoChange virFileOperation to return -errno (ie < 0) on error.
Laine Stump [Mon, 19 Jul 2010 23:25:58 +0000 (19:25 -0400)]
Change virFileOperation to return -errno (ie < 0) on error.

virFileOperation previously returned 0 on success, or the value of
errno on failure. Although there are other functions in libvirt that
use this convention, the preferred (and more common) convention is to
return 0 on success and -errno (or simply -1 in some cases) on
failure. This way the check for failure is always (ret < 0).

* src/util/util.c - change virFileOperation and virFileOperationNoFork to
                    return -errno on failure.

* src/storage/storage_backend.c, src/qemu/qemu_driver.c
  - change the hook functions passed to virFileOperation to return
    -errno on failure.

14 years agoDocument the memory balloon device
Daniel P. Berrange [Wed, 21 Jul 2010 10:53:56 +0000 (11:53 +0100)]
Document the memory balloon device

* formatdomain.html.in: Document <memballoon> element

14 years agoRe-arrange PCI device address assignment to match QEMU's default
Daniel P. Berrange [Tue, 20 Jul 2010 11:57:11 +0000 (12:57 +0100)]
Re-arrange PCI device address assignment to match QEMU's default

To try and ensure that people upgrading from old QEMU get guests
with the same PCI device ordering, change the way we assign addrs
to match QEMU's default order. This should make Windows less
annoyed.

* src/qemu/qemu_conf.c: Follow QEMU's default PCI ordering
  logic when assigning addresses
* tests/*.args: Update for changed PCI addresses

14 years agoExplicitly represent balloon device in XML and handle PCI address
Daniel P. Berrange [Thu, 15 Jul 2010 13:02:42 +0000 (14:02 +0100)]
Explicitly represent balloon device in XML and handle PCI address

To allow compatibility with older QEMU PCI device slot assignment
it is necessary to explicitly track the balloon device in the
XML. This introduces a new device

   <memballoon model='virtio|xen'/>

It can also have a PCI address, auto-assigned if necessary.

The memballoon will be automatically added to all Xen and QEMU
guests by default.

* docs/schemas/domain.rng: Add <memballoon> element
* src/conf/domain_conf.c, src/conf/domain_conf.h: parsing
  and formatting for memballoon device. Always add a memory
  balloon device to Xen/QEMU if none exists in XML
* src/libvirt_private.syms: Export memballoon model APIs
* src/qemu/qemu_conf.c, src/qemu/qemu_conf.h: Honour the
  PCI device address in memory balloon device
* tests/*: Update to test new functionality

14 years agoRearrange VGA/IDE controller address reservation
Daniel P. Berrange [Thu, 15 Jul 2010 13:44:18 +0000 (14:44 +0100)]
Rearrange VGA/IDE controller address reservation

The first VGA and IDE devices need to have fixed PCI address
reservations. Currently this is handled inline with the other
non-primary VGA/IDE devices. The fixed virtio balloon device
at slot 3, ensures auto-assignment skips the slots 1/2. The
virtio address will shortly become configurable though. This
means the reservation of fixed slots needs to be done upfront
to ensure that they don't get re-used for other devices.

This is more or less reverting the previous changeset:

  commit 83acdeaf173b2a1206b755c1ab317cac36facd90
  Author: Daniel P. Berrange <berrange@redhat.com>
  Date:   Wed Feb 3 16:11:29 2010 +0000

  Fix restore of QEMU guests with PCI device reservation

The difference is that this time, instead of unconditionally
reserving the address, we only reserve the address if it was
initially type=none. Addresses of type=pci were handled
earlier in process by qemuDomainPCIAddressSetCreate(). This
ensures restore step doesn't have problems

* src/qemu/qemu_conf.c: Reserve first VGA + IDE address
  upfront

14 years agoRemove inappropriate use of VIR_ERR_NO_SUPPORT
Daniel P. Berrange [Thu, 15 Jul 2010 13:27:27 +0000 (14:27 +0100)]
Remove inappropriate use of VIR_ERR_NO_SUPPORT

The VIR_ERR_NO_SUPPORT refers to an API which is not implemented.
There is a separate VIR_ERR_CONFIG_UNSUPPORTED for XML config
options that are not available with the current hypervisor.

* src/qemu/qemu_conf.c, src/qemu/qemu_driver.c: Remove
  many VIR_ERR_NO_SUPPORT replace with VIR_ERR_CONFIG_UNSUPPORTED

14 years agoFix a NULL dereference in the case that the arg in question
Chris Lalancette [Tue, 20 Jul 2010 16:51:12 +0000 (12:51 -0400)]
Fix a NULL dereference in the case that the arg in question
didn't exist.

Signed-off-by: Chris Lalancette <clalance@redhat.com>
14 years agoRemove bogus free of static strings
Daniel P. Berrange [Mon, 19 Jul 2010 17:59:34 +0000 (18:59 +0100)]
Remove bogus free of static strings

Remove bogus free of statically allocated strings introduced
in 03ca42046a54c5cfadb2e69194896abf06f6a10f

* src/conf/capabilities.c: Don't free static strings for
  default disk driver type/name

14 years agoFix a deadlock in bi-directional p2p concurrent migration.
Chris Lalancette [Thu, 15 Jul 2010 14:12:35 +0000 (10:12 -0400)]
Fix a deadlock in bi-directional p2p concurrent migration.

If you try to execute two concurrent migrations p2p
from A->B and B->A, the two libvirtd's will deadlock
trying to perform the migrations.  The reason for this is
that in p2p migration, the libvirtd's are responsible for
making the RPC Prepare, Migrate, and Finish calls.  However,
they are currently holding the driver lock while doing so,
which basically guarantees deadlock in this scenario.

This patch fixes the situation by adding
qemuDomainObjEnterRemoteWithDriver and
qemuDomainObjExitRemoteWithDriver helper methods.  The Enter
take an additional object reference, then drops both the
domain object lock and the driver lock.  The Exit takes
both the driver and domain object lock, then drops the
reference.  Adding calls to these Enter and Exit helpers
around remote calls in the various migration methods
seems to fix the problem for me in testing.

This should make the situation safe. The additional domain
object reference ensures that the domain object won't disappear
while this operation is happening.  The BeginJob that is called
inside of qemudDomainMigratePerform ensures that we can't execute a
second migrate (or shutdown, or save, etc) job while the
migration is active.  Finally, the additional check on the state
of the vm after we reacquire the locks ensures that we can't
be surprised by an external event (domain crash, etc).

Signed-off-by: Chris Lalancette <clalance@redhat.com>
14 years agoMake virsh setmaxmem balloon only when successful.
Chris Lalancette [Tue, 30 Mar 2010 13:03:26 +0000 (09:03 -0400)]
Make virsh setmaxmem balloon only when successful.

After playing around with virsh setmaxmem for a bit,
I ran into some surprising behavior; if a hypervisor does
not support the virDomainSetMaxMemory() API, but the value
specified for setmaxmem is less than the current amount
of memory in the domain, the domain would be ballooned
down *before* an error was reported.

To make this more consistent, run virDomainSetMaxMemory()
before trying to shrink; that way, if an error is thrown,
no changes to the running domain are made.

Signed-off-by: Chris Lalancette <clalance@redhat.com>
14 years agoUse unsigned long in cmdSetmem.
Chris Lalancette [Thu, 17 Jun 2010 19:36:36 +0000 (15:36 -0400)]
Use unsigned long in cmdSetmem.

The virsh command "setmem" takes as input a number that
should represent an unsigned long number of kilobytes.  Fix
cmdSetmem to properly parse this as an unsigned long instead
of an int.

Signed-off-by: Chris Lalancette <clalance@redhat.com>
14 years agofsync new storage volumes even if new volume was copied.
Laine Stump [Mon, 19 Jul 2010 22:59:07 +0000 (18:59 -0400)]
fsync new storage volumes even if new volume was copied.

Originally the storage volume files were opened with O_DSYNC to make
sure they were flushed to disk immediately. It turned out that this
was extremely slow in some cases, so the O_DSYNC was removed in favor
of just calling fsync() after all the data had been written. However,
this call to fsync was inside the block that is executed to zero-fill
the end of the volume file. In cases where the new volume is copied
from an old volume, and they are the same length, this fsync would
never take place.

Now the fsync is *always* done, unless there is an error (in which
case it isn't important, and is most likely inappropriate.

14 years agoDon't skip zero'ing end of volume file when inputvol is shorter than newvol
Laine Stump [Mon, 19 Jul 2010 22:53:38 +0000 (18:53 -0400)]
Don't skip zero'ing end of volume file when inputvol is shorter than newvol

A missing set of braces around an error condition caused us to skip
zero'ing out the remainder of a new volume file if the new volume was
longer than the original (the goto was supposed to be taken only in
the case of error, but was always being taken).

14 years agoAlways clear out the last_error in virshReportError.
Chris Lalancette [Fri, 18 Jun 2010 14:14:04 +0000 (10:14 -0400)]
Always clear out the last_error in virshReportError.

Otherwise you can get bogus "unknown error" printouts on
subsequent commands.

Signed-off-by: Chris Lalancette <clalance@redhat.com>
14 years agoFix up inconsistent virsh option error reporting.
Chris Lalancette [Thu, 17 Jun 2010 18:58:55 +0000 (14:58 -0400)]
Fix up inconsistent virsh option error reporting.

The virsh option error reporting was not being used
consistently; some commands would spit out errors on
missing required options while others would just silently fail.
However, vshCommandOptString knows which ones are required
and which ones aren't, so make it spit out an error where
appropriate.  The rest of the patch is just cleaning up
the uses of vshCommandOptString to deal with the new error
reporting.

Signed-off-by: Chris Lalancette <clalance@redhat.com>
14 years agoUse the extract backing store format in storage volume lookup
Daniel P. Berrange [Fri, 9 Jul 2010 10:28:40 +0000 (11:28 +0100)]
Use the extract backing store format in storage volume lookup

The storage volume lookup code was probing for the backing store
format, instead of using the format extracted from the file
itself. This meant it could report in accurate information. If
a format is included in the file, then use that in preference,
with probing as a fallback.

* src/storage/storage_backend_fs.c: Use extracted backing store
  format