linux-kernel - Re: [PATCH v2 1/6] NFS: use SUNRPC cache creation and destruction helper for DNS cache

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Fri, 8 Feb 2013 16:13:45 -0500
From:	"J. Bruce Fields" <bfields@...ldses.org>
To:	Stanislav Kinsbursky <skinsbursky@...allels.com>
Cc:	linux-nfs@...r.kernel.org, Trond.Myklebust@...app.com,
	linux-kernel@...r.kernel.org, devel@...nvz.org
Subject: Re: [PATCH v2 1/6] NFS: use SUNRPC cache creation and destruction
 helper for DNS cache

On Mon, Feb 04, 2013 at 02:02:35PM +0300, Stanislav Kinsbursky wrote:
> This cache was the first containerized and doesn't use net-aware cache
> creation and destruction helpers.
> This is a cleanup patch which just makes code looks clearer and reduce amount
> of lines of code.

This looks like uncontroversial cleanup, and should probably go in
with the rest through my tree, so I'm inclined to just merge it.

But an "ACK" from Trond would be reassuring.--b.

> 
> Signed-off-by: Stanislav Kinsbursky <skinsbursky@...allels.com>
> ---
>  fs/nfs/dns_resolve.c |   60 +++++++++++++++++++++-----------------------------
>  1 files changed, 25 insertions(+), 35 deletions(-)
> 
> diff --git a/fs/nfs/dns_resolve.c b/fs/nfs/dns_resolve.c
> index ca4b11e..5015447 100644
> --- a/fs/nfs/dns_resolve.c
> +++ b/fs/nfs/dns_resolve.c
> @@ -351,48 +351,39 @@ ssize_t nfs_dns_resolve_name(struct net *net, char *name,
>  }
>  EXPORT_SYMBOL_GPL(nfs_dns_resolve_name);
>  
> +static struct cache_detail nfs_dns_resolve_template = {
> +	.owner		= THIS_MODULE,
> +	.hash_size	= NFS_DNS_HASHTBL_SIZE,
> +	.name		= "dns_resolve",
> +	.cache_put	= nfs_dns_ent_put,
> +	.cache_upcall	= nfs_dns_upcall,
> +	.cache_parse	= nfs_dns_parse,
> +	.cache_show	= nfs_dns_show,
> +	.match		= nfs_dns_match,
> +	.init		= nfs_dns_ent_init,
> +	.update		= nfs_dns_ent_update,
> +	.alloc		= nfs_dns_ent_alloc,
> +};
> +
> +
>  int nfs_dns_resolver_cache_init(struct net *net)
>  {
> -	int err = -ENOMEM;
> +	int err;
>  	struct nfs_net *nn = net_generic(net, nfs_net_id);
> -	struct cache_detail *cd;
> -	struct cache_head **tbl;
>  
> -	cd = kzalloc(sizeof(struct cache_detail), GFP_KERNEL);
> -	if (cd == NULL)
> -		goto err_cd;
> -
> -	tbl = kzalloc(NFS_DNS_HASHTBL_SIZE * sizeof(struct cache_head *),
> -			GFP_KERNEL);
> -	if (tbl == NULL)
> -		goto err_tbl;
> -
> -	cd->owner = THIS_MODULE,
> -	cd->hash_size = NFS_DNS_HASHTBL_SIZE,
> -	cd->hash_table = tbl,
> -	cd->name = "dns_resolve",
> -	cd->cache_put = nfs_dns_ent_put,
> -	cd->cache_upcall = nfs_dns_upcall,
> -	cd->cache_parse = nfs_dns_parse,
> -	cd->cache_show = nfs_dns_show,
> -	cd->match = nfs_dns_match,
> -	cd->init = nfs_dns_ent_init,
> -	cd->update = nfs_dns_ent_update,
> -	cd->alloc = nfs_dns_ent_alloc,
> -
> -	nfs_cache_init(cd);
> -	err = nfs_cache_register_net(net, cd);
> +	nn->nfs_dns_resolve = cache_create_net(&nfs_dns_resolve_template, net);
> +	if (IS_ERR(nn->nfs_dns_resolve))
> +		return PTR_ERR(nn->nfs_dns_resolve);
> +
> +	nfs_cache_init(nn->nfs_dns_resolve);
> +	err = nfs_cache_register_net(net, nn->nfs_dns_resolve);
>  	if (err)
>  		goto err_reg;
> -	nn->nfs_dns_resolve = cd;
>  	return 0;
>  
>  err_reg:
> -	nfs_cache_destroy(cd);
> -	kfree(cd->hash_table);
> -err_tbl:
> -	kfree(cd);
> -err_cd:
> +	nfs_cache_destroy(nn->nfs_dns_resolve);
> +	cache_destroy_net(nn->nfs_dns_resolve, net);
>  	return err;
>  }
>  
> @@ -403,8 +394,7 @@ void nfs_dns_resolver_cache_destroy(struct net *net)
>  
>  	nfs_cache_unregister_net(net, cd);
>  	nfs_cache_destroy(cd);
> -	kfree(cd->hash_table);
> -	kfree(cd);
> +	cache_destroy_net(nn->nfs_dns_resolve, net);
>  }
>  
>  static int rpc_pipefs_event(struct notifier_block *nb, unsigned long event,
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/