linux-kernel - Re: [PATCH] x86 uv: lower UV rtc clocksource rating

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20120217141641.GA28063@sgi.com>
Date:	Fri, 17 Feb 2012 08:16:41 -0600
From:	Dimitri Sivanich <sivanich@....com>
To:	Ingo Molnar <mingo@...e.hu>
Cc:	Thomas Gleixner <tglx@...utronix.de>,
	Ingo Molnar <mingo@...hat.com>,
	"H. Peter Anvin" <hpa@...or.com>,
	John Stultz <johnstul@...ibm.com>, x86@...nel.org,
	linux-kernel@...r.kernel.org, Jack Steiner <steiner@....com>
Subject: Re: [PATCH] x86 uv: lower UV rtc clocksource rating

On Fri, Feb 17, 2012 at 01:02:42PM +0100, Ingo Molnar wrote:
> 
> * Dimitri Sivanich <sivanich@....com> wrote:
> 
> > Lower the rating of the UV rtc clocksource to just below that of the tsc.
> 
> This changelog is deficient for obvious reasons, mind updating 
> it?
>
OK.  Is this sufficient?


Lower the rating of the UV rtc clocksource to just below that of the tsc.

Reading the tsc clocksource has lower latency than reading the rtc, so favor
it in situations where it is synchronized and stable.  When the tsc is
unsynchronized, the rtc needs to be the chosen clocksource.

Signed-off-by: Dimitri Sivanich <sivanich@....com>
---
 arch/x86/platform/uv/uv_time.c |    6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

Index: linux/arch/x86/platform/uv/uv_time.c
===================================================================
--- linux.orig/arch/x86/platform/uv/uv_time.c
+++ linux/arch/x86/platform/uv/uv_time.c
@@ -37,7 +37,7 @@ static void uv_rtc_timer_setup(enum cloc
 
 static struct clocksource clocksource_uv = {
 	.name		= RTC_NAME,
-	.rating		= 400,
+	.rating		= 299,
 	.read		= uv_read_rtc,
 	.mask		= (cycle_t)UVH_RTC_REAL_TIME_CLOCK_MASK,
 	.flags		= CLOCK_SOURCE_IS_CONTINUOUS,
@@ -379,10 +379,6 @@ static __init int uv_rtc_setup_clock(voi
 	if (!is_uv_system())
 		return -ENODEV;
 
-	/* If single blade, prefer tsc */
-	if (uv_num_possible_blades() == 1)
-		clocksource_uv.rating = 250;
-
 	rc = clocksource_register_hz(&clocksource_uv, sn_rtc_cycles_per_second);
 	if (rc)
 		printk(KERN_INFO "UV RTC clocksource failed rc %d\n", rc);
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/