[LTP] [PATCH v2] Test statx syscall for SYNC_FLAGS

[Petr Vorel]

Hi Kewal,

> > One more thing, you need to define _GNU_SOURCE here as it's needed for
> > glibc and
> > thus used for detection in m4/ltp-statx.m4.


> > #define _GNU_SOURCE

> >> +#include <netdb.h>
> >> +#include <stdio.h>
> >> +#include <stdlib.h>
> >> +#include <linux/limits.h>
> >> +#include <sys/mount.h>
> >> +#include "tst_test.h"
> >> +#include "lapi/stat.h"

> > Without it it fails on glibc >= 2.28
> > https://travis-ci.org/pevik/ltp/builds/470077800

> > I thing it'd be better to remove _GNU_SOURCE from statx0*.c and put it
> > into
> > lapi/stat.h.

> These changes have already been made in v5 of this test case. This patch
> has been renamed to syscalls/statx: Add test for sync flags. Waiting for
> the tst_get_tmpdir() function to be ported to new library for the patch to
> be merged.
ok, I'm sorry, I got confused as you're both working on the same test.
But the problem remain even on v5 [1]: missing _GNU_SOURCE definition while
using lapi/stat.h.

> Regards,
> Vaishnavi D


Kind regards,
Petr

[1] https://patchwork.ozlabs.org/patch/1011904/