]> git.baikalelectronics.ru Git - kernel.git/commit
workqueue: remove PREPARE_[DELAYED_]WORK()
authorTejun Heo <tj@kernel.org>
Fri, 7 Mar 2014 15:24:50 +0000 (10:24 -0500)
committerTejun Heo <tj@kernel.org>
Fri, 7 Mar 2014 15:24:50 +0000 (10:24 -0500)
commitcca56f8d08658459dd84fb5272de39667d2be1ea
treef3a757aeaef233f894e2cc373061dcf5b3919d6c
parent6e0e361fc6439d13de52463550ea3a5f3f4f56c0
workqueue: remove PREPARE_[DELAYED_]WORK()

Peter Hurley noticed that since b5d4a929e988 ("workqueue: consider
work function when searching for busy work items"), a work item which
gets assigned a different work function would break out of the
non-reentrancy guarantee as workqueue would consider it a different
work item.

This is fragile and extremely subtle.  PREPARE_[DELAYED_]WORK() have
never been used widely and its semantics has always been somewhat
iffy.  If the work item is known not to be on queue when
PREPARE_WORK() is called, there's no difference from using
INIT_WORK().  If the work item may be queued at the time of
PREPARE_WORK(), we can't really tell whether the old or new function
will be executed the next time.

We really don't want this level of subtlety in workqueue interface for
such marginal use cases.  The previous patches converted all existing
users away from PREPARE_[DELAYED_]WORK().  Let's remove them.

Signed-off-by: Tejun Heo <tj@kernel.org>
Cc: Peter Hurley <peter@hurleysoftware.com>
Link: http://lkml.kernel.org/g/1392493119-9277-1-git-send-email-peter@hurleysoftware.com
include/linux/workqueue.h