diff options
author | David Dillow <dillowda@ornl.gov> | 2008-01-03 13:25:27 -0500 |
---|---|---|
committer | Roland Dreier <rolandd@cisco.com> | 2008-01-03 13:25:27 -0500 |
commit | b0e47c8b79154772a436f25bf7646733e1d6194c (patch) | |
tree | 48d4b1232975de4b12b57e4919665e7cc93ccb14 /drivers | |
parent | b8c9a18712f7b617fda66d878ce3759c9e575ba0 (diff) |
IB/srp: Fix list corruption/oops on module reload
Add a missing call to srp_remove_host() in srp_remove_one() so that we
don't leak SRP transport class list entries.
Tested-by: David Dillow <dillowda@ornl.gov>
Acked-by: FUJITA Tomonori <tomof@acm.org>
Signed-off-by: Roland Dreier <rolandd@cisco.com>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/infiniband/ulp/srp/ib_srp.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/infiniband/ulp/srp/ib_srp.c b/drivers/infiniband/ulp/srp/ib_srp.c index 950228fb009f..77e8b90dfbff 100644 --- a/drivers/infiniband/ulp/srp/ib_srp.c +++ b/drivers/infiniband/ulp/srp/ib_srp.c | |||
@@ -2054,6 +2054,7 @@ static void srp_remove_one(struct ib_device *device) | |||
2054 | list_for_each_entry_safe(target, tmp_target, | 2054 | list_for_each_entry_safe(target, tmp_target, |
2055 | &host->target_list, list) { | 2055 | &host->target_list, list) { |
2056 | scsi_remove_host(target->scsi_host); | 2056 | scsi_remove_host(target->scsi_host); |
2057 | srp_remove_host(target->scsi_host); | ||
2057 | srp_disconnect_target(target); | 2058 | srp_disconnect_target(target); |
2058 | ib_destroy_cm_id(target->cm_id); | 2059 | ib_destroy_cm_id(target->cm_id); |
2059 | srp_free_target_ib(target); | 2060 | srp_free_target_ib(target); |