[libvirt] [PATCH] virtperf: Revert PERF_COUNT_HW_REF_CPU_CYCLES conditional

Olaf Hering olaf at aepfle.de
Tue Mar 14 13:18:48 UTC 2017


Am Tue, 14 Mar 2017 14:05:37 +0100
schrieb Martin Kletzander <mkletzan at redhat.com>:

> However without that commit we'll be back to broken code that won't compile on some distros.  So unless I missed something (if I did, it should be added to the commit message), there should rather be a configure check that defines something similar to other code, e.g. HAVE_PERF_CPU_CYCLES and then that should be used in the conditional.

There are other places in that and other files which can not be compiled. The obvious fix is to just add a #ifndef x;#define x 0 to get libvirt going. The question is still unanswered if configure should just error out and refuse to go further, or if the few missing pieces should be supplied manually because the affected functionality is optional.

See attached patches for how I deal with it.

Olaf
-------------- next part --------------
A non-text attachment was scrubbed...
Name: libvirt-mount.patch
Type: text/x-patch
Size: 613 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20170314/a3a6d64f/attachment-0002.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: libvirt-perf.patch
Type: text/x-patch
Size: 523 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20170314/a3a6d64f/attachment-0003.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 195 bytes
Desc: Digitale Signatur von OpenPGP
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20170314/a3a6d64f/attachment-0001.sig>


More information about the libvir-list mailing list