On Mon, Feb 12, 2024 at 07:31:03PM +0000, Christophe Leroy wrote: > > > Le 09/02/2024 à 08:59, Naveen N Rao a écrit : > > 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
