From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Miller Subject: Re: [GIT] Networking Date: Thu, 16 Nov 2017 10:43:18 +0900 (KST) Message-ID: <20171116.104318.1816177872306749082.davem@davemloft.net> References: <20171115.203352.827797930375567595.davem@davemloft.net> Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit Cc: torvalds@linux-foundation.org, mingo@kernel.org, peterz@infradead.org, akpm@linux-foundation.org, netdev@vger.kernel.org, linux-kernel@vger.kernel.org, ast@kernel.org To: daniel@iogearbox.net Return-path: In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org List-Id: netdev.vger.kernel.org From: Daniel Borkmann Date: Wed, 15 Nov 2017 23:15:20 +0100 > On 11/15/2017 09:19 PM, Linus Torvalds wrote: >> On Wed, Nov 15, 2017 at 3:33 AM, David Miller wrote: >>> >>> Highlights: >> >> Lowlights: >> >> 1) it duplicated a commit from the hrtimer tree, which had been >> cleaned up and rewritten, but then merging the second copy of the >> commit re-introduced the bad code that had been cleaned up. >> >> I'm talking about commits >> >> - 7d9285e82db5: >> perf/bpf: Extend the perf_event_read_local() interface, a.k.a. >> "bpf: perf event change needed for subsequent bpf helpers" >> - 97562633bcba >> bpf: perf event change needed for subsequent bpf helpers >> >> where apparently there was no discussion between the groups about the >> subsequent changes. >> >> And this must have shown up in linux-next as a conflict, but no >> mention of it from either the perf event tree or the networking tree >> merge. >> >> Although it is of course possible that depending on merge order, the >> problem never showed up in next. > > Sorry about that, it was discussed that the patch in [1] would get > routed through net-next and again cherry-picked from tracing folks > due to conflicting changes in perf event tree that were being worked > on to avoid later merge conflicts - clearly that didn't give the > desired result. > > There was a subsequent discussion in [2] but not sure if cherry-picking > 0d3d73aac2ff ("perf/core: Rewrite event timekeeping") into net-next > would have made it better or worse. We'll have a bpf sub-tree up and > running soon for the next development cycle that can be pulled from > by different parties when needed; potentially this could reduce such > conflicts between trees in future. Sorry for the trouble. Yeah, sorry about all of this. I had hoped that since the patch was being appied to both trees in order to avoid merge problems, no modifications would have been made to the change at either end. This obviously didn't happen. I also didn't communicate the issue to you clearly in the pull request, and for this I apologize. As Daniel says, we realize that bpf is breaching multiple subsystems more and more so over time, and we hope a bpf GIT tree will help alleviate this moving forward. Thanks!