From mboxrd@z Thu Jan 1 00:00:00 1970 From: Hollis Blanchard Subject: Re: linux-next: manual merge of the kvm tree Date: Mon, 22 Dec 2008 19:02:31 -0600 Message-ID: <1229994151.7181.24.camel@localhost.localdomain> References: <20081222145255.17f885f2.sfr@canb.auug.org.au> <1229922804.13001.84.camel@pasglop> <1229981967.7181.5.camel@localhost.localdomain> <1229982095.4360.19.camel@pasglop> <1229983422.7181.7.camel@localhost.localdomain> <18768.5298.173232.227168@cargo.ozlabs.ibm.com> <1229985853.7181.10.camel@localhost.localdomain> <18768.11194.175787.360371@cargo.ozlabs.ibm.com> Mime-Version: 1.0 Content-Type: text/plain Content-Transfer-Encoding: 7bit Return-path: Received: from e3.ny.us.ibm.com ([32.97.182.143]:51689 "EHLO e3.ny.us.ibm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752902AbYLWBCe (ORCPT ); Mon, 22 Dec 2008 20:02:34 -0500 Received: from d01relay02.pok.ibm.com (d01relay02.pok.ibm.com [9.56.227.234]) by e3.ny.us.ibm.com (8.13.1/8.13.1) with ESMTP id mBN11ZDa011000 for ; Mon, 22 Dec 2008 20:01:35 -0500 Received: from d01av01.pok.ibm.com (d01av01.pok.ibm.com [9.56.224.215]) by d01relay02.pok.ibm.com (8.13.8/8.13.8/NCO v9.1) with ESMTP id mBN12X6E175182 for ; Mon, 22 Dec 2008 20:02:33 -0500 Received: from d01av01.pok.ibm.com (loopback [127.0.0.1]) by d01av01.pok.ibm.com (8.12.11.20060308/8.13.3) with ESMTP id mBN12Wpf002191 for ; Mon, 22 Dec 2008 20:02:33 -0500 In-Reply-To: <18768.11194.175787.360371@cargo.ozlabs.ibm.com> Sender: linux-next-owner@vger.kernel.org List-ID: To: Paul Mackerras Cc: Benjamin Herrenschmidt , Stephen Rothwell , Avi Kivity , linux-next@vger.kernel.org, Kumar Gala , Josh Boyer On Tue, 2008-12-23 at 11:07 +1100, Paul Mackerras wrote: > Hollis Blanchard writes: > > > On Tue, 2008-12-23 at 09:29 +1100, Paul Mackerras wrote: > > > Hollis Blanchard writes: > > > > > > > > Patch is in Paulus -next already so we'll have to do a manual resolve. > > > > > > > > Who has to do it? My code should "win". > > > > > > Do you have your own tree or is all your stuff in the main kvm tree? > > > > > > If there is a powerpc KVM tree, I could pull that and resolve the > > > conflict, but I don't want to pull all the queued up KVM stuff. > > > > It's just the main KVM tree. > > > > > Worst case is Linus gets to resolve the conflict in the merge window. > > > If that's just a matter of him taking your version of the file, it > > > should be within his powers to manage. :) > > > > Yeah, that's what I meant by "win"... whatever changes Ben made are to > > now-dead code, so the code in kvm.git should be used. Does Linus need > > special instruction to do this, or how does that work? > > ppc44x_defconfig now fails to build in my tree because of _tlbil_all > being undefined (it has CONFIG_KVM=y), so I need some sort of fix. > What would happen if I made arch/powerpc/kvm/powerpc.c look like the > version from the KVM tree? It still wouldn't build, because it would require new functions (e.g. kvmppc_core_vcpu_put) to be defined in other files. I don't really know what to say, because from my point of view the offending code was removed a long time ago, so it shouldn't be causing these problems. In hindsight, what is the best solution in these situations? Maybe if Ben had talked to me we could have applied the exact same patch to powerpc.git and kvm.git at the same time, which would keep both trees building and still be automatically merged later. If you commit a new change to your tree that reverses only the arch/powerpc/kvm/powerpc.c part of Ben's patch, would that solve the merge problem? ppc44x_defconfig would still fail in the meantime though. If you fix the ppc44x_defconfig build, the merge problem is compounded. Either way I guess it is something that needs to be sorted out in powerpc.git, so you can pick which style of breakage you'd prefer? -- Hollis Blanchard IBM Linux Technology Center