]> git.baikalelectronics.ru Git - kernel.git/commit
Revert "semaphore: fix"
authorLinus Torvalds <torvalds@linux-foundation.org>
Sun, 11 May 2008 03:43:22 +0000 (20:43 -0700)
committerLinus Torvalds <torvalds@linux-foundation.org>
Sun, 11 May 2008 03:43:22 +0000 (20:43 -0700)
commitb093ab3f1967a9d777f009a623f778899784214d
tree431f5019ce8dfd0f9c2b0e3816128dc4a34a49ff
parente0523af91388b45a645b8d5a24e282bc63735aef
Revert "semaphore: fix"

This reverts commit d54d3610453479f1270325ec3fe94a94e6ba738c, as it has
been reported to cause a regression with processes stuck in __down(),
apparently because some missing wakeup.

Quoth Sven Wegener:
 "I'm currently investigating a regression that has showed up with my
  last git pull yesterday.  Bisecting the commits showed bf726e
  "semaphore: fix" to be the culprit, reverting it fixed the issue.

  Symptoms: During heavy filesystem usage (e.g.  a kernel compile) I get
  several compiler processes in uninterruptible sleep, blocking all i/o
  on the filesystem.  System is an Intel Core 2 Quad running a 64bit
  kernel and userspace.  Filesystem is xfs on top of lvm.  See below for
  the output of sysrq-w."

See

http://lkml.org/lkml/2008/5/10/45

for full report.

In the meantime, we can just fix the BKL performance regression by
reverting back to the good old BKL spinlock implementation instead,
since any sleeping lock will generally perform badly, especially if it
tries to be fair.

Reported-by: Sven Wegener <sven.wegener@stealer.net>
Cc: Andrew Morton <akpm@linux-foundation.org>
Cc: Ingo Molnar <mingo@elte.hu>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
kernel/semaphore.c