From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Jin, Gordon" Subject: Re: The rebasing taboo Date: Wed, 15 Sep 2010 08:22:23 +0800 Message-ID: References: <8u3s94$ggkp3g@orsmga002.jf.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mga03.intel.com (mga03.intel.com [143.182.124.21]) by gabe.freedesktop.org (Postfix) with ESMTP id 6D3489E76A for ; Tue, 14 Sep 2010 17:23:03 -0700 (PDT) In-Reply-To: <8u3s94$ggkp3g@orsmga002.jf.intel.com> Content-Language: en-US List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org Errors-To: intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org To: Chris Wilson , Dave Airlie Cc: "intel-gfx@lists.freedesktop.org" List-Id: intel-gfx@lists.freedesktop.org Chris Wilson wrote on Tuesday, September 14, 2010 8:39 PM: > As I was pulling together the key branches that I wanted to base > -next on, I made a critical error and included a broken version of a > patch that itself would not be useful until much later. > > The patch in question is the start of pipeline support for > pageflipping, ba3d8d74, but alas was broken and in the process of > fixing it I realised we had further bugs in the surrounding areas > (some of which would have been fixed by pipelined fence registers the > code for which the broken patch was a precursor). > > Daniel summed up the case for rebasing very well: > > 1. It is early in the stabilization phase. > > 2. The patch is clearly broken. > > 3. The history makes more sense if the patch comes in the with other > cleanups. > > What is the best course of action here? As I've yet to send the pull > request to Dave, from his point of view it should be clean to rewrite > our history (as far back as our previous branch point). The other > side to rebasing is that it breaks regression testing, and QA quite > rightly frowns on that. > -Chris Rebasing causes a little pain for the history, but it's not severe nightmare. We want to avoid it, but once it happens as unintentional incident, QA can accept it. Thanks for the notification. Gordon