Messages in this thread | | | Date | Sun, 25 Apr 2010 10:09:27 +0200 | From | Tejun Heo <> | Subject | Re: [PATCHSET UPDATED] workqueue: concurrency managed workqueue, take#4 |
| |
Hello,
I've just updated the git tree.
git://git.kernel.org/pub/scm/linux/kernel/git/tj/wq.git review-cmwq
The original take#4 is now in branch review-cmwq-3. This will probably become take#5 soonish but I don't have access to my test machines for some days, so this is sort of take#5 pre-release. Changes are...
* The patchset is rebased on cpu_stop + sched/core. cpu_stop already reimplements stop_machine so that it doesn't use RT workqueue, so this patchset simply drops RT wq support.
* Oleg's clear work->data patch moved at the head of the queue and now lives in the for-next branch which will be pushed to mainline on the next merge window.
* David reported a bug where fscache show_work function caused panic by accessing already dead object. This turned out to be race condition between put() after execution and show(). If the put() after work execution is the last put, the object gets destroyed; however, show() could be called to describe the work is actually finished ending up dereferencing already freed object. Fixed by deferring put() to another work if debugfs support is enabled so that the object stays alive while the work is executing. Due to lack of test setup, I couldn't actually test this yet. I'll verify it works as soon as I have access to my stuff.
* Applied Oleg's review.
* Comments updated as suggested.
* work_flags_to_color() replaced w/ get_work_color()
* nr_cwqs_to_flush bug which could cause premature flush completion fixed.
* Replace rewind + list_for_each_entry_safe_continue() w/ list_for_each_entry_safe_from().
* Don't directly write to *work_data_bits() but use __set_bit() instead.
* Fixed cpu hotplug exclusion bug.
Thanks.
-- tejun
| |