[PATCH] staging: erofs: removing an extra call to iloc() in fill_inode()
Pratik Shinde
pratikshinde320 at gmail.com
Wed Aug 14 13:52:53 AEST 2019
Yes.since we already have a function with same name (and we are using it in
same context).
'inode_loc' was the most meaningful name I could come up with :)
--Pratik.
On Wed, Aug 14, 2019 at 7:37 AM Gao Xiang <gaoxiang25 at huawei.com> wrote:
> On Wed, Aug 14, 2019 at 09:56:09AM +0800, Chao Yu wrote:
> > On 2019/8/14 9:59, Gao Xiang wrote:
> > > Hi Pratik,
> > >
> > > On Wed, Aug 14, 2019 at 02:08:40AM +0530, Pratik Shinde wrote:
> > >> in fill_inode() we call iloc() twice.Avoiding the extra call by
> > >> storing the result.
> > >>
> > >> Signed-off-by: Pratik Shinde <pratikshinde320 at gmail.com>
> > >
> > > I have no objection of this patch, but I'd like to
> > > hear Chao/Greg's idea about this...
> >
> > It looks more clean. :)
> >
> > Nitpick, maybe change 'inode_loc' to shorter 'iloc' will be better.
>
> iloc is the name of static inline helper function in internal.h
> used for shorter lines...
>
> Thanks,
> Gao Xiang
>
> >
> > Reviewed-by: Chao Yu <yuchao0 at huawei.com>
> >
> > Thanks,
> >
> > >
> > > Thanks,
> > > Gao Xiang
> > >
> > >> ---
> > >> drivers/staging/erofs/inode.c | 7 ++++---
> > >> 1 file changed, 4 insertions(+), 3 deletions(-)
> > >>
> > >> diff --git a/drivers/staging/erofs/inode.c
> b/drivers/staging/erofs/inode.c
> > >> index 4c3d8bf..d82ba6c 100644
> > >> --- a/drivers/staging/erofs/inode.c
> > >> +++ b/drivers/staging/erofs/inode.c
> > >> @@ -167,11 +167,12 @@ static int fill_inode(struct inode *inode, int
> isdir)
> > >> int err;
> > >> erofs_blk_t blkaddr;
> > >> unsigned int ofs;
> > >> + erofs_off_t inode_loc;
> > >>
> > >> trace_erofs_fill_inode(inode, isdir);
> > >> -
> > >> - blkaddr = erofs_blknr(iloc(sbi, vi->nid));
> > >> - ofs = erofs_blkoff(iloc(sbi, vi->nid));
> > >> + inode_loc = iloc(sbi, vi->nid);
> > >> + blkaddr = erofs_blknr(inode_loc);
> > >> + ofs = erofs_blkoff(inode_loc);
> > >>
> > >> debugln("%s, reading inode nid %llu at %u of blkaddr %u",
> > >> __func__, vi->nid, ofs, blkaddr);
> > >> --
> > >> 2.9.3
> > >>
> > > .
> > >
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.ozlabs.org/pipermail/linux-erofs/attachments/20190814/7db46d5c/attachment.htm>
More information about the Linux-erofs
mailing list