From: Christoph Hellwig <hch@lst.de>
To: Andrew Morton <akpm@linux-foundation.org>
Cc: "Viacheslav Dubeyko" <slava@dubeyko.com>,
"Ernesto A. Fernández" <ernesto.mnd.fernandez@gmail.com>,
linux-fsdevel@vger.kernel.org, "Christoph Hellwig" <hch@lst.de>,
"Jan Kara" <jack@suse.cz>
Subject: Re: [PATCH] hfsplus: drop ACL support
Date: Tue, 17 Jul 2018 15:18:10 +0200 [thread overview]
Message-ID: <20180717131810.GE13582@lst.de> (raw)
In-Reply-To: <20180716170748.5264352de39271b67cc54143@linux-foundation.org>
On Mon, Jul 16, 2018 at 05:07:48PM -0700, Andrew Morton wrote:
> On Sat, 14 Jul 2018 12:11:03 -0700 Viacheslav Dubeyko <slava@dubeyko.com> wrote:
>
> > On Sat, 2018-07-14 at 16:06 -0300, Ernesto A. Fern�ndez wrote:
> > > The HFS+ Access Control Lists have not worked at all for the past
> > > five
> > > years, and nobody seems to have noticed. Besides, POSIX draft ACLs
> > > are
> > > not compatible with MacOS. Drop the feature entirely.
> > >
> >
> > Bugs need to be fixed but not to drop. Otherwise, it needs to drop the
> > whole HFS+ support from the kernel.
>
> Yes, we could make it depend on BROKEN and hope that someone comes
> along and fixes it - there's little cost in carrying such a thing.
>
> Or maybe it can never work for some reason and should indeed be
> dropped, I don't know. I was interested in understanding the reasoning
> behind hch's ack, but I can't find it on linux-fs-devel. Help?
The issue is that hfsplus supports it's own NFSv4-like ACLs, which
are nothing like the Posix ACLs supported by the Linux driver. Or
in other words: they were always a Linux-only incompatible extension
that should never have been merged.
next prev parent reply other threads:[~2018-07-17 13:48 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-07-14 19:06 [PATCH] hfsplus: drop ACL support Ernesto A. Fernández
2018-07-14 19:11 ` Viacheslav Dubeyko
2018-07-17 0:07 ` Andrew Morton
2018-07-17 0:21 ` Ernesto A. Fernández
2018-07-27 17:48 ` Ernesto A. Fernández
2018-07-17 13:18 ` Christoph Hellwig [this message]
2018-07-17 18:29 ` Viacheslav Dubeyko
2018-07-17 18:34 ` Eric Biggers
2018-07-14 19:29 ` Viacheslav Dubeyko
-- strict thread matches above, loose matches on Subject: below --
2017-12-03 1:41 Ernesto A. Fernández
2017-12-03 20:59 ` Viacheslav Dubeyko
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20180717131810.GE13582@lst.de \
--to=hch@lst.de \
--cc=akpm@linux-foundation.org \
--cc=ernesto.mnd.fernandez@gmail.com \
--cc=jack@suse.cz \
--cc=linux-fsdevel@vger.kernel.org \
--cc=slava@dubeyko.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.