Re: [GIT PULL] tracing: Updates for v4.12

From: Steven Rostedt
Date: Wed May 03 2017 - 22:10:07 EST

On Wed, 3 May 2017 18:59:05 -0700
Linus Torvalds <torvalds@xxxxxxxxxxxxxxxxxxxx> wrote:

> On Tue, May 2, 2017 at 4:41 PM, Steven Rostedt <rostedt@xxxxxxxxxxx> 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

> (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