Re: [PATCH] printk(): add KERN_CONT where needed

From: Joe Perches
Date: Tue Apr 03 2012 - 12:16:38 EST


On Tue, 2012-04-03 at 18:11 +0200, Kay Sievers wrote:
> On Tue, Apr 3, 2012 at 18:05, Joe Perches <joe@xxxxxxxxxxx> wrote:
> > On Tue, 2012-04-03 at 17:50 +0200, Kay Sievers wrote:
> >> I did not claim to address the problem of concurrent continuation line
> >> writers, and this patch has absolutely nothing to do with that
> >> problem. It _does_ fix encountered problems,
> >
> > No it doesn't. It fixes problems _you_ encounter
> > with an unpublished modification of the printk
> > subsystem.
>
> Exactly. And that is what is written in the changelog of the patch.
> But all that does not matter, the change results in more correct code
> than the current one is; and that is all that matters.
>
> Sure, I see your point, and support your effort, but I don't think
> your arguments are related to this patch and you are hijacking
> something unrelated, which should be discussed in a separate mail
> thread, that's all.

Conversations happen all over the place and making
separate discussions isn't that valuable nor are
they really controllable.

Just write the change log to simply state "add KERN_CONT"
without mentioning your unpublished stuff.

cheers, Joe

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