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

Ralph Böhme slow at samba.org
Sat Oct 28 17:09:03 UTC 2017


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?

-slow

-- 
Ralph Boehme, Samba Team       https://samba.org/
Samba Developer, SerNet GmbH   https://sernet.de/en/samba/



More information about the samba-technical mailing list