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

Shrikanth Hegde posted 1 patch 1 year, 1 month ago
There is a newer version of this series
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
[PATCH v3 0/1] powerpc: Enable dynamic preemption
Posted by Shrikanth Hegde 1 year, 1 month ago
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 

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

-- 
2.39.3
Re: [PATCH v3 0/1] powerpc: Enable dynamic preemption
Posted by Shrikanth Hegde 1 year ago

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.