[PATCH] SCSI: Update ASC list

2007-02-28 Thread Martin K. Petersen 

Add missing additional sense code and provide pointer to upstream
reference (from Doug Gilbert).

Signed-off-by: Martin K. Petersen <[EMAIL PROTECTED]>

--- 

 constants.c |5 +
 1 files changed, 5 insertions(+)


Index: scsi-misc-2.6/drivers/scsi/constants.c
===
--- scsi-misc-2.6.orig/drivers/scsi/constants.c
+++ scsi-misc-2.6/drivers/scsi/constants.c
@@ -404,6 +404,10 @@ struct error_info {
const char * text;
 };
 
+/*
+ * The canonical list of T10 Additional Sense Codes is available at:
+ * http://www.t10.org/lists/asc-num.txt
+ */
 static struct error_info additional[] =
 {
{0x, "No additional sense information"},
@@ -708,6 +712,7 @@ static struct error_info additional[] =
 
{0x2F00, "Commands cleared by another initiator"},
{0x2F01, "Commands cleared by power loss notification"},
+   {0x2F02, "Commands cleared by device server"},
 
{0x3000, "Incompatible medium installed"},
{0x3001, "Cannot read medium - unknown format"},
-
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH] SCSI: Update ASC list

2007-02-28 Thread Martin K. Petersen 
> "Michael" == Michael Tokarev <[EMAIL PROTECTED]> writes:

Michael> static CONST struct error_info additional[]... ?

Good spotting - updated patch coming...

-- 
Martin K. Petersen  Oracle Linux Engineering
-
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH] SCSI: Update ASC list

2007-02-28 Thread Michael Tokarev 
Martin K. Petersen wrote:
[]
> --- scsi-misc-2.6.orig/drivers/scsi/constants.c
> +++ scsi-misc-2.6/drivers/scsi/constants.c
[]
>  static struct error_info additional[] =

static CONST struct error_info additional[]... ?
>  {
>   {0x, "No additional sense information"},
[]

/mjt
-
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[PATCH] SCSI: Update ASC list and make it const

2007-02-28 Thread Martin K. Petersen 

Add missing additional sense code and provide pointer to upstream
reference (from Doug Gilbert).

Add missing const (from Michael Tokarev).

Signed-off-by: Martin K. Petersen <[EMAIL PROTECTED]>

--- 

 constants.c |7 ++-
 1 files changed, 6 insertions(+), 1 deletion(-)


Index: scsi-misc-2.6/drivers/scsi/constants.c
===
--- scsi-misc-2.6.orig/drivers/scsi/constants.c
+++ scsi-misc-2.6/drivers/scsi/constants.c
@@ -404,7 +404,11 @@ struct error_info {
const char * text;
 };
 
-static struct error_info additional[] =
+/*
+ * The canonical list of T10 Additional Sense Codes is available at:
+ * http://www.t10.org/lists/asc-num.txt
+ */
+static const struct error_info additional[] =
 {
{0x, "No additional sense information"},
{0x0001, "Filemark detected"},
@@ -708,6 +712,7 @@ static struct error_info additional[] =
 
{0x2F00, "Commands cleared by another initiator"},
{0x2F01, "Commands cleared by power loss notification"},
+   {0x2F02, "Commands cleared by device server"},
 
{0x3000, "Incompatible medium installed"},
{0x3001, "Cannot read medium - unknown format"},
-
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH] SCSI: Update ASC list and make it const

2007-02-28 Thread Daniel J. Priem 
"Martin K. Petersen" <[EMAIL PROTECTED]> writes:
> + * The canonical list of T10 Additional Sense Codes is available at:
> + * http://www.t10.org/lists/asc-num.txt+ */

Please do a

- * http://www.t10.org/lists/asc-num.txt+ */
+ * http://www.t10.org/lists/asc-num.txt */

its a typo.
Regards
Daniel

-- 
 .''`.
 : :' :  Daniel J. Priem
 `. `'   http://flexserv.de
   `-

-
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH] SCSI: Update ASC list and make it const

2007-02-28 Thread Martin K. Petersen 
> "Daniel" == Daniel J Priem <[EMAIL PROTECTED]> writes:

Daniel> - * http://www.t10.org/lists/asc-num.txt+ */ 
Daniel> + * http://www.t10.org/lists/asc-num.txt */

Daniel> its a typo.  

The diff looks fine both here and in the archives.  Maybe your mailer
mangled it?

-- 
Martin K. Petersen  Oracle Linux Engineering

-
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH] SCSI: Update ASC list and make it const

2007-02-28 Thread Daniel J. Priem 
"Martin K. Petersen" <[EMAIL PROTECTED]> writes:

>> "Daniel" == Daniel J Priem <[EMAIL PROTECTED]> writes:
>
> Daniel> - * http://www.t10.org/lists/asc-num.txt+ */ 
> Daniel> + * http://www.t10.org/lists/asc-num.txt */
>
> Daniel> its a typo.  
>
> The diff looks fine both here and in the archives.  Maybe your mailer
> mangled it?
Yes. you are right. i crosschecked against
http://permalink.gmane.org/gmane.linux.scsi/30188 ...

-- 
 .''`.
 : :' :  Daniel J. Priem
 `. `'   http://flexserv.de
   `-

-
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html