On Wed, Nov 09, 2016 at 09:39:21PM +0100, Luis R. Rodriguez wrote: > On Sun, Oct 30, 2016 at 03:50:48PM +0100, Yves-Alexis Perez wrote: > > From: Yves-Alexis Perez <corsac@xxxxxxxxxx> > > > > wait_for_completion_interruptible_timeout() return value is either > > -ERESTARTSYS (in case it was interrupted), 0 (in case the timeout expired) > > or the number of jiffies left until timeout. The return value is stored in > > a long, but in _request_firmware_load() it's silently casted to an int, > > which can overflow and give a negative value, indicating an error. > > > > Fix this by re-using the timeout variable and only set retval when it's > > safe. > > Please amend the commit log as I noted in the previous response, and > resend. > > > Signed-off-by: Yves-Alexis Perez <corsac@xxxxxxxxxx> > > Cc: Ming Lei <ming.lei@xxxxxxxxxxxxx> > > Cc: "Luis R. Rodriguez" <mcgrof@xxxxxxxxxx> > > Cc: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> > > Other than the commit log you can add on you resend: > > Acked-by: Luis R. Rodriguez. > > Modulo I don't personally thing this this is sable material but I'll let > Greg decide. Does it fix a regression? A reported issue with an older kernel version that people have hit? It shouldn't be hard to figure out if a patch should be in stable or not... 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