On Thu, May 31, 2007 at 02:07:53PM +0200, Andi Kleen wrote:
> > Yeah, that is a very sensible idea.
> >
> > > Possible further checks that might make sense:
> > > - panic() anywhere in drivers/*
> > > - externs in .c files without asmlinkage
> > > - general checking that everything in a fully visible {} block is the
> > > right
> > > indentation
> > >
>
> Here are some more warnings I would like to see:
>
> - Warning for any spinlock/mutex definition that doesn't have a comment
> nearby (all locks ought to be documented)
Also barriers. (Probably even moreso).
Dave
--
http://www.codemonkey.org.uk
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
