linux-kernel - RE: [PATCH 1/1] nfs: Move ACL xattr definitions to linux/xattr.h

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:   Wed, 2 Nov 2022 13:37:52 +0000
From:   Ondrej Valousek <ondrej.valousek.xm@...esas.com>
To:     Christoph Hellwig <hch@...radead.org>
CC:     "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "linux-nfs@...r.kernel.org" <linux-nfs@...r.kernel.org>,
        "trond.myklebust@...merspace.com" <trond.myklebust@...merspace.com>,
        "anna@...nel.org" <anna@...nel.org>,
        Chuck Lever III <chuck.lever@...cle.com>
Subject: RE: [PATCH 1/1] nfs: Move ACL xattr definitions to linux/xattr.h

Hi Christoph,
> Well, for that a proper formatted mail really helps.

Yeah for the first main I used Outlook to send, the second email with patch was sent directly using GIT so should be formatted well.
Sorry I am beginner here.

> And these identifiers do not belong into xattr.h.  They are a non-sanctioned NFS side API, not a VFS API.

Ah yes, I forgot about VFS, true. What is the best place to keep them then?
Or is it a big issue to extend VFS with them?
I mean NFSv4 - style ACLs are IMHO even better than Posix ACLs - if not for anything else then because Posix ACLs have never been ratified - maybe we'll see them on local FS at some stage, too.

Thanks,
Ondrej

-----Original Message-----
From: Christoph Hellwig <hch@...radead.org> 
Sent: pondělí 31. října 2022 9:03
To: Ondrej Valousek <ondrej.valousek.xm@...esas.com>
Cc: linux-kernel@...r.kernel.org; linux-nfs@...r.kernel.org; trond.myklebust@...merspace.com; anna@...nel.org; Chuck Lever III <chuck.lever@...cle.com>
Subject: Re: [PATCH 1/1] nfs: Move ACL xattr definitions to linux/xattr.h

On Thu, Oct 27, 2022 at 07:43:58AM +0000, Ondrej Valousek wrote:
> Hi kernel maintainers,
> 
> Please help to submit the following patch into kernel

Well, for that a proper formatted mail really helps.

And these identifiers do not belong into xattr.h.  They are a non-sanctioned NFS side API, not a VFS API.