Re: [PATCH 1/1] cciss: fix error reporting for SG_IO

2007-08-27 Thread Mike Miller (OS Dev)
On Fri, Aug 24, 2007 at 05:10:47PM -0700, Andrew Morton wrote: > On Fri, 24 Aug 2007 12:53:54 -0500 > "Mike Miller (OS Dev)" <[EMAIL PROTECTED]> wrote: > > > This fixes a problem with the way cciss was filling out the "errors" > > field of the request structure upon completion of requests. > >

Re: [PATCH 1/1] cciss: fix error reporting for SG_IO

2007-08-27 Thread Mike Miller (OS Dev)
On Fri, Aug 24, 2007 at 05:10:47PM -0700, Andrew Morton wrote: On Fri, 24 Aug 2007 12:53:54 -0500 Mike Miller (OS Dev) [EMAIL PROTECTED] wrote: This fixes a problem with the way cciss was filling out the errors field of the request structure upon completion of requests. Previously, it

Re: [PATCH 1/1] cciss: fix error reporting for SG_IO

2007-08-24 Thread Andrew Morton
On Fri, 24 Aug 2007 12:53:54 -0500 "Mike Miller (OS Dev)" <[EMAIL PROTECTED]> wrote: > This fixes a problem with the way cciss was filling out the "errors" > field of the request structure upon completion of requests. > Previously, it just put a 1 or a 0 in there and used the negation > of this

[PATCH 1/1] cciss: fix error reporting for SG_IO

2007-08-24 Thread Mike Miller (OS Dev)
- Forwarded message from Steve Cameron <[EMAIL PROTECTED]> - Date: Fri, 24 Aug 2007 11:10:44 -0500 From: Steve Cameron <[EMAIL PROTECTED]> To: [EMAIL PROTECTED] Reply-To: [EMAIL PROTECTED] Cc: [EMAIL PROTECTED] Subject: Re: [PATCH 1/1] cciss: fix error reporting for SG_IO

Re: [PATCH 1/1] cciss: fix error reporting for SG_IO

2007-08-24 Thread Jens Axboe
On Tue, Aug 14 2007, Mike Miller (OS Dev) wrote: > Patch 1/1 > Steve has been trying to send this out but it doesn't seem to be getting > anywhere. Please review this patch for accuracy. There's a couple of things > not clear to us. > > Thanks, > mikem > >

Re: [PATCH 1/1] cciss: fix error reporting for SG_IO

2007-08-24 Thread Jens Axboe
On Tue, Aug 14 2007, Mike Miller (OS Dev) wrote: Patch 1/1 Steve has been trying to send this out but it doesn't seem to be getting anywhere. Please review this patch for accuracy. There's a couple of things not clear to us. Thanks, mikem

[PATCH 1/1] cciss: fix error reporting for SG_IO

2007-08-24 Thread Mike Miller (OS Dev)
- Forwarded message from Steve Cameron [EMAIL PROTECTED] - Date: Fri, 24 Aug 2007 11:10:44 -0500 From: Steve Cameron [EMAIL PROTECTED] To: [EMAIL PROTECTED] Reply-To: [EMAIL PROTECTED] Cc: [EMAIL PROTECTED] Subject: Re: [PATCH 1/1] cciss: fix error reporting for SG_IO This fixes

Re: [PATCH 1/1] cciss: fix error reporting for SG_IO

2007-08-24 Thread Andrew Morton
On Fri, 24 Aug 2007 12:53:54 -0500 Mike Miller (OS Dev) [EMAIL PROTECTED] wrote: This fixes a problem with the way cciss was filling out the errors field of the request structure upon completion of requests. Previously, it just put a 1 or a 0 in there and used the negation of this as the

RE: [PATCH 1/1] cciss: fix error reporting for SG_IO

2007-08-16 Thread Cameron, Steve
Any feedback on my patch? Anybody know what the msg_byte in include/scsi/scsi.h is for? scsi.h says: * msg_byte= return status from host adapter itself. So, it's ok to use the msg_byte for device specific error codes? -- steve - To unsubscribe from this list: send the line

RE: [PATCH 1/1] cciss: fix error reporting for SG_IO

2007-08-16 Thread Cameron, Steve
Any feedback on my patch? Anybody know what the msg_byte in include/scsi/scsi.h is for? scsi.h says: * msg_byte= return status from host adapter itself. So, it's ok to use the msg_byte for device specific error codes? -- steve - To unsubscribe from this list: send the line

[PATCH 1/1] cciss: fix error reporting for SG_IO

2007-08-14 Thread Mike Miller (OS Dev)
Patch 1/1 Steve has been trying to send this out but it doesn't seem to be getting anywhere. Please review this patch for accuracy. There's a couple of things not clear to us. Thanks, mikem We found a problem with

[PATCH 1/1] cciss: fix error reporting for SG_IO

2007-08-14 Thread Mike Miller (OS Dev)
Patch 1/1 Steve has been trying to send this out but it doesn't seem to be getting anywhere. Please review this patch for accuracy. There's a couple of things not clear to us. Thanks, mikem We found a problem with