On Mon, Jul 31, 2017 at 04:58:33PM -0600, Ross Zwisler wrote:
> The following commit:
>
> commit 73b54bb6a2fb ("xfs_io: allow chattr & chproj on foreign filesystems")
>
> allowed chattr and chproj to be run on non-xfs filesystems now that
> FS_IOC_FSSETXATTR is a generic vfs call. It failed to enable the
> corresponding lsattr and lsproj commands for those filesystems, though.
>
> Signed-off-by: Ross Zwisler <ross.zwisler@xxxxxxxxxxxxxxx>
> Fixes: 73b54bb6a2fb ("xfs_io: allow chattr & chproj on foreign filesystems")
> Cc: Eric Sandeen <sandeen@xxxxxxxxxx>
> Cc: Dave Chinner <david@xxxxxxxxxxxxx>
> Cc: Jan Kara <jack@xxxxxxx>
Looks ok,
Reviewed-by: Darrick J. Wong <darrick.wong@xxxxxxxxxx>
--D
> ---
> io/attr.c | 2 +-
> io/open.c | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/io/attr.c b/io/attr.c
> index b8eec1b..728560e 100644
> --- a/io/attr.c
> +++ b/io/attr.c
> @@ -353,7 +353,7 @@ attr_init(void)
> lsattr_cmd.args = _("[-R|-D|-a|-v]");
> lsattr_cmd.argmin = 0;
> lsattr_cmd.argmax = 1;
> - lsattr_cmd.flags = CMD_NOMAP_OK;
> + lsattr_cmd.flags = CMD_NOMAP_OK | CMD_FOREIGN_OK;
> lsattr_cmd.oneline =
> _("list extended inode flags set on the currently open file");
> lsattr_cmd.help = lsattr_help;
> diff --git a/io/open.c b/io/open.c
> index b50f068..f2ea7c3 100644
> --- a/io/open.c
> +++ b/io/open.c
> @@ -812,7 +812,7 @@ open_init(void)
> lsproj_cmd.args = _("[-D | -R]");
> lsproj_cmd.argmin = 0;
> lsproj_cmd.argmax = -1;
> - lsproj_cmd.flags = CMD_NOMAP_OK;
> + lsproj_cmd.flags = CMD_NOMAP_OK | CMD_FOREIGN_OK;
> lsproj_cmd.oneline =
> _("list project identifier set on the currently open file");
> lsproj_cmd.help = lsproj_help;
> --
> 2.9.4
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-xfs" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-xfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
