Re: [PATCH] livepatch/module: make TAINT_LIVEPATCH module-specific

From: Jiri Kosina
Date: Thu Aug 25 2016 - 00:46:44 EST


On Wed, 24 Aug 2016, Josh Poimboeuf wrote:

> There's no reliable way to determine which module tainted the kernel
> with CONFIG_LIVEPATCH. For example, /sys/module/<klp module>/taint
> doesn't report it. Neither does the "mod -t" command in the crash tool.
>
> Make it crystal clear who the guilty party is by converting
> CONFIG_LIVEPATCH to a module taint flag.
>
> This changes the behavior a bit: now the the flag gets set when the
> module is loaded, rather than when it's enabled.
>
> Reviewed-by: Chunyu Hu <chuhu@xxxxxxxxxx>
> Signed-off-by: Josh Poimboeuf <jpoimboe@xxxxxxxxxx>

I like this change.

Rusty, in case you're okay with it as well, could you please either
provide your Ack so that I could take it through livepatching.git?
Alternatively, if you prefer to merge this through your tree, please feel
free to add

Acked-by: Jiri Kosina <jkosina@xxxxxxx>

Thanks,

--
Jiri Kosina
SUSE Labs