Patchwork [2/2,LUCID] (pre-upstream) Fix leapsecond triggered hrtimer/futex load spike issue

login
register
mail settings
Submitter Brad Figg
Date July 3, 2012, 2:17 a.m.
Message ID <1341281876-28601-3-git-send-email-brad.figg@canonical.com>
Download mbox | patch
Permalink /patch/168670/
State New
Headers show

Comments

Brad Figg - July 3, 2012, 2:17 a.m.
From: John Stultz <john.stultz@linaro.org>

BugLink: http://bugs.launchpad.net/bugs/1020285

Backport for 2.6.32.59

As widely reported on the internet, some Linux systems after
the leapsecond was inserted are experiencing futex related load
spikes (usually connected to MySQL, Firefox, Thunderbird, Java, etc).

An apparent  workaround for this issue is running:
$ date -s "`date`"

Credit: http://www.sheeri.com/content/mysql-and-leap-second-high-cpu-and-fix

I believe this issue is due to the leapsecond being added without
calling clock_was_set() to notify the hrtimer subsystem of the
change. (Although I've not yet chased all the way down to the
hrtimer code to validate exactly what's going on there).

The workaround functions as it forces a clock_was_set()
call from settimeofday().

This fix adds the required clock_was_set() calls to where
we adjust for leapseconds.

NOTE: This fix *depends* on the previous fix, which allows
clock_was_set to be called from atomic context. Do not try
to apply just this patch.

CC: Prarit Bhargava <prarit@redhat.com>
CC: stable@vger.kernel.org
CC: Thomas Gleixner <tglx@linutronix.de>
Reported-by: Jan Engelhardt <jengelh@inai.de>
Signed-off-by: John Stultz <johnstul@us.ibm.com>
Signed-off-by: Brad Figg <brad.figg@canonical.com>
---
 kernel/time/timekeeping.c |    1 +
 1 file changed, 1 insertion(+)

Patch

diff --git a/kernel/time/timekeeping.c b/kernel/time/timekeeping.c
index 4a71cff..7e6753a 100644
--- a/kernel/time/timekeeping.c
+++ b/kernel/time/timekeeping.c
@@ -189,6 +189,7 @@  void timekeeping_leap_insert(int leapsecond)
 	xtime.tv_sec += leapsecond;
 	wall_to_monotonic.tv_sec -= leapsecond;
 	update_vsyscall(&xtime, timekeeper.clock, timekeeper.mult);
+	clock_was_set();
 }
 
 #ifdef CONFIG_GENERIC_TIME