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  PHC 
Open Source and information security mailing list archives
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date:	Tue, 13 Oct 2015 15:39:15 +0200
From:	Andreas Gruenbacher <>
To:	Dave Chinner <>
Cc:	Andreas Gr├╝nbacher <>,
	Alexander Viro <>,
	"Theodore Ts'o" <>,
	Andreas Dilger <>,
	"J. Bruce Fields" <>,
	Jeff Layton <>,
	Trond Myklebust <>,
	Anna Schumaker <>,
	linux-ext4 <>,,
	Linux Kernel Mailing List <>,
	Linux FS-devel Mailing List <>,
	Linux NFS Mailing List <>,,
	Linux API Mailing List <>
Subject: Re: [PATCH v10 24/46] xfs: Add richacl support

On Mon, Oct 12, 2015 at 6:05 AM, Dave Chinner <> wrote:
> On Mon, Oct 12, 2015 at 03:51:15AM +0200, Andreas Gr├╝nbacher wrote:
>> 2015-10-12 2:10 GMT+02:00 Dave Chinner <>:
>> > Also, I really dislike the API where passing a NULL acl means to
>> > "set this acl" actually means "remove the existing ACL". Why no
>> > ->remove_acl method called from the generic code?
>> It's not uncommon, it saves inode operations and wiring-up code.
> I know it's common. All it does is put extra branches in the
> filesystem code to do this, because remove is a different operation
> to set. The API sucks, and we're not limited on inode operations,
> and the operator overloading makes the filesystem code unnecessarily
> complex as it has to detect when to branch out ot remove or not...

I've tried it out. The filesystem code could be simplified (see the
richacl-wip [*] branch until the next posting). Adding a
remove_richacl inode operation on top of that really doesn't help.


[*] git://
To unsubscribe from this list: send the line "unsubscribe linux-ext4" in
the body of a message to
More majordomo info at

Powered by blists - more mailing lists