Daniel Wagner ca8a91
From: Quinn Tran <qutran@marvell.com>
Daniel Wagner ca8a91
Date: Tue, 17 Dec 2019 14:06:16 -0800
Daniel Wagner ca8a91
Subject: scsi: qla2xxx: Fix mtcp dump collection failure
Michal Kubecek 35cf49
Patch-mainline: v5.6-rc1
Daniel Wagner ca8a91
Git-commit: 641e0efddcbde52461e017136acd3ce7f2ef0c14
Daniel Wagner ca8a91
References: jsc#SLE-9714 jsc#SLE-10327 jsc#SLE-10334
Daniel Wagner ca8a91
Daniel Wagner ca8a91
MTCP dump failed due to MB Reg 10 was picking garbage data from stack
Daniel Wagner ca8a91
memory.
Daniel Wagner ca8a91
Daniel Wagner ca8a91
Fixes: 81178772b636a ("[SCSI] qla2xxx: Implemetation of mctp.")
Daniel Wagner ca8a91
Cc: stable@vger.kernel.org
Daniel Wagner ca8a91
Link: https://lore.kernel.org/r/20191217220617.28084-14-hmadhani@marvell.com
Daniel Wagner ca8a91
Signed-off-by: Quinn Tran <qutran@marvell.com>
Daniel Wagner ca8a91
Signed-off-by: Himanshu Madhani <hmadhani@marvell.com>
Daniel Wagner ca8a91
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
Daniel Wagner ca8a91
Acked-by: Daniel Wagner <dwagner@suse.de>
Daniel Wagner ca8a91
---
Daniel Wagner ca8a91
 drivers/scsi/qla2xxx/qla_mbx.c |    3 +--
Daniel Wagner ca8a91
 1 file changed, 1 insertion(+), 2 deletions(-)
Daniel Wagner ca8a91
Daniel Wagner ca8a91
--- a/drivers/scsi/qla2xxx/qla_mbx.c
Daniel Wagner ca8a91
+++ b/drivers/scsi/qla2xxx/qla_mbx.c
Daniel Wagner ca8a91
@@ -6152,9 +6152,8 @@ qla2x00_dump_mctp_data(scsi_qla_host_t *
Daniel Wagner ca8a91
 	mcp->mb[7] = LSW(MSD(req_dma));
Daniel Wagner ca8a91
 	mcp->mb[8] = MSW(addr);
Daniel Wagner ca8a91
 	/* Setting RAM ID to valid */
Daniel Wagner ca8a91
-	mcp->mb[10] |= BIT_7;
Daniel Wagner ca8a91
 	/* For MCTP RAM ID is 0x40 */
Daniel Wagner ca8a91
-	mcp->mb[10] |= 0x40;
Daniel Wagner ca8a91
+	mcp->mb[10] = BIT_7 | 0x40;
Daniel Wagner ca8a91
 
Daniel Wagner ca8a91
 	mcp->out_mb |= MBX_10|MBX_8|MBX_7|MBX_6|MBX_5|MBX_4|MBX_3|MBX_2|MBX_1|
Daniel Wagner ca8a91
 	    MBX_0;