workqueue: Catch more locking problems with flush_work()
authorStephen Boyd <sboyd@codeaurora.org>
Sat, 21 Apr 2012 00:28:50 +0000 (17:28 -0700)
committerTejun Heo <tj@kernel.org>
Mon, 23 Apr 2012 18:06:42 +0000 (11:06 -0700)
commit0976dfc1d0cd80a4e9dfaf87bd8744612bde475a
tree0af2d79070fcb0a510187eb780db796c5702df20
parentf5b2552b4ebbeadcadde1532d7bbd3f850719046
workqueue: Catch more locking problems with flush_work()

If a workqueue is flushed with flush_work() lockdep checking can
be circumvented. For example:

 static DEFINE_MUTEX(mutex);

 static void my_work(struct work_struct *w)
 {
         mutex_lock(&mutex);
         mutex_unlock(&mutex);
 }

 static DECLARE_WORK(work, my_work);

 static int __init start_test_module(void)
 {
         schedule_work(&work);
         return 0;
 }
 module_init(start_test_module);

 static void __exit stop_test_module(void)
 {
         mutex_lock(&mutex);
         flush_work(&work);
         mutex_unlock(&mutex);
 }
 module_exit(stop_test_module);

would not always print a warning when flush_work() was called.
In this trivial example nothing could go wrong since we are
guaranteed module_init() and module_exit() don't run concurrently,
but if the work item is schedule asynchronously we could have a
scenario where the work item is running just at the time flush_work()
is called resulting in a classic ABBA locking problem.

Add a lockdep hint by acquiring and releasing the work item
lockdep_map in flush_work() so that we always catch this
potential deadlock scenario.

Signed-off-by: Stephen Boyd <sboyd@codeaurora.org>
Reviewed-by: Yong Zhang <yong.zhang0@gmail.com>
Signed-off-by: Tejun Heo <tj@kernel.org>
kernel/workqueue.c