scsi: ipr: Don't log unnecessary 9084 error details

A 9084 error gets logged by the ipr adapter when adapter raw mode gets
enabled. A bunch of unformatted hex data also gets logged for this
error, which is of little use, so let's avoid logging it by default in
order to avoid the log getting polluted with useless data.

Signed-off-by: Brian King <brking@linux.vnet.ibm.com>
Reviewed-by: Gabriel Krisman Bertazi <krisman@linux.vnet.ibm.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>

diff --git a/drivers/scsi/ipr.c b/drivers/scsi/ipr.c
index 5ecc32c..762627c 100644 (file)
--- a/drivers/scsi/ipr.c
+++ b/drivers/scsi/ipr.c
@@ -493,15 +493,15 @@ struct ipr_error_table_t ipr_error_table[] = {
        "9072: Link not operational transition"},
        {0x066B8200, 0, IPR_DEFAULT_LOG_LEVEL,
        "9032: Array exposed but still protected"},
-       {0x066B8300, 0, IPR_DEFAULT_LOG_LEVEL + 1,
+       {0x066B8300, 0, IPR_DEBUG_LOG_LEVEL,
        "70DD: Device forced failed by disrupt device command"},
        {0x066B9100, 0, IPR_DEFAULT_LOG_LEVEL,
        "4061: Multipath redundancy level got better"},
        {0x066B9200, 0, IPR_DEFAULT_LOG_LEVEL,
        "4060: Multipath redundancy level got worse"},
-       {0x06808100, 0, IPR_DEFAULT_LOG_LEVEL,
+       {0x06808100, 0, IPR_DEBUG_LOG_LEVEL,
        "9083: Device raw mode enabled"},
-       {0x06808200, 0, IPR_DEFAULT_LOG_LEVEL,
+       {0x06808200, 0, IPR_DEBUG_LOG_LEVEL,
        "9084: Device raw mode disabled"},
        {0x07270000, 0, 0,
        "Failure due to other device"},

diff --git a/drivers/scsi/ipr.h b/drivers/scsi/ipr.h
index 4dbeaaf..8995053 100644 (file)
--- a/drivers/scsi/ipr.h
+++ b/drivers/scsi/ipr.h
@@ -1506,6 +1506,7 @@ struct ipr_ioa_cfg {
        u8 log_level;
 #define IPR_MAX_LOG_LEVEL                      4
 #define IPR_DEFAULT_LOG_LEVEL          2
+#define IPR_DEBUG_LOG_LEVEL            3
 
 #define IPR_NUM_TRACE_INDEX_BITS       8
 #define IPR_NUM_TRACE_ENTRIES          (1 << IPR_NUM_TRACE_INDEX_BITS)