Re: [PATCH] headers: remove fs.h from poll.h
From: David Rientjes
Date: Wed Jul 02 2014 - 19:09:29 EST
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:
> * make allmodconfig
> * make
> * touch include/linux/fs.h
> * make
> * [count how many files were rebuilt]
>
> After more header tweaking the number of files that were rebuilt dropped
> by ~2000 but code still compiles, which means fs.h inclusions were useless.
>
Lots of #ifdefs in fs.h, I hope your series includes building for all
possible combinations.
> This is first patch in series. Same thing we did with sched.h earlier.
>
Is it possible to post the whole series together so it makes sense and
it's possible to put it through some thorough build testing? If you can
point me to a git tree that I can pull, then I can do that on
infrastructure that I have probably quicker than most.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/