Re: [LTP] [PATCH 1/1] fanotify.h: Include <linux/types.h>

[Petr Vorel <pvorel@suse.cz>]

> Hi!
> > to add kernel types (__s32, __u32, ...).

> > This fixes build on various Buildroot toolchains:

> > In file included from fanotify.h:14,
> >                  from fanotify04.c:25:
> > ../../../../include/lapi/fanotify.h:198:2: error: unknown type name ‘__s32’
> >   198 |  __s32 error;
> >       |  ^~~~~
> > ../../../../include/lapi/fanotify.h:199:2: error: unknown type name ‘__u32’
> >   199 |  __u32 error_count;
> >       |  ^~~~~

> > NOTE: Historically we tried to minimize number of linux includes,
> > because it would often lead to broken builds. Headers got better over
> > time and LTP also dropped support for archaic kernels over time.
> > Therefore we can keep original kernel types here.

> > Fixes: 67069fc46a ("syscalls/fanotify22: Validate the generic error info")

> > Signed-off-by: Petr Vorel <petr.vorel@gmail.com>
> > ---
> >  testcases/kernel/syscalls/fanotify/fanotify.h | 1 +
> >  1 file changed, 1 insertion(+)

> > diff --git a/testcases/kernel/syscalls/fanotify/fanotify.h b/testcases/kernel/syscalls/fanotify/fanotify.h
> > index 51078103eb..da78788378 100644
> > --- a/testcases/kernel/syscalls/fanotify/fanotify.h
> > +++ b/testcases/kernel/syscalls/fanotify/fanotify.h
> > @@ -11,6 +11,7 @@
> >  #include <sys/types.h>
> >  #include <sys/stat.h>
> >  #include <errno.h>
> > +#include <linux/types.h>

> Shouldn't that go into the lapi/fanotify.h instead? Otherwise the next
> file that includes the header will have the exactly same problem.

Ah yes, you're right, I don't know myself why I put it that into the other one.

Kind regards,
Petr

> >  #include "lapi/fanotify.h"
> >  #include "lapi/fcntl.h"

> > -- 
> > 2.39.1

-- 
Mailing list info: https://lists.linux.it/listinfo/ltp