login
Header Space

 
 

Re: [patch -mm 4/4] mempolicy: remove includes for duplicate headers

Score:
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: David Rientjes <rientjes@...>
Cc: <akpm@...>, <clameter@...>, <Lee.Schermerhorn@...>, <ak@...>, <linux-kernel@...>
Date: Thursday, March 6, 2008 - 4:19 pm

David R wrote:

I didn't see any problem offhand with the other patches 0 to 3 in this
set, but this patch 4/4 surprises me.

Perhaps I'm out of phase with what's customary in the kernel, but I
prefer directly including whatever headers I explicitly need.
Depending on some other header to drag in something you have explicit
need of anyway makes things more fragile.

-- 
                  I won't rest till it's the best ...
                  Programmer, Linux Scalability
                  Paul Jackson <pj@sgi.com> 1.940.382.4214
--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[patch -mm 1/4] mempolicy: move rebind functions, David Rientjes, (Thu Mar 6, 4:05 pm)
[PATCH] Mempolicy: fix parsing of tmpfs mpol mount option, Lee Schermerhorn, (Wed Mar 12, 3:33 pm)
Re: Regression: Re: [patch -mm 2/4] mempolicy: create mempo..., Lee Schermerhorn, (Mon Mar 10, 10:58 am)
[patch -mm 3/4] mempolicy: small header file cleanup, David Rientjes, (Thu Mar 6, 4:05 pm)
Re: [patch -mm 4/4] mempolicy: remove includes for duplicate..., Paul Jackson, (Thu Mar 6, 4:19 pm)
speck-geostationary