Messages in this thread | | | From | Sreekanth Reddy <> | Date | Fri, 6 Jan 2017 21:16:25 +0530 | Subject | Re: [PATCH] scsi: mpt3sas: fix hang on ata passthru commands |
| |
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
| |