Re: [PATCH] powerpc/ftrace: Ignore ftrace locations in exit text sections

On Mon, Feb 12, 2024 at 07:31:03PM +0000, Christophe Leroy wrote:

Le 09/02/2024 à 08:59, Naveen N Rao a écrit :

quoted

diff --git a/arch/powerpc/include/asm/sections.h 

b/arch/powerpc/include/asm/sections.h
index ea26665f82cf..d389dcecdb0b 100644

--- a/arch/powerpc/include/asm/sections.h
+++ b/arch/powerpc/include/asm/sections.h

@@ -14,6 +14,7 @@ typedef struct func_desc func_desc_t;
  
  extern char __head_end[];
  extern char __srwx_boundary[];
+extern char _sexittext[], _eexittext[];

Should we try to at least use the same symbols as others, or best try to 
move this into include/asm-generic/sections.h, just like inittext ?

I used this name based on what is used for init text start and end in 
the generic code: _sinittext and _einittext.

$ git grep exittext
arch/arm64/include/asm/sections.h:extern char __exittext_begin[], 
__exittext_end[];

Arm64 also uses the non-standard __inittext_begin/__inittext_end, so it 
looks to be something very specific to arm64.

I do agree it would be good to refactor and unify names across 
architectures.


- Naveen