Re: [PATCH 2/2] x86/tbl/trace: Do not trace on CPU that is offline

Sedat Dilek
2015-02-07T08:01:36Z

On Sat, Feb 7, 2015 at 5:02 AM, Steven Rostedt <rostedt@goodmis.org> wrote:
> On Fri, 6 Feb 2015 15:27:54 -0800
> "Paul E. McKenney" <paulmck@linux.vnet.ibm.com> wrote:
>
>> > Reported-by: Sedat Dilek <sedat.dilek@gmail.com>
>> > Suggested-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
>> > Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
>>
>> Acked-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
>
> Both patches survived all my tests, now I just need an ack by from Dave
> and/or Peter, and I'll push it off to Linus.
>

I took both patches from linux-trace.git/ftrace/urgent.

So, both patches were updated and especially 2/2 misses...

- Tested-by's
- Reference of 2/2 to 1/2
- CC: stable v3.17+/v3.18+ ? (1/2 has a hint for backporting in case
of 3.17+ which is BTW EOL, not sure if there are main distros
supporting linux-3.17.y)

BTW, do you happen to know if there is a someting like a
"Requires-tag" or "Precondition-tag" (for example the Fixes-tag is
very helpful)?
AFAICS people place such information into the commit-message.
I have seen "References:" but this is more to point to a
bug-tracking-system (BTS).

EXAMPLE:

commit 63b03e2d2477586440741677ecac45bcf28d7b1
"mutex: Always clear owner field upon mutex_unlock()"

References: https://bugs.freedesktop.org/show_bug.cgi?id=87955

Maybe ask godfather of SubmittingPatches greg-kh?

- Sedat -

[1] http://git.kernel.org/cgit/linux/kernel/git/rostedt/linux-trace.git/log/?h=ftrace/urgent