mbox series

[0/2] virtiofsd: Enable xattr if xattrmap is used

Message ID 20210414201207.3612432-1-jose.carlos.venegas.munoz@intel.com (mailing list archive)
Headers show
Series virtiofsd: Enable xattr if xattrmap is used | expand

Message

Venegas Munoz, Jose Carlos April 14, 2021, 8:12 p.m. UTC 
Using xattrmap for Kata Containers we found that xattr is should be used
or xattrmap wont work.  These patches enable xattr when -o xattrmap is
used. Also, they add help for the xattrmap  option on `virtiofsd --help` output.

Carlos Venegas (2):
  virtiofsd: Allow use "-o xattrmap" without "-o xattr"
  virtiofsd: Add help for -o xattr-mapping

 tools/virtiofsd/helper.c         | 3 +++
 tools/virtiofsd/passthrough_ll.c | 1 +
 2 files changed, 4 insertions(+)

Comments

Connor Kuehl April 14, 2021, 9:47 p.m. UTC | #1 
On Wed Apr 14, 2021 at 3:12 PM CDT, Carlos Venegas wrote:
>
> Using xattrmap for Kata Containers we found that xattr is should be used
> or xattrmap wont work. These patches enable xattr when -o xattrmap is
> used. Also, they add help for the xattrmap option on `virtiofsd --help`
> output.
>
> Carlos Venegas (2):
> virtiofsd: Allow use "-o xattrmap" without "-o xattr"
> virtiofsd: Add help for -o xattr-mapping

Good usability improvement.

For the series:

Reviewed-by: Connor Kuehl <ckuehl@redhat.com>
Dr. David Alan Gilbert May 6, 2021, 3:52 p.m. UTC | #2 
* Carlos Venegas (jose.carlos.venegas.munoz@intel.com) wrote:
> 
> Using xattrmap for Kata Containers we found that xattr is should be used
> or xattrmap wont work.  These patches enable xattr when -o xattrmap is
> used. Also, they add help for the xattrmap  option on `virtiofsd --help` output.

Queued.  You might like to submit some more patches to give the error
that Greg was suggesting and/or update some docs.

Dave

> Carlos Venegas (2):
>   virtiofsd: Allow use "-o xattrmap" without "-o xattr"
>   virtiofsd: Add help for -o xattr-mapping
> 
>  tools/virtiofsd/helper.c         | 3 +++
>  tools/virtiofsd/passthrough_ll.c | 1 +
>  2 files changed, 4 insertions(+)
> 
> -- 
> 2.25.1
> 
>
Venegas Munoz, Jose Carlos May 6, 2021, 3:55 p.m. UTC | #3 
Thank you both for the suggestions, I had not took a look recently, I will check today in my afternoon.

Cheers.

> -----Original Message-----
> From: Dr. David Alan Gilbert <dgilbert@redhat.com>
> Sent: Thursday, May 6, 2021 10:53 AM
> To: Venegas Munoz, Jose Carlos <jose.carlos.venegas.munoz@intel.com>
> Cc: virtio-fs@redhat.com; qemu-devel@nongnu.org; vgoyal@redhat.com
> Subject: Re: [PATCH 0/2] virtiofsd: Enable xattr if xattrmap is used
> 
> * Carlos Venegas (jose.carlos.venegas.munoz@intel.com) wrote:
> >
> > Using xattrmap for Kata Containers we found that xattr is should be
> > used or xattrmap wont work.  These patches enable xattr when -o
> > xattrmap is used. Also, they add help for the xattrmap  option on `virtiofsd -
> -help` output.
> 
> Queued.  You might like to submit some more patches to give the error that
> Greg was suggesting and/or update some docs.
> 
> Dave
> 
> > Carlos Venegas (2):
> >   virtiofsd: Allow use "-o xattrmap" without "-o xattr"
> >   virtiofsd: Add help for -o xattr-mapping
> >
> >  tools/virtiofsd/helper.c         | 3 +++
> >  tools/virtiofsd/passthrough_ll.c | 1 +
> >  2 files changed, 4 insertions(+)
> >
> > --
> > 2.25.1
> >
> >
> --
> Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK