On Mon, Jul 30, 2018 at 6:38 PM Eric Blake <[email protected]> wrote:

> On 07/29/2018 06:35 AM, Nir Soffer wrote:
> > On RHEL 7.5 we need to include <linux/falloc.h> for FALLOC_FL_* macros.
> > Without the macros, fallocate is never used and we fall back to manual
> > zeroing.
> >
>
> > @@ -41,6 +41,7 @@
> >   #include <sys/types.h>
> >   #include <sys/stat.h>
> >   #include <errno.h>
> > +#include <linux/falloc.h>   /* For FALLOC_FL_* on RHEL, glibc < 2.18 */
>
> This header is not available on all platforms; does it need a configure
> check, or do we not care about non-Linux compilation?


I'm not sure if nbdkit tries to be portable, but I think we should care.

Is #if defined(__linux__) good enough for this?

Nir
_______________________________________________
Libguestfs mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/libguestfs

Reply via email to