mirror of
https://github.com/hardkernel/linux.git
synced 2026-06-06 10:58:48 +09:00
scsi: mpt3sas: Remove scsi_dma_map() error messages
commit 0c25422d34 upstream.
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.
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>
Cc: Paul Menzel <pmenzel@molgen.mpg.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
committed by
Greg Kroah-Hartman
parent
e2ea555642
commit
63c2fa09b8
@@ -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 -
|
||||
|
||||
Reference in New Issue
Block a user