On Tue, Jun 22, 2021 at 10:47 PM Keith Busch <kbusch@xxxxxxxxxx> wrote: > > linux/fileattr.h has an implicit requirement that linux/fs.h be included > first. Make that dependency explicit. > > Cc: Miklos Szeredi <mszeredi@xxxxxxxxxx> > Cc: Matthew Wilcox <willy@xxxxxxxxxxxxx> > Cc: Jens Axboe: <axboe@xxxxxxxxx> > Signed-off-by: Keith Busch <kbusch@xxxxxxxxxx> > --- > include/linux/fileattr.h | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/include/linux/fileattr.h b/include/linux/fileattr.h > index 9e37e063ac69..34e153172a85 100644 > --- a/include/linux/fileattr.h > +++ b/include/linux/fileattr.h > @@ -3,6 +3,8 @@ > #ifndef _LINUX_FILEATTR_H > #define _LINUX_FILEATTR_H > > +#include <linux/fs.h> > + > /* Flags shared betwen flags/xflags */ > #define FS_COMMON_FL \ > (FS_SYNC_FL | FS_IMMUTABLE_FL | FS_APPEND_FL | \ > -- Thinking out loud: Would it be better to split out the flag definitions to uapi/linux/fileattr.h and include it from here and from uapi/linux/fs.h? Will that cause UAPI pain?? Thanks, Amir.
