Re: [PATCH v3 0/9] pseries/papr-hvpipe: Fix deadlock, races and misc cleanups

[Aboorva Devarajan]

On Fri, 2026-05-01 at 09:41 +0530, Ritesh Harjani (IBM) wrote:
> While going over papr-hvpipe code, there were a few fixes which were identified.
> This patch series is an attempt to fix those along with some misc cleanups.
> Me and Haren are trying to get these patches verified on a real HW. The tests
> are not straight forward and we are waiting for the results.
> Will update on the test results once we hear back from the internal test team.
> 
> v2->v3:
> ======
> 1. Rearranged the patches in such a way that it is easier to backport the fixes
>    if required.
> 2. Clubbed patch-8 and patch-10 (of v2) since they both were changing the same function.
> 3. Handled ret>=0 case in copy_to_user patch, when the user itself may request
>    for 0 effective bytes (after the HDR_LEN).


Since this is CCed to stable, it is currently being evaluated by RSCT.
We can merge it once we receive an Acked-by from RSCT.

Thanks,
Aboorva

> 
> [v2]: https://lore.kernel.org/linuxppc-dev/cover.1775648406.git.ritesh.list@xxxxxxxxx/
> 
> v1->v2:
> ========
> 1. Fix a possible deadlock due to use of spin_lock instead of spin_lock_irqsave.
> 2. Prevent kernel stack uninit memory leak to userspace
> 3. Fix the race condition in null-ptr-deref case where there may be an
>    msg pending to be consumed from the hvpipe.
> 4. Fixed error handling in init routine in patch-10
> 
> [v1]: https://lore.kernel.org/linuxppc-dev/cover.1775569027.git.ritesh.list@xxxxxxxxx/#t
> 
> Ritesh Harjani (IBM) (9):
>   pseries/papr-hvpipe: Fix race with interrupt handler
>   pseries/papr-hvpipe: Prevent kernel stack memory leak to userspace
>   pseries/papr-hvpipe: Fix null ptr deref in papr_hvpipe_dev_create_handle()
>   pseries/papr-hvpipe: Fix & simplify error handling in papr_hvpipe_init()
>   pseries/papr-hvpipe: Fix the usage of copy_to_user()
>   pseries/papr-hvpipe: Simplify spin unlock usage in papr_hvpipe_handle_release()
>   pseries/papr-hvpipe: Kill task_struct pointer from struct hvpipe_source_info
>   pseries/papr-hvpipe: Refactor and simplify hvpipe_rtas_recv_msg()
>   pseries/papr-hvpipe: Fix style and checkpatch issues in enable_hvpipe_IRQ()
> 
>  arch/powerpc/platforms/pseries/papr-hvpipe.c | 181 ++++++++++---------
>  arch/powerpc/platforms/pseries/papr-hvpipe.h |   1 -
>  2 files changed, 97 insertions(+), 85 deletions(-)
> 
> --
> 2.39.5