bhyvexml2argvdata: Add mac address to <interface/>

In the next patch we will need a stable mac address for
<interface/>.

Signed-off-by: Michal Privoznik <mprivozn@redhat.com>

bhyvexml2argvdata: Fix <drive/> addresses

After c07d1c1c4f got merged it uncovered couple of broken domain
XMLs for bhyvexml2argv test. Some disk drives had incompatible
type of address configured.

Signed-off-by: Michal Privoznik <mprivozn@redhat.com>

networkxml2conftest: s/lo/lo0/ on non-Linux

After 478ddedc12 a bug is fixed where we wrongly presumed loopack
device name on non-Linux systems. It's lo0. However, the fix is
not reflected in the tests which are failing now.

Signed-off-by: Michal Privoznik <mprivozn@redhat.com>

src: Build libvirt_nss.la iff WITH_NSS

If the nss module is disabled we don't need to build the
supplementary library for it either.

Signed-off-by: Michal Privoznik <mprivozn@redhat.com>

tests: Run nss*test iff WITH_NSS

If the nss module is disabled we shouldn't run the tests that
include it either.

Signed-off-by: Michal Privoznik <mprivozn@redhat.com>

virt-nss: Introduce more checks before enabling NSS

The plugin depends on more modules than we currently check for,
i.e. network driver and yajl library.

Signed-off-by: Michal Privoznik <mprivozn@redhat.com>

tests: Move test_libraries a bit higher

In the Makefile in tests/ we initialize couple of variables like
test_programs, test_libraries and test_helpers. These variables
contain all the targets that we need to build in order to run
the test suite. So we initialize test_programs and test_helpers
and then conditionally add targets to them depending on what we
are building with. Then we repeat the same process with
test_libraries. It makes no sense to have two separate if-endif
sequences.

Signed-off-by: Michal Privoznik <mprivozn@redhat.com>

tests: Run virmacmaptest iff WITH_YAJL

Since the internal implementation relies on a json parser being
available, it make no sense to run this test if there's none
available.

Signed-off-by: Michal Privoznik <mprivozn@redhat.com>

virmacmap: Don't use hash table dataFree callback

Due to nature of operations we do over the string list (more
precisely due to how virStringListRemove() works), it is not the
best idea to use dataFree callback. Problem is, on MAC address
remove, the string list remove function modifies the original
list in place. Then, virHashUpdateEntry() is called which frees
all the data stored in the list rendering @newMacsList point to
freed data.

==16002== Invalid read of size 8
==16002==    at 0x50BC083: virFree (viralloc.c:582)
==16002==    by 0x513DC39: virStringListFree (virstring.c:251)
==16002==    by 0x51089B4: virMacMapHashFree (virmacmap.c:67)
==16002==    by 0x50EF30B: virHashAddOrUpdateEntry (virhash.c:352)
==16002==    by 0x50EF4FD: virHashUpdateEntry (virhash.c:415)
==16002==    by 0x5108BED: virMacMapRemoveLocked (virmacmap.c:129)
==16002==    by 0x51092D5: virMacMapRemove (virmacmap.c:346)
==16002==    by 0x402F02: testMACRemove (virmacmaptest.c:107)
==16002==    by 0x403F15: virTestRun (testutils.c:180)
==16002==    by 0x4032C4: mymain (virmacmaptest.c:205)
==16002==    by 0x405A3B: virTestMain (testutils.c:992)
==16002==    by 0x403D87: main (virmacmaptest.c:237)
==16002==  Address 0xdd5a4d0 is 0 bytes inside a block of size 24 free'd
==16002==    at 0x4C2AD6F: realloc (vg_replace_malloc.c:693)
==16002==    by 0x50BB99B: virReallocN (viralloc.c:245)
==16002==    by 0x513DC0B: virStringListRemove (virstring.c:235)
==16002==    by 0x5108BA6: virMacMapRemoveLocked (virmacmap.c:124)
==16002==    by 0x51092D5: virMacMapRemove (virmacmap.c:346)
==16002==    by 0x402F02: testMACRemove (virmacmaptest.c:107)
==16002==    by 0x403F15: virTestRun (testutils.c:180)
==16002==    by 0x4032C4: mymain (virmacmaptest.c:205)
==16002==    by 0x405A3B: virTestMain (testutils.c:992)
==16002==    by 0x403D87: main (virmacmaptest.c:237)

Signed-off-by: Michal Privoznik <mprivozn@redhat.com>

