Commit dff16268 authored by NeilBrown's avatar NeilBrown Committed by Greg Kroah-Hartman

staging: lustre: simplfy lov_finish_set()

When deleting everything from a list, a while loop
is cleaner than list_for_each_safe().
Signed-off-by: default avatarNeilBrown <neilb@suse.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 557d001a
...@@ -49,15 +49,13 @@ static void lov_init_set(struct lov_request_set *set) ...@@ -49,15 +49,13 @@ static void lov_init_set(struct lov_request_set *set)
static void lov_finish_set(struct lov_request_set *set) static void lov_finish_set(struct lov_request_set *set)
{ {
struct list_head *pos, *n; struct lov_request *req;
LASSERT(set); LASSERT(set);
list_for_each_safe(pos, n, &set->set_list) { while ((req = list_first_entry_or_null(&set->set_list,
struct lov_request *req = list_entry(pos, struct lov_request,
struct lov_request, rq_link)) != NULL) {
rq_link);
list_del_init(&req->rq_link); list_del_init(&req->rq_link);
kfree(req->rq_oi.oi_osfs); kfree(req->rq_oi.oi_osfs);
kfree(req); kfree(req);
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment