Re: [PATCH] livepatch: make klp_mutex proper part of API

From: Josh Poimboeuf
Date: Wed Mar 08 2017 - 12:22:36 EST


On Wed, Mar 08, 2017 at 10:16:00AM +0100, Jiri Kosina wrote:
> From: Jiri Kosina <jkosina@xxxxxxx>
>
> klp_mutex is shared between core.c and transition.c, and as such would
> rather be properly located in livepatch.h so that we don't have to play
> 'extern' games from .c sources.
>
> This also silences sparse warning (wrongly) suggesting that klp_mutex
> should be defined static.

include/linux/livepatch.h is the public interface, whereas klp_mutex is
a private implementation detail. I think it would be a better fit in
kernel/livepatch/core.h (though that file doesn't yet exist).

--
Josh