The patch titled
     revert "cancel_delayed_work: use del_timer() instead of del_timer_sync()"
has been removed from the -mm tree.  Its filename was
     revert-cancel_delayed_work-use-del_timer-instead-of-del_timer_sync.patch

This patch was dropped because it was merged into mainline or a subsystem tree

------------------------------------------------------
Subject: revert "cancel_delayed_work: use del_timer() instead of 
del_timer_sync()"
From: Oleg Nesterov <[EMAIL PROTECTED]>

As pointed out by Jarek Poplawski, the patch

        [WORKQUEUE]: cancel_delayed_work: use del_timer() instead of 
del_timer_sync()
        commit: 071b638689464c6b39407025eedd810d5b5e6f5d

was wrong, it was merged by mistake after that.

>From the changelog:

        after this patch:
                ...
                delayed_work_timer_fn->__queue_work() in progress.

                The latter doesn't differ from the caller's POV,

it does make a difference if the caller calls flush_workqueue() after
cancel_delayed_work(), in that case flush_workqueue() can miss this
work_struct.

Signed-off-by: Oleg Nesterov <[EMAIL PROTECTED]>
Cc: Jarek Poplawski <[EMAIL PROTECTED]>
Cc: David Howells <[EMAIL PROTECTED]>
Signed-off-by: Andrew Morton <[EMAIL PROTECTED]>
---

 include/linux/workqueue.h |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff -puN 
include/linux/workqueue.h~revert-cancel_delayed_work-use-del_timer-instead-of-del_timer_sync
 include/linux/workqueue.h
--- 
a/include/linux/workqueue.h~revert-cancel_delayed_work-use-del_timer-instead-of-del_timer_sync
+++ a/include/linux/workqueue.h
@@ -160,7 +160,7 @@ static inline int cancel_delayed_work(st
 {
        int ret;
 
-       ret = del_timer(&work->timer);
+       ret = del_timer_sync(&work->timer);
        if (ret)
                work_clear_pending(&work->work);
        return ret;
_

Patches currently in -mm which might be from [EMAIL PROTECTED] are

libata-core-convert-to-use-cancel_rearming_delayed_work.patch
recalc_sigpending_tsk-fixes.patch
simplify-cleanup_workqueue_thread.patch
simplify-cleanup_workqueue_thread-fix.patch
freezer-close-potential-race-between-refrigerator-and-thaw_tasks.patch
freezer-fix-vfork-problem.patch
freezer-take-kernel_execve-into-consideration.patch
freezer-fix-kthread_create-vs-freezer-theoretical-race.patch
freezer-fix-pf_nofreeze-vs-freezeable-race.patch
freezer-move-frozen_process-to-kernel-power-processc.patch
clone-flag-clone_parent_tidptr-leaves-invalid-results-in-memory.patch
use-write_trylock_irqsave-in-ptrace_attach.patch
tidy-up-usermode-helper-waiting-a-bit.patch
fix-stop_machine_run-problem-with-naughty-real-time-process.patch
cpu-hotplug-fix-ksoftirqd-termination-on-cpu-hotplug-with-naughty-realtime-process.patch
cpu-hotplug-fix-ksoftirqd-termination-on-cpu-hotplug-with-naughty-realtime-process-fix.patch
percpu_counters-use-cpu-notifiers.patch
percpu_counters-use-for_each_online_cpu.patch

-
To unsubscribe from this list: send the line "unsubscribe mm-commits" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to