[SRU][Bionic][Cosmic][PATCH 1/1] powerpc/livepatch: Implement reliable stack tracing for the consistency model

Khaled Elmously khalid.elmously at canonical.com
Thu Jun 7 19:37:13 UTC 2018


On 2018-06-07 15:03:25 , Joseph Salisbury wrote:
> On 06/07/2018 01:37 PM, Khaled Elmously wrote:
> > On 2018-06-05 11:52:15 , Joseph Salisbury wrote:
> >> On 06/04/2018 10:11 PM, Michael Ellerman wrote:
> >>> Joseph Salisbury <joseph.salisbury at canonical.com> writes:
> >>>> diff --git a/arch/powerpc/kernel/stacktrace.c b/arch/powerpc/kernel/stacktrace.c
> >>>> index d534ed901538..26a50603177c 100644
> >>>> --- a/arch/powerpc/kernel/stacktrace.c
> >>>> +++ b/arch/powerpc/kernel/stacktrace.c
> >>>> @@ -76,3 +81,115 @@ save_stack_trace_regs(struct pt_regs *regs, struct stack_trace *trace)
> >>>>  	save_context_stack(trace, regs->gpr[1], current, 0);
> >>>>  }
> >>>>  EXPORT_SYMBOL_GPL(save_stack_trace_regs);
> >>>> +
> >>>> +#ifdef CONFIG_HAVE_RELIABLE_STACKTRACE
> >>>> +int
> >>>> +save_stack_trace_tsk_reliable(struct task_struct *tsk,
> >>>> +				struct stack_trace *trace)
> >>>> +{
> >>> ...
> >>>> +		/*
> >>>> +		 * Mark stacktraces with kretprobed functions on them
> >>>> +		 * as unreliable.
> >>>> +		 */
> >>>> +		if (ip == (unsigned long)kretprobe_trampoline)
> >>>> +			return 1;
> >>> You may also want:
> >>>
> >>> 5e3f0d15ae5f ("powerpc/livepatch: Fix build error with kprobes disabled.")
> >>>
> >>> cheers
> >> Thanks for the feedback, Michael.  Would it be possible for you to post
> >> this comment to the bug?  That way we can get feedback from the original
> >> bug reporter and IBM.
> >>
> >>
> >> Thanks,
> >>
> >>
> >> Joe
> > Joe, I did not apply this patch as it is not clear to me if a V2 is on the way or if it needs more work. Please advise.
> >
> > Thanks
> >
> > Khalid
> I don't plan on sending a V2.  IBM tested what is in this SRU request
> and it was confirmed to fix the bug for them.  I would say apply what is
> in this request.
> 
> Michael, it's probably best to open a new bug if you believe additional
> commits are needed.

Cool, thanks

> 
> >
> >>
> >>
> >> -- 
> >> kernel-team mailing list
> >> kernel-team at lists.ubuntu.com
> >> https://lists.ubuntu.com/mailman/listinfo/kernel-team
> 
> 




More information about the kernel-team mailing list