Re: [PATCH v3 0/6] powerpc: use lock guards for mutex Set 1

[Venkat Rao Bagalkote <venkat88@linux.ibm.com>]

On 05/05/25 1:23 pm, Shrikanth Hegde wrote:
> This is an effort to make the code simpler by making use of lock
> guards which were introduced in [1], which works by using __cleanup
> attributes. More details in v1 cover letter
>
> v2->v3:
> - Collects tags (Andrew Donnellan & Srikar Dronamraju)
> - Address comments from Srikar Dronamraju
>
> v1->v2:
> - Fix changelog of powernv (Andrew Donnellan)
> - use scoped_guard in couple of places to avoid holding mutex
>    un-necessarily (Peter Zijlstra)
>
> [1]: https://lkml.kernel.org/r/20230612093537.614161713%40infradead.org
> v1: https://lore.kernel.org/all/20250314054544.1998928-1-sshegde@linux.ibm.com/#t
> v2: https://lore.kernel.org/all/20250314114502.2083434-1-sshegde@linux.ibm.com/
>
> Shrikanth Hegde (6):
>    powerpc: eeh: use lock guard for mutex
>    powerpc: rtas: use lock guard for mutex
>    powerpc: fadump: use lock guard for mutex
>    powerpc: book3s: vas: use lock guard for mutex
>    powerpc: powernv: ocxl: use lock guard for mutex
>    powerpc: sysdev: use lock guard for mutex
>
>   arch/powerpc/kernel/eeh.c                   | 20 +++----
>   arch/powerpc/kernel/fadump.c                | 11 ++--
>   arch/powerpc/kernel/rtas_flash.c            | 64 +++++++--------------
>   arch/powerpc/platforms/book3s/vas-api.c     | 32 +++++------
>   arch/powerpc/platforms/powernv/ocxl.c       | 12 +---
>   arch/powerpc/sysdev/fsl_mpic_timer_wakeup.c | 14 +----
>   6 files changed, 50 insertions(+), 103 deletions(-)
>
Tested this patch-set by applying on top of powerpc repo, with HEAD: 
b4432656b36e5cc1d50a1f2dc15357543add530e, and Build and boot test is 
successful.

Also FADUMP and DLPAR operations are successful. Hence, for the series,


Tested-by: Venkat Rao Bagalkote <venkat88@linux.ibm.com>


Regards,

Venkat.