]> www.infradead.org Git - users/jedix/linux-maple.git/commit
qla2xxx: Fix crash in qla2x00_abort_all_cmds() on unload
authorRoland Dreier <roland@purestorage.com>
Thu, 22 Sep 2011 07:06:05 +0000 (00:06 -0700)
committerMaxim Uvarov <maxim.uvarov@oracle.com>
Thu, 8 Dec 2011 19:16:11 +0000 (11:16 -0800)
commit0bfda19dbfccc8352f24dcada98309c2724b14ef
tree0d1df4d0bc8495106a90fc18bc42dff46357f0cb
parent39fcec9e7161b8cf134365b16436fbd9804c321a
qla2xxx: Fix crash in qla2x00_abort_all_cmds() on unload

commit 9bfacd01dc9b7519e1e6da12b01963550b9d09a2 upstream.

I hit a crash in qla2x00_abort_all_cmds() if the qla2xxx module is
unloaded right after it is loaded.  I debugged this down to the abort
handling improperly treating a command of type SRB_ADISC_CMD as if it
had a bsg_job to complete when that command actually uses the iocb_cmd
part of the union.  (I guess to hit this one has to unload the module
while the async FC initialization is still in progress)

It seems we should only look for a bsg_job if type is SRB_ELS_CMD_RPT,
SRB_ELS_CMD_HST or SRB_CT_CMD, so switch the test to make that explicit.

Signed-off-by: Roland Dreier <roland@purestorage.com>
Acked-by: Chad Dupuis <chad.dupuis@qlogic.com>
Signed-off-by: James Bottomley <JBottomley@Parallels.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/scsi/qla2xxx/qla_os.c