Re: [PATCH v4 05/40] fs: add attr_flags_to_mnt_flags helper

[Christoph Hellwig <hch@xxxxxx>]

> @@ -3450,6 +3450,28 @@ SYSCALL_DEFINE5(mount, char __user *, dev_name, char __user *, dir_name,
>  	return ret;
>  }
>  
> +#define FSMOUNT_VALID_FLAGS                                                    \
> +	(MOUNT_ATTR_RDONLY | MOUNT_ATTR_NOSUID | MOUNT_ATTR_NODEV |            \
> +	 MOUNT_ATTR_NOEXEC | MOUNT_ATTR__ATIME | MOUNT_ATTR_NODIRATIME)

Any good reason for aligning the \ using spaces all the way out?

Otherwise looks good:

Reviewed-by: Christoph Hellwig <hch@xxxxxx>