Re: ktap inclusion in drivers/staging/?

From: Ingo Molnar
Date: Fri Oct 25 2013 - 06:10:17 EST



* Linus Torvalds <torvalds@xxxxxxxxxxxxxxxxxxxx> wrote:

> On Thu, Oct 24, 2013 at 9:46 AM, Steven Rostedt <rostedt@xxxxxxxxxxx> wrote:
> >
> > But I agree, this should not be just dumped into the staging
> > tree until the patches themselves have been posted and reviewed.
>
> Btw, it's not just the commit history. The actual file layout is
> terminally horrible too. The actual LWN article made it look like
> ktap was just a user-space tool, and I was thinking that it was
> like tools/pert/, just in staging.
>
> But looking at the tree, it looks like parts of it is a kernel
> module, and parts of it is the user space thing, and it's totally
> impossible to see which is which, it's just all mixed up in the
> same directory structure.
>
> Maybe I misunderstood, but that was my reaction from a very quick
> look.

Yes, that's what I saw as well. I'd be less worried about it all if
it was tooling alone, but this is actually mostly kernel side code,
which was not apparent at all to me either, from the structure of
it.

Thanks,

Ingo
--
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/