]> git.baikalelectronics.ru Git - kernel.git/commit
[PATCH] ia64: cpu_idle performance bug fix
authorChen, Kenneth W <kenneth.w.chen@intel.com>
Tue, 15 Nov 2005 22:37:05 +0000 (14:37 -0800)
committerLinus Torvalds <torvalds@g5.osdl.org>
Tue, 15 Nov 2005 23:50:51 +0000 (15:50 -0800)
commitb69afc7cf4d1ec8e506ec2221d39a1796dff7c06
tree0184e24ae1e9247b7dee62838b981c2d2a741d2e
parenta5d7377e9903ec381215012ed1dda3b3255e1ca6
[PATCH] ia64: cpu_idle performance bug fix

Our performance validation on 2.6.15-rc1 caught a disastrous performance
regression on ia64 with netperf (-98%) and volanomark (-58%) compares to
previous kernel version 2.6.14-git7.  See the following chart (result
group 1 & 2).

  http://kernel-perf.sourceforge.net/results.machine_id=26.html

We have root caused it to commit 6f38ee2ebe42ecaf9128ab8513358faa5a4b175e

This changeset broke the ia64 task resched notification.  In
sched.c:resched_task(), a reschedule IPI is conditioned upon
TIF_POLLING_NRFLAG.  However, the above changeset unconditionally set
the polling thread flag for idle tasks regardless whether pal_halt_light
is in use or not.  As a result, resched IPI is not sent from
resched_task().  And since the default behavior on ia64 is to use
pal_halt_light, we end up delaying the rescheduling task until next
timer tick, and thus cause the performance regression.

This fixes the performance bug.  I'm glad our performance suite is
turning up bad performance bug like this in time.

Signed-off-by: Ken Chen <kenneth.w.chen@intel.com>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
arch/ia64/kernel/process.c