From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ingo Molnar Subject: Re: linux-next: manual merge of the tip tree with Linus' tree Date: Wed, 21 May 2014 08:10:55 +0200 Message-ID: <20140521061055.GA6091@gmail.com> References: <20140521141249.6f87b7e7@canb.auug.org.au> <537C2A72.9030503@zytor.com> <20140521060100.GA5484@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from mail-ee0-f52.google.com ([74.125.83.52]:61591 "EHLO mail-ee0-f52.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750796AbaEUGLA (ORCPT ); Wed, 21 May 2014 02:11:00 -0400 Content-Disposition: inline In-Reply-To: <20140521060100.GA5484@gmail.com> Sender: linux-next-owner@vger.kernel.org List-ID: To: "H. Peter Anvin" Cc: Stephen Rothwell , Thomas Gleixner , Ingo Molnar , Peter Zijlstra , linux-next@vger.kernel.org, linux-kernel@vger.kernel.org, Linus * Ingo Molnar wrote: > > * H. Peter Anvin wrote: > > > On 05/20/2014 09:12 PM, Stephen Rothwell wrote: > > > Hi all, > > > > > > Today's linux-next merge of the tip tree got a conflict in > > > arch/x86/kernel/ldt.c between commit fa81511bb0bb ("x86-64, > > > modify_ldt: Make support for 16-bit segments a runtime option") > > > from Linus' tree and commit 34273f41d57e ("x86, espfix: Make it > > > possible to disable 16-bit support") from the tip tree. > > > > > > I fixed it up (see below) and can carry the fix as necessary (no > > > action is required). > > > > > > > This (and the previous one) is not the correct fix, although it will > > work. The correct fix is instead to completely revert fa81511bb0bb > > before merging in tip:x86/espfix. > > > > Sorry for the inconvenience. Linus generally doesn't like it when we > > fix up merges for him, or I'd set up a "clean" tip:x86/espfix branch. > > Please merge x86/urgent into x86/vdso while memories are still fresh > - fixing up conflicts between our own branches is entirely fine (I'm > doing it all the time to help the development flow) and it will make > life easier. > > What Linus dislikes most are merges between completely _unrelated_ > topic branches, especially if they cross maintenance domains. Also, 'pre pull request' conflict resolution merges are frowned upon mostly, as they tend to be of lower quality, come with little testing, and they also hide the various conflict interactions, many of which are canaries of bad development flow. Here the development flow is healthy: what conflicted was a quick and simple short term urgent fix for upstream which came through you, with the longer term fix coming in the next window, through you as well. Resolving those conflicts in the development branch, to make developer life easier, is pretty natural and I'd say is even considered good practice in a clean Git flow. Thanks, Ingo