RE: [PATCH 5/8][SCSI] mpt2sas: Infinite loop can occur if MPI2_IOCSTATUS_CONFIG_INVALID_PAGE is not returned

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi James,

Sorry  for the delay. 

we support the first method i.e. "strip the cc: stable from this patch and send it to the misc branch".

Thanks,
Sreekanth.

>-----Original Message-----
>From: James Bottomley [mailto:James.Bottomley@xxxxxxxxxxxxxxxxxxxxx]
>Sent: Monday, August 26, 2013 2:11 PM
>To: Reddy, Sreekanth
>Cc: linux-scsi; Prakash, Sathya; Nandigama, Nagalakshmi
>Subject: Re: [PATCH 5/8][SCSI] mpt2sas: Infinite loop can occur if
>MPI2_IOCSTATUS_CONFIG_INVALID_PAGE is not returned
>
>On Thu, 2013-07-25 at 07:00 -0700, James Bottomley wrote:
>> On Thu, 2013-07-25 at 11:29 +0530, Sreekanth Reddy wrote:
>> > Infinite loop can occur if IOCStatus is not equal to
>> > MPI2_IOCSTATUS_CONFIG_INVALID_PAGE value in the while loops in
>> > functions _scsih_search_responding_sas_devices,
>> > _scsih_search_responding_raid_devices and
>> > _scsih_search_responding_expanders
>> >
>> > So, Instead of checking for MPI2_IOCSTATUS_CONFIG_INVALID_PAGE
>> > value, in this patch code is modified to check for IOCStatus not
>> > equals to MPI2_IOCSTATUS_SUCCESS to break the while loop.
>> >
>> > Signed-off-by: Sreekanth Reddy <Sreekanth.Reddy@xxxxxxx>
>> > Cc: stable@xxxxxxxxxxxxxxx
>>
>> Patches like this aren't acceptable any more: Greg will find this and
>> reject it from Stable.
>>
>> If there's a bug in the driver, it needs to be in a separate patch set
>> for the fixes branch of SCSI to go upstream now rather than waiting
>> for the merge window.  Only patches for the fixes branch can be tagged
>> for stable.
>
>Ping on this, please.  What do you want me to do?  I can strip the cc:
>stable from the series and send it to the misc branch or you can resubmit as
>two series: one for fixes with the stable tags and the rest for misc.
>
>James
>
>


--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [SCSI Target Devel]     [Linux SCSI Target Infrastructure]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Linux IIO]     [Samba]     [Device Mapper]
  Powered by Linux