Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1168985AbdDXK4I (ORCPT ); Mon, 24 Apr 2017 06:56:08 -0400 Received: from hqemgate16.nvidia.com ([216.228.121.65]:19140 "EHLO hqemgate16.nvidia.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1168443AbdDXK4B (ORCPT ); Mon, 24 Apr 2017 06:56:01 -0400 X-PGP-Universal: processed; by hqnvupgp08.nvidia.com on Mon, 24 Apr 2017 03:51:56 -0700 Subject: Re: [PATCH] soc/tegra: pmc: Don't allocate struct tegra_powergate on stack To: Viresh Kumar References: <3fe40fcd427e49cbeac31e14721fea569d230b6e.1490073884.git.viresh.kumar@linaro.org> <7d0425c1-de9a-cb1a-fde7-71e903579562@nvidia.com> <20170321103950.GS25659@vireshk-i7> <20170417055035.GI28191@vireshk-i7> CC: Stephen Warren , Thierry Reding , Alexandre Courbot , , , From: Jon Hunter Message-ID: Date: Mon, 24 Apr 2017 11:55:55 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.8.0 MIME-Version: 1.0 In-Reply-To: <20170417055035.GI28191@vireshk-i7> X-Originating-IP: [10.26.11.190] X-ClientProxiedBy: UKMAIL101.nvidia.com (10.26.138.13) To UKMAIL101.nvidia.com (10.26.138.13) Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1397 Lines: 35 On 17/04/17 06:50, Viresh Kumar wrote: > On 21-03-17, 16:09, Viresh Kumar wrote: >> On 21-03-17, 10:37, Jon Hunter wrote: >>> >>> On 21/03/17 05:24, Viresh Kumar wrote: >>>> The size of the struct tegra_powergate is quite big and if any more >>>> fields are added to the internal genpd structure, following warnings are >>>> thrown: >>>> >>>> drivers/soc/tegra/pmc.c:577:1: warning: the frame size of 1176 bytes is larger than 1024 bytes [-Wframe-larger-than=] >>> >>> Hmmm ... AFAICT the size of the tegra_powergate struct is 312 bytes >>> (based upon next-20170321) and so it looks like something massive needs >>> to be added to the genpd struct to blow this up to over 1024 bytes. Are >>> there some genpd changes in-flight that are causing this? >> >> https://marc.info/?l=linux-kernel&m=149000247329743&w=2 >> >> This is up for discussion right now though and we don't know if it >> will surely get merged or not. > > @Jon: Regardless of the above series, do you want this patch to be merged as it > will still be better to avoid keeping large structures on stack. Given that it is currently much less than the default threshold, it seems ok to me as-is. However, if it looks like you patch to add the device struct to the gpd struct is going to be accepted, then it is fine with me. Maybe we should wait for you patch to be accepted then this can be applied as a fix. Jon -- nvpublic