Re: [GIT PULL] libsas fixes for 3.4-rc4

[Dan Williams]

On Sat, Apr 21, 2012 at 5:28 AM, James Bottomley
<James.Bottomley@xxxxxxxxxxxxxxxxxxxxx> wrote:
> On Fri, 2012-04-20 at 15:29 -0700, Dan Williams wrote:
>> These patches, save for the new "scsi: fix eh wakeup (scsi_schedule_eh
>> vs scsi_restart_operations)" and "Revert "[SCSI] libsas: fix sas port
>> naming", were all originally posted before the merge
>> window opened, and have also appeared in -next for the same timeframe.
>>
>> The commit dates are not that aged (9 days old) because they were
>> rebased out of larger set of updates that were pending for 3.4.
>>
>> There is a mix of pure regression fixes and fixes for long-standing bugs
>> in libsas.  Some of the long-standing bug fixes are made worse / easier
>> to trigger by the new async error handling scheme.
>>
>> The largest patch in the series is "libata, libsas: introduce sched_eh
>> and end_eh port ops" it has been on the list since March 10th.
>>
>> Jack Wang has independently tested this set with pm8001 and reports
>> success. [1]
>>
>> Apologies if scsi-rc-fixes was in the process of picking these up.  With
>> -rc4 looming I lost my nerve and pulled the trigger.
>
> Right, so as a point of process, these are SCSI fixes and are supposed
> to be going through the SCSI tree.  The only urgent one is the revert; I
> still have outstanding questions about some of the others.

Up until this feedback I thought the branch was going to be pulled
intact.  So, not trying to subvert process, just thought this request
would be seen as helpful since you seem to have been rather busy of
late.  Yes, I'd much rather these go through the SCSI tree.

> We're at least 3 weeks away from 3.4 (and that's only if -rc6 is final),
> so there's time to address all of this via the usual process.

My concern is that:

 19 files changed, 344 insertions(+), 172 deletions(-)

...is something that should have went in at -rc2.  Waiting longer
likely means cutting down the list even more and either turning it
into something nobody has given serious test time to, or leaving known
bugs un-addressed for 3.4-final.  Some of these we have lived with for
a while, but are worse with the 3.4-rc1 reworked libsas workqueue and
new async error handling changes.  For grey area "is this a pure
regression fix vs old bug that could maybe wait" I thought the earlier
the rc the better.

--
Dan
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/