On 09/08/2016 09:44 AM, Gabriel Krisman Bertazi wrote:
> Brian King <brk...@linux.vnet.ibm.com> writes:
> 
>> 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 <brk...@linux.vnet.ibm.com>
>> ---
>>
>>  drivers/scsi/ipr.c |    4 ++--
>>  1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff -puN drivers/scsi/ipr.c~ipr_9084 drivers/scsi/ipr.c
>> --- linux-2.6.git/drivers/scsi/ipr.c~ipr_9084        2016-08-26 
>> 06:09:39.047244907 -0500
>> +++ linux-2.6.git-bjking1/drivers/scsi/ipr.c 2016-08-30 20:50:20.495406875 
>> -0500
>> @@ -499,9 +499,9 @@ struct ipr_error_table_t ipr_error_table
>>      "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_DEFAULT_LOG_LEVEL+1,
>>      "9083: Device raw mode enabled"},
>> -    {0x06808200, 0, IPR_DEFAULT_LOG_LEVEL,
>> +    {0x06808200, 0, IPR_DEFAULT_LOG_LEVEL+1,
> 
> Can we have a new IPR_LOG_LEVEL_DEBUG here instead?
> 
> #define IPR_LOG_LEVEL_DEBUG 3

Good idea. Done.

Thanks,

Brian

-- 
Brian King
Power Linux I/O
IBM Linux Technology Center


------------------------------------------------------------------------------
_______________________________________________
Iprdd-devel mailing list
Iprdd-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/iprdd-devel

Reply via email to