Re: [PATCH] scsi: mpt3sas: fix hang on ata passthru commands

From: Sreekanth Reddy
Date: Fri Jan 06 2017 - 10:47:03 EST


On Fri, Jan 6, 2017 at 7:29 AM, Martin K. Petersen
<martin.petersen@xxxxxxxxxx> wrote:
>>>>>> "James" == James Bottomley <jejb@xxxxxxxxxxxxxxxxxx> 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@xxxxxxxxxxxx>

>
> --
> Martin K. Petersen Oracle Linux Engineering