Date: Wed, 20 Sep 2006 23:28:15 +0200
From: Willy Tarreau <w@1wt.eu>
To: Toyo Abe <toyoa@mvista.com>
Cc: Andi Kleen <ak@suse.de>, linux-kernel@vger.kernel.org
Subject: Re: [PATCH for 2.4] x86_64: Fix missing delay when the TSC counter just overflowed
Message-ID: <20060920212815.GA9397@1wt.eu>
References: <200609202143.k8KLhsxg007647@dhcp119.mvista.com>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <200609202143.k8KLhsxg007647@dhcp119.mvista.com>
User-Agent: Mutt/1.5.11
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1138
Lines: 39

On Wed, Sep 20, 2006 at 02:43:54PM -0700, Toyo Abe wrote:
> I'd seen a problem that *delay functions return in too short delay.
> It happens when the lower 32bit of TSC counter is overflowed.
> This patch fixes the problem. This is back-port of Andi Kleen's
> 2.6 fix.
> 
> http://www.kernel.org/git/?p=linux/kernel/git/tglx/history.git;a=commit;h=6c51e28ffbbebf49437ec63ac4f9e385d60827e5
> 
> Signed-off-by: Toyo Abe <toyoa@mvista.com>
> 
> ---
> 
>  arch/x86_64/lib/delay.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/arch/x86_64/lib/delay.c b/arch/x86_64/lib/delay.c
> index cc845d2..91345ee 100644
> --- a/arch/x86_64/lib/delay.c
> +++ b/arch/x86_64/lib/delay.c
> @@ -19,7 +19,7 @@ #endif
>  
>  void __delay(unsigned long loops)
>  {
> -	unsigned long bclock, now;
> +	unsigned bclock, now;
>  	
>  	rdtscl(bclock);
>  	do

Queued.

Thanks Toyo,
Willy

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/