Date: Sun, 29 Mar 2009 13:46:57 +0200 (CEST)
From: Thomas Gleixner <tglx@linutronix.de>
To: Jaswinder Singh Rajput <jaswinder@kernel.org>
cc: Ingo Molnar <mingo@elte.hu>, Hugh Dickins <hugh@veritas.com>,
       x86 maintainers <x86@kernel.org>, LKML <linux-kernel@vger.kernel.org>
Subject: Re: [git-pull -tip] x86: include inverse Xmas tree patches
In-Reply-To: <1238308734.3074.3.camel@ht.satnam>
Message-ID: <alpine.LFD.2.00.0903291345300.3397@localhost.localdomain>
References: <1238251730.2526.2.camel@ht.satnam>  <Pine.LNX.4.64.0903281508410.21235@blonde.anvils>  <20090328234012.GC4957@elte.hu> <1238308734.3074.3.camel@ht.satnam>
User-Agent: Alpine 2.00 (LFD 1167 2008-08-23)
MIME-Version: 1.0
Content-Type: TEXT/PLAIN; charset=US-ASCII
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1309
Lines: 34

On Sun, 29 Mar 2009, Jaswinder Singh Rajput wrote:
> On Sun, 2009-03-29 at 00:40 +0100, Ingo Molnar wrote:
> > * Hugh Dickins <hugh@veritas.com> wrote:
> > > 
> > > Please find a better use for your time.
> > > 
> > > Imagine, there might be developers out there with real work
> > > to do in these files: all you are doing is throwing silly
> > > little obstacles in their way.
> > 
> > Actually, now is the right time to do such changes as we've just 
> > pushed all pending x86 changes upstream. And since these changes 
> > reduce the chance of future patch conflicts, they do help 
> > development. (in a small way)
> > 
> > Nevertheless i'd suggest to make this a single commit, to not 
> > clobber the shortlog with it.
> > 
> 
> I think individual patches will be better option, in case of unavoidable
> situation we can simply revert that particular patch.

What unavoidable situation ? That you broke stuff with your changes ?
If you are so unsure about the quality of your changes, then the best
option is to not pull that stuff at all.

Thanks,

	tglx
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/