scsi: mpt3sas: Remove scsi_dma_map() error messages

When scsi_dma_map() fails by returning a sges_left value less than zero,
the amount of logging produced can be extremely high.  In a recent end-user
environment, 1200 messages per second were being sent to the log buffer.
This eventually overwhelmed the system and it stalled.

These error messages are not needed. Remove them.

BugLink: https://bugs.launchpad.net/bugs/1965927
Link: https://lore.kernel.org/r/20220303140203.12642-1-sreekanth.reddy@broadcom.com
Suggested-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Sreekanth Reddy <sreekanth.reddy@broadcom.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
(cherry picked from commit 0c25422d34b4726b2707d5f38560943155a91b80)
Signed-off-by: Michael Reed <Michael.Reed@canonical.com>
Acked-by: Tim Gardner <tim.gardner@canonical.com>
Acked-by: Stefan Bader <stefan.bader@canonical.com>
Signed-off-by: Stefan Bader <stefan.bader@canonical.com>

diff --git a/drivers/scsi/mpt3sas/mpt3sas_base.c b/drivers/scsi/mpt3sas/mpt3sas_base.c
index fafa9fbf3b107c5bab925e7bede0a22134b89460..916e7874a746616953528fca113b009546acbef1 100644 (file)
--- a/drivers/scsi/mpt3sas/mpt3sas_base.c
+++ b/drivers/scsi/mpt3sas/mpt3sas_base.c
@@ -2594,12 +2594,8 @@ _base_check_pcie_native_sgl(struct MPT3SAS_ADAPTER *ioc,
 
        /* Get the SG list pointer and info. */
        sges_left = scsi_dma_map(scmd);
-       if (sges_left < 0) {
-               sdev_printk(KERN_ERR, scmd->device,
-                       "scsi_dma_map failed: request for %d bytes!\n",
-                       scsi_bufflen(scmd));
+       if (sges_left < 0)
                return 1;
-       }
 
        /* Check if we need to build a native SG list. */
        if (!base_is_prp_possible(ioc, pcie_device,
@@ -2706,12 +2702,8 @@ _base_build_sg_scmd(struct MPT3SAS_ADAPTER *ioc,
 
        sg_scmd = scsi_sglist(scmd);
        sges_left = scsi_dma_map(scmd);
-       if (sges_left < 0) {
-               sdev_printk(KERN_ERR, scmd->device,
-                "scsi_dma_map failed: request for %d bytes!\n",
-                scsi_bufflen(scmd));
+       if (sges_left < 0)
                return -ENOMEM;
-       }
 
        sg_local = &mpi_request->SGL;
        sges_in_segment = ioc->max_sges_in_main_message;
@@ -2854,12 +2846,8 @@ _base_build_sg_scmd_ieee(struct MPT3SAS_ADAPTER *ioc,
 
        sg_scmd = scsi_sglist(scmd);
        sges_left = scsi_dma_map(scmd);
-       if (sges_left < 0) {
-               sdev_printk(KERN_ERR, scmd->device,
-                       "scsi_dma_map failed: request for %d bytes!\n",
-                       scsi_bufflen(scmd));
+       if (sges_left < 0)
                return -ENOMEM;
-       }
 
        sg_local = &mpi_request->SGL;
        sges_in_segment = (ioc->request_sz -