Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752506AbdFNQKe (ORCPT ); Wed, 14 Jun 2017 12:10:34 -0400 Received: from mail-pf0-f172.google.com ([209.85.192.172]:34651 "EHLO mail-pf0-f172.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751624AbdFNQKc (ORCPT ); Wed, 14 Jun 2017 12:10:32 -0400 Date: Wed, 14 Jun 2017 09:10:24 -0700 From: Stephen Hemminger To: Jork Loeser Cc: Vitaly Kuznetsov , Stephen Hemminger , Haiyang Zhang , "x86@kernel.org" , "linux-kernel@vger.kernel.org" , Steven Rostedt , Andy Shevchenko , "Ingo Molnar" , Andy Lutomirski , "H. Peter Anvin" , "devel@linuxdriverproject.org" , Thomas Gleixner , "Greg Kroah-Hartman" Subject: Re: [PATCH v8 07/10] hyper-v: globalize vp_index Message-ID: <20170614091024.723bd7f4@xeon-e3> In-Reply-To: References: <20170609132736.13699-1-vkuznets@redhat.com> <20170609132736.13699-8-vkuznets@redhat.com> <20170613162143.71ca0cc8@xeon-e3> <8760fzxpxb.fsf@vitty.brq.redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2543 Lines: 55 On Wed, 14 Jun 2017 04:31:32 +0000 Jork Loeser wrote: > > From: Vitaly Kuznetsov [mailto:vkuznets@redhat.com] > > Sent: Tuesday, June 13, 2017 19:29 > > > > Stephen Hemminger writes: > > > > > On Fri, 9 Jun 2017 15:27:33 +0200 > > > Vitaly Kuznetsov wrote: > > > > > >> To support implementing remote TLB flushing on Hyper-V with a > > >> hypercall we need to make vp_index available outside of vmbus module. > > >> Rename and globalize. > > >> > > >> Signed-off-by: Vitaly Kuznetsov > > >> Reviewed-by: Andy Shevchenko > > > > > > This is correct, but needs to be rebased. > > > It conflicts with the PCI protocol version 1.2 patches that are in the > > > PCI tree. > > > > :-( > > > > The question is - what do we do? As far as I understand the intent was to push > > this through Greg's char-misc tree. If I rebase it to Bjorn's pci tree patches won't > > apply to char-misc and Greg won't take them. I see three possible ways to go: > > 1) Take them into char-misc and resolve the conflict in merge window (Linus will > > hate us all :-( ) > > 2) Ask Greg to merge with Bjorn _now_ so we can send the rebased version. > > 3) Postpone these patches to the next kernel release. No guarantee we won't > > clash with something else :-( > > > > So I'm a bit lost. With Hyper-V drivers scattered across multiple trees we're > > doomed to have such issues with every relatively big series. > > I would like to see Vitaly's patch-set being integrated shortly (option 1). > > In anticipation of this, the PCI protocol version 1.2 patches duplicate the CPU-ID/vCPU-ID mapping. The conflict thus is "just" a re-naming conflict - taking either old or new is fine (one occurrence of conflict). Is this acceptable for conflict management without instilling undue despise? > > That said, I am more than happy to help in the resolution. Also, once both changes are merged, I'll remove the duplicated logic. > > Regards, > Jork > There a few other options: 1) Work with Stephen to resolve merge conflict in linux-next. This means any conflict would get resolved before merge window 2) Figure out how to get enabling code in (maybe duplicate functions) and then delete the extra later. For example 1-6 could go in now. 3) Just wait. The hypercall patches are optimizations and could be deferred the pain with this is carrying more patches and managing the backlog gets to be a real nuisance.