Re: [PATCH -tip v6 1/2] irq: Make irqentry text section unconditional

[Masami Hiramatsu]

On Fri, 28 Jul 2017 21:24:54 +0900
Masami Hiramatsu <mhiramat@xxxxxxxxxx> wrote:

> Generate irqentry and softirqentry text sections without
> any configs. This will add above extra sections, but
> no performace impact.
> 
> Signed-off-by: Masami Hiramatsu <mhiramat@xxxxxxxxxx>
> Suggested-by: Ingo Molnar <mingo@xxxxxxxxxx>
> ---
>  arch/arm/include/asm/traps.h      |    7 -------
>  arch/arm64/include/asm/traps.h    |    7 -------
>  arch/x86/entry/entry_64.S         |    9 ++-------
>  include/asm-generic/sections.h    |    4 ++++
>  include/asm-generic/vmlinux.lds.h |    8 --------
>  include/linux/interrupt.h         |   14 +-------------
>  6 files changed, 7 insertions(+), 42 deletions(-)

Oops, I forgot to fix x86 unwind code.
I'll update this to fix in_entry_code() unconditional.

Thanks,

> 
> diff --git a/arch/arm/include/asm/traps.h b/arch/arm/include/asm/traps.h
> index f555bb3664dc..683d9230984a 100644
> --- a/arch/arm/include/asm/traps.h
> +++ b/arch/arm/include/asm/traps.h
> @@ -18,7 +18,6 @@ struct undef_hook {
>  void register_undef_hook(struct undef_hook *hook);
>  void unregister_undef_hook(struct undef_hook *hook);
>  
> -#ifdef CONFIG_FUNCTION_GRAPH_TRACER
>  static inline int __in_irqentry_text(unsigned long ptr)
>  {
>  	extern char __irqentry_text_start[];
> @@ -27,12 +26,6 @@ static inline int __in_irqentry_text(unsigned long ptr)
>  	return ptr >= (unsigned long)&__irqentry_text_start &&
>  	       ptr < (unsigned long)&__irqentry_text_end;
>  }
> -#else
> -static inline int __in_irqentry_text(unsigned long ptr)
> -{
> -	return 0;
> -}
> -#endif
>  
>  static inline int in_exception_text(unsigned long ptr)
>  {
> diff --git a/arch/arm64/include/asm/traps.h b/arch/arm64/include/asm/traps.h
> index 02e9035b0685..47a9066f7c86 100644
> --- a/arch/arm64/include/asm/traps.h
> +++ b/arch/arm64/include/asm/traps.h
> @@ -37,18 +37,11 @@ void unregister_undef_hook(struct undef_hook *hook);
>  
>  void arm64_notify_segfault(struct pt_regs *regs, unsigned long addr);
>  
> -#ifdef CONFIG_FUNCTION_GRAPH_TRACER
>  static inline int __in_irqentry_text(unsigned long ptr)
>  {
>  	return ptr >= (unsigned long)&__irqentry_text_start &&
>  	       ptr < (unsigned long)&__irqentry_text_end;
>  }
> -#else
> -static inline int __in_irqentry_text(unsigned long ptr)
> -{
> -	return 0;
> -}
> -#endif
>  
>  static inline int in_exception_text(unsigned long ptr)
>  {
> diff --git a/arch/x86/entry/entry_64.S b/arch/x86/entry/entry_64.S
> index aa58155187c5..5e41396efbbf 100644
> --- a/arch/x86/entry/entry_64.S
> +++ b/arch/x86/entry/entry_64.S
> @@ -766,13 +766,8 @@ apicinterrupt3 \num trace(\sym) smp_trace(\sym)
>  #endif
>  
>  /* Make sure APIC interrupt handlers end up in the irqentry section: */
> -#if defined(CONFIG_FUNCTION_GRAPH_TRACER) || defined(CONFIG_KASAN)
> -# define PUSH_SECTION_IRQENTRY	.pushsection .irqentry.text, "ax"
> -# define POP_SECTION_IRQENTRY	.popsection
> -#else
> -# define PUSH_SECTION_IRQENTRY
> -# define POP_SECTION_IRQENTRY
> -#endif
> +#define PUSH_SECTION_IRQENTRY	.pushsection .irqentry.text, "ax"
> +#define POP_SECTION_IRQENTRY	.popsection
>  
>  .macro apicinterrupt num sym do_sym
>  PUSH_SECTION_IRQENTRY
> diff --git a/include/asm-generic/sections.h b/include/asm-generic/sections.h
> index 532372c6cf15..e5da44eddd2f 100644
> --- a/include/asm-generic/sections.h
> +++ b/include/asm-generic/sections.h
> @@ -27,6 +27,8 @@
>   *	__kprobes_text_start, __kprobes_text_end
>   *	__entry_text_start, __entry_text_end
>   *	__ctors_start, __ctors_end
> + *	__irqentry_text_start, __irqentry_text_end
> + *	__softirqentry_text_start, __softirqentry_text_end
>   */
>  extern char _text[], _stext[], _etext[];
>  extern char _data[], _sdata[], _edata[];
> @@ -39,6 +41,8 @@ extern char __per_cpu_load[], __per_cpu_start[], __per_cpu_end[];
>  extern char __kprobes_text_start[], __kprobes_text_end[];
>  extern char __entry_text_start[], __entry_text_end[];
>  extern char __start_rodata[], __end_rodata[];
> +extern char __irqentry_text_start[], __irqentry_text_end[];
> +extern char __softirqentry_text_start[], __softirqentry_text_end[];
>  
>  /* Start and end of .ctors section - used for constructor calls. */
>  extern char __ctors_start[], __ctors_end[];
> diff --git a/include/asm-generic/vmlinux.lds.h b/include/asm-generic/vmlinux.lds.h
> index fffc9bdae025..438b014c192f 100644
> --- a/include/asm-generic/vmlinux.lds.h
> +++ b/include/asm-generic/vmlinux.lds.h
> @@ -483,25 +483,17 @@
>  		*(.entry.text)						\
>  		VMLINUX_SYMBOL(__entry_text_end) = .;
>  
> -#if defined(CONFIG_FUNCTION_GRAPH_TRACER) || defined(CONFIG_KASAN)
>  #define IRQENTRY_TEXT							\
>  		ALIGN_FUNCTION();					\
>  		VMLINUX_SYMBOL(__irqentry_text_start) = .;		\
>  		*(.irqentry.text)					\
>  		VMLINUX_SYMBOL(__irqentry_text_end) = .;
> -#else
> -#define IRQENTRY_TEXT
> -#endif
>  
> -#if defined(CONFIG_FUNCTION_GRAPH_TRACER) || defined(CONFIG_KASAN)
>  #define SOFTIRQENTRY_TEXT						\
>  		ALIGN_FUNCTION();					\
>  		VMLINUX_SYMBOL(__softirqentry_text_start) = .;		\
>  		*(.softirqentry.text)					\
>  		VMLINUX_SYMBOL(__softirqentry_text_end) = .;
> -#else
> -#define SOFTIRQENTRY_TEXT
> -#endif
>  
>  /* Section used for early init (in .S files) */
>  #define HEAD_TEXT  *(.head.text)
> diff --git a/include/linux/interrupt.h b/include/linux/interrupt.h
> index a2fddddb0d60..59ba11661b6e 100644
> --- a/include/linux/interrupt.h
> +++ b/include/linux/interrupt.h
> @@ -18,6 +18,7 @@
>  #include <linux/atomic.h>
>  #include <asm/ptrace.h>
>  #include <asm/irq.h>
> +#include <asm/sections.h>
>  
>  /*
>   * These correspond to the IORESOURCE_IRQ_* defines in
> @@ -726,7 +727,6 @@ extern int early_irq_init(void);
>  extern int arch_probe_nr_irqs(void);
>  extern int arch_early_irq_init(void);
>  
> -#if defined(CONFIG_FUNCTION_GRAPH_TRACER) || defined(CONFIG_KASAN)
>  /*
>   * We want to know which function is an entrypoint of a hardirq or a softirq.
>   */
> @@ -734,16 +734,4 @@ extern int arch_early_irq_init(void);
>  #define __softirq_entry  \
>  	__attribute__((__section__(".softirqentry.text")))
>  
> -/* Limits of hardirq entrypoints */
> -extern char __irqentry_text_start[];
> -extern char __irqentry_text_end[];
> -/* Limits of softirq entrypoints */
> -extern char __softirqentry_text_start[];
> -extern char __softirqentry_text_end[];
> -
> -#else
> -#define __irq_entry
> -#define __softirq_entry
> -#endif
> -
>  #endif
> 


-- 
Masami Hiramatsu <mhiramat@xxxxxxxxxx>