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/infiniband | |
| 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/infiniband')
| -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 950228fb00..77e8b90dfb 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); |