virmacmap: Fix variable handling

In virMacMapRemoveLocked() we have two variables: @macsList and
@newMacsList. Obviously, @newMacsList is supposed to hold pointer
to modified list but in fact it holds pointer to the old list.
It's confusing.

Signed-off-by: Michal Privoznik <mprivozn@redhat.com>

maint: Sync bootstrap after gnulib update

7fa7fe22d844 updated gnulib to the latest upstream commit,
but forgot to include the corresponding bootstrap changes.

maint: update to latest gnulib

Required for the copyright year bump to keep 'make syntax-check'
happy, and also pulls in several portability fixes.

* .gnulib: Update to latest.
* bootstrap: Resync from upstream.

Signed-off-by: Michal Privoznik <mprivozn@redhat.com>

NEWS: Update after qemu namespace fix

Signed-off-by: Michal Privoznik <mprivozn@redhat.com>

docs: Update docs for recently added physical adjustments

Add the description for <physical>... One is a new feature - the API
and the other is an improvement for the volume xml output.

vz: get disks statistics for CTs

A CT disk statistics is reported with prefix "hdd" and we should use
it to extract data.

Signed-off-by: Maxim Nestratov <mnestratov@virtuozzo.com>

vz: set boot from disk for CT only when there is no root filesystem

Before, boot devices information for CTs was always empty and we
didn't indicate that containers can boot from disk.

Signed-off-by: Maxim Nestratov <mnestratov@virtuozzo.com>

vz: report disks either as disks or filesystems depending on original xml

Virtuozzo SDK interface doesn't differ filesystems from disks and sees them as disks.
Before, we always mistakenly presented disks based on files as filesystems, which is
not completely correct. Now we are going to show either disks or filesystems depending
on a hint, which uses boot device section of VZ config. Though this information
doesn't change booting order of a CT, it is used by vz libvirt interface as a hint
for libvirt representation of disks. Since now, if we have filesystems in input xml,
then we add them to VZ booting devices list and rely on this information to show
corresponding libvirt xml.

Signed-off-by: Maxim Nestratov <mnestratov@virtuozzo.com>

vz: don't add implicit devices for CTs

Implicit devices like controllers are confusing for CTs and
function virDomainDefAddImplicitDevices never intended to be called
for CTs.

Signed-off-by: Maxim Nestratov <mnestratov@virtuozzo.com>

vz: report "scsi" bus for disks when nothing was set explixitly

This is necessary to show CTs created out of libvirt correctly.

Signed-off-by: Maxim Nestratov <mnestratov@virtuozzo.com>

tests: enable virpolkittest on any ELF platform

This tests uses preload, which should work on any ELF-based platform
(and indeed it passes on Linux, GNU/kFreeBSD, and FreeBSD).

Also remove the WITH_DBUS conditional, as the test is already built
based on that conditional.

m4/virt-arg: Rename LIBVIRT_ARG_WITH* macros

LIBVIRT_ARG_WITH_ALT is more generic than LIBVIRT_ARG_WITH, which
is tailored at switching features on and off.

Rename the macros according to their intended purpose, and add
some documentation to help developers pick between the two.

m4/virt-arg: Move LIBVIRT_ARG_* macros to their own file

They're not just used, or supposed to be just used, for
library-related arguments, so they deserve to have their
own separate file.

m4/virt-gnutls: properly backup CFLAGS and LIBS for AC_CHECK_* macros

Commit 943ddcb7120 partially fixed this bug in gnutls configure code.
However we also need to backup and modify CFLAGS and do if for
AC_CHECK_HEADERS as well.

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

NEWS: Update after virtio console on ppc64 fix

m4/virt-gnutls: use AC_CHECK_FUNCS to detect availability of functions

Sigh, autoconf has AC_CHECK_FUNC and AC_CHECK_FUNCS and both check
whether some function exists or not.  However in addition to the obvious
difference that the later is able to check multiple functions it also
defines HAVE_FUNCTION_NAME for each function.

Commit 680d2f49da moved the code to separate file, but also blindly
changed the macro to AC_CHECK_FUNC without confronting the
documentation.

Reported-by: John Ferlan <jferlan@redhat.com>
<https://www.redhat.com/archives/libvir-list/2016-December/msg01086.html>

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

m4/virt-lib: use correct format of package name for AC_ARG_* macros

Introduced by commit 25034b3c40 and reused by commit 4519e94b743.
This patch silences warning printed by configure:

  configure: WARNING: unrecognized options: --with-qemu-user

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

