Hi Saurav, Will these changes conflict with those submitted in August [1] to Christoph's drivers-for-3.18 branch? In particular, "qla2xxx: Fix shost use-after-free on device removal" [2] fixed this same driver unload issue in a slightly different manner. That change was marked for stable as the bug was introduced by fe1b806f4f71 ("qla2xxx: Refactor shutdown code so some functionality can be reused") in previous releases. Regards, -- Joe [1] http://thread.gmane.org/gmane.linux.scsi/93859 [2] http://git.infradead.org/users/hch/scsi-queue.git/commitdiff/db7157d4cfce6edf052452fb1d327d4d11b67f4c On Wed, 24 Sep 2014 03:08:23 -0400 Saurav Kashyap <saurav.kashyap@xxxxxxxxxx> wrote: > From: Arun Easi <arun.easi@xxxxxxxxxx> > > Signed-off-by: Arun Easi <arun.easi@xxxxxxxxxx> > Signed-off-by: Saurav Kashyap <saurav.kashyap@xxxxxxxxxx> > --- > drivers/scsi/qla2xxx/qla_os.c | 4 ++-- > 1 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/scsi/qla2xxx/qla_os.c b/drivers/scsi/qla2xxx/qla_os.c > index 8846883..6539513 100644 > --- a/drivers/scsi/qla2xxx/qla_os.c > +++ b/drivers/scsi/qla2xxx/qla_os.c > @@ -3190,10 +3190,10 @@ qla2x00_remove_one(struct pci_dev *pdev) > > qla2x00_free_device(base_vha); > > - scsi_host_put(base_vha->host); > - > qla2x00_clear_drv_active(base_vha); > > + scsi_host_put(base_vha->host); > + > qla2x00_unmap_iobases(ha); > > pci_release_selected_regions(ha->pdev, ha->bars); -- 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