bcachefs: drop unused posix acl handlers
authorYouling Tang <tangyouling@kylinos.cn>
Thu, 15 Aug 2024 08:57:43 +0000 (16:57 +0800)
committerKent Overstreet <kent.overstreet@linux.dev>
Mon, 9 Sep 2024 13:41:49 +0000 (09:41 -0400)
Remove struct nop_posix_acl_{access,default} for bcachefs filesystem
that don't depend on the xattr handler in their inode->i_op->listxattr()
method in any way. There's nothing more to do than to simply remove the
handler. It's been effectively unused ever since we introduced the new
posix acl api. See [1] for details.

Link [1]: https://patchwork.kernel.org/project/linux-fsdevel/cover/20230125-fs-acl-remove-generic-xattr-handlers-v3-0-f760cc58967d@kernel.org/

Signed-off-by: Youling Tang <tangyouling@kylinos.cn>
Signed-off-by: Kent Overstreet <kent.overstreet@linux.dev>
fs/bcachefs/xattr.c

index 9f810fb36ba1b24676b833241a82ef961eec810a..7b3d9fdae244f009ea93f07443c73e4daf3ced83 100644 (file)
@@ -601,10 +601,6 @@ static const struct xattr_handler bch_xattr_bcachefs_effective_handler = {
 
 const struct xattr_handler *bch2_xattr_handlers[] = {
        &bch_xattr_user_handler,
-#ifdef CONFIG_BCACHEFS_POSIX_ACL
-       &nop_posix_acl_access,
-       &nop_posix_acl_default,
-#endif
        &bch_xattr_trusted_handler,
        &bch_xattr_security_handler,
 #ifndef NO_BCACHEFS_FS