On Tue, Jul 23, 2013 at 04:12:21PM +0530, Reddy, Sreekanth wrote: > Hi Greg, > > Here is the summary of this patch. > > After host reset, the driver breaks from the device scan loop only > when ioc_status of config page request equals > MPI2_IOCSTATUS_CONFIG_INVALID_PAGE. If the firmware returns some > other ioc_status such as MPI2_IOCSTATUS_INVALID_FUNCTION then this > results to infinite loop. > > As this issue is critical since the driver may go in to infinite loop, > so we thought to add this patch in stable kernel also. Why was this text not in the changelog information for the patch itself? That would have help justified why it was submitted to the stable tree. Why would the firmware return some other status values? Is this a firmware failure, or a driver failure to handle valid firmware errors? Note, the patch is bigger than I like to see in stable patches, are you _sure_ it's needed? thanks, greg k-h -- To unsubscribe from this list: send the line "unsubscribe stable" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html