[libvirt] [PATCH v2] Move the FIPS detection from capabilities
Eric Blake
eblake at redhat.com
Thu Sep 18 19:57:34 UTC 2014
On 09/18/2014 12:01 PM, Pavel Hrdina wrote:
> We are not detecting the presence of FIPS from QEMU, but from procfs and
> that means it's not QEMU capability. It was decided that we will pass
> this flag to QEMU even if it's not supported by old QEMU binaries.
>
> This patch also reverts changes done by commit a21cfb0f to
> qemucapabilitestest and implements a new test case in qemuxml2argvtest.
>
> Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1135431
>
> Signed-off-by: Pavel Hrdina <phrdina at redhat.com>
> ---
> @@ -183,19 +176,16 @@ mymain(void)
>
> data.xmlopt = xmlopt;
>
> -#define DO_TEST_FULL(name, use_fips) \
> - data.base = name; \
> - data.fips = use_fips; \
> - if (virtTestRun(name, testQemuCaps, &data) < 0) \
> +#define DO_TEST(name) \
> + data.base = name; \
> + if (virtTestRun(name, testQemuCaps, &data) < 0) \
We are not very consistent on whether multiline macros should align the
\ to the same column.
> ret = -1
Eww - we really did that in a multiline macro? I'd much rather fix
things to use:
do {
data.base = name;
if (virtTestRun(name, testQemuCaps, &data) < 0)
ret = -1;
} while (0)
as long as we are touching the code.
ACK.
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 539 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20140918/c3b9da13/attachment-0001.sig>
More information about the libvir-list
mailing list