On Fri, May 27, 2005 at 03:06:09AM +0200, Blaisorblade wrote: > On Friday 27 May 2005 02:38, [email protected] wrote: > > After porting this fixlet to UML: > > > > http://linux.bkbits.net:8080/linux-2.5/cset@41791ab52lfMuF2i3V-eTIGRBbDYKQ > > > > , I've also added a warning which should refuse compilation with insane > > values for PREEMPT_ACTIVE... maybe we should simply move PREEMPT_ACTIVE out > > of architectures using GENERIC_IRQS. > Ok, a grep shows that possible culprits (i.e. giving success to > grep GENERIC_HARDIRQS arch/*/Kconfig, and using 0x4000000 as PREEMPT_ACTIVE, > as given by grep PREEMPT_ACTIVE include/asm-*/thread_info.h) are (at a first > glance): frv, sh, sh64. > Yeah, that's bogus for sh and sh64 anyways, this should do it. It would be nice to move PRREMPT_ACTIVE so it isn't per-arch anymore, there's not many users that use a different value (at least for the ones using generic hardirqs, ia64 seems to be the only one?). Signed-off-by: Paul Mundt <[email protected]> include/asm-sh/thread_info.h: needs update include/asm-sh64/thread_info.h: needs update Index: include/asm-sh/thread_info.h =================================================================== --- 3ac9a34948049bff79a2b2ce49c0a3c84e35a748/include/asm-sh/thread_info.h (mode:100644) +++ uncommitted/include/asm-sh/thread_info.h (mode:100644) @@ -27,7 +27,7 @@ #endif -#define PREEMPT_ACTIVE 0x4000000 +#define PREEMPT_ACTIVE 0x10000000 /* * macros/functions for gaining access to the thread information structure Index: include/asm-sh64/thread_info.h =================================================================== --- 3ac9a34948049bff79a2b2ce49c0a3c84e35a748/include/asm-sh64/thread_info.h (mode:100644) +++ uncommitted/include/asm-sh64/thread_info.h (mode:100644) @@ -73,7 +73,7 @@ #define THREAD_SIZE 8192 -#define PREEMPT_ACTIVE 0x4000000 +#define PREEMPT_ACTIVE 0x10000000 /* thread information flags */ #define TIF_SYSCALL_TRACE 0 /* syscall trace active */
Attachment:
pgpMj3ASw9lE5.pgp
Description: PGP signature
- Follow-Ups:
- Re: [patch 4/8] irq code: Add coherence test for PREEMPT_ACTIVE
- From: Blaisorblade <[email protected]>
- Re: [patch 4/8] irq code: Add coherence test for PREEMPT_ACTIVE
- References:
- Prev by Date: Error in building modules into kernel.
- Next by Date: nfsroot question - laptop
- Previous by thread: Re: [patch 4/8] irq code: Add coherence test for PREEMPT_ACTIVE
- Next by thread: Re: [patch 4/8] irq code: Add coherence test for PREEMPT_ACTIVE
- Index(es):