From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-pf0-f170.google.com ([209.85.192.170]:50161 "EHLO mail-pf0-f170.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751988AbdJQPU3 (ORCPT ); Tue, 17 Oct 2017 11:20:29 -0400 Received: by mail-pf0-f170.google.com with SMTP id i5so1574097pfe.6 for ; Tue, 17 Oct 2017 08:20:29 -0700 (PDT) Message-ID: <1508253627.3006.29.camel@dubeyko.com> Subject: Re: [PATCH 2/2] hfsplus: add mount option to enable ACL support From: Viacheslav Dubeyko To: Christoph Hellwig , Jan Kara Cc: "Ernesto A." =?ISO-8859-1?Q?Fern=E1ndez?= , linux-fsdevel@vger.kernel.org, Andreas Gruenbacher , Alexander Viro Date: Tue, 17 Oct 2017 08:20:27 -0700 In-Reply-To: <20171017080307.GB27649@infradead.org> References: <4d0758dad0ee32bda49113a957a8b5d9ce882e9b.1501124092.git.ernesto.mnd.fernandez@gmail.com> <7f2987dd3b8eace03dece289d35011ef65170c11.1501124092.git.ernesto.mnd.fernandez@gmail.com> <20170727094147.GE21100@quack2.suse.cz> <20170727095053.GA8136@infradead.org> <20171014024318.GA8995@debian.home> <20171016074210.GA1053@infradead.org> <1508169030.3006.24.camel@dubeyko.com> <20171017075346.GM9762@quack2.suse.cz> <20171017080307.GB27649@infradead.org> Content-Type: text/plain; charset="UTF-8" Mime-Version: 1.0 Content-Transfer-Encoding: 8bit Sender: linux-fsdevel-owner@vger.kernel.org List-ID: On Tue, 2017-10-17 at 01:03 -0700, Christoph Hellwig wrote: > On Tue, Oct 17, 2017 at 09:53:46AM +0200, Jan Kara wrote: > > > > With xattr support the situation is different. It has been broken > > for four > > years and nobody complained. That is a good indication nobody uses > > that > > code and we can drop it. Furthermore, Christoph pointed out that > > original > > HFS+ in MacOS never supported xattrs which makes it even less > > likely that > > someone uses that functionality (as you cannot use it if you want > > your > > filesystem to stay compatible with MacOS). > s/xattrs/Posix ACLs/g The extended attributes supported in Mac OS X many years already [1]. Of course, it looks like more than Richacls [2]. But no Richacls was available during the implementation phase. [1] https://developer.apple.com/legacy/library/documentation/Darwin/Reference/ManPages/man1/xattr.1.html [2] https://en.wikipedia.org/wiki/Richacls