Re: [discuss] Re: [PATCH] [6/6] i386: Move CONFIG_DOUBLEFAULT into arch/i386 where it belongs.

From: Adrian Bunk
Date: Thu Apr 20 2006 - 09:47:49 EST


On Thu, Apr 20, 2006 at 03:26:21PM +0200, Andi Kleen wrote:
> On Thursday 20 April 2006 13:49, Adrian Bunk wrote:
>
> > My main problem with his patch is still the way he did it - sending a
> > patch reverting a recently included patch with neither discussion before
> > the patch nor mentioning in the patch that it's a revert nor a Cc to the
> > people involved with the patch.
>
> I just noticed a problem (bogus symbols in my x86-64 config) and fixed it. I normally
> don't look at which patch it introduced for such trivial changes.

First of all, the "problem" of an unset config variable in the .config
is at most a cosmetical issue.

And even if it had been a real problem, discussing such an issue and
convincing people why you consider it to be bad is always better, since
whoever gets conviced in the situation will have learned for similar
future cases.

There are ways how things should work and we should follow them.

As an example, there is one trivial x86_64 specific patch removing the
small bloat caused by an unused export I made in -mm Andrew has
forwarded at least twice to you.

Until now, I'd have considered it an unfriendly act to forward such a
patch to Linus through my trivial tree bypassing you as the maintainer,
but looking at your statement it seems to be OK for you if I send such
trivial patches directly to Linus without bothering to go through you as
the maintainer.

> -Andi

cu
Adrian

--

"Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
"Only a promise," Lao Er said.
Pearl S. Buck - Dragon Seed

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