> -----Original Message----- > From: Chengguang Xu <cgxu...@gmx.com> > Sent: Sunday, November 4, 2018 7:28 PM > To: j...@linux.vnet.ibm.com; martin.peter...@oracle.com > Cc: Dept-Eng QLogic Storage Upstream <QLogic-Storage- > upstr...@cavium.com>; linux-scsi@vger.kernel.org; Chengguang Xu > <cgxu...@gmx.com> > Subject: [PATCH 2/2] scsi: qla4xxx: remvoe unnecessary condition check for > mempool_destroy() > > External Email > > mempool_destroy() can handle NULL pointer correctly, so there is no need > to check NULL pointer before calling mempool_destroy(). > > Signed-off-by: Chengguang Xu <cgxu...@gmx.com> > --- > drivers/scsi/qla4xxx/ql4_os.c | 4 +--- > 1 file changed, 1 insertion(+), 3 deletions(-) > > diff --git a/drivers/scsi/qla4xxx/ql4_os.c b/drivers/scsi/qla4xxx/ql4_os.c > index 039950ab1cbc..1c702cd22359 100644 > --- a/drivers/scsi/qla4xxx/ql4_os.c > +++ b/drivers/scsi/qla4xxx/ql4_os.c > @@ -4160,9 +4160,7 @@ static void qla4xxx_mem_free(struct scsi_qla_host > *ha) > ha->fw_dump_size = 0; > > /* Free srb pool. */ > - if (ha->srb_mempool) > - mempool_destroy(ha->srb_mempool); > - > + mempool_destroy(ha->srb_mempool); > ha->srb_mempool = NULL; > > dma_pool_destroy(ha->chap_dma_pool); > --
Thanks Acked-by: Manish Rangankar <manish.rangan...@cavium.com>
