Messages in this thread | | | Subject | Re: [PATCH] likely cleanup: remove unlikely for kfree(NULL) | From | Arjan van de Ven <> | Date | Wed, 26 Apr 2006 10:27:18 +0200 |
| |
On Wed, 2006-04-26 at 11:16 +0300, Pekka J Enberg wrote: > On 4/25/06, Hua Zhong <hzhong@gmail.com> wrote: > > > > diff --git a/mm/slab.c b/mm/slab.c > > > > index e6ef9bd..0fbc854 100644 > > > > --- a/mm/slab.c > > > > +++ b/mm/slab.c > > > > @@ -3380,7 +3380,7 @@ void kfree(const void *objp) > > > > struct kmem_cache *c; > > > > unsigned long flags; > > > > > > > > - if (unlikely(!objp)) > > > > + if (!objp) > > > > return; > > > On Wed, 2006-04-26 at 10:30 +0300, Pekka Enberg wrote: > > > NAK. Fix the callers instead. > > On Wed, 26 Apr 2006, Arjan van de Ven wrote: > > eh dude... they are being fixed... to remove the NULL check :) > > Most of which are on error paths. The problem we're seeing is in handful > of fastpath offenders which should be fixed either by re-design or adding > the NULL check along with a big fat comment like Andrew is doing.
what I would like is kfree to become an inline wrapper that does the null check inline, that way gcc can optimize it out (and it will in 4.1 with the VRP pass) if gcc can prove it's not NULL.
- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |