[PATCH] riscv: Prevent early kernel panic in instrumented apply_early_boot_alternatives

[PATCH] riscv: Prevent early kernel panic in instrumented apply_early_boot_alternatives

[Changbin Du]

Under FTRACE=y, DYNAMIC_FTRACE=n, and RISCV_ALTERNATIVE_EARLY=n, the kernel
panics upon returning from _mcount() in the early boot path. This occurs
during _start_kernel() -> setup_vm() -> apply_early_boot_alternatives().

The CC_FLAGS_FTRACE is only removed from alternative.c when
CONFIG_RISCV_ALTERNATIVE_EARLY=y. Therefore, no function calls should be
made to alternative.c during early boot in this configuration.

Fixes: a35707c3d850 ("riscv: add memory-type errata for T-Head")
Signed-off-by: Changbin Du <changbin.du@huawei.com>
---
 arch/riscv/include/asm/alternative.h | 6 ++++++
 arch/riscv/kernel/alternative.c      | 4 ++--
 2 files changed, 8 insertions(+), 2 deletions(-)

diff --git a/arch/riscv/include/asm/alternative.h b/arch/riscv/include/asm/alternative.h
index 3c2b59b25017..c670b0cc55f4 100644
--- a/arch/riscv/include/asm/alternative.h
+++ b/arch/riscv/include/asm/alternative.h
@@ -31,7 +31,13 @@
 #define ALT_ALT_PTR(a)			__ALT_PTR(a, alt_offset)
 
 void __init apply_boot_alternatives(void);
+
+# ifdef CONFIG_RISCV_ALTERNATIVE_EARLY
 void __init apply_early_boot_alternatives(void);
+# else
+static inline void apply_early_boot_alternatives(void) { }
+# endif
+
 void apply_module_alternatives(void *start, size_t length);
 
 void riscv_alternative_fix_offsets(void *alt_ptr, unsigned int len,
diff --git a/arch/riscv/kernel/alternative.c b/arch/riscv/kernel/alternative.c
index 7eb3cb1215c6..5406c3301627 100644
--- a/arch/riscv/kernel/alternative.c
+++ b/arch/riscv/kernel/alternative.c
@@ -205,6 +205,7 @@ void __init apply_boot_alternatives(void)
 	apply_vdso_alternatives();
 }
 
+#ifdef CONFIG_RISCV_ALTERNATIVE_EARLY
 /*
  * apply_early_boot_alternatives() is called from setup_vm() with MMU-off.
  *
@@ -219,12 +220,11 @@ void __init apply_boot_alternatives(void)
  */
 void __init apply_early_boot_alternatives(void)
 {
-#ifdef CONFIG_RISCV_ALTERNATIVE_EARLY
 	_apply_alternatives((struct alt_entry *)__alt_start,
 			    (struct alt_entry *)__alt_end,
 			    RISCV_ALTERNATIVES_EARLY_BOOT);
-#endif
 }
+#endif
 
 #ifdef CONFIG_MODULES
 void apply_module_alternatives(void *start, size_t length)
-- 
2.43.0


_______________________________________________
linux-riscv mailing list
linux-riscv@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-riscv

[PATCH] riscv: Prevent early kernel panic in instrumented apply_early_boot_alternatives

[Changbin Du]

Under FTRACE=y, DYNAMIC_FTRACE=n, and RISCV_ALTERNATIVE_EARLY=n, the kernel
panics upon returning from _mcount() in the early boot path. This occurs
during _start_kernel() -> setup_vm() -> apply_early_boot_alternatives().

The CC_FLAGS_FTRACE is only removed from alternative.c when
CONFIG_RISCV_ALTERNATIVE_EARLY=y. Therefore, no function calls should be
made to alternative.c during early boot in this configuration.

Fixes: a35707c3d850 ("riscv: add memory-type errata for T-Head")
Signed-off-by: Changbin Du <changbin.du@huawei.com>
---
 arch/riscv/include/asm/alternative.h | 6 ++++++
 arch/riscv/kernel/alternative.c      | 4 ++--
 2 files changed, 8 insertions(+), 2 deletions(-)

diff --git a/arch/riscv/include/asm/alternative.h b/arch/riscv/include/asm/alternative.h
index 3c2b59b25017..c670b0cc55f4 100644
--- a/arch/riscv/include/asm/alternative.h
+++ b/arch/riscv/include/asm/alternative.h
@@ -31,7 +31,13 @@
 #define ALT_ALT_PTR(a)			__ALT_PTR(a, alt_offset)
 
 void __init apply_boot_alternatives(void);
+
+# ifdef CONFIG_RISCV_ALTERNATIVE_EARLY
 void __init apply_early_boot_alternatives(void);
+# else
+static inline void apply_early_boot_alternatives(void) { }
+# endif
+
 void apply_module_alternatives(void *start, size_t length);
 
 void riscv_alternative_fix_offsets(void *alt_ptr, unsigned int len,
diff --git a/arch/riscv/kernel/alternative.c b/arch/riscv/kernel/alternative.c
index 7eb3cb1215c6..5406c3301627 100644
--- a/arch/riscv/kernel/alternative.c
+++ b/arch/riscv/kernel/alternative.c
@@ -205,6 +205,7 @@ void __init apply_boot_alternatives(void)
 	apply_vdso_alternatives();
 }
 
+#ifdef CONFIG_RISCV_ALTERNATIVE_EARLY
 /*
  * apply_early_boot_alternatives() is called from setup_vm() with MMU-off.
  *
@@ -219,12 +220,11 @@ void __init apply_boot_alternatives(void)
  */
 void __init apply_early_boot_alternatives(void)
 {
-#ifdef CONFIG_RISCV_ALTERNATIVE_EARLY
 	_apply_alternatives((struct alt_entry *)__alt_start,
 			    (struct alt_entry *)__alt_end,
 			    RISCV_ALTERNATIVES_EARLY_BOOT);
-#endif
 }
+#endif
 
 #ifdef CONFIG_MODULES
 void apply_module_alternatives(void *start, size_t length)
-- 
2.43.0

Re: [PATCH] riscv: Prevent early kernel panic in instrumented apply_early_boot_alternatives

[Alexandre Ghiti]

Hi Changbin,

On 6/24/25 13:30, Changbin Du wrote:
> Under FTRACE=y, DYNAMIC_FTRACE=n, and RISCV_ALTERNATIVE_EARLY=n, the kernel


Your above config works fine for me, I guess you meant FUNCTION_TRACER 
&& !DYNAMIC_FTRACE (which fails).

We were just talking with Andy about this configuration (FUNCTION_TRACER 
&& !DYNAMIC_FTRACE): do we really want to support static ftrace? Andy 
should send a patch soon to remove this possibility as IMO we don't want 
to support it. Let's wait for this patch and the discussion that will 
follow before merging your fix. I'll keep it in my list for 6.16 just in 
case someone comes up with a good argument to keep it.

Thanks,

Alex


> panics upon returning from _mcount() in the early boot path. This occurs
> during _start_kernel() -> setup_vm() -> apply_early_boot_alternatives().
>
> The CC_FLAGS_FTRACE is only removed from alternative.c when
> CONFIG_RISCV_ALTERNATIVE_EARLY=y. Therefore, no function calls should be
> made to alternative.c during early boot in this configuration.
>
> Fixes: a35707c3d850 ("riscv: add memory-type errata for T-Head")
> Signed-off-by: Changbin Du <changbin.du@huawei.com>
> ---
>   arch/riscv/include/asm/alternative.h | 6 ++++++
>   arch/riscv/kernel/alternative.c      | 4 ++--
>   2 files changed, 8 insertions(+), 2 deletions(-)
>
> diff --git a/arch/riscv/include/asm/alternative.h b/arch/riscv/include/asm/alternative.h
> index 3c2b59b25017..c670b0cc55f4 100644
> --- a/arch/riscv/include/asm/alternative.h
> +++ b/arch/riscv/include/asm/alternative.h
> @@ -31,7 +31,13 @@
>   #define ALT_ALT_PTR(a)			__ALT_PTR(a, alt_offset)
>   
>   void __init apply_boot_alternatives(void);
> +
> +# ifdef CONFIG_RISCV_ALTERNATIVE_EARLY
>   void __init apply_early_boot_alternatives(void);
> +# else
> +static inline void apply_early_boot_alternatives(void) { }
> +# endif
> +
>   void apply_module_alternatives(void *start, size_t length);
>   
>   void riscv_alternative_fix_offsets(void *alt_ptr, unsigned int len,
> diff --git a/arch/riscv/kernel/alternative.c b/arch/riscv/kernel/alternative.c
> index 7eb3cb1215c6..5406c3301627 100644
> --- a/arch/riscv/kernel/alternative.c
> +++ b/arch/riscv/kernel/alternative.c
> @@ -205,6 +205,7 @@ void __init apply_boot_alternatives(void)
>   	apply_vdso_alternatives();
>   }
>   
> +#ifdef CONFIG_RISCV_ALTERNATIVE_EARLY
>   /*
>    * apply_early_boot_alternatives() is called from setup_vm() with MMU-off.
>    *
> @@ -219,12 +220,11 @@ void __init apply_boot_alternatives(void)
>    */
>   void __init apply_early_boot_alternatives(void)
>   {
> -#ifdef CONFIG_RISCV_ALTERNATIVE_EARLY
>   	_apply_alternatives((struct alt_entry *)__alt_start,
>   			    (struct alt_entry *)__alt_end,
>   			    RISCV_ALTERNATIVES_EARLY_BOOT);
> -#endif
>   }
> +#endif
>   
>   #ifdef CONFIG_MODULES
>   void apply_module_alternatives(void *start, size_t length)

_______________________________________________
linux-riscv mailing list
linux-riscv@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-riscv

Re: [PATCH] riscv: Prevent early kernel panic in instrumented apply_early_boot_alternatives

[Alexandre Ghiti]

Hi Changbin,

On 6/24/25 13:30, Changbin Du wrote:
> Under FTRACE=y, DYNAMIC_FTRACE=n, and RISCV_ALTERNATIVE_EARLY=n, the kernel


Your above config works fine for me, I guess you meant FUNCTION_TRACER 
&& !DYNAMIC_FTRACE (which fails).

We were just talking with Andy about this configuration (FUNCTION_TRACER 
&& !DYNAMIC_FTRACE): do we really want to support static ftrace? Andy 
should send a patch soon to remove this possibility as IMO we don't want 
to support it. Let's wait for this patch and the discussion that will 
follow before merging your fix. I'll keep it in my list for 6.16 just in 
case someone comes up with a good argument to keep it.

Thanks,

Alex


> panics upon returning from _mcount() in the early boot path. This occurs
> during _start_kernel() -> setup_vm() -> apply_early_boot_alternatives().
>
> The CC_FLAGS_FTRACE is only removed from alternative.c when
> CONFIG_RISCV_ALTERNATIVE_EARLY=y. Therefore, no function calls should be
> made to alternative.c during early boot in this configuration.
>
> Fixes: a35707c3d850 ("riscv: add memory-type errata for T-Head")
> Signed-off-by: Changbin Du <changbin.du@huawei.com>
> ---
>   arch/riscv/include/asm/alternative.h | 6 ++++++
>   arch/riscv/kernel/alternative.c      | 4 ++--
>   2 files changed, 8 insertions(+), 2 deletions(-)
>
> diff --git a/arch/riscv/include/asm/alternative.h b/arch/riscv/include/asm/alternative.h
> index 3c2b59b25017..c670b0cc55f4 100644
> --- a/arch/riscv/include/asm/alternative.h
> +++ b/arch/riscv/include/asm/alternative.h
> @@ -31,7 +31,13 @@
>   #define ALT_ALT_PTR(a)			__ALT_PTR(a, alt_offset)
>   
>   void __init apply_boot_alternatives(void);
> +
> +# ifdef CONFIG_RISCV_ALTERNATIVE_EARLY
>   void __init apply_early_boot_alternatives(void);
> +# else
> +static inline void apply_early_boot_alternatives(void) { }
> +# endif
> +
>   void apply_module_alternatives(void *start, size_t length);
>   
>   void riscv_alternative_fix_offsets(void *alt_ptr, unsigned int len,
> diff --git a/arch/riscv/kernel/alternative.c b/arch/riscv/kernel/alternative.c
> index 7eb3cb1215c6..5406c3301627 100644
> --- a/arch/riscv/kernel/alternative.c
> +++ b/arch/riscv/kernel/alternative.c
> @@ -205,6 +205,7 @@ void __init apply_boot_alternatives(void)
>   	apply_vdso_alternatives();
>   }
>   
> +#ifdef CONFIG_RISCV_ALTERNATIVE_EARLY
>   /*
>    * apply_early_boot_alternatives() is called from setup_vm() with MMU-off.
>    *
> @@ -219,12 +220,11 @@ void __init apply_boot_alternatives(void)
>    */
>   void __init apply_early_boot_alternatives(void)
>   {
> -#ifdef CONFIG_RISCV_ALTERNATIVE_EARLY
>   	_apply_alternatives((struct alt_entry *)__alt_start,
>   			    (struct alt_entry *)__alt_end,
>   			    RISCV_ALTERNATIVES_EARLY_BOOT);
> -#endif
>   }
> +#endif
>   
>   #ifdef CONFIG_MODULES
>   void apply_module_alternatives(void *start, size_t length)

Re: [PATCH] riscv: Prevent early kernel panic in instrumented apply_early_boot_alternatives

[duchangbin]

On Tue, Jun 24, 2025 at 02:54:32PM +0200, Alexandre Ghiti wrote:
> Hi Changbin,
> 
> On 6/24/25 13:30, Changbin Du wrote:
> > Under FTRACE=y, DYNAMIC_FTRACE=n, and RISCV_ALTERNATIVE_EARLY=n, the kernel
> 
> 
> Your above config works fine for me, I guess you meant FUNCTION_TRACER &&
> !DYNAMIC_FTRACE (which fails).
>
Yes, it's FUNCTION_TRACER.

> We were just talking with Andy about this configuration (FUNCTION_TRACER &&
> !DYNAMIC_FTRACE): do we really want to support static ftrace? Andy should
> send a patch soon to remove this possibility as IMO we don't want to support
> it. Let's wait for this patch and the discussion that will follow before
> merging your fix. I'll keep it in my list for 6.16 just in case someone
> comes up with a good argument to keep it.
> 
No problem. I'm unable to enable DYNAMIC_FTRACE because GCC_SUPPORTS_DYNAMIC_FTRACE=n
and CC_HAS_MIN_FUNCTION_ALIGNMENT=n are set. It seems that my GCC version (13.3.0)
does not support the option -fmin-function-alignment=8.

By the way, this change also eliminates an empty function call.

> Thanks,
> 
> Alex
> 
> 
> > panics upon returning from _mcount() in the early boot path. This occurs
> > during _start_kernel() -> setup_vm() -> apply_early_boot_alternatives().
> > 
> > The CC_FLAGS_FTRACE is only removed from alternative.c when
> > CONFIG_RISCV_ALTERNATIVE_EARLY=y. Therefore, no function calls should be
> > made to alternative.c during early boot in this configuration.
> > 
> > Fixes: a35707c3d850 ("riscv: add memory-type errata for T-Head")
> > Signed-off-by: Changbin Du <changbin.du@huawei.com>
> > ---
> >   arch/riscv/include/asm/alternative.h | 6 ++++++
> >   arch/riscv/kernel/alternative.c      | 4 ++--
> >   2 files changed, 8 insertions(+), 2 deletions(-)
> > 
> > diff --git a/arch/riscv/include/asm/alternative.h b/arch/riscv/include/asm/alternative.h
> > index 3c2b59b25017..c670b0cc55f4 100644
> > --- a/arch/riscv/include/asm/alternative.h
> > +++ b/arch/riscv/include/asm/alternative.h
> > @@ -31,7 +31,13 @@
> >   #define ALT_ALT_PTR(a)			__ALT_PTR(a, alt_offset)
> >   void __init apply_boot_alternatives(void);
> > +
> > +# ifdef CONFIG_RISCV_ALTERNATIVE_EARLY
> >   void __init apply_early_boot_alternatives(void);
> > +# else
> > +static inline void apply_early_boot_alternatives(void) { }
> > +# endif
> > +
> >   void apply_module_alternatives(void *start, size_t length);
> >   void riscv_alternative_fix_offsets(void *alt_ptr, unsigned int len,
> > diff --git a/arch/riscv/kernel/alternative.c b/arch/riscv/kernel/alternative.c
> > index 7eb3cb1215c6..5406c3301627 100644
> > --- a/arch/riscv/kernel/alternative.c
> > +++ b/arch/riscv/kernel/alternative.c
> > @@ -205,6 +205,7 @@ void __init apply_boot_alternatives(void)
> >   	apply_vdso_alternatives();
> >   }
> > +#ifdef CONFIG_RISCV_ALTERNATIVE_EARLY
> >   /*
> >    * apply_early_boot_alternatives() is called from setup_vm() with MMU-off.
> >    *
> > @@ -219,12 +220,11 @@ void __init apply_boot_alternatives(void)
> >    */
> >   void __init apply_early_boot_alternatives(void)
> >   {
> > -#ifdef CONFIG_RISCV_ALTERNATIVE_EARLY
> >   	_apply_alternatives((struct alt_entry *)__alt_start,
> >   			    (struct alt_entry *)__alt_end,
> >   			    RISCV_ALTERNATIVES_EARLY_BOOT);
> > -#endif
> >   }
> > +#endif
> >   #ifdef CONFIG_MODULES
> >   void apply_module_alternatives(void *start, size_t length)

-- 
Cheers,
Changbin Du

_______________________________________________
linux-riscv mailing list
linux-riscv@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-riscv

Re: [PATCH] riscv: Prevent early kernel panic in instrumented apply_early_boot_alternatives

[duchangbin]

On Tue, Jun 24, 2025 at 02:54:32PM +0200, Alexandre Ghiti wrote:
> Hi Changbin,
> 
> On 6/24/25 13:30, Changbin Du wrote:
> > Under FTRACE=y, DYNAMIC_FTRACE=n, and RISCV_ALTERNATIVE_EARLY=n, the kernel
> 
> 
> Your above config works fine for me, I guess you meant FUNCTION_TRACER &&
> !DYNAMIC_FTRACE (which fails).
>
Yes, it's FUNCTION_TRACER.

> We were just talking with Andy about this configuration (FUNCTION_TRACER &&
> !DYNAMIC_FTRACE): do we really want to support static ftrace? Andy should
> send a patch soon to remove this possibility as IMO we don't want to support
> it. Let's wait for this patch and the discussion that will follow before
> merging your fix. I'll keep it in my list for 6.16 just in case someone
> comes up with a good argument to keep it.
> 
No problem. I'm unable to enable DYNAMIC_FTRACE because GCC_SUPPORTS_DYNAMIC_FTRACE=n
and CC_HAS_MIN_FUNCTION_ALIGNMENT=n are set. It seems that my GCC version (13.3.0)
does not support the option -fmin-function-alignment=8.

By the way, this change also eliminates an empty function call.

> Thanks,
> 
> Alex
> 
> 
> > panics upon returning from _mcount() in the early boot path. This occurs
> > during _start_kernel() -> setup_vm() -> apply_early_boot_alternatives().
> > 
> > The CC_FLAGS_FTRACE is only removed from alternative.c when
> > CONFIG_RISCV_ALTERNATIVE_EARLY=y. Therefore, no function calls should be
> > made to alternative.c during early boot in this configuration.
> > 
> > Fixes: a35707c3d850 ("riscv: add memory-type errata for T-Head")
> > Signed-off-by: Changbin Du <changbin.du@huawei.com>
> > ---
> >   arch/riscv/include/asm/alternative.h | 6 ++++++
> >   arch/riscv/kernel/alternative.c      | 4 ++--
> >   2 files changed, 8 insertions(+), 2 deletions(-)
> > 
> > diff --git a/arch/riscv/include/asm/alternative.h b/arch/riscv/include/asm/alternative.h
> > index 3c2b59b25017..c670b0cc55f4 100644
> > --- a/arch/riscv/include/asm/alternative.h
> > +++ b/arch/riscv/include/asm/alternative.h
> > @@ -31,7 +31,13 @@
> >   #define ALT_ALT_PTR(a)			__ALT_PTR(a, alt_offset)
> >   void __init apply_boot_alternatives(void);
> > +
> > +# ifdef CONFIG_RISCV_ALTERNATIVE_EARLY
> >   void __init apply_early_boot_alternatives(void);
> > +# else
> > +static inline void apply_early_boot_alternatives(void) { }
> > +# endif
> > +
> >   void apply_module_alternatives(void *start, size_t length);
> >   void riscv_alternative_fix_offsets(void *alt_ptr, unsigned int len,
> > diff --git a/arch/riscv/kernel/alternative.c b/arch/riscv/kernel/alternative.c
> > index 7eb3cb1215c6..5406c3301627 100644
> > --- a/arch/riscv/kernel/alternative.c
> > +++ b/arch/riscv/kernel/alternative.c
> > @@ -205,6 +205,7 @@ void __init apply_boot_alternatives(void)
> >   	apply_vdso_alternatives();
> >   }
> > +#ifdef CONFIG_RISCV_ALTERNATIVE_EARLY
> >   /*
> >    * apply_early_boot_alternatives() is called from setup_vm() with MMU-off.
> >    *
> > @@ -219,12 +220,11 @@ void __init apply_boot_alternatives(void)
> >    */
> >   void __init apply_early_boot_alternatives(void)
> >   {
> > -#ifdef CONFIG_RISCV_ALTERNATIVE_EARLY
> >   	_apply_alternatives((struct alt_entry *)__alt_start,
> >   			    (struct alt_entry *)__alt_end,
> >   			    RISCV_ALTERNATIVES_EARLY_BOOT);
> > -#endif
> >   }
> > +#endif
> >   #ifdef CONFIG_MODULES
> >   void apply_module_alternatives(void *start, size_t length)

-- 
Cheers,
Changbin Du

Re: [PATCH] riscv: Prevent early kernel panic in instrumented apply_early_boot_alternatives

[Paul Walmsley]

Hi Alex, duchangbin, Andy,

On Wed, 25 Jun 2025, duchangbin wrote:

> On Tue, Jun 24, 2025 at 02:54:32PM +0200, Alexandre Ghiti wrote:
> > Hi Changbin,
> > 
> > On 6/24/25 13:30, Changbin Du wrote:
> > > Under FTRACE=y, DYNAMIC_FTRACE=n, and RISCV_ALTERNATIVE_EARLY=n, the kernel
> > 
> > 
> > Your above config works fine for me, I guess you meant FUNCTION_TRACER &&
> > !DYNAMIC_FTRACE (which fails).
> >
> Yes, it's FUNCTION_TRACER.
> 
> > We were just talking with Andy about this configuration (FUNCTION_TRACER &&
> > !DYNAMIC_FTRACE): do we really want to support static ftrace? Andy should
> > send a patch soon to remove this possibility as IMO we don't want to support
> > it. Let's wait for this patch and the discussion that will follow before
> > merging your fix. I'll keep it in my list for 6.16 just in case someone
> > comes up with a good argument to keep it.
> > 
> No problem. I'm unable to enable DYNAMIC_FTRACE because GCC_SUPPORTS_DYNAMIC_FTRACE=n
> and CC_HAS_MIN_FUNCTION_ALIGNMENT=n are set. It seems that my GCC version (13.3.0)
> does not support the option -fmin-function-alignment=8.
> 
> By the way, this change also eliminates an empty function call.

Working on cleaning out Patchwork.  Was there any further conclusion 
reached on this patch, or more broadly, static ftrace ?


- Paul

_______________________________________________
linux-riscv mailing list
linux-riscv@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-riscv

Re: [PATCH] riscv: Prevent early kernel panic in instrumented apply_early_boot_alternatives

[Paul Walmsley]

Hi Alex, duchangbin, Andy,

On Wed, 25 Jun 2025, duchangbin wrote:

> On Tue, Jun 24, 2025 at 02:54:32PM +0200, Alexandre Ghiti wrote:
> > Hi Changbin,
> > 
> > On 6/24/25 13:30, Changbin Du wrote:
> > > Under FTRACE=y, DYNAMIC_FTRACE=n, and RISCV_ALTERNATIVE_EARLY=n, the kernel
> > 
> > 
> > Your above config works fine for me, I guess you meant FUNCTION_TRACER &&
> > !DYNAMIC_FTRACE (which fails).
> >
> Yes, it's FUNCTION_TRACER.
> 
> > We were just talking with Andy about this configuration (FUNCTION_TRACER &&
> > !DYNAMIC_FTRACE): do we really want to support static ftrace? Andy should
> > send a patch soon to remove this possibility as IMO we don't want to support
> > it. Let's wait for this patch and the discussion that will follow before
> > merging your fix. I'll keep it in my list for 6.16 just in case someone
> > comes up with a good argument to keep it.
> > 
> No problem. I'm unable to enable DYNAMIC_FTRACE because GCC_SUPPORTS_DYNAMIC_FTRACE=n
> and CC_HAS_MIN_FUNCTION_ALIGNMENT=n are set. It seems that my GCC version (13.3.0)
> does not support the option -fmin-function-alignment=8.
> 
> By the way, this change also eliminates an empty function call.

