]> git.proxmox.com Git - mirror_qemu.git/commit
crypto: move crypto objects out of libqemuutil.la
authorDaniel P. Berrange <berrange@redhat.com>
Wed, 2 Sep 2015 09:57:27 +0000 (10:57 +0100)
committerDaniel P. Berrange <berrange@redhat.com>
Tue, 15 Sep 2015 13:18:18 +0000 (14:18 +0100)
commitfb37726db77b21f3731b90693d2c93ade1777528
tree03e6990ab7b59cd867f3a27925246107ffe9600a
parentb124533e069a6624316da2096d170b0bd9197d86
crypto: move crypto objects out of libqemuutil.la

Future patches will be adding more crypto related APIs which
rely on QOM infrastructure. This creates a problem, because
QOM relies on library constructors to register objects. When
you have a file in a static .a library though which is only
referenced by a constructor the linker is dumb and will drop
that file when linking to the final executable :-( The only
workaround for this is to link the .a library to the executable
using the -Wl,--whole-archive flag, but this creates its own
set of problems because QEMU is relying on lazy linking for
libqemuutil.a. Using --whole-archive majorly increases the
size of final executables as they now contain a bunch of
object code they don't actually use.

The least bad option is to thus not include the crypto objects
in libqemuutil.la, and instead define a crypto-obj-y variable
that is referenced directly by all the executables that need
this code (tools + softmmu, but not qemu-ga). We avoid pulling
entire of crypto-obj-y into the userspace emulators as that
would force them to link to gnutls too, which is not required.

Signed-off-by: Daniel P. Berrange <berrange@redhat.com>
Makefile
Makefile.objs
Makefile.target
crypto/Makefile.objs
tests/Makefile