On Fri, Mar 13, 2020 at 01:49:57PM -0400, Waiman Long wrote:
> >> if (!tmpbuf || unlikely(ret > tmpbuflen)) {
> >> if (unlikely(tmpbuf))
> >> - kzfree(tmpbuf);
> >> + __kvzfree(tmpbuf, tmpbuflen);
> > Both kzfree() and __kvzfree() handle a NULL pointer, so there's no need for the
> > NULL check first.
> >
> I would like to keep this one because of the unlikely annotation.
What (measurable) gain does it bring anyway?
/Jarkko
