Re: [PATCH 2/2] afs: Add metadata xattrs

From	Jeffrey Altman <jaltman@auristor.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH 2/2] afs: Add metadata xattrs
Date	2017-07-09 04:50 +0200
Message-ID	<u1ash-1XK-3@gated-at.bofh.it> (permalink)
References	(2 earlier) <u0gq6-6g7-13@gated-at.bofh.it> <u0hFv-7Q8-5@gated-at.bofh.it> <u0jHk-HR-23@gated-at.bofh.it> <u13TP-6gZ-3@gated-at.bofh.it> <u18Tv-15v-1@gated-at.bofh.it>
Organization	AuriStor, Inc. (formerly Your File System, Inc.)

Show all headers | View raw

[Multipart message — attachments visible in raw view] - view raw

On 7/8/2017 9:01 PM, Theodore Ts'o wrote:
> On Sat, Jul 08, 2017 at 12:44:54PM -0700, Linus Torvalds wrote:
>> Yeah, I think attributes are likely much better than some random crazy
>> ioctl interface. They can be listed with generic tools, and have
>> various scripting interfaces in ways that ioctl's do not sanely have.
> 
> I personally don't have a particular problem with these xattrs.  For
> one thing, they are read-only.  You use them just to find out the AFS
> cell, the AFS "fid", and the AFS volume name.
> 
> I think the place where people will start getting nervous is when we
> start adding "write-only" xattrs or where writing to an xattr causes a
> side-effect to take place.

Ted,

The list of AFS pioctls and the proposed alternatives for kAFS are
listed at

  https://www.infradead.org/~dhowells/kafs/user_interface.html

While it is true that the majority of the proposed xattrs are read-only
properties of AFS objects (cell, volume, fid, servers, sec_class,
sec_mode) there is one exception that is read-write (acls).  AuriStorFS
permits acls to be set per-file; there was some per-file acl work begun
for OpenAFS but it was never completed.

Is there an alternative for fetching and setting ACLs that should be
considered?

Jeffrey Altman

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

[PATCH 0/2] AFS: Changes David Howells <dhowells@redhat.com> - 2017-07-06 17:00 +0200
  [PATCH 2/2] afs: Add metadata xattrs David Howells <dhowells@redhat.com> - 2017-07-06 17:00 +0200
    Re: [PATCH 2/2] afs: Add metadata xattrs Christoph Hellwig <hch@infradead.org> - 2017-07-06 17:30 +0200
      Re: [PATCH 2/2] afs: Add metadata xattrs David Howells <dhowells@redhat.com> - 2017-07-06 18:20 +0200
        Re: [PATCH 2/2] afs: Add metadata xattrs Andreas Dilger <adilger@dilger.ca> - 2017-07-06 20:30 +0200
          Re: [PATCH 2/2] afs: Add metadata xattrs Linus Torvalds <torvalds@linux-foundation.org> - 2017-07-08 21:50 +0200
            Re: [PATCH 2/2] afs: Add metadata xattrs Theodore Ts'o <tytso@mit.edu> - 2017-07-09 03:10 +0200
              Re: [PATCH 2/2] afs: Add metadata xattrs Jeffrey Altman <jaltman@auristor.com> - 2017-07-09 04:50 +0200

csiph-web