On Fri, May 10, 2019 at 06:29:56PM +0800, Peng Tao wrote: > It is always nr_ranges.
This looks like a good cleanup. Will squash it with existing patches. Vivek > > Signed-off-by: Peng Tao <[email protected]> > --- > fs/fuse/inode.c | 7 +++---- > 1 file changed, 3 insertions(+), 4 deletions(-) > > diff --git a/fs/fuse/inode.c b/fs/fuse/inode.c > index dd16c7f6a561..cb56572c4b26 100644 > --- a/fs/fuse/inode.c > +++ b/fs/fuse/inode.c > @@ -640,7 +640,7 @@ static int fuse_dax_mem_range_init(struct fuse_conn *fc, > phys_addr_t phys_addr; > int ret = 0, id; > size_t dax_size = -1; > - unsigned long allocated_ranges = 0, i; > + unsigned long i; > > id = dax_read_lock(); > nr_pages = dax_direct_access(dax_dev, 0, PHYS_PFN(dax_size), &kaddr, > @@ -670,12 +670,11 @@ static int fuse_dax_mem_range_init(struct fuse_conn *fc, > range->length = FUSE_DAX_MEM_RANGE_SZ; > list_add_tail(&range->list, &mem_ranges); > INIT_LIST_HEAD(&range->busy_list); > - allocated_ranges++; > } > > list_replace_init(&mem_ranges, &fc->free_ranges); > - fc->nr_free_ranges = allocated_ranges; > - fc->nr_ranges = allocated_ranges; > + fc->nr_free_ranges = nr_ranges; > + fc->nr_ranges = nr_ranges; > return 0; > out_err: > /* Free All allocated elements */ > -- > 2.17.1 >
