On Mon, 2011-05-16 at 14:01 -0700, Joe Perches wrote: > On Mon, 2011-05-16 at 22:23 +0200, Alessio Igor Bogani wrote: > > Signed-off-by: Alessio Igor Bogani <abogani@xxxxxxxxxx> > > diff --git a/kernel/module.c b/kernel/module.c > > @@ -2055,10 +2055,8 @@ static const struct kernel_symbol *lookup_symbol(const char *name, > > const struct kernel_symbol *stop) > > { > > const struct kernel_symbol *ks = start; > > - for (; ks < stop; ks++) > > - if (strcmp(ks->name, name) == 0) > > - return ks; > > - return NULL; > > + return bsearch(name, start, stop - start, > > + sizeof(struct kernel_symbol), cmp_name); [] > why cmp_name and not strcmp? Nevermind, cmp_name is obviously correct. -- To unsubscribe from this list: send the line "unsubscribe linux-embedded" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html