2023-11-20 10:59:39

by Valentin Schneider

Subject: [PATCH 0/5] jump_label: Fix __ro_after_init keys for modules & annotate some keys

Hi folks,

After chatting about deferring IPIs [1] at LPC I had another look at my patches
and realized a handful of them could already be sent as-is.

This series contains the __ro_after_init static_key bits, which fixes
__ro_after_init keys used in modules (courtesy of PeterZ) and flags more keys as
__ro_after_init.

[1]: https://lore.kernel.org/lkml/[email protected]/

Cheers,
Valentin

Peter Zijlstra (1):
jump_label,module: Don't alloc static_key_mod for __ro_after_init keys

Valentin Schneider (4):
context_tracking: Make context_tracking_key __ro_after_init
x86/kvm: Make kvm_async_pf_enabled __ro_after_init
x86/speculation: Make mds_user_clear __ro_after_init
x86/tsc: Make __use_tsc __ro_after_init

arch/x86/kernel/cpu/bugs.c | 2 +-
arch/x86/kernel/kvm.c | 2 +-
arch/x86/kernel/tsc.c | 2 +-
include/asm-generic/sections.h | 5 ++++
include/linux/jump_label.h | 1 +
init/main.c | 1 +
kernel/context_tracking.c | 2 +-
kernel/jump_label.c | 49 ++++++++++++++++++++++++++++++++++
8 files changed, 60 insertions(+), 4 deletions(-)

--
2.41.0

2023-11-20 10:59:53

by Valentin Schneider

Subject: [PATCH 3/5] x86/kvm: Make kvm_async_pf_enabled __ro_after_init

kvm_async_pf_enabled is only ever enabled in __init kvm_guest_init(), so
mark it as __ro_after_init.

Signed-off-by: Valentin Schneider <[email protected]>
---
arch/x86/kernel/kvm.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/x86/kernel/kvm.c b/arch/x86/kernel/kvm.c
index 0ddb3bd0f1aac..146e16f420edf 100644
--- a/arch/x86/kernel/kvm.c
+++ b/arch/x86/kernel/kvm.c
@@ -44,7 +44,7 @@
#include <asm/svm.h>
#include <asm/e820/api.h>

-DEFINE_STATIC_KEY_FALSE(kvm_async_pf_enabled);
+DEFINE_STATIC_KEY_FALSE_RO(kvm_async_pf_enabled);

static int kvmapf = 1;

--
2.41.0

2023-11-27 22:23:31

by Sean Christopherson

Subject: Re: [PATCH 3/5] x86/kvm: Make kvm_async_pf_enabled __ro_after_init

On Mon, Nov 20, 2023, Valentin Schneider wrote:
> kvm_async_pf_enabled is only ever enabled in __init kvm_guest_init(), so
> mark it as __ro_after_init.
>
> Signed-off-by: Valentin Schneider <[email protected]>
> ---

Reviewed-by: Sean Christopherson <[email protected]>

2023-12-02 16:36:55

by Josh Poimboeuf

Subject: Re: [PATCH 0/5] jump_label: Fix __ro_after_init keys for modules & annotate some keys

On Mon, Nov 20, 2023 at 11:55:23AM +0100, Valentin Schneider wrote:
> Hi folks,
>
> After chatting about deferring IPIs [1] at LPC I had another look at my patches
> and realized a handful of them could already be sent as-is.
>
> This series contains the __ro_after_init static_key bits, which fixes
> __ro_after_init keys used in modules (courtesy of PeterZ) and flags more keys as
> __ro_after_init.
>
> [1]: https://lore.kernel.org/lkml/[email protected]/

Acked-by: Josh Poimboeuf <[email protected]>

--
Josh