Re: [discuss] Re: [PATCH for 2.6.18] [2/8] x86_64: On Intel systems when CPU has C3 don't use TSC

[Sergio Monteiro Basto <sergio@sergiomb.no-ip.org>]

[-- Attachment #1.1: Type: text/plain, Size: 2043 bytes --]

ok , here it is dmesg version 16 
I just add this printtk 
--- linux-2.6.17.x86_64/arch/x86_64/kernel/time.c.orig	2006-08-01
01:34:29.000000000 +0100
+++ linux-2.6.17.x86_64/arch/x86_64/kernel/time.c	2006-08-01 23:31:29.000000000 +0100
@@ -959,6 +959,9 @@ __cpuinit int unsynchronized_tsc(void)
 	   multi node systems */
  	if (boot_cpu_data.x86_vendor == X86_VENDOR_INTEL) {
 #ifdef CONFIG_ACPI
+		printk(KERN_INFO "time.c:SB acpi_fadt.length=%d, acpi_fadt.plvl3_lat=%d unsynchronized_tsc=%d\n",
+					            acpi_fadt.length, acpi_fadt.plvl3_lat,(acpi_fadt.length > 0 && acpi_fadt.plvl3_lat <
+									100) );
 		/* But TSC doesn't tick in C3 so don't use it there */
 		if (acpi_fadt.length > 0 && acpi_fadt.plvl3_lat < 100)
 			return 1;

but this time the function unsynchronized_tsc was just call one time.

time.c:SB acpi_fadt.length=0, acpi_fadt.plvl3_lat=0 unsynchronized_tsc=0

The patch works like I have use boot parameter notsc, which is better.
Now I don't need any parameter to boot.

Big Thanks 

On Wed, 2006-08-02 at 00:21 +0200, Andi Kleen wrote:
> On Wednesday 02 August 2006 00:06, Sergio Monteiro Basto wrote:
> > On Tue, 2006-08-01 at 23:56 +0200, Andi Kleen wrote:
> > > Lost timer ticks print a rip. Do you have some samples?
> 
> Can you send dmesg with the following patch applied too?
> 
> cc'ing Suresh because he might have an explanation too then.
> 
> -Andi
> 
> Index: linux-2.6.18-rc3-work/arch/x86_64/kernel/smpboot.c
> ===================================================================
> --- linux-2.6.18-rc3-work.orig/arch/x86_64/kernel/smpboot.c
> +++ linux-2.6.18-rc3-work/arch/x86_64/kernel/smpboot.c
> @@ -345,7 +345,7 @@ static void __cpuinit tsc_sync_wait(void
>  	 * mess up a possible perfect synchronization with a
>  	 * not-quite-perfect algorithm.
>  	 */
> -	if (notscsync || !cpu_has_tsc || !unsynchronized_tsc())
> +	if (0 && (notscsync || !cpu_has_tsc || !unsynchronized_tsc()))
>  		return;
>  	sync_tsc(0);
>  }
-- 
Sérgio M. B.

[-- Attachment #1.2: dmesg16.bz2 --]
[-- Type: application/x-bzip, Size: 7845 bytes --]

[-- Attachment #2: smime.p7s --]
[-- Type: application/x-pkcs7-signature, Size: 2166 bytes --]