2021-05-16 16:36:50

by Jisheng Zhang

Subject: [PATCH] riscv: Add __init section marker to some functions again

From: Jisheng Zhang <[email protected]>

These functions are not needed after booting, so mark them as __init
to move them to the __init section.

Signed-off-by: Jisheng Zhang <[email protected]>
---
arch/riscv/kernel/cpufeature.c | 2 +-
arch/riscv/mm/context.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/riscv/kernel/cpufeature.c b/arch/riscv/kernel/cpufeature.c
index a2848dc36927..d959d207a40d 100644
--- a/arch/riscv/kernel/cpufeature.c
+++ b/arch/riscv/kernel/cpufeature.c
@@ -59,7 +59,7 @@ bool __riscv_isa_extension_available(const unsigned long *isa_bitmap, int bit)
}
EXPORT_SYMBOL_GPL(__riscv_isa_extension_available);

-void riscv_fill_hwcap(void)
+void __init riscv_fill_hwcap(void)
{
struct device_node *node;
const char *isa;
diff --git a/arch/riscv/mm/context.c b/arch/riscv/mm/context.c
index 6d445f2888ec..63dc0b9bd8aa 100644
--- a/arch/riscv/mm/context.c
+++ b/arch/riscv/mm/context.c
@@ -213,7 +213,7 @@ static inline void set_mm(struct mm_struct *mm, unsigned int cpu)
set_mm_noasid(mm);
}

-static int asids_init(void)
+static int __init asids_init(void)
{
unsigned long old;

--
2.31.0

2021-05-29 21:15:15

by Palmer Dabbelt

Subject: Re: [PATCH] riscv: Add __init section marker to some functions again

On Sun, 16 May 2021 05:59:42 PDT (-0700), [email protected] wrote:
> From: Jisheng Zhang <[email protected]>
>
> These functions are not needed after booting, so mark them as __init
> to move them to the __init section.
>
> Signed-off-by: Jisheng Zhang <[email protected]>
> ---
> arch/riscv/kernel/cpufeature.c | 2 +-
> arch/riscv/mm/context.c | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/arch/riscv/kernel/cpufeature.c b/arch/riscv/kernel/cpufeature.c
> index a2848dc36927..d959d207a40d 100644
> --- a/arch/riscv/kernel/cpufeature.c
> +++ b/arch/riscv/kernel/cpufeature.c
> @@ -59,7 +59,7 @@ bool __riscv_isa_extension_available(const unsigned long *isa_bitmap, int bit)
> }
> EXPORT_SYMBOL_GPL(__riscv_isa_extension_available);
>
> -void riscv_fill_hwcap(void)
> +void __init riscv_fill_hwcap(void)
> {
> struct device_node *node;
> const char *isa;
> diff --git a/arch/riscv/mm/context.c b/arch/riscv/mm/context.c
> index 6d445f2888ec..63dc0b9bd8aa 100644
> --- a/arch/riscv/mm/context.c
> +++ b/arch/riscv/mm/context.c
> @@ -213,7 +213,7 @@ static inline void set_mm(struct mm_struct *mm, unsigned int cpu)
> set_mm_noasid(mm);
> }
>
> -static int asids_init(void)
> +static int __init asids_init(void)
> {
> unsigned long old;

Thanks, this is on for-next.