[PATCH] staging: erofs: remove needless CONFIG_EROFS_FS_SECURITY
Greg KH
gregkh at linuxfoundation.org
Fri Jun 21 17:01:25 AEST 2019
On Thu, Jun 20, 2019 at 04:30:04PM +0800, Yue Hu wrote:
> From: Yue Hu <huyue2 at yulong.com>
>
> erofs_xattr_security_handler is already marked __maybe_unused, no need
> to add CONFIG_EROFS_FS_SECURITY condition.
>
> Signed-off-by: Yue Hu <huyue2 at yulong.com>
> ---
> drivers/staging/erofs/xattr.c | 2 --
> 1 file changed, 2 deletions(-)
>
> diff --git a/drivers/staging/erofs/xattr.c b/drivers/staging/erofs/xattr.c
> index df40654..06024ac 100644
> --- a/drivers/staging/erofs/xattr.c
> +++ b/drivers/staging/erofs/xattr.c
> @@ -499,13 +499,11 @@ static int erofs_xattr_generic_get(const struct xattr_handler *handler,
> .get = erofs_xattr_generic_get,
> };
>
> -#ifdef CONFIG_EROFS_FS_SECURITY
> const struct xattr_handler __maybe_unused erofs_xattr_security_handler = {
> .prefix = XATTR_SECURITY_PREFIX,
> .flags = EROFS_XATTR_INDEX_SECURITY,
> .get = erofs_xattr_generic_get,
> };
> -#endif
It's nicer just to leave this as-is for now, the memory savings isn't
much at all.
thanks,
greg k-h
More information about the Linux-erofs
mailing list