Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1552471 > unrolled thread
| Started by | "Martin K. Petersen" <martin.petersen@oracle.com> |
|---|---|
| First post | 2017-01-06 03:10 +0100 |
| Last post | 2017-01-10 06:00 +0100 |
| Articles | 3 — 2 participants |
Back to article view | Back to linux.kernel
This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by
below is the oldest one visible, not the original post.
Re: [PATCH] scsi: mpt3sas: fix hang on ata passthru commands "Martin K. Petersen" <martin.petersen@oracle.com> - 2017-01-06 03:10 +0100
Re: [PATCH] scsi: mpt3sas: fix hang on ata passthru commands Sreekanth Reddy <sreekanth.reddy@broadcom.com> - 2017-01-06 16:50 +0100
Re: [PATCH] scsi: mpt3sas: fix hang on ata passthru commands "Martin K. Petersen" <martin.petersen@oracle.com> - 2017-01-10 06:00 +0100
| From | "Martin K. Petersen" <martin.petersen@oracle.com> |
|---|---|
| Date | 2017-01-06 03:10 +0100 |
| Subject | Re: [PATCH] scsi: mpt3sas: fix hang on ata passthru commands |
| Message-ID | <sWs8G-3cr-11@gated-at.bofh.it> |
>>>>> "James" == James Bottomley <jejb@linux.vnet.ibm.com> writes: James> Now that I look at the reviews, each of the reviewers said what James> the correct thing to do was: return SAM_STAT_BUSY if SATL James> commands are outstanding like the spec says. You all get James> negative brownie points for not insisting on a rework. James> Does this patch (compile tested only) fix the problems for James> everyone? I also like this approach better. Broadcom folks: Please comment and test. -- Martin K. Petersen Oracle Linux Engineering
[toc] | [next] | [standalone]
| From | Sreekanth Reddy <sreekanth.reddy@broadcom.com> |
|---|---|
| Date | 2017-01-06 16:50 +0100 |
| Message-ID | <sWEWe-3VU-41@gated-at.bofh.it> |
| In reply to | #1552471 |
On Fri, Jan 6, 2017 at 7:29 AM, Martin K. Petersen <martin.petersen@oracle.com> wrote: >>>>>> "James" == James Bottomley <jejb@linux.vnet.ibm.com> writes: > > James> Now that I look at the reviews, each of the reviewers said what > James> the correct thing to do was: return SAM_STAT_BUSY if SATL > James> commands are outstanding like the spec says. You all get > James> negative brownie points for not insisting on a rework. > > James> Does this patch (compile tested only) fix the problems for > James> everyone? > > I also like this approach better. > > Broadcom folks: Please comment and test. Matin, We are fine with this patch. Can we rename function 'set_satl_pending()' name to '_scsih_set_satl_pending()' and can add headers to this function. other wise I am OK. Acked-by: Sreekanth Reddy <Sreekanth.Reddy@broadcom.com> > > -- > Martin K. Petersen Oracle Linux Engineering
[toc] | [prev] | [next] | [standalone]
| From | "Martin K. Petersen" <martin.petersen@oracle.com> |
|---|---|
| Date | 2017-01-10 06:00 +0100 |
| Message-ID | <sXWHo-5cU-7@gated-at.bofh.it> |
| In reply to | #1552894 |
>>>>> "Sreekanth" == Sreekanth Reddy <sreekanth.reddy@broadcom.com> writes: Sreekanth> We are fine with this patch. Can we rename function Sreekanth> 'set_satl_pending()' name to '_scsih_set_satl_pending()' and Sreekanth> can add headers to this function. Sreekanth> other wise I am OK. Sreekanth> Acked-by: Sreekanth Reddy <Sreekanth.Reddy@broadcom.com> James: Please tweak and post an updated patch. -- Martin K. Petersen Oracle Linux Engineering
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web