Messages in this thread | | | Date | Sun, 17 Aug 2014 15:45:22 -0700 (PDT) | From | David Rientjes <> | Subject | Re: [PATCH] headers: remove fs.h from poll.h |
| |
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 xyz" or some other > > explanation. > > This particular patch doesn't change situation because it is so small and > removes only one header from one file. > > I can send bigger patch removing fs.h from many possible places. >
I extensively build tested your series before and reported the single point of failure. Once that's fixed and the patches have appropriate changelogs and signed-off-by lines then I can build test and push them for either Andrew or Linus.
| |