From: Youling Tang Date: Thu, 15 Aug 2024 08:57:43 +0000 (+0800) Subject: bcachefs: drop unused posix acl handlers X-Git-Tag: v6.12-rc1~101^2~50 X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=d3f30f16294cc8e58ef14f934874ed7b3ec15b71;p=thirdparty%2Fkernel%2Flinux.git bcachefs: drop unused posix acl handlers 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 Signed-off-by: Kent Overstreet --- diff --git a/fs/bcachefs/xattr.c b/fs/bcachefs/xattr.c index 9f810fb36ba1b..7b3d9fdae244f 100644 --- a/fs/bcachefs/xattr.c +++ b/fs/bcachefs/xattr.c @@ -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