Date: Tue, 14 Feb 2006 13:37:27 +0100
From: Ingo Molnar <mingo@elte.hu>
To: Heiko Carstens <heiko.carstens@de.ibm.com>
Cc: Olaf Hering <olh@suse.de>, Hannes Reinecke <hare@suse.de>,
       linux-kernel@vger.kernel.org, Andrew Morton <akpm@osdl.org>
Subject: Re: calibrate_migration_costs takes ages on s390
Message-ID: <20060214123727.GA5451@elte.hu>
References: <20060213102634.GA4677@osiris.boeblingen.de.ibm.com> <20060213104645.GA17173@elte.hu> <20060213234254.GA5368@suse.de> <20060214000807.GA6188@suse.de> <20060214080942.GC19896@osiris.boeblingen.de.ibm.com> <20060214105608.GD19896@osiris.boeblingen.de.ibm.com> <20060214123521.GB3138@elte.hu>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20060214123521.GB3138@elte.hu>
User-Agent: Mutt/1.4.2.1i
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 859
Lines: 26


* Ingo Molnar <mingo@elte.hu> wrote:

> * Heiko Carstens <heiko.carstens@de.ibm.com> wrote:
> 
> > --- a/arch/s390/lib/delay.c
> > +++ b/arch/s390/lib/delay.c
> > @@ -30,7 +30,7 @@ void __delay(unsigned long loops)
> >           */
> >          __asm__ __volatile__(
> >                  "0: brct %0,0b"
> > -                : /* no outputs */ : "r" (loops/2) );
> > +                : /* no outputs */ : "r" ((loops/2) + 1));
> >  }
> 
> ahh ... that explains the delays indeed!

just to make sure, i've checked all the other __delay() implementations 
in the kernel, and none seems to have such problems.

	Ingo
-
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/