On 05/24/2017 11:48 PM, Guilherme G. Piccoli wrote: > We might have a NULL pring in lpfc_els_abort(), for example on > error recovery path, since queues are destroyed during error > recovery mechanism. > > In this case, we should just drop the abort since the queues will > be recreated anyway. This patch just verifies for NULL pointer > and stop the abortion of the queue in case of a NULL pring. > > Also, this patch converts return type of lpfc_els_abort() from int > to void, since it's not checked anywhere. > > Reported-by: Harsha Thyagaraja <hathy...@in.ibm.com> > Reported-by: Naresh Bannoth <nbann...@in.ibm.com> > Tested-by: Raphael Silva <rapha...@linux.vnet.ibm.com> > Signed-off-by: Guilherme G. Piccoli <gpicc...@linux.vnet.ibm.com> > ---
Looks good, Reviewed-by: Johannes Thumshirn <jthumsh...@suse.de> -- Johannes Thumshirn Storage jthumsh...@suse.de +49 911 74053 689 SUSE LINUX GmbH, Maxfeldstr. 5, 90409 Nürnberg GF: Felix Imendörffer, Jane Smithard, Graham Norton HRB 21284 (AG Nürnberg) Key fingerprint = EC38 9CAB C2C4 F25D 8600 D0D0 0393 969D 2D76 0850
