Re: [PATCH 03/10] jump label v11: base patch

Andi Kleen
2010-09-21T14:41:21Z

>
> So there are ~150 tracepoints, but this code is also being proposed for
> use with 'dynamic debug' of which there are > 1000, and I'm hoping for
> more users moving forward.

Even 1000 is fine to walk, but if it was sorted a binary search
would be much faster anyways. That is then you would still
need to search for each module, but that is a relatively small
number (< 100)

> Also, I think the hash table deals nicely with modules.

Maybe but it's also a lot of code. And it seems to me
that it is optimizing the wrong thing. Simpler is nicer.

-Andi