lkml.org 
[lkml]   [2016]   [Apr]   [16]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH 1/2] lightnvm: rename nvm_targets to nvm_tgt_type
From
Date
On 04/13/2016 02:15 PM, Simon A. F. Lund wrote:
> The functions nvm_register_target(), nvm_unregister_target() and
> associated list refers to a target type that is being registered by a
> target type module. Rename nvm_*_targets() to nvm_*_tgt_type(), so that
> the intension is clear.
>
> This enables target instances to use the _nvm_*_targets() naming.
>
> Signed-off-by: Simon A. F. Lund <slund@cnexlabs.com>
> ---
> drivers/lightnvm/core.c | 16 ++++++++--------
> drivers/lightnvm/rrpc.c | 4 ++--
> include/linux/lightnvm.h | 4 ++--
> 3 files changed, 12 insertions(+), 12 deletions(-)
>
> diff --git a/drivers/lightnvm/core.c b/drivers/lightnvm/core.c
> index 74fb049..240b473 100644
> --- a/drivers/lightnvm/core.c
> +++ b/drivers/lightnvm/core.c
> @@ -30,7 +30,7 @@
> #include <linux/sched/sysctl.h>
> #include <uapi/linux/lightnvm.h>
>
> -static LIST_HEAD(nvm_targets);
> +static LIST_HEAD(nvm_tgt_types);
> static LIST_HEAD(nvm_mgrs);
> static LIST_HEAD(nvm_devices);
> static DECLARE_RWSEM(nvm_lock);
> @@ -39,14 +39,14 @@ static struct nvm_tgt_type *nvm_find_target_type(const char *name)
> {
> struct nvm_tgt_type *tt;
>
> - list_for_each_entry(tt, &nvm_targets, list)
> + list_for_each_entry(tt, &nvm_tgt_types, list)
> if (!strcmp(name, tt->name))
> return tt;
>
> return NULL;
> }
>
> -int nvm_register_target(struct nvm_tgt_type *tt)
> +int nvm_register_tgt_type(struct nvm_tgt_type *tt)
> {
> int ret = 0;
>
> @@ -54,14 +54,14 @@ int nvm_register_target(struct nvm_tgt_type *tt)
> if (nvm_find_target_type(tt->name))
> ret = -EEXIST;
> else
> - list_add(&tt->list, &nvm_targets);
> + list_add(&tt->list, &nvm_tgt_types);
> up_write(&nvm_lock);
>
> return ret;
> }
> -EXPORT_SYMBOL(nvm_register_target);
> +EXPORT_SYMBOL(nvm_register_tgt_type);
>
> -void nvm_unregister_target(struct nvm_tgt_type *tt)
> +void nvm_unregister_tgt_type(struct nvm_tgt_type *tt)
> {
> if (!tt)
> return;
> @@ -70,7 +70,7 @@ void nvm_unregister_target(struct nvm_tgt_type *tt)
> list_del(&tt->list);
> up_write(&nvm_lock);
> }
> -EXPORT_SYMBOL(nvm_unregister_target);
> +EXPORT_SYMBOL(nvm_unregister_tgt_type);
>
> void *nvm_dev_dma_alloc(struct nvm_dev *dev, gfp_t mem_flags,
> dma_addr_t *dma_handler)
> @@ -1020,7 +1020,7 @@ static long nvm_ioctl_info(struct file *file, void __user *arg)
> info->version[2] = NVM_VERSION_PATCH;
>
> down_write(&nvm_lock);
> - list_for_each_entry(tt, &nvm_targets, list) {
> + list_for_each_entry(tt, &nvm_tgt_types, list) {
> struct nvm_ioctl_info_tgt *tgt = &info->tgts[tgt_iter];
>
> tgt->version[0] = tt->version[0];
> diff --git a/drivers/lightnvm/rrpc.c b/drivers/lightnvm/rrpc.c
> index 3ab6495..4625e9c 100644
> --- a/drivers/lightnvm/rrpc.c
> +++ b/drivers/lightnvm/rrpc.c
> @@ -1468,12 +1468,12 @@ static struct nvm_tgt_type tt_rrpc = {
>
> static int __init rrpc_module_init(void)
> {
> - return nvm_register_target(&tt_rrpc);
> + return nvm_register_tgt_type(&tt_rrpc);
> }
>
> static void rrpc_module_exit(void)
> {
> - nvm_unregister_target(&tt_rrpc);
> + nvm_unregister_tgt_type(&tt_rrpc);
> }
>
> module_init(rrpc_module_init);
> diff --git a/include/linux/lightnvm.h b/include/linux/lightnvm.h
> index dacaa28..497da91 100644
> --- a/include/linux/lightnvm.h
> +++ b/include/linux/lightnvm.h
> @@ -453,8 +453,8 @@ struct nvm_tgt_type {
> struct list_head list;
> };
>
> -extern int nvm_register_target(struct nvm_tgt_type *);
> -extern void nvm_unregister_target(struct nvm_tgt_type *);
> +extern int nvm_register_tgt_type(struct nvm_tgt_type *);
> +extern void nvm_unregister_tgt_type(struct nvm_tgt_type *);
>
> extern void *nvm_dev_dma_alloc(struct nvm_dev *, gfp_t, dma_addr_t *);
> extern void nvm_dev_dma_free(struct nvm_dev *, void *, dma_addr_t);
>

Thanks Simon. Applied for 4.7.

\
 
 \ /
  Last update: 2016-04-16 16:41    [W:0.059 / U:0.164 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site