pseries/papr-hvpipe: Simplify spin unlock usage in papr_hvpipe_handle_release()

Once the src_info is removed from the global list, no one can access it.
This simplies the usage of spin_unlock_irqrestore() in
papr_hvpipe_handle_release()

Signed-off-by: Ritesh Harjani (IBM) <ritesh.list@gmail.com>
Signed-off-by: Madhavan Srinivasan <maddy@linux.ibm.com>
Link: https://patch.msgid.link/4a980331557af3d10aada8576aaa16cddc691c65.1777606826.git.ritesh.list@gmail.com

diff --git a/arch/powerpc/platforms/pseries/papr-hvpipe.c b/arch/powerpc/platforms/pseries/papr-hvpipe.c
index c007560d2d8ce9b4f5212a24ee39a6e904678ab6..5aa37f6ad8c998808b64199e533d6271fb385620 100644 (file)
--- a/arch/powerpc/platforms/pseries/papr-hvpipe.c
+++ b/arch/powerpc/platforms/pseries/papr-hvpipe.c
@@ -460,6 +460,7 @@ static int papr_hvpipe_handle_release(struct inode *inode,
        src_info = file->private_data;
        list_del(&src_info->list);
        file->private_data = NULL;
+       spin_unlock_irqrestore(&hvpipe_src_list_lock, flags);
        /*
         * If the pipe for this specific source has any pending
         * payload, issue recv HVPIPE RTAS so that pipe will not
@@ -467,10 +468,8 @@ static int papr_hvpipe_handle_release(struct inode *inode,
         */
        if (src_info->hvpipe_status & HVPIPE_MSG_AVAILABLE) {
                src_info->hvpipe_status = 0;
-               spin_unlock_irqrestore(&hvpipe_src_list_lock, flags);
                hvpipe_rtas_recv_msg(NULL, 0);
-       } else
-               spin_unlock_irqrestore(&hvpipe_src_list_lock, flags);
+       }
 
        kfree(src_info);
        return 0;