Re: [PATCH 3/4] x86/unwind: include __schedule() in stack traces

2017-01-10 Thread Miroslav Benes
On Tue, 10 Jan 2017, Josh Poimboeuf wrote: > On Tue, Jan 10, 2017 at 11:14:51AM +0100, Miroslav Benes wrote: > > On Mon, 9 Jan 2017, Josh Poimboeuf wrote: > > > Signed-off-by: Josh Poimboeuf > > > --- > > > arch/x86/include/asm/stacktrace.h | 5 + > > > arch/x86/include/asm/switch_to.h | 1

Re: [PATCH 3/4] x86/unwind: include __schedule() in stack traces

2017-01-10 Thread Josh Poimboeuf
On Tue, Jan 10, 2017 at 11:14:51AM +0100, Miroslav Benes wrote: > On Mon, 9 Jan 2017, Josh Poimboeuf wrote: > > Signed-off-by: Josh Poimboeuf > > --- > > arch/x86/include/asm/stacktrace.h | 5 + > > arch/x86/include/asm/switch_to.h | 10 +- > > 2 files changed, 10 insertions(+), 5 d

Re: [PATCH 3/4] x86/unwind: include __schedule() in stack traces

2017-01-10 Thread Miroslav Benes
On Mon, 9 Jan 2017, Josh Poimboeuf wrote: > In the following commit: > > 0100301bfdf5 ("sched/x86: Rewrite the switch_to() code") > > ... the layout of the 'inactive_task_frame' struct was designed to have > a frame pointer header embedded in it, so that the unwinder could use > the 'bp' and '

[PATCH 3/4] x86/unwind: include __schedule() in stack traces

2017-01-09 Thread Josh Poimboeuf
In the following commit: 0100301bfdf5 ("sched/x86: Rewrite the switch_to() code") ... the layout of the 'inactive_task_frame' struct was designed to have a frame pointer header embedded in it, so that the unwinder could use the 'bp' and 'ret_addr' fields to report __schedule() on the stack (or