m4/virt-loader-nvram: use quotation for list of loader:nvram pairs

The bug was introduced by commit 08c2d1480b.  The string must be quoted
because it is used as function argument.

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

NEWS: Fix indentation

Allow virtio-console on PPC64

virQEMUCapsSupportsChardev existing checks returns true
for spapr-vty alone. Instead verify spapr-vty validity
and let the logic to return true for other device types
so that virtio-console passes.

The non-pseries machines dont have spapr-vio-bus. So, the
function always returned false for them before.

Fixes - https://bugzilla.redhat.com/show_bug.cgi?id=1257813

Signed-off-by: Shivaprasad G Bhat <sbhat@linux.vnet.ibm.com>

qemu: clean out unused migrate to unix

configure: group all storage driver checks together

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

configure: move storage ZFS check to its own file

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

configure: move storage Gluster check to its own file

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

configure: move storage Sheepdog check to its own file

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

configure: move storage RBD check to its own file

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

configure: move storage disk check to its own file

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

configure: move storage mpath check to its own file

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

configure: move storage SCSI check to its own file

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

configure: move storage iSCSI check to its own file

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

configure: move storage LVM check to its own file

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

configure: move storage fs check to its own file

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

configure: move storage dir check to its own file

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

configure: move AppArmor secdriver check to its own file

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

configure: move SELinux secdriver check into its own file

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

configure: group all virtualization drivers checks together

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

configure: move parallels version definition to m4/virt-driver-vz

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

configure: move interface driver check to its own file

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

configure: move network driver check to its own file

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

configure: move libvirtd driver check to its own file

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

configure: move Hyper-V driver check to its own file

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

configure: move ESX driver check to its own file

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

configure: move vbox driver check to its own file

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

configure: move remote driver check to its own file

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

configure: move test driver check to its own file

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

configure: move PHYP driver check to its own file

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

m4/virt-driver-xenapi: use LIBVIRT_CHECK_LIB macro

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

configure: move XenAPI driver check to its own file

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

m4/virt-libxl: cleanup storing CFLAGS and LIBS variables

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

m4/virt-libxl: properly check for required libxl version

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

m4/virt-libxl: use LIBVIRT_CHECK_(PKG|LIB) macros

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

configure: move libxl driver check to its own file

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

configure: move XEN driver check to its own file

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

configure: move LXC driver check to its own file

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

configure: move qemu bridge helper check to qemu driver file

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

configure: move qemu privileges check to qemu driver file

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

configure: move QEMU driver check to its own file

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

configure: move VMware driver check to its own file

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

configure: move OpenVZ driver check to its own file

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

configure: group all Miscellaneous checks together

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

configure: move debug check to its own file

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

configure: move xml catalog file check to its own file

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

configure: move TLS priority check to its own file

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

configure: move default editor check to its own file

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

configure: move loader/nvram check to its own file

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

configure: move chrdev lock files check to its own file

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

configure: move sysctl config check to its own file

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

configure: move numad check to its own file

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

configure: move DTrace check to its own file

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

configure: group all library checks together

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

m4/virt-parted: use LIBVIRT_CHECK_PKG

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

configure: move libparted check to its own file

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

m4/virt-devmapper: use LIBVIRT_CHECK_(PKG|LIB)

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

configure: move devmapper check to its own file

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

configure: move pm-utils check to its own file

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

configure: move nwfilter check to its own file

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

configure: move version script check to one place

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

configure: move windres check to its own file

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

configure: move windows symbols check to its own file

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

configure: move MinGW check to its own file

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

configure: move Cygwin check to its own file

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

configure: move windows common check to its own file

This renames MSCOM_LIBS to WIN32_EXTRA_LIBS to make it consistent with
WIN32_EXTRA_CFLAGS.

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

configure: move libnl check to its own file

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

configure: move virtualport check to its own file

This patch moves the virtualport result out of *Driver* section because
it's not a driver.

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

configure: move macvtap check to its own file

This patch moves the macvtap result out of *Driver* section because
it's not a driver.

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

m4/virt-libpcap: error out if asked for libpcap but it was not found

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

m4/virt-libpcap: remove unreasonable dependency on QEMU

Libpcap is used by nwfilter driver and it could be build without QEMU
driver.

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

configure: move libpcap check to its own file

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

configure: move XDR check to its own file

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

configure: move atomic check to its own file

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>

configure: move pthread check it its own file

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>