summaryrefslogtreecommitdiffstats
path: root/kernel
diff options
context:
space:
mode:
authorGreg Thelen <gthelen@google.com>2012-11-28 13:50:45 -0800
committerTejun Heo <tj@kernel.org>2012-11-28 13:52:14 -0800
commit9718ceb3431acdeee9bdec5c18e18266333970aa (patch)
tree5f8cf33738076409cf0c311100b2972c0571ec27 /kernel
parent205a872bd6f9a9a09ef035ef1e90185a8245cc58 (diff)
downloadop-kernel-dev-9718ceb3431acdeee9bdec5c18e18266333970aa.zip
op-kernel-dev-9718ceb3431acdeee9bdec5c18e18266333970aa.tar.gz
cgroup: list_del_init() on removed events
Use list_del_init() rather than list_del() to remove events from cgrp->event_list. No functional change. This is just defensive coding. Signed-off-by: Greg Thelen <gthelen@google.com> Signed-off-by: Tejun Heo <tj@kernel.org>
Diffstat (limited to 'kernel')
-rw-r--r--kernel/cgroup.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/kernel/cgroup.c b/kernel/cgroup.c
index 589433f..46dcdbd 100644
--- a/kernel/cgroup.c
+++ b/kernel/cgroup.c
@@ -3767,7 +3767,7 @@ static int cgroup_event_wake(wait_queue_t *wait, unsigned mode,
if (flags & POLLHUP) {
__remove_wait_queue(event->wqh, &event->wait);
spin_lock(&cgrp->event_list_lock);
- list_del(&event->list);
+ list_del_init(&event->list);
spin_unlock(&cgrp->event_list_lock);
/*
* We are in atomic context, but cgroup_event_remove() may
@@ -4341,7 +4341,7 @@ static int cgroup_destroy_locked(struct cgroup *cgrp)
list_splice_init(&cgrp->event_list, &tmp_list);
spin_unlock(&cgrp->event_list_lock);
list_for_each_entry_safe(event, tmp, &tmp_list, list) {
- list_del(&event->list);
+ list_del_init(&event->list);
remove_wait_queue(event->wqh, &event->wait);
eventfd_signal(event->eventfd, 1);
schedule_work(&event->remove);
OpenPOWER on IntegriCloud