diff mbox

scsi : Adding all the definitions of host bytes in hostbyte_table

Message ID 77471C95FAFD844C8CA02DD4F4C5FE2B0BC9EDE3@SACEXCMBX02-PRD.hq.netapp.com (mailing list archive)
State Deferred, archived
Headers show

Commit Message

babu moger July 27, 2012, 5:28 p.m. UTC
Submitted this patch earlier. But it still did not make it to kernel tree.
Resubmitting again(recreated on top of 3.5 kernel).
 
Adding all the definitions of host bytes in hostbyte_table.

Without this patch, scsi_show_result prints hostbyte as invalid
for statuses that are not defined in hostbyte_table(when scsi logging is enabled).

Look at scsi_print_result function for better understanding.

Signed-off-by: Babu Moger <babu.moger@netapp.com>
---



--
dm-devel mailing list
dm-devel@redhat.com
https://www.redhat.com/mailman/listinfo/dm-devel

Comments

Hannes Reinecke July 30, 2012, 6:15 a.m. UTC | #1
On 07/27/2012 07:28 PM, Moger, Babu wrote:
> Submitted this patch earlier. But it still did not make it to kernel tree.
> Resubmitting again(recreated on top of 3.5 kernel).
>  
> Adding all the definitions of host bytes in hostbyte_table.
> 
> Without this patch, scsi_show_result prints hostbyte as invalid
> for statuses that are not defined in hostbyte_table(when scsi logging is enabled).
> 
> Look at scsi_print_result function for better understanding.
> 
> Signed-off-by: Babu Moger <babu.moger@netapp.com>
Acked-by: Hannes Reinecke <hare@suse.de

Cheers,

Hannes
diff mbox

Patch

--- linux-3.5-rc7/drivers/scsi/constants.c.orig	2012-07-27 13:02:28.000000000 -0500
+++ linux-3.5-rc7/drivers/scsi/constants.c	2012-07-27 13:03:26.000000000 -0500
@@ -1422,7 +1422,8 @@  static const char * const hostbyte_table
 "DID_OK", "DID_NO_CONNECT", "DID_BUS_BUSY", "DID_TIME_OUT", "DID_BAD_TARGET",
 "DID_ABORT", "DID_PARITY", "DID_ERROR", "DID_RESET", "DID_BAD_INTR",
 "DID_PASSTHROUGH", "DID_SOFT_ERROR", "DID_IMM_RETRY", "DID_REQUEUE",
-"DID_TRANSPORT_DISRUPTED", "DID_TRANSPORT_FAILFAST" };
+"DID_TRANSPORT_DISRUPTED", "DID_TRANSPORT_FAILFAST", "DID_TARGET_FAILURE",
+"DID_NEXUS_FAILURE" };
 #define NUM_HOSTBYTE_STRS ARRAY_SIZE(hostbyte_table)
 
 static const char * const driverbyte_table[]={