Re: [PATCH 1/3] jump_label: Warn on failed jump_label patch

From: Josh Poimboeuf
Date: Wed Feb 14 2018 - 16:17:15 EST


On Wed, Feb 14, 2018 at 12:40:34PM -0500, Steven Rostedt wrote:
> On Wed, 14 Feb 2018 11:34:07 -0600
> Josh Poimboeuf <jpoimboe@xxxxxxxxxx> wrote:
>
>
> > And in patch 1 the warning conditions need to be tweaked a bit to
> > exclude the __init case.
> >
>
> Hmm, I wonder if we could simply remove the references from the jump
> label table when init is freed.

Interesting idea. The static key's entries (i.e. tracepoint sites) are
in a static array. Maybe we could rewrite the affected arrays when init
memory is freed. I'm not sure whether that would be overkill though. I
can tinker with it and see how it compares to the current solution.

--
Josh