* [PATCH] Correct NO_EXTENDED_MSGS definition
@ 2005-12-13 13:30 Matthew Wilcox
2005-12-13 13:53 ` Christoph Hellwig
2005-12-15 3:29 ` Douglas Gilbert
0 siblings, 2 replies; 3+ messages in thread
From: Matthew Wilcox @ 2005-12-13 13:30 UTC (permalink / raw)
To: linux-scsi
Use the ARRAY_SIZE macro. Also fixes the definition of NO_EXTENDED_MSGS
(I diagnose excessive cut-and-paste-itis).
Signed-off-by: Matthew Wilcox <matthew@wil.cx>
Index: drivers/scsi/constants.c
===================================================================
RCS file: /var/cvs/linux-2.6/drivers/scsi/constants.c,v
retrieving revision 1.12
retrieving revision 1.13
diff -u -p -r1.12 -r1.13
--- a/drivers/scsi/constants.c 12 Nov 2005 04:08:09 -0000 1.12
+++ b/drivers/scsi/constants.c 13 Dec 2005 00:01:16 -0000 1.13
@@ -114,8 +114,7 @@ static const struct value_name_pair main
{0xd, "Report supported task management functions"},
{0xe, "Report priority"},
};
-#define MAINT_IN_SZ \
- (int)(sizeof(maint_in_arr) / sizeof(maint_in_arr[0]))
+#define MAINT_IN_SZ (int)ARRAY_SIZE(maint_in_arr)
static const struct value_name_pair maint_out_arr[] = {
{0x6, "Set device identifier"},
@@ -123,34 +122,29 @@ static const struct value_name_pair main
{0xb, "Change aliases"},
{0xe, "Set priority"},
};
-#define MAINT_OUT_SZ \
- (int)(sizeof(maint_out_arr) / sizeof(maint_out_arr[0]))
+#define MAINT_OUT_SZ (int)ARRAY_SIZE(maint_out_arr)
static const struct value_name_pair serv_in12_arr[] = {
{0x1, "Read media serial number"},
};
-#define SERV_IN12_SZ \
- (int)(sizeof(serv_in12_arr) / sizeof(serv_in12_arr[0]))
+#define SERV_IN12_SZ (int)ARRAY_SIZE(serv_in12_arr)
static const struct value_name_pair serv_out12_arr[] = {
{-1, "dummy entry"},
};
-#define SERV_OUT12_SZ \
- (int)(sizeof(serv_out12_arr) / sizeof(serv_in12_arr[0]))
+#define SERV_OUT12_SZ (int)ARRAY_SIZE(serv_out12_arr)
static const struct value_name_pair serv_in16_arr[] = {
{0x10, "Read capacity(16)"},
{0x11, "Read long(16)"},
};
-#define SERV_IN16_SZ \
- (int)(sizeof(serv_in16_arr) / sizeof(serv_in16_arr[0]))
+#define SERV_IN16_SZ (int)ARRAY_SIZE(serv_in16_arr)
static const struct value_name_pair serv_out16_arr[] = {
{0x11, "Write long(16)"},
{0x1f, "Notify data transfer device(16)"},
};
-#define SERV_OUT16_SZ \
- (int)(sizeof(serv_out16_arr) / sizeof(serv_in16_arr[0]))
+#define SERV_OUT16_SZ (int)ARRAY_SIZE(serv_out16_arr)
static const struct value_name_pair variable_length_arr[] = {
{0x1, "Rebuild(32)"},
@@ -190,8 +184,7 @@ static const struct value_name_pair vari
{0x8f7e, "Perform SCSI command (osd)"},
{0x8f7f, "Perform task management function (osd)"},
};
-#define VARIABLE_LENGTH_SZ \
- (int)(sizeof(variable_length_arr) / sizeof(variable_length_arr[0]))
+#define VARIABLE_LENGTH_SZ (int)ARRAY_SIZE(variable_length_arr)
static const char * get_sa_name(const struct value_name_pair * arr,
int arr_sz, int service_action)
@@ -1287,19 +1280,19 @@ static const char *one_byte_msgs[] = {
/* 0x0c */ "Bus device reset", "Abort Tag", "Clear Queue",
/* 0x0f */ "Initiate Recovery", "Release Recovery"
};
-#define NO_ONE_BYTE_MSGS (sizeof(one_byte_msgs) / sizeof (const char *))
+#define NO_ONE_BYTE_MSGS ARRAY_SIZE(one_byte_msgs)
static const char *two_byte_msgs[] = {
/* 0x20 */ "Simple Queue Tag", "Head of Queue Tag", "Ordered Queue Tag",
/* 0x23 */ "Ignore Wide Residue"
};
-#define NO_TWO_BYTE_MSGS (sizeof(two_byte_msgs) / sizeof (const char *))
+#define NO_TWO_BYTE_MSGS ARRAY_SIZE(two_byte_msgs)
static const char *extended_msgs[] = {
/* 0x00 */ "Modify Data Pointer", "Synchronous Data Transfer Request",
/* 0x02 */ "SCSI-I Extended Identify", "Wide Data Transfer Request"
};
-#define NO_EXTENDED_MSGS (sizeof(two_byte_msgs) / sizeof (const char *))
+#define NO_EXTENDED_MSGS ARRAY_SIZE(extended_msgs)
int scsi_print_msg (const unsigned char *msg)
@@ -1401,7 +1399,7 @@ static const char * 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"};
-#define NUM_HOSTBYTE_STRS (sizeof(hostbyte_table) / sizeof(const char *))
+#define NUM_HOSTBYTE_STRS ARRAY_SIZE(hostbyte_table)
void scsi_print_hostbyte(int scsiresult)
{
@@ -1425,12 +1423,12 @@ void scsi_print_hostbyte(int scsiresult)
static const char * driverbyte_table[]={
"DRIVER_OK", "DRIVER_BUSY", "DRIVER_SOFT", "DRIVER_MEDIA", "DRIVER_ERROR",
"DRIVER_INVALID", "DRIVER_TIMEOUT", "DRIVER_HARD", "DRIVER_SENSE"};
-#define NUM_DRIVERBYTE_STRS (sizeof(driverbyte_table) / sizeof(const char *))
+#define NUM_DRIVERBYTE_STRS ARRAY_SIZE(driverbyte_table)
static const char * driversuggest_table[]={"SUGGEST_OK",
"SUGGEST_RETRY", "SUGGEST_ABORT", "SUGGEST_REMAP", "SUGGEST_DIE",
"SUGGEST_5", "SUGGEST_6", "SUGGEST_7", "SUGGEST_SENSE"};
-#define NUM_SUGGEST_STRS (sizeof(driversuggest_table) / sizeof(const char *))
+#define NUM_SUGGEST_STRS ARRAY_SIZE(driversuggest_table)
void scsi_print_driverbyte(int scsiresult)
{
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] Correct NO_EXTENDED_MSGS definition
2005-12-13 13:30 [PATCH] Correct NO_EXTENDED_MSGS definition Matthew Wilcox
@ 2005-12-13 13:53 ` Christoph Hellwig
2005-12-15 3:29 ` Douglas Gilbert
1 sibling, 0 replies; 3+ messages in thread
From: Christoph Hellwig @ 2005-12-13 13:53 UTC (permalink / raw)
To: Matthew Wilcox; +Cc: linux-scsi
On Tue, Dec 13, 2005 at 06:30:24AM -0700, Matthew Wilcox wrote:
> +#define MAINT_IN_SZ (int)ARRAY_SIZE(maint_in_arr)
I'd suggest to just opencode the ARRAY_SIZE in the callers like we
do in most places. And please kill the (int) cast..
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] Correct NO_EXTENDED_MSGS definition
2005-12-13 13:30 [PATCH] Correct NO_EXTENDED_MSGS definition Matthew Wilcox
2005-12-13 13:53 ` Christoph Hellwig
@ 2005-12-15 3:29 ` Douglas Gilbert
1 sibling, 0 replies; 3+ messages in thread
From: Douglas Gilbert @ 2005-12-15 3:29 UTC (permalink / raw)
To: Matthew Wilcox; +Cc: linux-scsi
Matthew Wilcox wrote:
> Use the ARRAY_SIZE macro. Also fixes the definition of NO_EXTENDED_MSGS
> (I diagnose excessive cut-and-paste-itis).
Guilty as charged.
While you are at it, perhaps you could look at sg_lib.c
from sg3_utils-1.18 and update the SCSI opcode name and
asc/ascq tables. I maintain those tables in sg_lib.c
close to the latest drafts, but it has been over a year
since since constants.c was updated.
There is one SCSI command not listed in constants.c that
may become very popular soon (and cause the odd problem):
ATA COMMAND PASS THROUGH (16).
Also SCSI opcode names should take into account the
peripheral device type where applicable.
Doug Gilbert
> Signed-off-by: Matthew Wilcox <matthew@wil.cx>
>
> Index: drivers/scsi/constants.c
> ===================================================================
> RCS file: /var/cvs/linux-2.6/drivers/scsi/constants.c,v
> retrieving revision 1.12
> retrieving revision 1.13
> diff -u -p -r1.12 -r1.13
> --- a/drivers/scsi/constants.c 12 Nov 2005 04:08:09 -0000 1.12
> +++ b/drivers/scsi/constants.c 13 Dec 2005 00:01:16 -0000 1.13
> @@ -114,8 +114,7 @@ static const struct value_name_pair main
> {0xd, "Report supported task management functions"},
> {0xe, "Report priority"},
> };
> -#define MAINT_IN_SZ \
> - (int)(sizeof(maint_in_arr) / sizeof(maint_in_arr[0]))
> +#define MAINT_IN_SZ (int)ARRAY_SIZE(maint_in_arr)
>
> static const struct value_name_pair maint_out_arr[] = {
> {0x6, "Set device identifier"},
> @@ -123,34 +122,29 @@ static const struct value_name_pair main
> {0xb, "Change aliases"},
> {0xe, "Set priority"},
> };
> -#define MAINT_OUT_SZ \
> - (int)(sizeof(maint_out_arr) / sizeof(maint_out_arr[0]))
> +#define MAINT_OUT_SZ (int)ARRAY_SIZE(maint_out_arr)
>
> static const struct value_name_pair serv_in12_arr[] = {
> {0x1, "Read media serial number"},
> };
> -#define SERV_IN12_SZ \
> - (int)(sizeof(serv_in12_arr) / sizeof(serv_in12_arr[0]))
> +#define SERV_IN12_SZ (int)ARRAY_SIZE(serv_in12_arr)
>
> static const struct value_name_pair serv_out12_arr[] = {
> {-1, "dummy entry"},
> };
> -#define SERV_OUT12_SZ \
> - (int)(sizeof(serv_out12_arr) / sizeof(serv_in12_arr[0]))
> +#define SERV_OUT12_SZ (int)ARRAY_SIZE(serv_out12_arr)
>
> static const struct value_name_pair serv_in16_arr[] = {
> {0x10, "Read capacity(16)"},
> {0x11, "Read long(16)"},
> };
> -#define SERV_IN16_SZ \
> - (int)(sizeof(serv_in16_arr) / sizeof(serv_in16_arr[0]))
> +#define SERV_IN16_SZ (int)ARRAY_SIZE(serv_in16_arr)
>
> static const struct value_name_pair serv_out16_arr[] = {
> {0x11, "Write long(16)"},
> {0x1f, "Notify data transfer device(16)"},
> };
> -#define SERV_OUT16_SZ \
> - (int)(sizeof(serv_out16_arr) / sizeof(serv_in16_arr[0]))
> +#define SERV_OUT16_SZ (int)ARRAY_SIZE(serv_out16_arr)
>
> static const struct value_name_pair variable_length_arr[] = {
> {0x1, "Rebuild(32)"},
> @@ -190,8 +184,7 @@ static const struct value_name_pair vari
> {0x8f7e, "Perform SCSI command (osd)"},
> {0x8f7f, "Perform task management function (osd)"},
> };
> -#define VARIABLE_LENGTH_SZ \
> - (int)(sizeof(variable_length_arr) / sizeof(variable_length_arr[0]))
> +#define VARIABLE_LENGTH_SZ (int)ARRAY_SIZE(variable_length_arr)
>
> static const char * get_sa_name(const struct value_name_pair * arr,
> int arr_sz, int service_action)
> @@ -1287,19 +1280,19 @@ static const char *one_byte_msgs[] = {
> /* 0x0c */ "Bus device reset", "Abort Tag", "Clear Queue",
> /* 0x0f */ "Initiate Recovery", "Release Recovery"
> };
> -#define NO_ONE_BYTE_MSGS (sizeof(one_byte_msgs) / sizeof (const char *))
> +#define NO_ONE_BYTE_MSGS ARRAY_SIZE(one_byte_msgs)
>
> static const char *two_byte_msgs[] = {
> /* 0x20 */ "Simple Queue Tag", "Head of Queue Tag", "Ordered Queue Tag",
> /* 0x23 */ "Ignore Wide Residue"
> };
> -#define NO_TWO_BYTE_MSGS (sizeof(two_byte_msgs) / sizeof (const char *))
> +#define NO_TWO_BYTE_MSGS ARRAY_SIZE(two_byte_msgs)
>
> static const char *extended_msgs[] = {
> /* 0x00 */ "Modify Data Pointer", "Synchronous Data Transfer Request",
> /* 0x02 */ "SCSI-I Extended Identify", "Wide Data Transfer Request"
> };
> -#define NO_EXTENDED_MSGS (sizeof(two_byte_msgs) / sizeof (const char *))
> +#define NO_EXTENDED_MSGS ARRAY_SIZE(extended_msgs)
>
>
> int scsi_print_msg (const unsigned char *msg)
> @@ -1401,7 +1399,7 @@ static const char * 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"};
> -#define NUM_HOSTBYTE_STRS (sizeof(hostbyte_table) / sizeof(const char *))
> +#define NUM_HOSTBYTE_STRS ARRAY_SIZE(hostbyte_table)
>
> void scsi_print_hostbyte(int scsiresult)
> {
> @@ -1425,12 +1423,12 @@ void scsi_print_hostbyte(int scsiresult)
> static const char * driverbyte_table[]={
> "DRIVER_OK", "DRIVER_BUSY", "DRIVER_SOFT", "DRIVER_MEDIA", "DRIVER_ERROR",
> "DRIVER_INVALID", "DRIVER_TIMEOUT", "DRIVER_HARD", "DRIVER_SENSE"};
> -#define NUM_DRIVERBYTE_STRS (sizeof(driverbyte_table) / sizeof(const char *))
> +#define NUM_DRIVERBYTE_STRS ARRAY_SIZE(driverbyte_table)
>
> static const char * driversuggest_table[]={"SUGGEST_OK",
> "SUGGEST_RETRY", "SUGGEST_ABORT", "SUGGEST_REMAP", "SUGGEST_DIE",
> "SUGGEST_5", "SUGGEST_6", "SUGGEST_7", "SUGGEST_SENSE"};
> -#define NUM_SUGGEST_STRS (sizeof(driversuggest_table) / sizeof(const char *))
> +#define NUM_SUGGEST_STRS ARRAY_SIZE(driversuggest_table)
>
> void scsi_print_driverbyte(int scsiresult)
> {
> -
> To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2005-12-15 3:28 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2005-12-13 13:30 [PATCH] Correct NO_EXTENDED_MSGS definition Matthew Wilcox
2005-12-13 13:53 ` Christoph Hellwig
2005-12-15 3:29 ` Douglas Gilbert
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.