Re: [PATCH 2/8 ] ltt for 2.6.10 : core headers

[Karim Yaghmour]

Hello Sam,

Sam Ravnborg wrote:
> include/linux/*.h is supposed to include only definitions used by other
> parts of the kernel.
> Definitions used only internally by ltt shall stay in kernel/
> 
> This is generally agreed upon, but not yet common practice.

Should there be a kernel/ltt-core.h or should I just put all required
definitions in kernel/ltt-core.c? The latter would result in a
cluttered C file, I think. Though there aren't any .h's in kernel/,
so I'm not sure what's the best way to proceed here.

> Btw. did you run it through sparse?
> Not that I found something, but did not see sparse annotation at first
> sight.

No I haven't. I've added it to my to-do list.

Karim
-- 
Author, Speaker, Developer, Consultant
Pushing Embedded and Real-Time Linux Systems Beyond the Limits
http://www.opersys.com || karim@xxxxxxxxxxx || 1-866-677-4546
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/