On Thu, Apr 27, 2023 at 12:00:59PM +0200, Andrea Bolognani wrote:
On Thu, Apr 27, 2023 at 11:50:05AM +0200, Michal Prívozník wrote:
On 4/27/23 11:41, Andrea Bolognani wrote:
+# headers for which we need to check actual usability. in most +# cases, checking for presence is enough (and it's way faster) +check_headers = [ + 'linux/kvm.h', +] + +foreach name : check_headers + if cc.check_header(name) + conf.set('WITH_@0@'.format(name.underscorify().to_upper()), 1) + endif +endforeach
One could argue, that the semantics of has_header() is broken then. What good is there to check if the host has a header file when compiler rejects it subsequently?
But leaving meson aside, shouldn't we just use check_header() for every header file then? I mean, this fixes this particular instance of the problem, but can't we hit it again with say ifaddrs.h or any other header file on the list?
We could. The Meson documentation recommends not doing this, as check_header() is (understandably) slower than has_header().
I haven't tried to see how much slower we're actually talking... Maybe for the 20-ish headers that we care about, it wouldn't make a measurable difference? Especially since the results of the check are cached between Meson runs.
I tried testing: diff --git a/meson.build b/meson.build index c15003ce02..3ae02ce4df 100644 --- a/meson.build +++ b/meson.build @@ -633,14 +633,14 @@ if host_machine.system() == 'freebsd' endif foreach name : headers - if cc.has_header(name) + if cc.check_header(name) conf.set('WITH_@0@'.format(name.underscorify().to_upper()), 1) endif endforeach # check for kernel header required by src/util/virnetdevbridge.c if host_machine.system() == 'linux' - if not cc.has_header('linux/sockios.h') + if not cc.check_header('linux/sockios.h') error('You must install kernel-headers in order to compile libvirt with QEMU or LXC support') endif endif and could not measure any deterministic change in execution time before/after With regards, Daniel -- |: https://berrange.com -o- https://www.flickr.com/photos/dberrange :| |: https://libvirt.org -o- https://fstop138.berrange.com :| |: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|