[RFC] vfs_nfs4acl_xattr NFS4.1 support, support for XDR encoding and more...

[Andrew Bartlett]

On Sat, 2017-10-28 at 19:09 +0200, Ralph Böhme wrote:
> On Fri, Oct 27, 2017 at 01:56:04PM -0700, Jeremy Allison wrote:
> > On Fri, Oct 27, 2017 at 10:32:33PM +0200, Ralph Böhme wrote:
> > > On Wed, Oct 25, 2017 at 10:59:38AM +0200, Ralph Böhme wrote:
> > > > I'm in favour of changing the default for the old NDR backend as well. What do
> > > > you think? Andrew? It's your code. :)
> > > 
> > > ping.
> > > 
> > > > > 4). [PATCH 22/25] vfs_nfs4acl_xattr: add XDR backend
> > > > > In nfs4acl_get_xdrblob_size() - integer wrap checks
> > > > > please on this:
> > > > > size += nacl->na41_aces.na41_aces_len * sizeof(struct nfsace4);
> > > > 
> > > > Done. Thanks for spotting this! I hope I guarded all relevant places in the
> > > > attached updated patchset.
> > > 
> > > Another ping. :=)
> > 
> > Next on my list to review once I've looked at the kernel
> > oplock bugs, honest ! :-).
> 
> Oh, so I hope you come out of that in a sane state of mind. :)
> 
> > FYI. Is this merely a WIP, or do you want me to push if
> > it passes review ?
> 
> Still waiting for feedback on the questions I brought up in the first mail, from
> you and especially from Andrew as it's his code.
> 
> Andrew?

I'm fine with the concept - my role was to dispel the myth that this
was un-testable, I'm not wedded to the NDR encoding. 

I'm very happy for the defaults to change, I can't imagine it has been
used in production.

I've got a busy week this week so I'm not sure I'll be able to get you
a formal review however.

Sorry,

Andrew Bartlett
-- 
Andrew Bartlett                       http://samba.org/~abartlet/
Authentication Developer, Samba Team  http://samba.org
Samba Developer, Catalyst IT          http://catalyst.net.nz/services/samba