[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Virtio-fs] [PATCH 1/2] virtiofsd: Allow use "-o xattrmap" without "
From: |
Greg Kurz |
Subject: |
Re: [Virtio-fs] [PATCH 1/2] virtiofsd: Allow use "-o xattrmap" without "-o xattr" |
Date: |
Fri, 16 Apr 2021 13:05:37 +0200 |
On Wed, 14 Apr 2021 20:12:06 +0000
Carlos Venegas <jose.carlos.venegas.munoz@intel.com> wrote:
> When -o xattrmap is used, it will not work unless xattr is enabled.
>
> This patch enables xattr when -o xattrmap is used.
>
> Signed-off-by: Carlos Venegas <jose.carlos.venegas.munoz@intel.com>
> ---
> tools/virtiofsd/passthrough_ll.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/tools/virtiofsd/passthrough_ll.c
> b/tools/virtiofsd/passthrough_ll.c
> index ddaf57305c..2337ea5a58 100644
> --- a/tools/virtiofsd/passthrough_ll.c
> +++ b/tools/virtiofsd/passthrough_ll.c
> @@ -3939,6 +3939,7 @@ int main(int argc, char *argv[])
> }
>
> if (lo.xattrmap) {
> + lo.xattr = 1;
> parse_xattrmap(&lo);
> }
>
This seems reasonable. I'm just wondering if we should also
add an explicit error if the user tries something silly
like -o xattrmap=MAPPING,no_xattr instead of silently
ignoring no_xattr...
Re: [PATCH 0/2] virtiofsd: Enable xattr if xattrmap is used, Connor Kuehl, 2021/04/14