From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753982AbdEDCKD (ORCPT ); Wed, 3 May 2017 22:10:03 -0400 Received: from smtprelay0053.hostedemail.com ([216.40.44.53]:41688 "EHLO smtprelay.hostedemail.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1753474AbdEDCJ4 (ORCPT ); Wed, 3 May 2017 22:09:56 -0400 X-Session-Marker: 726F737465647440676F6F646D69732E6F7267 X-Spam-Summary: 2,0,0,,d41d8cd98f00b204,rostedt@goodmis.org,:::::::,RULES_HIT:41:355:379:541:599:800:960:968:973:982:988:989:1260:1277:1311:1313:1314:1345:1359:1437:1515:1516:1518:1534:1541:1593:1594:1711:1730:1747:1777:1792:2198:2199:2393:2553:2559:2562:2693:3138:3139:3140:3141:3142:3354:3622:3865:3866:3867:3868:3870:3871:3872:3873:3874:4605:5007:6261:7875:7903:10004:10400:10848:10967:11232:11658:11914:12043:12266:12294:12438:12555:12663:12740:12760:12895:13069:13161:13229:13311:13357:13439:14096:14097:14181:14659:14721:14777:21080:21451:21627:30012:30054:30083:30090:30091,0,RBL:none,CacheIP:none,Bayesian:0.5,0.5,0.5,Netcheck:none,DomainCache:0,MSF:not bulk,SPF:,MSBL:0,DNSBL:none,Custom_rules:0:0:0,LFtime:2,LUA_SUMMARY:none X-HE-Tag: watch75_733b41f841d5e X-Filterd-Recvd-Size: 2915 Date: Wed, 3 May 2017 22:09:50 -0400 From: Steven Rostedt To: Linus Torvalds Cc: LKML , Ingo Molnar , Andrew Morton Subject: Re: [GIT PULL] tracing: Updates for v4.12 Message-ID: <20170503220950.45cf3c6f@grimm.local.home> In-Reply-To: References: <20170502194152.434d1b4a@gandalf.local.home> X-Mailer: Claws Mail 3.14.1 (GTK+ 2.24.31; x86_64-pc-linux-gnu) 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 On Wed, 3 May 2017 18:59:05 -0700 Linus Torvalds wrote: > On Tue, May 2, 2017 at 4:41 PM, Steven Rostedt wrote: > > > > This will conflict with changes I have already sent to you. They may > > not be so trivial to fix. I merged my urgent branch when pushing to > > linux-next. You can look at how I resolved the conflicts in my > > "for-next" branch, specifically sha1: f96d18dee6f09486b944b75f6151d36381f396b5 > > Hmm. My merge resolution is different, but I think I did it right. > > Yours does > > ret = alloc_snapshot(&global_trace); > > and I think it should be > > ret = alloc_snapshot(tr); No you are right. Damn, I'm upset at myself that I missed it. Another good reason to have you do the merge and not the subsystem maintainers ;-) > > but you should double-check it. I only looked at the code, I didn't > actually *test* anything. I have a couple of small patches I'm about to test and send to you later while the merge window is open. Would you be fine if I just add them on top of your merge commit? That would kill two birds with one test. > > (There's a few other differences, but they are just ordering of the > function declarations). I'll have to look at that (haven't looked at your merge commit yet). Because there were two functions that made more sense to go together, and the merge put the odd ball one in the middle. > > Btw, I'd prefer to *not* see the full patch in the pull request if > it's this big. For small stuff, sure. For a multi-thousand-line patch? > I'm not reading those in a mail-reader anyway. It's part of my script that prepares the pull request. I could remove it. I kept it there because it also shows others the code that I am asking to be changed. It's just my way of being open. But I'm not too hard set to keep it. If you prefer, I can stop doing that. -- Steve