Re: [PATCH v3 0/1] powerpc: Enable dynamic preemption

[Shrikanth Hegde <sshegde@linux.ibm.com>]

On 1/6/25 10:49, Shrikanth Hegde wrote:
> Now that preempt=lazy patches[1] are in powerpc-next tree, sending out the
> patch to support dynamic preemption based on DYNAMIC_KEY.
> 
> base: powerpc-next
> 

+ankur, sebastian; sorry for not cc'ing earlier.

> Once the arch supports static inline calls, it would be needed to
> evaluate to see if that gives better performance.
> 
> v2->v3:
> - fixed a build error reported by linux test robot by including jump
>    label header.
> 
> v1->v2:
> - Instead of copying asm-generic preempt.h content include it in
>    arch/asm preempt.h. (Christophe Leroy)
> - Merge the patches into one patch (Christophe Leroy)
> 
> v1: https://lore.kernel.org/all/20241125042212.1522315-1-sshegde@linux.ibm.com/
> v2: https://lore.kernel.org/all/20250102191856.499424-1-sshegde@linux.ibm.com/
> [1]: https://lore.kernel.org/all/173572211264.1875638.9927288574435880962.b4-ty@linux.ibm.com/
> 
> Shrikanth Hegde (1):
>    powerpc: Enable dynamic preemption
> 
>   arch/powerpc/Kconfig               |  1 +
>   arch/powerpc/include/asm/preempt.h | 12 ++++++++++++
>   arch/powerpc/kernel/interrupt.c    |  6 +++++-
>   arch/powerpc/kernel/traps.c        |  6 +++++-
>   arch/powerpc/lib/vmx-helper.c      |  2 +-
>   5 files changed, 24 insertions(+), 3 deletions(-)
>   create mode 100644 arch/powerpc/include/asm/preempt.h
> 

Hi maddy, mpe, christophe.

Now that preempt=lazy is merged upstream, is would be start to enable 
dynamic preemption to make use of all preemption models at runtime.

Are there any concerns with this patch?

Thanks.