[PATCH] erofs-utils: avoid unnecessary insert behavior when not deduplicating

Gao Xiang hsiangkao at linux.alibaba.com
Fri Oct 14 13:15:27 AEDT 2022


On Fri, Oct 14, 2022 at 09:48:46AM +0800, Yue Hu wrote:
> On Thu, 13 Oct 2022 16:59:26 +0800
> Gao Xiang <hsiangkao at linux.alibaba.com> wrote:
> 
> > Hi Yue,
> > 
> > On Thu, Oct 13, 2022 at 12:00:11PM +0800, Yue Hu wrote:
> > > From: Yue Hu <huyue2 at coolpad.com>
> > > 
> > > We should do nothing in dedupe inserting when it's not configured.
> > > 
> > > Signed-off-by: Yue Hu <huyue2 at coolpad.com>
> > > ---  
> > 
> > Thanks for the patch, do you observe some strange happening? 
> 
> I can see malloc/memcpy at runtime when dedupe is disabled. So, just skip.

Would you mind confirming the numbers of e->length and window_size 
at that time?

Thanks,
Gao Xiang

> 
> > 
> > IMO, If dedupe is not enabled, window_size will be 0 I think.
> > However, I think we might need to disable it explicitly like below.
> > 
> > So,
> > Reviewed-by: Gao Xiang <hsiangkao at linux.alibaba.com>
> > 
> > Thanks,
> > Gao Xiang


More information about the Linux-erofs mailing list