2013-03-24 07:11:31 +08:00
|
|
|
/*
|
|
|
|
* Asynchronous refcounty things
|
|
|
|
*
|
|
|
|
* Copyright 2010, 2011 Kent Overstreet <kent.overstreet@gmail.com>
|
|
|
|
* Copyright 2012 Google, Inc.
|
|
|
|
*/
|
|
|
|
|
|
|
|
#include <linux/debugfs.h>
|
|
|
|
#include <linux/module.h>
|
|
|
|
#include <linux/seq_file.h>
|
2018-01-09 04:21:26 +08:00
|
|
|
#include <linux/sched/debug.h>
|
2013-03-24 07:11:31 +08:00
|
|
|
|
|
|
|
#include "closure.h"
|
|
|
|
|
|
|
|
static inline void closure_put_after_sub(struct closure *cl, int flags)
|
|
|
|
{
|
|
|
|
int r = flags & CLOSURE_REMAINING_MASK;
|
|
|
|
|
|
|
|
BUG_ON(flags & CLOSURE_GUARD_MASK);
|
2013-11-02 09:03:08 +08:00
|
|
|
BUG_ON(!r && (flags & ~CLOSURE_DESTRUCTOR));
|
2013-03-24 07:11:31 +08:00
|
|
|
|
|
|
|
if (!r) {
|
|
|
|
if (cl->fn && !(flags & CLOSURE_DESTRUCTOR)) {
|
|
|
|
atomic_set(&cl->remaining,
|
|
|
|
CLOSURE_REMAINING_INITIALIZER);
|
|
|
|
closure_queue(cl);
|
|
|
|
} else {
|
|
|
|
struct closure *parent = cl->parent;
|
2013-07-11 09:04:21 +08:00
|
|
|
closure_fn *destructor = cl->fn;
|
2013-03-24 07:11:31 +08:00
|
|
|
|
|
|
|
closure_debug_destroy(cl);
|
|
|
|
|
2013-07-11 09:04:21 +08:00
|
|
|
if (destructor)
|
|
|
|
destructor(cl);
|
2013-03-24 07:11:31 +08:00
|
|
|
|
|
|
|
if (parent)
|
|
|
|
closure_put(parent);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
/* For clearing flags with the same atomic op as a put */
|
|
|
|
void closure_sub(struct closure *cl, int v)
|
|
|
|
{
|
|
|
|
closure_put_after_sub(cl, atomic_sub_return(v, &cl->remaining));
|
|
|
|
}
|
2013-11-02 09:03:08 +08:00
|
|
|
EXPORT_SYMBOL(closure_sub);
|
2013-03-24 07:11:31 +08:00
|
|
|
|
2018-03-19 08:36:29 +08:00
|
|
|
/*
|
2013-12-21 07:55:23 +08:00
|
|
|
* closure_put - decrement a closure's refcount
|
|
|
|
*/
|
2013-03-24 07:11:31 +08:00
|
|
|
void closure_put(struct closure *cl)
|
|
|
|
{
|
|
|
|
closure_put_after_sub(cl, atomic_dec_return(&cl->remaining));
|
|
|
|
}
|
2013-11-02 09:03:08 +08:00
|
|
|
EXPORT_SYMBOL(closure_put);
|
2013-03-24 07:11:31 +08:00
|
|
|
|
2018-03-19 08:36:29 +08:00
|
|
|
/*
|
2013-12-21 07:55:23 +08:00
|
|
|
* closure_wake_up - wake up all closures on a wait list, without memory barrier
|
|
|
|
*/
|
2013-03-24 07:11:31 +08:00
|
|
|
void __closure_wake_up(struct closure_waitlist *wait_list)
|
|
|
|
{
|
|
|
|
struct llist_node *list;
|
bcache: use llist_for_each_entry_safe() in __closure_wake_up()
Commit 09b3efec ("bcache: Don't reinvent the wheel but use existing llist
API") replaces the following while loop by llist_for_each_entry(),
-
- while (reverse) {
- cl = container_of(reverse, struct closure, list);
- reverse = llist_next(reverse);
-
+ llist_for_each_entry(cl, reverse, list) {
closure_set_waiting(cl, 0);
closure_sub(cl, CLOSURE_WAITING + 1);
}
This modification introduces a potential race by iterating a corrupted
list. Here is how it happens.
In the above modification, closure_sub() may wake up a process which is
waiting on reverse list. If this process decides to wait again by calling
closure_wait(), its cl->list will be added to another wait list. Then
when llist_for_each_entry() continues to iterate next node, it will travel
on another new wait list which is added in closure_wait(), not the
original reverse list in __closure_wake_up(). It is more probably to
happen on UP machine because the waked up process may preempt the process
which wakes up it.
Use llist_for_each_entry_safe() will fix the issue, the safe version fetch
next node before waking up a process. Then the copy of next node will make
sure list iteration stays on original reverse list.
Fixes: 09b3efec81de ("bcache: Don't reinvent the wheel but use existing llist API")
Signed-off-by: Coly Li <colyli@suse.de>
Reported-by: Michael Lyle <mlyle@lyle.org>
Reviewed-by: Byungchul Park <byungchul.park@lge.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
2017-09-26 17:54:12 +08:00
|
|
|
struct closure *cl, *t;
|
2013-03-24 07:11:31 +08:00
|
|
|
struct llist_node *reverse = NULL;
|
|
|
|
|
|
|
|
list = llist_del_all(&wait_list->list);
|
|
|
|
|
|
|
|
/* We first reverse the list to preserve FIFO ordering and fairness */
|
2017-09-06 14:25:54 +08:00
|
|
|
reverse = llist_reverse_order(list);
|
2013-03-24 07:11:31 +08:00
|
|
|
|
|
|
|
/* Then do the wakeups */
|
bcache: use llist_for_each_entry_safe() in __closure_wake_up()
Commit 09b3efec ("bcache: Don't reinvent the wheel but use existing llist
API") replaces the following while loop by llist_for_each_entry(),
-
- while (reverse) {
- cl = container_of(reverse, struct closure, list);
- reverse = llist_next(reverse);
-
+ llist_for_each_entry(cl, reverse, list) {
closure_set_waiting(cl, 0);
closure_sub(cl, CLOSURE_WAITING + 1);
}
This modification introduces a potential race by iterating a corrupted
list. Here is how it happens.
In the above modification, closure_sub() may wake up a process which is
waiting on reverse list. If this process decides to wait again by calling
closure_wait(), its cl->list will be added to another wait list. Then
when llist_for_each_entry() continues to iterate next node, it will travel
on another new wait list which is added in closure_wait(), not the
original reverse list in __closure_wake_up(). It is more probably to
happen on UP machine because the waked up process may preempt the process
which wakes up it.
Use llist_for_each_entry_safe() will fix the issue, the safe version fetch
next node before waking up a process. Then the copy of next node will make
sure list iteration stays on original reverse list.
Fixes: 09b3efec81de ("bcache: Don't reinvent the wheel but use existing llist API")
Signed-off-by: Coly Li <colyli@suse.de>
Reported-by: Michael Lyle <mlyle@lyle.org>
Reviewed-by: Byungchul Park <byungchul.park@lge.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
2017-09-26 17:54:12 +08:00
|
|
|
llist_for_each_entry_safe(cl, t, reverse, list) {
|
2013-12-21 07:55:23 +08:00
|
|
|
closure_set_waiting(cl, 0);
|
2013-03-24 07:11:31 +08:00
|
|
|
closure_sub(cl, CLOSURE_WAITING + 1);
|
|
|
|
}
|
|
|
|
}
|
2013-11-02 09:03:08 +08:00
|
|
|
EXPORT_SYMBOL(__closure_wake_up);
|
2013-03-24 07:11:31 +08:00
|
|
|
|
2013-12-21 07:55:23 +08:00
|
|
|
/**
|
|
|
|
* closure_wait - add a closure to a waitlist
|
2018-03-19 08:36:29 +08:00
|
|
|
* @waitlist: will own a ref on @cl, which will be released when
|
2013-12-21 07:55:23 +08:00
|
|
|
* closure_wake_up() is called on @waitlist.
|
2018-03-19 08:36:29 +08:00
|
|
|
* @cl: closure pointer.
|
2013-12-21 07:55:23 +08:00
|
|
|
*
|
|
|
|
*/
|
|
|
|
bool closure_wait(struct closure_waitlist *waitlist, struct closure *cl)
|
2013-03-24 07:11:31 +08:00
|
|
|
{
|
|
|
|
if (atomic_read(&cl->remaining) & CLOSURE_WAITING)
|
|
|
|
return false;
|
|
|
|
|
2013-12-21 07:55:23 +08:00
|
|
|
closure_set_waiting(cl, _RET_IP_);
|
2013-03-24 07:11:31 +08:00
|
|
|
atomic_add(CLOSURE_WAITING + 1, &cl->remaining);
|
2013-12-21 07:55:23 +08:00
|
|
|
llist_add(&cl->list, &waitlist->list);
|
2013-03-24 07:11:31 +08:00
|
|
|
|
|
|
|
return true;
|
|
|
|
}
|
2013-11-02 09:03:08 +08:00
|
|
|
EXPORT_SYMBOL(closure_wait);
|
2013-03-24 07:11:31 +08:00
|
|
|
|
2018-01-09 04:21:25 +08:00
|
|
|
struct closure_syncer {
|
|
|
|
struct task_struct *task;
|
|
|
|
int done;
|
|
|
|
};
|
|
|
|
|
|
|
|
static void closure_sync_fn(struct closure *cl)
|
2013-03-24 07:11:31 +08:00
|
|
|
{
|
2018-01-09 04:21:25 +08:00
|
|
|
cl->s->done = 1;
|
|
|
|
wake_up_process(cl->s->task);
|
|
|
|
}
|
2013-03-24 07:11:31 +08:00
|
|
|
|
2018-01-09 04:21:26 +08:00
|
|
|
void __sched __closure_sync(struct closure *cl)
|
2018-01-09 04:21:25 +08:00
|
|
|
{
|
|
|
|
struct closure_syncer s = { .task = current };
|
2013-03-24 07:11:31 +08:00
|
|
|
|
2018-01-09 04:21:25 +08:00
|
|
|
cl->s = &s;
|
|
|
|
continue_at(cl, closure_sync_fn, NULL);
|
|
|
|
|
|
|
|
while (1) {
|
|
|
|
set_current_state(TASK_UNINTERRUPTIBLE);
|
|
|
|
if (s.done)
|
|
|
|
break;
|
2013-03-24 07:11:31 +08:00
|
|
|
schedule();
|
|
|
|
}
|
|
|
|
|
2018-01-09 04:21:25 +08:00
|
|
|
__set_current_state(TASK_RUNNING);
|
2013-03-24 07:11:31 +08:00
|
|
|
}
|
2018-01-09 04:21:25 +08:00
|
|
|
EXPORT_SYMBOL(__closure_sync);
|
2013-03-24 07:11:31 +08:00
|
|
|
|
|
|
|
#ifdef CONFIG_BCACHE_CLOSURES_DEBUG
|
|
|
|
|
|
|
|
static LIST_HEAD(closure_list);
|
|
|
|
static DEFINE_SPINLOCK(closure_list_lock);
|
|
|
|
|
|
|
|
void closure_debug_create(struct closure *cl)
|
|
|
|
{
|
|
|
|
unsigned long flags;
|
|
|
|
|
|
|
|
BUG_ON(cl->magic == CLOSURE_MAGIC_ALIVE);
|
|
|
|
cl->magic = CLOSURE_MAGIC_ALIVE;
|
|
|
|
|
|
|
|
spin_lock_irqsave(&closure_list_lock, flags);
|
|
|
|
list_add(&cl->all, &closure_list);
|
|
|
|
spin_unlock_irqrestore(&closure_list_lock, flags);
|
|
|
|
}
|
2013-11-02 09:03:08 +08:00
|
|
|
EXPORT_SYMBOL(closure_debug_create);
|
2013-03-24 07:11:31 +08:00
|
|
|
|
|
|
|
void closure_debug_destroy(struct closure *cl)
|
|
|
|
{
|
|
|
|
unsigned long flags;
|
|
|
|
|
|
|
|
BUG_ON(cl->magic != CLOSURE_MAGIC_ALIVE);
|
|
|
|
cl->magic = CLOSURE_MAGIC_DEAD;
|
|
|
|
|
|
|
|
spin_lock_irqsave(&closure_list_lock, flags);
|
|
|
|
list_del(&cl->all);
|
|
|
|
spin_unlock_irqrestore(&closure_list_lock, flags);
|
|
|
|
}
|
2013-11-02 09:03:08 +08:00
|
|
|
EXPORT_SYMBOL(closure_debug_destroy);
|
2013-03-24 07:11:31 +08:00
|
|
|
|
2018-03-19 08:36:23 +08:00
|
|
|
static struct dentry *closure_debug;
|
2013-03-24 07:11:31 +08:00
|
|
|
|
|
|
|
static int debug_seq_show(struct seq_file *f, void *data)
|
|
|
|
{
|
|
|
|
struct closure *cl;
|
|
|
|
spin_lock_irq(&closure_list_lock);
|
|
|
|
|
|
|
|
list_for_each_entry(cl, &closure_list, all) {
|
|
|
|
int r = atomic_read(&cl->remaining);
|
|
|
|
|
|
|
|
seq_printf(f, "%p: %pF -> %pf p %p r %i ",
|
|
|
|
cl, (void *) cl->ip, cl->fn, cl->parent,
|
|
|
|
r & CLOSURE_REMAINING_MASK);
|
|
|
|
|
2018-01-09 04:21:25 +08:00
|
|
|
seq_printf(f, "%s%s\n",
|
2015-10-05 20:39:52 +08:00
|
|
|
test_bit(WORK_STRUCT_PENDING_BIT,
|
2013-03-24 07:11:31 +08:00
|
|
|
work_data_bits(&cl->work)) ? "Q" : "",
|
2018-01-09 04:21:25 +08:00
|
|
|
r & CLOSURE_RUNNING ? "R" : "");
|
2013-03-24 07:11:31 +08:00
|
|
|
|
|
|
|
if (r & CLOSURE_WAITING)
|
|
|
|
seq_printf(f, " W %pF\n",
|
|
|
|
(void *) cl->waiting_on);
|
|
|
|
|
|
|
|
seq_printf(f, "\n");
|
|
|
|
}
|
|
|
|
|
|
|
|
spin_unlock_irq(&closure_list_lock);
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
|
|
|
static int debug_seq_open(struct inode *inode, struct file *file)
|
|
|
|
{
|
|
|
|
return single_open(file, debug_seq_show, NULL);
|
|
|
|
}
|
|
|
|
|
|
|
|
static const struct file_operations debug_ops = {
|
|
|
|
.owner = THIS_MODULE,
|
|
|
|
.open = debug_seq_open,
|
|
|
|
.read = seq_read,
|
|
|
|
.release = single_release
|
|
|
|
};
|
|
|
|
|
2018-08-09 15:48:42 +08:00
|
|
|
void __init closure_debug_init(void)
|
2013-03-24 07:11:31 +08:00
|
|
|
{
|
2018-08-09 15:48:42 +08:00
|
|
|
if (!IS_ERR_OR_NULL(bcache_debug))
|
|
|
|
/*
|
|
|
|
* it is unnecessary to check return value of
|
|
|
|
* debugfs_create_file(), we should not care
|
|
|
|
* about this.
|
|
|
|
*/
|
|
|
|
closure_debug = debugfs_create_file(
|
|
|
|
"closures", 0400, bcache_debug, NULL, &debug_ops);
|
2013-03-24 07:11:31 +08:00
|
|
|
}
|
|
|
|
#endif
|
|
|
|
|
|
|
|
MODULE_AUTHOR("Kent Overstreet <koverstreet@google.com>");
|
|
|
|
MODULE_LICENSE("GPL");
|