Re: [PATCH] headers: remove fs.h from poll.h

2014-08-17 Thread David Rientjes
On Sun, 17 Aug 2014, Alexey Dobriyan wrote: > > Doors this actually help anything? fs.h tends to get included other ways > > anyway, and the double inclusion protection means that the compiler doesn't > > even spend extra time on it.. > > > > IOW, I'd like to see "this improves compile times by x

Re: [PATCH] headers: remove fs.h from poll.h

2014-08-17 Thread Alexey Dobriyan
On Sat, Aug 16, 2014 at 06:41:35PM -0600, Linus Torvalds wrote: > Doors this actually help anything? fs.h tends to get included other ways > anyway, and the double inclusion protection means that the compiler doesn't > even spend extra time on it.. > > IOW, I'd like to see "this improves compile t

[PATCH] headers: remove fs.h from poll.h

2014-08-16 Thread Alexey Dobriyan
* remove fs.h from poll.h it is not necessary to include full header, * fixup compilation with additional includes where needed. Cross-compile tested on hundreds of configs across 15 archs. Signed-off-by: Alexey Dobriyan --- arch/m68k/bvme6000/rtc.c|1 + arch/m68k/mv

Re: [PATCH] headers: remove fs.h from poll.h

2014-08-04 Thread David Rientjes
On Tue, 29 Jul 2014, David Rientjes wrote: > On Sat, 5 Jul 2014, Alexey Dobriyan wrote: > > > On Wed, Jul 02, 2014 at 04:09:17PM -0700, David Rientjes wrote: > > > Lots of #ifdefs in fs.h, I hope your series includes building for all > > > possible combinations. > > > > If google can donate 1 m

Re: [PATCH] headers: remove fs.h from poll.h

2014-07-29 Thread David Rientjes
On Sat, 5 Jul 2014, Alexey Dobriyan wrote: > On Wed, Jul 02, 2014 at 04:09:17PM -0700, David Rientjes wrote: > > Lots of #ifdefs in fs.h, I hope your series includes building for all > > possible combinations. > > If google can donate 1 million of cpu hours to me, then yeah :-) > > > > This is

Re: [PATCH] headers: remove fs.h from poll.h

2014-07-05 Thread Alexey Dobriyan
On Wed, Jul 02, 2014 at 04:09:17PM -0700, David Rientjes wrote: > Lots of #ifdefs in fs.h, I hope your series includes building for all > possible combinations. If google can donate 1 million of cpu hours to me, then yeah :-) > > This is first patch in series. Same thing we did with sched.h earl

Re: [PATCH] headers: remove fs.h from poll.h

2014-07-02 Thread David Rientjes
On Wed, 2 Jul 2014, Alexey Dobriyan wrote: > fs.h inclusion is unnecessary because it is only needed > for "struct file" forward declaration. > > Those fs.h inclusions are fixing inevitable compilation failures. > There maybe more compilation failures. > > What I did to get 2000 number is: > * m

Re: [PATCH] headers: remove fs.h from poll.h

2014-07-01 Thread Alexey Dobriyan
On Tue, Jul 01, 2014 at 01:43:38PM -0700, Andrew Morton wrote: > On Tue, 1 Jul 2014 23:01:50 +0300 Alexey Dobriyan wrote: > > > fs.h is new sched.h! > > > > Start removing unnecessary fs.h inclusions. > > > > In the end of the whole series ~2000 useless fs.h inclusions are gone > > (just need m

Re: [PATCH] headers: remove fs.h from poll.h

2014-07-01 Thread Andrew Morton
On Tue, 1 Jul 2014 23:01:50 +0300 Alexey Dobriyan wrote: > fs.h is new sched.h! > > Start removing unnecessary fs.h inclusions. > > In the end of the whole series ~2000 useless fs.h inclusions are gone > (just need more comple testing). > This changelog is rather confusing. The patch adds lo

[PATCH] headers: remove fs.h from poll.h

2014-07-01 Thread Alexey Dobriyan
fs.h is new sched.h! Start removing unnecessary fs.h inclusions. In the end of the whole series ~2000 useless fs.h inclusions are gone (just need more comple testing). Signed-off-by: Alexey Dobriyan --- drivers/gpu/vga/vgaarb.c|1 + drivers/input/serio/serio_raw.c