Re: [PATCH -mm 1/5] Blackfin: blackfin architecture patch update

2007-03-21 Thread Paul Mundt
On Wed, Mar 21, 2007 at 11:44:27AM -0400, Mike Frysinger wrote: > On 3/5/07, Paul Mundt <[EMAIL PROTECTED]> wrote: > >> +int __init oprofile_arch_init(struct oprofile_operations *ops) > >> +{ > >> +#ifdef CONFIG_HARDWARE_PM > >[snip] > >> +#else > >> + return -1; > >> +#endif > >> +} > > >

Re: [PATCH -mm 1/5] Blackfin: blackfin architecture patch update

2007-03-21 Thread Mike Frysinger
On 3/5/07, Paul Mundt <[EMAIL PROTECTED]> wrote: > +int __init oprofile_arch_init(struct oprofile_operations *ops) > +{ > +#ifdef CONFIG_HARDWARE_PM [snip] > +#else > + return -1; > +#endif > +} Uh.. fix your dependencies. can you elaborate ? -mike - To unsubscribe from this list: send

Re: [PATCH -mm 1/5] Blackfin: blackfin architecture patch update

2007-03-21 Thread Mike Frysinger
On 3/5/07, Paul Mundt [EMAIL PROTECTED] wrote: +int __init oprofile_arch_init(struct oprofile_operations *ops) +{ +#ifdef CONFIG_HARDWARE_PM [snip] +#else + return -1; +#endif +} Uh.. fix your dependencies. can you elaborate ? -mike - To unsubscribe from this list: send the line

Re: [PATCH -mm 1/5] Blackfin: blackfin architecture patch update

2007-03-21 Thread Paul Mundt
On Wed, Mar 21, 2007 at 11:44:27AM -0400, Mike Frysinger wrote: On 3/5/07, Paul Mundt [EMAIL PROTECTED] wrote: +int __init oprofile_arch_init(struct oprofile_operations *ops) +{ +#ifdef CONFIG_HARDWARE_PM [snip] +#else + return -1; +#endif +} Uh.. fix your dependencies.

Re: [PATCH -mm 1/5] Blackfin: blackfin architecture patch update

2007-03-05 Thread Mike Frysinger
On 3/3/07, Arnd Bergmann <[EMAIL PROTECTED]> wrote: > +#if defined(CONFIG_MTD_UCLINUX) > + /* generic memory mapped MTD driver */ > + memory_mtd_end = memory_end; > + > + mtd_phys = _ramstart; > + mtd_size = PAGE_ALIGN(*((unsigned long *)(mtd_phys + 8))); > + > +# if

Re: [PATCH -mm 1/5] Blackfin: blackfin architecture patch update

2007-03-05 Thread Mike Frysinger
On 3/5/07, Paul Mundt <[EMAIL PROTECTED]> wrote: > +config DEBUG_HUNT_FOR_ZERO > + bool "Catch NULL pointer reads/writes" > + default y Is this sane to have conditional? yes ... with the ability to change the load address of the kernel, you can create a reserved chunk of memory for

Re: [PATCH -mm 1/5] Blackfin: blackfin architecture patch update

2007-03-05 Thread Robin Getz
On Mon 5 Mar 2007 12:32, Paul Mundt pondered: > On Mon, Mar 05, 2007 at 11:29:19AM -0500, Robin Getz wrote: > > On Mon 5 Mar 2007 09:00, Paul Mundt pondered: > > > Throwing this all at the user simply shows that the functions being > > > relocated haven't been profiled adequately with real

Re: [PATCH -mm 1/5] Blackfin: blackfin architecture patch update

2007-03-05 Thread Paul Mundt
On Mon, Mar 05, 2007 at 11:29:19AM -0500, Robin Getz wrote: > On Mon 5 Mar 2007 09:00, Paul Mundt pondered: > > Throwing this all at the user simply shows that the functions being > > relocated haven't been profiled adequately with real workloads. > > Actually - that is not true at all - we have

Re: [PATCH -mm 1/5] Blackfin: blackfin architecture patch update

2007-03-05 Thread Arnd Bergmann
On Monday 05 March 2007, Wu, Bryan wrote: > Maybe NUMA is a solution, but it is not a wonderful solution. NUMA doesn't help you. Linux only runs on cache-coherent NUMA, which this isn't. > In some application product, BF561 core A is running Linux kernel > +Applications while BF561 core B is

Re: [PATCH -mm 1/5] Blackfin: blackfin architecture patch update

2007-03-05 Thread Robin Getz
On Mon 5 Mar 2007 09:00, Paul Mundt pondered: > On Mon, Mar 05, 2007 at 08:26:56AM -0500, Robin Getz wrote: > > On Mon 5 Mar 2007 07:39, Paul Mundt pondered: > > > On Mon, Mar 05, 2007 at 01:32:07PM +0100, Bernd Schmidt wrote: > > > > That is what this does - it is just a easy to use knob. > >

Re: [PATCH -mm 1/5] Blackfin: blackfin architecture patch update

2007-03-05 Thread Paul Mundt
On Mon, Mar 05, 2007 at 08:26:56AM -0500, Robin Getz wrote: > On Mon 5 Mar 2007 07:39, Paul Mundt pondered: > > On Mon, Mar 05, 2007 at 01:32:07PM +0100, Bernd Schmidt wrote: > > > This is done intentionally, because it's also possible for user code to > > > be loaded into L1 memory.?? We want

Re: [PATCH -mm 1/5] Blackfin: blackfin architecture patch update

2007-03-05 Thread Robin Getz
On Mon 5 Mar 2007 07:39, Paul Mundt pondered: > On Mon, Mar 05, 2007 at 01:32:07PM +0100, Bernd Schmidt wrote: > > Paul Mundt wrote: > > >>+comment "Memory Optimizations" > > >>+ > > >>+config I_ENTRY_L1 > > >>+   bool "Locate interrupt entry code in L1 Memory" > > >>+   default y > > >>+  

Re: [PATCH -mm 1/5] Blackfin: blackfin architecture patch update

2007-03-05 Thread Bernd Schmidt
Paul Mundt wrote: +comment "Memory Optimizations" + +config I_ENTRY_L1 + bool "Locate interrupt entry code in L1 Memory" + default y + help + If enabled interrupt entry code (STORE/RESTORE CONTEXT) is linked + into L1 instruction memory.(less latency) + Wow,

Re: [PATCH -mm 1/5] Blackfin: blackfin architecture patch update

2007-03-05 Thread Paul Mundt
On Mon, Mar 05, 2007 at 01:32:07PM +0100, Bernd Schmidt wrote: > Paul Mundt wrote: > >>+comment "Memory Optimizations" > >>+ > >>+config I_ENTRY_L1 > >>+ bool "Locate interrupt entry code in L1 Memory" > >>+ default y > >>+ help > >>+ If enabled interrupt entry code (STORE/RESTORE

Re: [PATCH -mm 1/5] Blackfin: blackfin architecture patch update

2007-03-05 Thread Paul Mundt
On Thu, Mar 01, 2007 at 12:14:40PM +0800, Wu, Bryan wrote: > Here is the update version of blackfin-arch.patch in -mm tree. > simply add support to utrace and it was tested on blackfin STAMP board > as well as other following patches. > > The whole patch is located at URL: >

Re: [PATCH -mm 1/5] Blackfin: blackfin architecture patch update

2007-03-05 Thread Wu, Bryan
On Mon, 2007-03-05 at 09:47 +0100, Arnd Bergmann wrote: > On Monday 05 March 2007, Aubrey Li wrote: > > On 3/4/07, Arnd Bergmann <[EMAIL PROTECTED]> wrote: > > > > In general, please put EXPORT_SYMBOL lines below the definition > > > of the symbol itself. This list of exports should only be used

Re: [PATCH -mm 1/5] Blackfin: blackfin architecture patch update

2007-03-05 Thread Arnd Bergmann
On Monday 05 March 2007, Aubrey Li wrote: > On 3/4/07, Arnd Bergmann <[EMAIL PROTECTED]> wrote: > > In general, please put EXPORT_SYMBOL lines below the definition > > of the symbol itself. This list of exports should only be used > > for symbols that come from assembly files. > > What is the

Re: [PATCH -mm 1/5] Blackfin: blackfin architecture patch update

2007-03-05 Thread Arnd Bergmann
On Monday 05 March 2007, Wu, Bryan wrote: > So could please give us some information about the merge window > schedule, we may try to catch this. The merge window opens after 2.6.21 gets released and is open for two weeks aftre that. The idea is however that you have everything ready at the

Re: [PATCH -mm 1/5] Blackfin: blackfin architecture patch update

2007-03-05 Thread Arnd Bergmann
On Monday 05 March 2007, Wu, Bryan wrote: So could please give us some information about the merge window schedule, we may try to catch this. The merge window opens after 2.6.21 gets released and is open for two weeks aftre that. The idea is however that you have everything ready at the

Re: [PATCH -mm 1/5] Blackfin: blackfin architecture patch update

2007-03-05 Thread Arnd Bergmann
On Monday 05 March 2007, Aubrey Li wrote: On 3/4/07, Arnd Bergmann [EMAIL PROTECTED] wrote: In general, please put EXPORT_SYMBOL lines below the definition of the symbol itself. This list of exports should only be used for symbols that come from assembly files. What is the right way to

Re: [PATCH -mm 1/5] Blackfin: blackfin architecture patch update

2007-03-05 Thread Wu, Bryan
On Mon, 2007-03-05 at 09:47 +0100, Arnd Bergmann wrote: On Monday 05 March 2007, Aubrey Li wrote: On 3/4/07, Arnd Bergmann [EMAIL PROTECTED] wrote: In general, please put EXPORT_SYMBOL lines below the definition of the symbol itself. This list of exports should only be used for

Re: [PATCH -mm 1/5] Blackfin: blackfin architecture patch update

2007-03-05 Thread Paul Mundt
On Thu, Mar 01, 2007 at 12:14:40PM +0800, Wu, Bryan wrote: Here is the update version of blackfin-arch.patch in -mm tree. simply add support to utrace and it was tested on blackfin STAMP board as well as other following patches. The whole patch is located at URL:

Re: [PATCH -mm 1/5] Blackfin: blackfin architecture patch update

2007-03-05 Thread Paul Mundt
On Mon, Mar 05, 2007 at 01:32:07PM +0100, Bernd Schmidt wrote: Paul Mundt wrote: +comment Memory Optimizations + +config I_ENTRY_L1 + bool Locate interrupt entry code in L1 Memory + default y + help + If enabled interrupt entry code (STORE/RESTORE CONTEXT) is linked + into

Re: [PATCH -mm 1/5] Blackfin: blackfin architecture patch update

2007-03-05 Thread Bernd Schmidt
Paul Mundt wrote: +comment Memory Optimizations + +config I_ENTRY_L1 + bool Locate interrupt entry code in L1 Memory + default y + help + If enabled interrupt entry code (STORE/RESTORE CONTEXT) is linked + into L1 instruction memory.(less latency) + Wow, this

Re: [PATCH -mm 1/5] Blackfin: blackfin architecture patch update

2007-03-05 Thread Robin Getz
On Mon 5 Mar 2007 07:39, Paul Mundt pondered: On Mon, Mar 05, 2007 at 01:32:07PM +0100, Bernd Schmidt wrote: Paul Mundt wrote: +comment Memory Optimizations + +config I_ENTRY_L1 +   bool Locate interrupt entry code in L1 Memory +   default y +   help +     If enabled

Re: [PATCH -mm 1/5] Blackfin: blackfin architecture patch update

2007-03-05 Thread Paul Mundt
On Mon, Mar 05, 2007 at 08:26:56AM -0500, Robin Getz wrote: On Mon 5 Mar 2007 07:39, Paul Mundt pondered: On Mon, Mar 05, 2007 at 01:32:07PM +0100, Bernd Schmidt wrote: This is done intentionally, because it's also possible for user code to be loaded into L1 memory.?? We want to give

Re: [PATCH -mm 1/5] Blackfin: blackfin architecture patch update

2007-03-05 Thread Robin Getz
On Mon 5 Mar 2007 09:00, Paul Mundt pondered: On Mon, Mar 05, 2007 at 08:26:56AM -0500, Robin Getz wrote: On Mon 5 Mar 2007 07:39, Paul Mundt pondered: On Mon, Mar 05, 2007 at 01:32:07PM +0100, Bernd Schmidt wrote: That is what this does - it is just a easy to use knob. This is hardly

Re: [PATCH -mm 1/5] Blackfin: blackfin architecture patch update

2007-03-05 Thread Arnd Bergmann
On Monday 05 March 2007, Wu, Bryan wrote: Maybe NUMA is a solution, but it is not a wonderful solution. NUMA doesn't help you. Linux only runs on cache-coherent NUMA, which this isn't. In some application product, BF561 core A is running Linux kernel +Applications while BF561 core B is just

Re: [PATCH -mm 1/5] Blackfin: blackfin architecture patch update

2007-03-05 Thread Paul Mundt
On Mon, Mar 05, 2007 at 11:29:19AM -0500, Robin Getz wrote: On Mon 5 Mar 2007 09:00, Paul Mundt pondered: Throwing this all at the user simply shows that the functions being relocated haven't been profiled adequately with real workloads. Actually - that is not true at all - we have

Re: [PATCH -mm 1/5] Blackfin: blackfin architecture patch update

2007-03-05 Thread Robin Getz
On Mon 5 Mar 2007 12:32, Paul Mundt pondered: On Mon, Mar 05, 2007 at 11:29:19AM -0500, Robin Getz wrote: On Mon 5 Mar 2007 09:00, Paul Mundt pondered: Throwing this all at the user simply shows that the functions being relocated haven't been profiled adequately with real workloads.

Re: [PATCH -mm 1/5] Blackfin: blackfin architecture patch update

2007-03-05 Thread Mike Frysinger
On 3/5/07, Paul Mundt [EMAIL PROTECTED] wrote: +config DEBUG_HUNT_FOR_ZERO + bool Catch NULL pointer reads/writes + default y Is this sane to have conditional? yes ... with the ability to change the load address of the kernel, you can create a reserved chunk of memory for users to

Re: [PATCH -mm 1/5] Blackfin: blackfin architecture patch update

2007-03-05 Thread Mike Frysinger
On 3/3/07, Arnd Bergmann [EMAIL PROTECTED] wrote: +#if defined(CONFIG_MTD_UCLINUX) + /* generic memory mapped MTD driver */ + memory_mtd_end = memory_end; + + mtd_phys = _ramstart; + mtd_size = PAGE_ALIGN(*((unsigned long *)(mtd_phys + 8))); + +# if

Re: [PATCH -mm 1/5] Blackfin: blackfin architecture patch update

2007-03-04 Thread Wu, Bryan
On Sat, 2007-03-03 at 17:30 -0500, Arnd Bergmann wrote: > On Thursday 01 March 2007 05:14:40 Wu, Bryan wrote: > > Here is the update version of blackfin-arch.patch in -mm tree. > > simply add support to utrace and it was tested on blackfin STAMP > board > > as well as other following patches. > >

Re: [PATCH -mm 1/5] Blackfin: blackfin architecture patch update

2007-03-04 Thread Aubrey Li
On 3/4/07, Arnd Bergmann <[EMAIL PROTECTED]> wrote: On Thursday 01 March 2007 05:14:40 Wu, Bryan wrote: > Here is the update version of blackfin-arch.patch in -mm tree. > simply add support to utrace and it was tested on blackfin STAMP board > as well as other following patches. Wow, this has

Re: [PATCH -mm 1/5] Blackfin: blackfin architecture patch update

2007-03-04 Thread Aubrey Li
On 3/4/07, Arnd Bergmann [EMAIL PROTECTED] wrote: On Thursday 01 March 2007 05:14:40 Wu, Bryan wrote: Here is the update version of blackfin-arch.patch in -mm tree. simply add support to utrace and it was tested on blackfin STAMP board as well as other following patches. Wow, this has come a

Re: [PATCH -mm 1/5] Blackfin: blackfin architecture patch update

2007-03-04 Thread Wu, Bryan
On Sat, 2007-03-03 at 17:30 -0500, Arnd Bergmann wrote: On Thursday 01 March 2007 05:14:40 Wu, Bryan wrote: Here is the update version of blackfin-arch.patch in -mm tree. simply add support to utrace and it was tested on blackfin STAMP board as well as other following patches. Wow, this

Re: [PATCH -mm 1/5] Blackfin: blackfin architecture patch update

2007-03-03 Thread Arnd Bergmann
On Saturday 03 March 2007 23:50:02 bert hubert wrote: > >   for (;;) > >   asm volatile ("idle"); > > This looks remarkably like relax_cpu() Actually not: cpu_relax() is defined as barrier(), it can't call idle because that might make it sleep for a indefinite amount of time

Re: [PATCH -mm 1/5] Blackfin: blackfin architecture patch update

2007-03-03 Thread bert hubert
On Sat, Mar 03, 2007 at 11:30:49PM +0100, Arnd Bergmann wrote: > It might be nicer to make this > > for (;;) > asm volatile ("idle"); This looks remarkably like relax_cpu() -- http://www.PowerDNS.com Open source, database driven DNS Software http://netherlabs.nl

Re: [PATCH -mm 1/5] Blackfin: blackfin architecture patch update

2007-03-03 Thread Arnd Bergmann
On Thursday 01 March 2007 05:14:40 Wu, Bryan wrote: > Here is the update version of blackfin-arch.patch in -mm tree. > simply add support to utrace and it was tested on blackfin STAMP board > as well as other following patches. Wow, this has come a long way since I looked at the patches last

Re: [PATCH -mm 1/5] Blackfin: blackfin architecture patch update

2007-03-03 Thread Arnd Bergmann
On Thursday 01 March 2007 05:14:40 Wu, Bryan wrote: > The whole patch is located at URL: > https://blackfin.uclinux.org/gf/download/frsrelease/39/2583/blackfin-arch.p >atch The incremental patch is located at URL: > https://blackfin.uclinux.org/gf/download/frsrelease/39/2584/blackfin-arch-m

Re: [PATCH -mm 1/5] Blackfin: blackfin architecture patch update

2007-03-03 Thread Arnd Bergmann
On Thursday 01 March 2007 05:14:40 Wu, Bryan wrote: The whole patch is located at URL: https://blackfin.uclinux.org/gf/download/frsrelease/39/2583/blackfin-arch.p atch The incremental patch is located at URL: https://blackfin.uclinux.org/gf/download/frsrelease/39/2584/blackfin-arch-m

Re: [PATCH -mm 1/5] Blackfin: blackfin architecture patch update

2007-03-03 Thread Arnd Bergmann
On Thursday 01 March 2007 05:14:40 Wu, Bryan wrote: Here is the update version of blackfin-arch.patch in -mm tree. simply add support to utrace and it was tested on blackfin STAMP board as well as other following patches. Wow, this has come a long way since I looked at the patches last year,

Re: [PATCH -mm 1/5] Blackfin: blackfin architecture patch update

2007-03-03 Thread bert hubert
On Sat, Mar 03, 2007 at 11:30:49PM +0100, Arnd Bergmann wrote: It might be nicer to make this for (;;) asm volatile (idle); This looks remarkably like relax_cpu() -- http://www.PowerDNS.com Open source, database driven DNS Software http://netherlabs.nl

Re: [PATCH -mm 1/5] Blackfin: blackfin architecture patch update

2007-03-03 Thread Arnd Bergmann
On Saturday 03 March 2007 23:50:02 bert hubert wrote:   for (;;)   asm volatile (idle); This looks remarkably like relax_cpu() Actually not: cpu_relax() is defined as barrier(), it can't call idle because that might make it sleep for a indefinite amount of time (until the

[PATCH -mm 1/5] Blackfin: blackfin architecture patch update

2007-02-28 Thread Wu, Bryan
Hi folks, Here is the update version of blackfin-arch.patch in -mm tree. simply add support to utrace and it was tested on blackfin STAMP board as well as other following patches. The whole patch is located at URL: https://blackfin.uclinux.org/gf/download/frsrelease/39/2583/blackfin-arch.patch

[PATCH -mm 1/5] Blackfin: blackfin architecture patch update

2007-02-28 Thread Wu, Bryan
Hi folks, Here is the update version of blackfin-arch.patch in -mm tree. simply add support to utrace and it was tested on blackfin STAMP board as well as other following patches. The whole patch is located at URL: https://blackfin.uclinux.org/gf/download/frsrelease/39/2583/blackfin-arch.patch