ntb: intel: Fix using link status DB's

maquefel · jonmason · commit 8144e9c8f30f · 2025-03-18T10:22:48.000-04:00
Make sure we are not using DB's which were remapped for link status. Fixes: f6e51c3 ("ntb: intel: split out the gen3 code") Signed-off-by: Nikita Shubin <n.shubin@yadro.com> Reviewed-by: Dave Jiang <dave.jiang@intel.com> Signed-off-by: Jon Mason <jdmason@kudzu.us>
diff --git a/drivers/ntb/hw/intel/ntb_hw_gen3.c b/drivers/ntb/hw/intel/ntb_hw_gen3.c
@@ -215,6 +215,9 @@ static int gen3_init_ntb(struct intel_ntb_dev *ndev)
 	}
 
 	ndev->db_valid_mask = BIT_ULL(ndev->db_count) - 1;
+	/* Make sure we are not using DB's used for link status */
+	if (ndev->hwerr_flags & NTB_HWERR_MSIX_VECTOR32_BAD)
+		ndev->db_valid_mask &= ~ndev->db_link_mask;
 
 	ndev->reg->db_iowrite(ndev->db_valid_mask,
 			      ndev->self_mmio +

Original file line number	Diff line number	Diff line change
`@@ -215,6 +215,9 @@ static int gen3_init_ntb(struct intel_ntb_dev *ndev)`
`215`	`215`	`}`
`216`	`216`
`217`	`217`	`ndev->db_valid_mask = BIT_ULL(ndev->db_count) - 1;`
	`218`	`+ /* Make sure we are not using DB's used for link status */`
	`219`	`+ if (ndev->hwerr_flags & NTB_HWERR_MSIX_VECTOR32_BAD)`
	`220`	`+ ndev->db_valid_mask &= ~ndev->db_link_mask;`
`218`	`221`
`219`	`222`	`ndev->reg->db_iowrite(ndev->db_valid_mask,`
`220`	`223`	`ndev->self_mmio +`