From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756430AbcA2RM6 (ORCPT ); Fri, 29 Jan 2016 12:12:58 -0500 Received: from e06smtp09.uk.ibm.com ([195.75.94.105]:46360 "EHLO e06smtp09.uk.ibm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751448AbcA2RM4 (ORCPT ); Fri, 29 Jan 2016 12:12:56 -0500 X-IBM-Helo: d06dlp03.portsmouth.uk.ibm.com X-IBM-MailFrom: heiko.carstens@de.ibm.com X-IBM-RcptTo: linux-kernel@vger.kernel.org;linux-s390@vger.kernel.org Date: Fri, 29 Jan 2016 18:12:49 +0100 From: Heiko Carstens To: Steven Rostedt Cc: Pratyush Anand , schwidefsky@de.ibm.com, linux-s390@vger.kernel.org, Chunyu Hu , open list Subject: Re: [PATCH] s390:ftrace: add save_stack_trace_regs() Message-ID: <20160129171249.GG3896@osiris> References: <86d380d172bba0416aef64448b0e1b6a50fad326.1454044645.git.panand@redhat.com> <20160129085447.GA3896@osiris> <20160129125749.GD3896@osiris> <20160129085936.09458371@gandalf.local.home> <20160129144522.GE3896@osiris> <20160129102241.1b3c9c04@gandalf.local.home> <20160129164917.GF3896@osiris> <20160129115659.1b31f9c5@gandalf.local.home> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20160129115659.1b31f9c5@gandalf.local.home> User-Agent: Mutt/1.5.21 (2010-09-15) X-TM-AS-MML: disable X-Content-Scanned: Fidelis XPS MAILER x-cbid: 16012917-0037-0000-0000-0000056D1598 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Jan 29, 2016 at 11:56:59AM -0500, Steven Rostedt wrote: > On Fri, 29 Jan 2016 17:49:18 +0100 > Heiko Carstens wrote: > > > Which looks much better. The stack tracer also works again! > > Great! Although, it shouldn't have affected the stack tracing of kprobe > events :-/ Yes, you're absolutely right. I mixed up my trees (one 4.4 based and one Linus' master). > Although the previous commit I showed would. So, the save_stack_trace_regs() patch from Pratyush does work out-of-the-box on Linus' master tree and the stack tracer only if your additional patch is applied. Sorry for the confusion!