[lkml]   [2002]   [Jan]   [14]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [RFC][PATCH] cleanup file.h and INIT_TASK a bit

On Mon, 14 Jan 2002, Benjamin LaHaise wrote:

> Resend. Several people pointed out they like the cleanup, nobody complained.
> On Sun, Jan 13, 2002 at 06:59:47PM -0500, Benjamin LaHaise wrote:
> > This patches does a couple of things: first off, it removes the sched.h
> > include from file.h that was added recently, as we really don't need yet
> > another include file chain mess. To make this a bit more palatable, a
> > few of the inlines are moved out of file.h and into fcntl.c, plus the
> > files_struct is moved to file.h from sched.h. Since this meant adding
> > file.h to the various arch/*/kernel/init_task.c files, I took the time
> > to move the INIT_* bits for initializing the init task out of sched.h
> > and into init_task.h. If this is okay, please apply the patch. There
> > are other cleanups to do if people are interested: the #define for init_task
> > is currently duplicated in *all* asm-*/processor.h files to be exactly
> > the same thing... This is a way of testing the waters on include file
> > cleanups. Done properly, they shave ~10-15% off of the kernel compile
> > time on my machine.
> >
> > Oh, the file.h cleanup exposed a mess (bug): usb.c was duplicating code
> > from daemonize().

Please, split it in two chunks - init_task.h stuff and everything else.
The former is obviously good thing, the latter may need more - I understand
why you do the #define tricks, but I'm less than sure that it's the best

As for the USB... There is a bunch of other places that should do
daemonize() - I'll dig the patch out and send it.

To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to
More majordomo info at
Please read the FAQ at

 \ /
  Last update: 2005-03-22 13:22    [W:0.649 / U:3.488 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site