Re: [PATCH v2 2/2] hostfs: store permissions in extended attributes

2023-04-25 Thread Marko Petrović
On Tue Apr 18, 2023 at 10:26 AM CEST, Johannes Berg wrote: Hello, Thank you for your review! > On Fri, 2023-04-14 at 19:19 +0200, Marko Petrović wrote: > > > > +++ b/fs/hostfs/hostfs.h > > > > @@ -37,6 +37,7 @@ > > > > * is on, and remove the appropriate bits from attr->ia_mode (attr is a > > > >

Re: [PATCH v2 2/2] hostfs: store permissions in extended attributes

2023-04-18 Thread Johannes Berg
On Fri, 2023-04-14 at 19:19 +0200, Marko Petrović wrote: > > > +++ b/fs/hostfs/hostfs.h > > > @@ -37,6 +37,7 @@ > > > * is on, and remove the appropriate bits from attr->ia_mode (attr is a > > > * "struct iattr *"). -BlaisorBlade > > > */ > > > +extern int use_xattr; > > > > > > That seems

Re: [PATCH v2 2/2] hostfs: store permissions in extended attributes

2023-04-14 Thread Richard Weinberger
- Ursprüngliche Mail - > Von: "Marko Petrović" > About using -1 for uid and gid, in documentation for chown it is stated > that -1 shall be used if the desired value should not be changed, since > normally chown(2) accepts both uid and gid for changing in one system > call. > In the concre

Re: [PATCH v2 2/2] hostfs: store permissions in extended attributes

2023-04-14 Thread Marko Petrović
On Fri Apr 14, 2023 at 12:54 PM CEST, Richard Weinberger wrote: > - Ursprüngliche Mail - > > Von: "Marko Petrović" > > +static int uml_chown(const char *pathname, unsigned int owner, unsigned int > > group) > > Is there a specific reason why you are not using uid_t and gid_t? > > > +{ > >

Re: [PATCH v2 2/2] hostfs: store permissions in extended attributes

2023-04-14 Thread Marko Petrović
On Fri Apr 14, 2023 at 9:40 AM CEST, Johannes Berg wrote: > On Fri, 2023-04-14 at 04:33 +0200, Marko Petrović wrote: > > Fix GID assignment error in uml_chown() and add support for correct > > behavior when parent directory has SETGID bit. > > > That was the change for 'v2' I guess, but you should

Re: [PATCH v2 2/2] hostfs: store permissions in extended attributes

2023-04-14 Thread Richard Weinberger
- Ursprüngliche Mail - > Von: "Marko Petrović" > +static int uml_chown(const char *pathname, unsigned int owner, unsigned int > group) Is there a specific reason why you are not using uid_t and gid_t? > +{ > + int status; > + > + if (use_xattr) { > + if (owner != -1)

Re: [PATCH v2 2/2] hostfs: store permissions in extended attributes

2023-04-14 Thread Johannes Berg
On Fri, 2023-04-14 at 04:33 +0200, Marko Petrović wrote: > Fix GID assignment error in uml_chown() and add support for correct > behavior when parent directory has SETGID bit. That was the change for 'v2' I guess, but you should document here what the new code does and why, probably a good chunk