lkml.org 
[lkml]   [2014]   [Jul]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] headers: remove fs.h from poll.h
On Tue, Jul 01, 2014 at 01:43:38PM -0700, Andrew Morton wrote:
> On Tue, 1 Jul 2014 23:01:50 +0300 Alexey Dobriyan <adobriyan@gmail.com> 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 lots *more* fs.h inclusions. I *think* the intent is to
> remove one nested inclusion in poll.h. But that contradicts your third
> paragraph.
>
> Help us out here, OK?

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.

This is first patch in series. Same thing we did with sched.h earlier.

Alexey


\
 
 \ /
  Last update: 2014-07-02 05:21    [W:1.798 / U:0.012 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site