Re: [patch 3/8] Immediate Values - Kconfig menu in EMBEDDED

2007-09-12 Thread Andi Kleen
> Yeah, but then you would make immediate.o an obj-y (both for > kernel/Makefile and arch/*/kernel/Makefile) and it would be built > even on systems that would be configured not to use immediate values. You can stick a define into generic-immediate.h and put an ifndef for that around immediate.c

Re: [patch 3/8] Immediate Values - Kconfig menu in EMBEDDED

2007-09-11 Thread Mathieu Desnoyers
* Andi Kleen ([EMAIL PROTECTED]) wrote: > On Fri, Sep 07, 2007 at 08:46:48AM -0400, Mathieu Desnoyers wrote: > > * Andi Kleen ([EMAIL PROTECTED]) wrote: > > > > +config IMMEDIATE > > > > + default y if !DISABLE_IMMEDIATE > > > > > > It's still unclear to me why DISABLE_IMMEDIATE is needed. I

Re: [patch 3/8] Immediate Values - Kconfig menu in EMBEDDED

2007-09-07 Thread Andi Kleen
On Fri, Sep 07, 2007 at 08:46:48AM -0400, Mathieu Desnoyers wrote: > * Andi Kleen ([EMAIL PROTECTED]) wrote: > > > +config IMMEDIATE > > > + default y if !DISABLE_IMMEDIATE > > > > It's still unclear to me why DISABLE_IMMEDIATE is needed. It would > > be better to make it just the default. > > >

Re: [patch 3/8] Immediate Values - Kconfig menu in EMBEDDED

2007-09-07 Thread Mathieu Desnoyers
* Andi Kleen ([EMAIL PROTECTED]) wrote: > > +config IMMEDIATE > > + default y if !DISABLE_IMMEDIATE > > It's still unclear to me why DISABLE_IMMEDIATE is needed. It would > be better to make it just the default. > It is actually the default on any non embedded configuration. Do you think we sh

Re: [patch 3/8] Immediate Values - Kconfig menu in EMBEDDED

2007-09-06 Thread Andi Kleen
> +config IMMEDIATE > + default y if !DISABLE_IMMEDIATE It's still unclear to me why DISABLE_IMMEDIATE is needed. It would be better to make it just the default. > +config DISABLE_IMMEDIATE > + default y if EMBEDDED > + bool "Disable immediate values" if EMBEDDED > + depends on X8

[patch 3/8] Immediate Values - Kconfig menu in EMBEDDED

2007-09-06 Thread Mathieu Desnoyers
Immediate values provide a way to use dynamic code patching to update variables sitting within the instruction stream. It saves caches lines normally used by static read mostly variables. Enable it by default, but let users disable it through the EMBEDDED menu with the "Disable immediate values" su

[patch 3/8] Immediate Values - Kconfig menu in EMBEDDED

2007-08-27 Thread Mathieu Desnoyers
Immediate values provide a way to use dynamic code patching to update variables sitting within the instruction stream. It saves caches lines normally used by static read mostly variables. Enable it by default, but let users disable it through the EMBEDDED menu with the "Disable immediate values" su

[patch 3/8] Immediate Values - Kconfig menu in EMBEDDED

2007-08-20 Thread Mathieu Desnoyers
Immediate values provide a way to use dynamic code patching to update variables sitting within the instruction stream. It saves caches lines normally used by static read mostly variables. Enable it by default, but let users disable it through the EMBEDDED menu with the "Disable immediate values" su

[patch 3/8] Immediate Values - Kconfig menu in EMBEDDED

2007-08-12 Thread Mathieu Desnoyers
Immediate values provide a way to use dynamic code patching to update variables sitting within the instruction stream. It saves caches lines normally used by static read mostly variables. Enable it by default, but let users disable it through the EMBEDDED menu with the "Disable immediate values" su