From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751823AbdCCP14 (ORCPT ); Fri, 3 Mar 2017 10:27:56 -0500 Received: from mail.kernel.org ([198.145.29.136]:37278 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751560AbdCCP1y (ORCPT ); Fri, 3 Mar 2017 10:27:54 -0500 Date: Fri, 3 Mar 2017 09:31:23 -0500 From: Steven Rostedt To: Todd Brandt Cc: linux-kernel@vger.kernel.org, todd.e.brandt@intel.com, Andrew Morton Subject: Re: [PATCH] ftrace: add ftrace_graph_max_depth kernel parameter Message-ID: <20170303093123.5fa413bf@gandalf.local.home> In-Reply-To: <1488504931.7212.41.camel@linux.intel.com> References: <1488499935-23216-1-git-send-email-todd.e.brandt@linux.intel.com> <20170302193827.6f5ee2eb@gandalf.local.home> <20170302193943.3117fca7@gandalf.local.home> <1488502497.7212.24.camel@linux.intel.com> <20170302200950.6d16c91a@gandalf.local.home> <1488504931.7212.41.camel@linux.intel.com> X-Mailer: Claws Mail 3.14.0 (GTK+ 2.24.31; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, 02 Mar 2017 17:35:31 -0800 Todd Brandt wrote: > > I could toy with the idea, implement it, and see what breaks. > If it was possible that would be great. The reason I ask is because I've > implemented a tool called analyze_boot which I intend to upgrade to use > early callgraph trace (it just uses initcall_debug now): > > https://marc.info/?l=linux-pm&m=148667508500967&w=2 > > I'll have it do the same thing for boot that analyze_suspend does for > suspend/resume: display the callgraphs for each initcall instance in > html. The max_graph_depth param is necessary to keep the html outputs > manageable. Most laptop/desktop machines running standard linux seem to > take at least 5 seconds for boot, but we've got some servers optimized > to boot in less than a second (which means I'd only be able to trace > half of the boot timeline or less). The more coverage I can get the > better. > > P.S. I submitted the analyze_boot tool on the pm list but Raphael didn't > think it would fit in the PM tree. Would you be willing to accept this > script? (I can resubmit but nothing's changed since the RESEND). That may be something to ask Andrew about. He's sorta the one to dump changes to that don't seem to fit anywhere else. -- Steve