On Tue, Apr 08, 2014 at 04:59:48PM +0800, ZhangZhen wrote: > You are right, this patch is useless. I'm sorry for this spam mail. > > But btrfs_set_acl checks the validity of the ACL too. Should we delete it? > Another choice is we can check the validity in every fs tree, and delete the > check from posix_acl_xattr_set. Doing it in common code is the right thing. I suspect btrfs having it is an oversight, but please verify it with the btrfs maintainers. -- To unsubscribe from this list: send the line "unsubscribe linux-ext4" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html