Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
MIME-Version: 1.0
In-Reply-To: <73dae166-2e71-7fc9-7985-89cc44b38bf6@gmail.com>
References: <1520705944-6723-1-git-send-email-jix024@eng.ucsd.edu>
 <1520705944-6723-15-git-send-email-jix024@eng.ucsd.edu> <73dae166-2e71-7fc9-7985-89cc44b38bf6@gmail.com>
From:   Andiry Xu <jix024@eng.ucsd.edu>
Date:   Sun, 11 Mar 2018 14:31:15 -0700
Message-ID: <CAD4Szjurt=QyWr=yO-z_p56uvMAcv2mb5M-j21Lmy3we0Xo4mg@mail.gmail.com>
Subject: Re: [RFC v2 14/83] Add range node kmem cache.
To:     Nikolay Borisov <n.borisov.lkml@gmail.com>
Cc:     Linux FS Devel <linux-fsdevel@vger.kernel.org>,
        linux-kernel@vger.kernel.org,
        "linux-nvdimm@lists.01.org" <linux-nvdimm@lists.01.org>,
        Dan Williams <dan.j.williams@intel.com>,
        "Rudoff, Andy" <andy.rudoff@intel.com>, coughlan@redhat.com,
        Steven Swanson <swanson@cs.ucsd.edu>,
        Dave Chinner <david@fromorbit.com>, jack@suse.com,
        swhiteho@redhat.com, miklos@szeredi.hu,
        Jian Xu <andiry.xu@gmail.com>, Andiry Xu <jix024@cs.ucsd.edu>
Content-Type: text/plain; charset="UTF-8"
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On Sun, Mar 11, 2018 at 4:55 AM, Nikolay Borisov
<n.borisov.lkml@gmail.com> wrote:
>
>
> On 10.03.2018 20:17, Andiry Xu wrote:
>> From: Andiry Xu <jix024@cs.ucsd.edu>
>>
>> Range node specifies a range of [start, end]. and is managed by a red-black tree.
>> NOVA uses range node to manage NVM allocator and inodes being used.
>>
>> Signed-off-by: Andiry Xu <jix024@cs.ucsd.edu>
>> ---
>>  fs/nova/nova.h  |  8 ++++++++
>>  fs/nova/super.c | 45 ++++++++++++++++++++++++++++++++++++++++++---
>>  fs/nova/super.h |  2 ++
>>  3 files changed, 52 insertions(+), 3 deletions(-)
>>
>> diff --git a/fs/nova/nova.h b/fs/nova/nova.h
>> index ba7ffca..e0e85fb 100644
>> --- a/fs/nova/nova.h
>> +++ b/fs/nova/nova.h
>> @@ -301,6 +301,14 @@ static inline u64 nova_get_epoch_id(struct super_block *sb)
>>  }
>>
>>  #include "inode.h"
>> +
>> +/* A node in the RB tree representing a range of pages */
>> +struct nova_range_node {
>> +     struct rb_node node;
>> +     unsigned long range_low;
>> +     unsigned long range_high;
>> +};
>> +
>>  #include "bbuild.h"
>>
>>  /* ====================================================== */
>> diff --git a/fs/nova/super.c b/fs/nova/super.c
>> index f41cc04..aec1cd3 100644
>> --- a/fs/nova/super.c
>> +++ b/fs/nova/super.c
>> @@ -52,6 +52,7 @@ MODULE_PARM_DESC(nova_dbgmask, "Control debugging output");
>>  static struct super_operations nova_sops;
>>
>>  static struct kmem_cache *nova_inode_cachep;
>> +static struct kmem_cache *nova_range_node_cachep;
>>
>>
>>  /* FIXME: should the following variable be one per NOVA instance? */
>> @@ -686,6 +687,20 @@ static void nova_put_super(struct super_block *sb)
>>       sb->s_fs_info = NULL;
>>  }
>>
>> +inline void nova_free_range_node(struct nova_range_node *node)
>> +{
>> +     kmem_cache_free(nova_range_node_cachep, node);
>> +}
>> +
>> +inline struct nova_range_node *nova_alloc_range_node(struct super_block *sb)
>> +{
>> +     struct nova_range_node *p;
>> +
>> +     p = (struct nova_range_node *)
> nit: needless cast

Thanks. Will fix.

Andiry

>> +             kmem_cache_zalloc(nova_range_node_cachep, GFP_NOFS);
>> +     return p;
>> +}
>> +
>>  static struct inode *nova_alloc_inode(struct super_block *sb)
>>  {
>>       struct nova_inode_info *vi;
>> @@ -719,6 +734,17 @@ static void init_once(void *foo)
>>       inode_init_once(&vi->vfs_inode);
>>  }
>>
>> +static int __init init_rangenode_cache(void)
>> +{
>> +     nova_range_node_cachep = kmem_cache_create("nova_range_node_cache",
>> +                                     sizeof(struct nova_range_node),
>> +                                     0, (SLAB_RECLAIM_ACCOUNT |
>
>> +                                     SLAB_MEM_SPREAD), NULL);
>> +     if (nova_range_node_cachep == NULL)
>> +             return -ENOMEM;
>> +     return 0;
>> +}
>> +
>>  static int __init init_inodecache(void)
>>  {
>>       nova_inode_cachep = kmem_cache_create("nova_inode_cache",
>> @@ -740,6 +766,11 @@ static void destroy_inodecache(void)
>>       kmem_cache_destroy(nova_inode_cachep);
>>  }
>>
>> +static void destroy_rangenode_cache(void)
>> +{
>> +     kmem_cache_destroy(nova_range_node_cachep);
>> +}
>> +
>>
>>  /*
>>   * the super block writes are all done "on the fly", so the
>> @@ -781,20 +812,27 @@ static int __init init_nova_fs(void)
>>       nova_info("Arch new instructions support: CLWB %s\n",
>>                       support_clwb ? "YES" : "NO");
>>
>> -     rc = init_inodecache();
>> +     rc = init_rangenode_cache();
>>       if (rc)
>>               goto out;
>>
>> -     rc = register_filesystem(&nova_fs_type);
>> +     rc = init_inodecache();
>>       if (rc)
>>               goto out1;
>>
>> +     rc = register_filesystem(&nova_fs_type);
>> +     if (rc)
>> +             goto out2;
>> +
>>  out:
>>       NOVA_END_TIMING(init_t, init_time);
>>       return rc;
>>
>> -out1:
>> +out2:
>>       destroy_inodecache();
>> +
>> +out1:
>> +     destroy_rangenode_cache();
>>       goto out;
>>  }
>>
>> @@ -802,6 +840,7 @@ static void __exit exit_nova_fs(void)
>>  {
>>       unregister_filesystem(&nova_fs_type);
>>       destroy_inodecache();
>> +     destroy_rangenode_cache();
>>  }
>>
>>  MODULE_AUTHOR("Andiry Xu <jix024@cs.ucsd.edu>");
>> diff --git a/fs/nova/super.h b/fs/nova/super.h
>> index cb53908..b478080 100644
>> --- a/fs/nova/super.h
>> +++ b/fs/nova/super.h
>> @@ -145,5 +145,7 @@ static inline struct nova_super_block *nova_get_super(struct super_block *sb)
>>  }
>>
>>  extern void nova_error_mng(struct super_block *sb, const char *fmt, ...);
>> +extern struct nova_range_node *nova_alloc_range_node(struct super_block *sb);
>> +extern void nova_free_range_node(struct nova_range_node *node);
>>
>>  #endif
>>