Commit 7b3871fd authored by Brian King's avatar Brian King Committed by Martin K. Petersen
Browse files

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: default avatarBrian King <brking@linux.vnet.ibm.com>
Reviewed-by: default avatarGabriel Krisman Bertazi <krisman@linux.vnet.ibm.com>
Signed-off-by: default avatarMartin K. Petersen <martin.petersen@oracle.com>
parent e018ef57

drivers/scsi/ipr.c

+3 −3
Original line number Diff line number Diff line
@@ -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"},

drivers/scsi/ipr.h

+1 −0
Original line number Diff line number Diff line
@@ -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)

CRA Git | Maintained and supported by SUSTech CRA and CCSE