<div dir="ltr"><div>Yes.since we already have a function with same name (and we are using it in same context).</div><div>'inode_loc' was the most meaningful name I could come up with :)</div><div><br></div><div>--Pratik.<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Aug 14, 2019 at 7:37 AM Gao Xiang <<a href="mailto:gaoxiang25@huawei.com">gaoxiang25@huawei.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Wed, Aug 14, 2019 at 09:56:09AM +0800, Chao Yu wrote:<br>
> On 2019/8/14 9:59, Gao Xiang wrote:<br>
> > Hi Pratik,<br>
> > <br>
> > On Wed, Aug 14, 2019 at 02:08:40AM +0530, Pratik Shinde wrote:<br>
> >> in fill_inode() we call iloc() twice.Avoiding the extra call by<br>
> >> storing the result.<br>
> >><br>
> >> Signed-off-by: Pratik Shinde <<a href="mailto:pratikshinde320@gmail.com" target="_blank">pratikshinde320@gmail.com</a>><br>
> > <br>
> > I have no objection of this patch, but I'd like to<br>
> > hear Chao/Greg's idea about this...<br>
> <br>
> It looks more clean. :)<br>
> <br>
> Nitpick, maybe change 'inode_loc' to shorter 'iloc' will be better.<br>
<br>
iloc is the name of static inline helper function in internal.h<br>
used for shorter lines...<br>
<br>
Thanks,<br>
Gao Xiang<br>
<br>
> <br>
> Reviewed-by: Chao Yu <<a href="mailto:yuchao0@huawei.com" target="_blank">yuchao0@huawei.com</a>><br>
> <br>
> Thanks,<br>
> <br>
> > <br>
> > Thanks,<br>
> > Gao Xiang<br>
> > <br>
> >> ---<br>
> >>  drivers/staging/erofs/inode.c | 7 ++++---<br>
> >>  1 file changed, 4 insertions(+), 3 deletions(-)<br>
> >><br>
> >> diff --git a/drivers/staging/erofs/inode.c b/drivers/staging/erofs/inode.c<br>
> >> index 4c3d8bf..d82ba6c 100644<br>
> >> --- a/drivers/staging/erofs/inode.c<br>
> >> +++ b/drivers/staging/erofs/inode.c<br>
> >> @@ -167,11 +167,12 @@ static int fill_inode(struct inode *inode, int isdir)<br>
> >>    int err;<br>
> >>    erofs_blk_t blkaddr;<br>
> >>    unsigned int ofs;<br>
> >> +  erofs_off_t inode_loc;<br>
> >>  <br>
> >>    trace_erofs_fill_inode(inode, isdir);<br>
> >> -<br>
> >> -  blkaddr = erofs_blknr(iloc(sbi, vi->nid));<br>
> >> -  ofs = erofs_blkoff(iloc(sbi, vi->nid));<br>
> >> +  inode_loc = iloc(sbi, vi->nid);<br>
> >> +  blkaddr = erofs_blknr(inode_loc);<br>
> >> +  ofs = erofs_blkoff(inode_loc);<br>
> >>  <br>
> >>    debugln("%s, reading inode nid %llu at %u of blkaddr %u",<br>
> >>            __func__, vi->nid, ofs, blkaddr);<br>
> >> -- <br>
> >> 2.9.3<br>
> >><br>
> > .<br>
> > <br>
</blockquote></div>