Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S967962AbWK3XNx (ORCPT ); Thu, 30 Nov 2006 18:13:53 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S967967AbWK3XNx (ORCPT ); Thu, 30 Nov 2006 18:13:53 -0500 Received: from agminet01.oracle.com ([141.146.126.228]:25815 "EHLO agminet01.oracle.com") by vger.kernel.org with ESMTP id S967962AbWK3XNw (ORCPT ); Thu, 30 Nov 2006 18:13:52 -0500 Date: Thu, 30 Nov 2006 15:14:05 -0800 From: Randy Dunlap To: Andrew Morton Cc: Ingo Molnar , lkml , ak@suse.de Subject: Re: [PATCH -mm] x86_64 UP needs smp_call_function_single Message-Id: <20061130151405.d7e2dd08.randy.dunlap@oracle.com> In-Reply-To: <20061130142719.7474b4c0.akpm@osdl.org> References: <20061129170111.a0ffb3f4.randy.dunlap@oracle.com> <20061129174558.3dfd13df.akpm@osdl.org> <1164870000.11036.23.camel@earth> <20061130141140.a1b7d7cc.randy.dunlap@oracle.com> <20061130142719.7474b4c0.akpm@osdl.org> Organization: Oracle Linux Eng. X-Mailer: Sylpheed version 2.2.9 (GTK+ 2.8.10; x86_64-unknown-linux-gnu) Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-Brightmail-Tracker: AAAAAQAAAAI= X-Brightmail-Tracker: AAAAAQAAAAI= X-Whitelist: TRUE X-Whitelist: TRUE Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3473 Lines: 95 On Thu, 30 Nov 2006 14:27:19 -0800 Andrew Morton wrote: > On Thu, 30 Nov 2006 14:11:40 -0800 > Randy Dunlap wrote: > > > On Thu, 30 Nov 2006 08:00:00 +0100 Ingo Molnar wrote: > > > > > On Wed, 2006-11-29 at 17:45 -0800, Andrew Morton wrote: > > > > No, I think this patch is right - the declaration of the CONFIG_SMP > > > > smp_call_function_single() is in linux/smp.h so the !CONFIG_SMP > > > > declaration > > > > or definition should be there too. > > > > > > > > It's still buggy though. It should disable local interrupts around > > > > the > > > > call to match the SMP version. I'll fix that separately. > > > > > > hm, didnt i send an updated patch for that already? See the patch below, > > > from many days ago. I sent it after the tsc-sync-rewrite patch. > > > > Hi Ingo, > > > > Has there been a patch for this one? (UP again, not SMP) > > > > drivers/input/ff-memless.c:384: warning: implicit declaration of function 'local_bh_disable' > > drivers/input/ff-memless.c:393: warning: implicit declaration of function 'local_bh_enable' > > > > Thanks, > > --- > > ~Randy > > config: http://oss.oracle.com/~rdunlap/configs/config-input-up-header > > eww.. I guess linux/spinlock.h should really include linux/interrupt.h. > But interrupt.h includes stuff like sched.h which will want spinlock.h. > > This, maybe? Ack. Tested on UP and SMP x86_64. > include/linux/bottom_half.h | 5 +++++ > include/linux/interrupt.h | 7 +------ > include/linux/spinlock.h | 1 + > 3 files changed, 7 insertions(+), 6 deletions(-) > > diff -puN /dev/null include/linux/bottom_half.h > --- /dev/null > +++ a/include/linux/bottom_half.h > @@ -0,0 +1,5 @@ > +extern void local_bh_disable(void); > +extern void __local_bh_enable(void); > +extern void _local_bh_enable(void); > +extern void local_bh_enable(void); > +extern void local_bh_enable_ip(unsigned long ip); > diff -puN include/linux/interrupt.h~add-bottom_half.h include/linux/interrupt.h > --- a/include/linux/interrupt.h~add-bottom_half.h > +++ a/include/linux/interrupt.h > @@ -11,6 +11,7 @@ > #include > #include > #include > +#include > #include > #include > #include > @@ -217,12 +218,6 @@ static inline void __deprecated save_and > #define save_and_cli(x) save_and_cli(&x) > #endif /* CONFIG_SMP */ > > -extern void local_bh_disable(void); > -extern void __local_bh_enable(void); > -extern void _local_bh_enable(void); > -extern void local_bh_enable(void); > -extern void local_bh_enable_ip(unsigned long ip); > - > /* PLEASE, avoid to allocate new softirqs, if you need not _really_ high > frequency threaded job scheduling. For almost all the purposes > tasklets are more than enough. F.e. all serial device BHs et > diff -puN include/linux/spinlock.h~add-bottom_half.h include/linux/spinlock.h > --- a/include/linux/spinlock.h~add-bottom_half.h > +++ a/include/linux/spinlock.h > @@ -52,6 +52,7 @@ > #include > #include > #include > +#include > > #include --- ~Randy - 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/