diff options
author | Joern Engel <joern@logfs.org> | 2014-09-16 16:23:13 -0400 |
---|---|---|
committer | Nicholas Bellinger <nab@linux-iscsi.org> | 2014-10-01 17:39:10 -0400 |
commit | f81ccb489a7a641c1bed41b49cf8d72c199c68d5 (patch) | |
tree | 8ab987efb52a2495bafdcc399f465ae15c47ae7a /drivers/target | |
parent | 33940d09937276cd3c81f2874faf43e37c2db0e2 (diff) |
target: simplify core_tmr_abort_task
list_for_each_entry_safe is necessary if list objects are deleted from
the list while traversing it. Not the case here, so we can use the base
list_for_each_entry variant.
Signed-off-by: Joern Engel <joern@logfs.org>
Signed-off-by: Nicholas Bellinger <nab@linux-iscsi.org>
Diffstat (limited to 'drivers/target')
-rw-r--r-- | drivers/target/target_core_tmr.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/drivers/target/target_core_tmr.c b/drivers/target/target_core_tmr.c index 83de7aec4aac..fa5e157db47b 100644 --- a/drivers/target/target_core_tmr.c +++ b/drivers/target/target_core_tmr.c | |||
@@ -115,13 +115,12 @@ void core_tmr_abort_task( | |||
115 | struct se_tmr_req *tmr, | 115 | struct se_tmr_req *tmr, |
116 | struct se_session *se_sess) | 116 | struct se_session *se_sess) |
117 | { | 117 | { |
118 | struct se_cmd *se_cmd, *tmp_cmd; | 118 | struct se_cmd *se_cmd; |
119 | unsigned long flags; | 119 | unsigned long flags; |
120 | int ref_tag; | 120 | int ref_tag; |
121 | 121 | ||
122 | spin_lock_irqsave(&se_sess->sess_cmd_lock, flags); | 122 | spin_lock_irqsave(&se_sess->sess_cmd_lock, flags); |
123 | list_for_each_entry_safe(se_cmd, tmp_cmd, | 123 | list_for_each_entry(se_cmd, &se_sess->sess_cmd_list, se_cmd_list) { |
124 | &se_sess->sess_cmd_list, se_cmd_list) { | ||
125 | 124 | ||
126 | if (dev != se_cmd->se_dev) | 125 | if (dev != se_cmd->se_dev) |
127 | continue; | 126 | continue; |