From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932467AbcHCOWf (ORCPT ); Wed, 3 Aug 2016 10:22:35 -0400 Received: from mx1.redhat.com ([209.132.183.28]:33714 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932285AbcHCOWI (ORCPT ); Wed, 3 Aug 2016 10:22:08 -0400 Date: Wed, 3 Aug 2016 09:13:52 -0500 From: Josh Poimboeuf To: Steven Rostedt Cc: Thomas Gleixner , Ingo Molnar , "H . Peter Anvin" , x86@kernel.org, linux-kernel@vger.kernel.org, Andy Lutomirski , Linus Torvalds , Brian Gerst , Kees Cook , Peter Zijlstra , Frederic Weisbecker , Byungchul Park Subject: Re: [PATCH 05/19] x86/dumpstack: fix function graph tracing stack dump reliability issues Message-ID: <20160803141352.tp6z4j4h25pd4lsd@treble> References: <20160802191622.466f53d2@gandalf.local.home> <20160803015656.j6fxrgucmghh5gfn@treble> <20160802223011.6209e28c@grimm.local.home> <20160803025012.bmte6hjhcd6lotod@treble> <20160802225936.5b6dad93@grimm.local.home> <20160803031233.b73xz5jftxj7crqh@treble> <20160802231857.4c082831@grimm.local.home> <20160802232104.31730733@grimm.local.home> <20160803033125.uxyrs3aaic2w5pvq@treble> <20160802234530.74d603f9@grimm.local.home> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <20160802234530.74d603f9@grimm.local.home> User-Agent: Mutt/1.6.0.1 (2016-04-01) X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.27]); Wed, 03 Aug 2016 14:13:55 +0000 (UTC) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Aug 02, 2016 at 11:45:30PM -0400, Steven Rostedt wrote: > On Tue, 2 Aug 2016 22:31:25 -0500 > Josh Poimboeuf wrote: > > Yeah, we do need to add a parameter to ftrace_push_return_trace(). But > > callers which don't implement it could just pass zero like they do with > > 'fp'. > > > > Right, if zero is passed in, then just ignore it. I still think we need the define though, because there will be two versions of ftrace_graph_ret_addr(). -- Josh