On Mon, Jul 30, 2018 at 6:38 PM Eric Blake <eblake(a)redhat.com> 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