Working on cleaning out Patchwork.  Was there any further conclusion 
reached on this patch, or more broadly, static ftrace ?


- Paul

Re: [PATCH] riscv: Prevent early kernel panic in instrumented apply_early_boot_alternatives

[Vivian Wang]

On 10/9/25 10:01, Paul Walmsley wrote:
> Hi Alex, duchangbin, Andy,
>
> On Wed, 25 Jun 2025, duchangbin wrote:
>
> [...]
>
> Working on cleaning out Patchwork.  Was there any further conclusion 
> reached on this patch, or more broadly, static ftrace ?

Static ftrace for riscv is no longer supported in 6.17. config RISCV is now:

    select DYNAMIC_FTRACE if FUNCTION_TRACER

Since commit 5874ca4c6280 ("riscv: Stop supporting static ftrace"). This
patch should be obsolete.

Vivian "dramforever" Wang


_______________________________________________
linux-riscv mailing list
linux-riscv@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-riscv

Re: [PATCH] riscv: Prevent early kernel panic in instrumented apply_early_boot_alternatives

[Vivian Wang]

On 10/9/25 10:01, Paul Walmsley wrote:
> Hi Alex, duchangbin, Andy,
>
> On Wed, 25 Jun 2025, duchangbin wrote:
>
> [...]
>
> Working on cleaning out Patchwork.  Was there any further conclusion 
> reached on this patch, or more broadly, static ftrace ?

Static ftrace for riscv is no longer supported in 6.17. config RISCV is now:

    select DYNAMIC_FTRACE if FUNCTION_TRACER

Since commit 5874ca4c6280 ("riscv: Stop supporting static ftrace"). This
patch should be obsolete.

Vivian "dramforever" Wang

Re: [PATCH] riscv: Prevent early kernel panic in instrumented apply_early_boot_alternatives

[Paul Walmsley]

[-- Attachment #1: Type: text/plain, Size: 560 bytes --]

Hi Vivian,

On Thu, 9 Oct 2025, Vivian Wang wrote:
> On 10/9/25 10:01, Paul Walmsley wrote:
> > On Wed, 25 Jun 2025, duchangbin wrote:
> >
> > [...]
> >
> > Working on cleaning out Patchwork.  Was there any further conclusion 
> > reached on this patch, or more broadly, static ftrace ?
> 
> Static ftrace for riscv is no longer supported in 6.17. config RISCV is now:
> 
>     select DYNAMIC_FTRACE if FUNCTION_TRACER
> 
> Since commit 5874ca4c6280 ("riscv: Stop supporting static ftrace"). This
> patch should be obsolete.

Thanks for the followup.

- Paul

[-- Attachment #2: Type: text/plain, Size: 161 bytes --]

_______________________________________________
linux-riscv mailing list
linux-riscv@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-riscv

Re: [PATCH] riscv: Prevent early kernel panic in instrumented apply_early_boot_alternatives

[Paul Walmsley]

[-- Attachment #1: Type: text/plain, Size: 560 bytes --]

Hi Vivian,

On Thu, 9 Oct 2025, Vivian Wang wrote:
> On 10/9/25 10:01, Paul Walmsley wrote:
> > On Wed, 25 Jun 2025, duchangbin wrote:
> >
> > [...]
> >
> > Working on cleaning out Patchwork.  Was there any further conclusion 
> > reached on this patch, or more broadly, static ftrace ?
> 
> Static ftrace for riscv is no longer supported in 6.17. config RISCV is now:
> 
>     select DYNAMIC_FTRACE if FUNCTION_TRACER
> 
> Since commit 5874ca4c6280 ("riscv: Stop supporting static ftrace"). This
> patch should be obsolete.

Thanks for the followup.

- Paul