Re: [PATCH v3 02/16] powerpc: Override __ALIGN and __ALIGN_STR macros

2022-09-13 Thread Peter Zijlstra
On Tue, Sep 13, 2022 at 12:21:51PM +, Christophe Leroy wrote: > > Like mentioned last time; I'm fixing this (but you're right to not wait > > on that), that said, would it make sense to write it like: > > > > #define __ALIGN .balign 4 > > #define __ALIGN_STR __stringify(__ALIGN) >

Re: [PATCH v3 02/16] powerpc: Override __ALIGN and __ALIGN_STR macros

2022-09-13 Thread Christophe Leroy
Le 13/09/2022 à 14:03, Peter Zijlstra a écrit : > On Mon, Sep 12, 2022 at 01:50:06PM +0530, Sathvika Vasireddy wrote: >> In a subsequent patch, we would want to annotate powerpc assembly functions >> with SYM_FUNC_START_LOCAL macro. This macro depends on __ALIGN macro. >> >> The default expansion

Re: [PATCH v3 02/16] powerpc: Override __ALIGN and __ALIGN_STR macros

2022-09-13 Thread Peter Zijlstra
On Mon, Sep 12, 2022 at 01:50:06PM +0530, Sathvika Vasireddy wrote: > In a subsequent patch, we would want to annotate powerpc assembly functions > with SYM_FUNC_START_LOCAL macro. This macro depends on __ALIGN macro. > > The default expansion of __ALIGN macro is: > #define __ALIGN .a

[PATCH v3 02/16] powerpc: Override __ALIGN and __ALIGN_STR macros

2022-09-12 Thread Sathvika Vasireddy
In a subsequent patch, we would want to annotate powerpc assembly functions with SYM_FUNC_START_LOCAL macro. This macro depends on __ALIGN macro. The default expansion of __ALIGN macro is: #define __ALIGN .align 4,0x90 So, override __ALIGN and __ALIGN_STR macros to use the same align