Re: [PATCH 1/1] Smack:- Remove redundant inode_smack cache since now new cache is being used for inode_smack structure allocations

[Casey Schaufler]

On 4/26/2020 9:12 PM, Vishal Goel wrote:
> Signed-off-by: Vishal Goel <vishal.goel@xxxxxxxxxxx>

I've taken this after adding a commit message.
Please include a commit message, even if you believe
the "subject line says it all" in the future.

> ---
>  security/smack/smack_lsm.c | 9 +--------
>  1 file changed, 1 insertion(+), 8 deletions(-)
>
> diff --git a/security/smack/smack_lsm.c b/security/smack/smack_lsm.c
> index 2862fc3..d0f87d4 100644
> --- a/security/smack/smack_lsm.c
> +++ b/security/smack/smack_lsm.c
> @@ -55,7 +55,6 @@
>  DEFINE_MUTEX(smack_ipv6_lock);
>  static LIST_HEAD(smk_ipv6_port_list);
>  #endif
> -static struct kmem_cache *smack_inode_cache;
>  struct kmem_cache *smack_rule_cache;
>  int smack_enabled;
>  
> @@ -4841,15 +4840,9 @@ static __init int smack_init(void)
>  	struct cred *cred = (struct cred *) current->cred;
>  	struct task_smack *tsp;
>  
> -	smack_inode_cache = KMEM_CACHE(inode_smack, 0);
> -	if (!smack_inode_cache)
> -		return -ENOMEM;
> -
>  	smack_rule_cache = KMEM_CACHE(smack_rule, 0);
> -	if (!smack_rule_cache) {
> -		kmem_cache_destroy(smack_inode_cache);
> +	if (!smack_rule_cache)
>  		return -ENOMEM;
> -	}
>  
>  	/*
>  	 * Set the security state for the initial task.