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>

configure: move polkit check to its own file

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

configure: move firewalld check to its own file

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

configure: move with-driver-modules check to its own file

Rename DRIVER_MODULE_(LDFLAGS|LIBS|CFLAGS) to unify the naming.

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

configure: move dlopen check to its own file

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

m4/virt-libxml: use LIBVIRT_CHECK_PKG

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

m4/virt-libxml: use only pkg-config to check for libxml2

Pkg-config file libxml-2.0.pc was introduced in libxml2-2.2.4 and we
require at least 2.6.0 so the crazy xml2-config check can be dropped.

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

configure: move libxml check to its own file

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

m4/virt-lib: add new parameter to override default fail action

This allows to use two or more LIBVIRT_CHECK_(LIB|LIB_ALT|PKG) macros
for one library.  For example, when we check for existence of
pgk-config file of some library and we know that not all versions of
that library that libvirt requires provides this file, we can fallback
to use library check.

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

m4/virt: introduce LIBVIRT_ARG_* macros for all library checks

Usage of AC_REQUIRE will mess with order how LIBVIRT_CHECK_* macros
are composed into configure.ac.  This ensures that the output of
configure --help is properly ordered and grouped into sections.

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

m4/virt-lib: move LIBVIRT_ARG_WITH out of LIBVIRT_CHECK_(LIB|LIB_ALT|PKG)

Following patch will update LIBVIRT_CHECK_(LIB|LIB_ALT|PKG) macros in
a way that you will be able to call a another macro as a fallback if
the first one fails.  To allow that, we need to move the
LIBVIRT_ARG_WITH out of those macro to not have two or more same lines
in output of "configure --help".

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

configure: use LIBVIRT_ARG_ENABLE macro

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

configure: use LIBVIRT_ARG_WITH(_ALT) macros

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

m4/virt-lib: introduce LIBVIRT_ARG_ENABLE macro

This macro helps create unified output of "configure --help".

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

m4/virt-lib: introduce LIBVIRT_ARG_WITH(_ALT) macro

These macros help create unified output of "configure --help".

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

m4/virt: use LIBVIRT_RESULT macro instead of AC_MSG_NOTICE

The LIBVIRT_RESULT does a nice formatting of the output and ensures
that the format is unified.

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

m4/virt: create result macros for the remaining virt check

All checks that prints result at the end of configure uses
LIBVIRT_RESULT_${CHECK_NAME}.  Create those macros for remaining check.

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

configure: move external programs check to its own file

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