]> git.baikalelectronics.ru Git - kernel.git/commit
Revert "cgroup: use an ordered workqueue for cgroup destruction"
authorTejun Heo <tj@kernel.org>
Thu, 13 Feb 2014 00:06:19 +0000 (19:06 -0500)
committerTejun Heo <tj@kernel.org>
Thu, 13 Feb 2014 00:08:28 +0000 (19:08 -0500)
commitfcea9f98dfae574526f250a1e2b589e3b2879ab5
treef7d31231685d2c52af4804d265956660f0f849fb
parent144e7ec5f1718ea39b7bc1c248d4b1114133a584
Revert "cgroup: use an ordered workqueue for cgroup destruction"

This reverts commit df33e165a96df2794d7145a28726e51b60278c06.
Explanation from Hugh:

  It's because more thorough testing, by others here, found that it
  wasn't always solving the problem: so I asked Tejun privately to
  hold off from sending it in, until we'd worked out why not.

  Most of our testing being on a v3,11-based kernel, it was perfectly
  possible that the problem was merely our own e.g. missing Tejun's
  3bf7c752e88e ("workqueue: fix ordered workqueues in NUMA setups").

  But that turned out not to be enough to fix it either. Then Filipe
  pointed out how percpu_ref_kill_and_confirm() uses call_rcu_sched()
  before we ever get to put the offline on to the workqueue: by the
  time we get to the workqueue, the ordering has already been lost.

  So, thanks for the Acks, but I'm afraid that this ordered workqueue
  solution is just not good enough: we should simply forget that patch
  and provide a different answer."

Signed-off-by: Tejun Heo <tj@kernel.org>
Cc: Hugh Dickins <hughd@google.com>
kernel/cgroup.c