scsi: mpt3sas: Suppress unnecessary IOCLogInfo on CONFIG_INVALID_PAGE

Avoid unconditional IOCLogInfo prints for CONFIG_INVALID_PAGE.  Log only
if MPT_DEBUG_REPLY is enabled or when loginfo represents other
errors. This reduces uncessary logging without losing useful error
reporting.

Signed-off-by: Ranjan Kumar <ranjan.kumar@broadcom.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>

diff --git a/drivers/scsi/mpt3sas/mpt3sas_base.c b/drivers/scsi/mpt3sas/mpt3sas_base.c
index bd3efa5b46c780d43fae58c12f0bce5057dcda85..0d652db8fe249e5e455fbbba3a41686b23bce9df 100644 (file)
--- a/drivers/scsi/mpt3sas/mpt3sas_base.c
+++ b/drivers/scsi/mpt3sas/mpt3sas_base.c
@@ -1420,7 +1420,13 @@ _base_display_reply_info(struct MPT3SAS_ADAPTER *ioc, u16 smid, u8 msix_index,
 
        if (ioc_status & MPI2_IOCSTATUS_FLAG_LOG_INFO_AVAILABLE) {
                loginfo = le32_to_cpu(mpi_reply->IOCLogInfo);
-               _base_sas_log_info(ioc, loginfo);
+               if (ioc->logging_level & MPT_DEBUG_REPLY)
+                       _base_sas_log_info(ioc, loginfo);
+               else {
+                       if (!((ioc_status & MPI2_IOCSTATUS_MASK) &
+                       MPI2_IOCSTATUS_CONFIG_INVALID_PAGE))
+                               _base_sas_log_info(ioc, loginfo);
+               }
        }
 
        if (ioc_status || loginfo) {