Re: [PATCH] Remove fs.h from mm.h

2007-07-30 Thread Paul Mundt
On Mon, Jul 30, 2007 at 02:36:13AM +0400, Alexey Dobriyan wrote: > 0) Remove fs.h from mm.h. For this, > 1) Uninline vma_wants_writenotify(). It's pretty huge anyway. > 2) Add back fs.h or less bloated headers (err.h) to files that need it. > sh ended up breaking all over the place, and sh64 in a

Re: [PATCH] Remove fs.h from mm.h

2007-07-30 Thread Bryan Wu
On Mon, 2007-07-30 at 23:43 +0400, Alexey Dobriyan wrote: > On Mon, Jul 30, 2007 at 12:42:07PM +0800, Bryan Wu wrote: > > Can I do something to help this regression testing? > > > > Please feel free to ask me. > > Sorry, blackfin toolchain doesn't like me, so I can't test this myself. > Check curr

Re: [PATCH] Remove fs.h from mm.h

2007-07-30 Thread Hirokazu Takata
From: Alexey Dobriyan <[EMAIL PROTECTED]> > Apologies. I just finished adding m32r to my cross-build setup, so there > shouldn't be any more such breakages. Many thanks for your erabolation of cross-build check. -- Takata - To unsubscribe from this list: send the line "unsubscribe linux-kernel" i

Re: [PATCH] Remove fs.h from mm.h

2007-07-30 Thread Mike Frysinger
On 7/30/07, Alexey Dobriyan <[EMAIL PROTECTED]> wrote: > On Sun, Jul 29, 2007 at 05:08:20PM -0700, Linus Torvalds wrote: > > On Mon, 30 Jul 2007, Alexey Dobriyan wrote: > > > > > > Cross-compile tested without regressions on my two usual configs and > > > (sigh): > > > > > > alpha arm

Re: [PATCH] Remove fs.h from mm.h

2007-07-30 Thread Alexey Dobriyan
On Mon, Jul 30, 2007 at 10:00:47PM +0900, Hirokazu Takata wrote: > This change breaks m32r, too. Apologies. I just finished adding m32r to my cross-build setup, so there shouldn't be any more such breakages. - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of

Re: [PATCH] Remove fs.h from mm.h

2007-07-30 Thread Alexey Dobriyan
On Sun, Jul 29, 2007 at 05:08:20PM -0700, Linus Torvalds wrote: > On Mon, 30 Jul 2007, Alexey Dobriyan wrote: > > > > Cross-compile tested without regressions on my two usual configs and (sigh): > > > > alpha arm-mx1adsmips-bigsur powerpc-ebony > .. > > Heh. > > K

Re: [PATCH] Remove fs.h from mm.h

2007-07-30 Thread Hirokazu Takata
Hi, Alexery, This change breaks m32r, too. Don't forget updating the other archs, please. From: Linus Torvalds <[EMAIL PROTECTED]> Subject: Re: [PATCH] Remove fs.h from mm.h Date: Sun, 29 Jul 2007 17:08:20 -0700 (PDT) > > On Mon, 30 Jul 2007, Alexey Dobriyan wrote: > > &

Re: [PATCH] Remove fs.h from mm.h

2007-07-29 Thread Bryan Wu
On Mon, 2007-07-30 at 08:08 +0800, Linus Torvalds wrote: > > > On Mon, 30 Jul 2007, Alexey Dobriyan wrote: > > > > Cross-compile tested without regressions on my two usual configs and > (sigh): > > > > alpha arm-mx1adsmips-bigsur > powerpc-ebony > .. > > Heh. > > Ku

Re: [PATCH] Remove fs.h from mm.h

2007-07-29 Thread Linus Torvalds
On Mon, 30 Jul 2007, Alexey Dobriyan wrote: > > Cross-compile tested without regressions on my two usual configs and (sigh): > > alpha arm-mx1adsmips-bigsur powerpc-ebony .. Heh. Kudos for going above and beyond. "But where is blackfin and frv?" Thanks

[PATCH] Remove fs.h from mm.h

2007-07-29 Thread Alexey Dobriyan
0) Remove fs.h from mm.h. For this, 1) Uninline vma_wants_writenotify(). It's pretty huge anyway. 2) Add back fs.h or less bloated headers (err.h) to files that need it. As result, on x86_64 allyesconfig, fs.h dependencies cut down from 3929 files rebuilt down to 3444 (-12.3%). Cross-compile test