Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1030708AbWI0TjB (ORCPT ); Wed, 27 Sep 2006 15:39:01 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1030703AbWI0TjA (ORCPT ); Wed, 27 Sep 2006 15:39:00 -0400 Received: from ns2.suse.de ([195.135.220.15]:32434 "EHLO mx2.suse.de") by vger.kernel.org with ESMTP id S1030682AbWI0Ti7 (ORCPT ); Wed, 27 Sep 2006 15:38:59 -0400 To: Kyle McMartin Cc: linux-kernel@vger.kernel.org, linux-acpi@vger.kernel.org, akpm@osdl.org, torvalds@osdl.org Subject: Re: [BUG] Oops on boot (probably ACPI related) References: <200609271424.47824.eike-kernel@sf-tec.de> <20060927184037.GA3306@athena.road.mcmartin.ca> From: Andi Kleen Date: 27 Sep 2006 21:38:56 +0200 In-Reply-To: <20060927184037.GA3306@athena.road.mcmartin.ca> Message-ID: User-Agent: Gnus/5.09 (Gnus v5.9.0) Emacs/21.3 MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1692 Lines: 51 Kyle McMartin writes: > On Wed, Sep 27, 2006 at 07:56:13PM +0200, Markus Dahms wrote: > > > I get this on my machine. SMP kernel, linus git from this morning. .config > > > and test available on request. > > > > I saw this as well. > > Reverting, > > i386: Remove lock section support in semaphore.h > > Fixes it for me (and apparently akpm too from Message-Id: > <20060926224114.5ca873ec.akpm@osdl.org>) > > Linus, please revert 01215ad8d83e18321d99e9b5750a6f21cac243a2 for now... I expect this patch to fix it. -Andi i386: Use early clobbers for semaphores now The new code does clobber the result early, so make sure to tell gcc to not put it into the same register as a input argument Signed-off-by: Andi Kleen Index: linux/include/asm-i386/semaphore.h =================================================================== --- linux.orig/include/asm-i386/semaphore.h +++ linux/include/asm-i386/semaphore.h @@ -126,7 +126,7 @@ static inline int down_interruptible(str "lea %1,%%eax\n\t" "call __down_failed_interruptible\n" "2:" - :"=a" (result), "+m" (sem->count) + :"=&a" (result), "+m" (sem->count) : :"memory"); return result; @@ -148,7 +148,7 @@ static inline int down_trylock(struct se "lea %1,%%eax\n\t" "call __down_failed_trylock\n\t" "2:\n" - :"=a" (result), "+m" (sem->count) + :"=&a" (result), "+m" (sem->count) : :"memory"); return result; - 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/