From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755304AbdCGJpe (ORCPT ); Tue, 7 Mar 2017 04:45:34 -0500 Received: from smtp.codeaurora.org ([198.145.29.96]:47160 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754852AbdCGJoa (ORCPT ); Tue, 7 Mar 2017 04:44:30 -0500 DMARC-Filter: OpenDMARC Filter v1.3.2 smtp.codeaurora.org 32F6960739 Authentication-Results: pdx-caf-mail.web.codeaurora.org; dmarc=none (p=none dis=none) header.from=codeaurora.org Authentication-Results: pdx-caf-mail.web.codeaurora.org; spf=none smtp.mailfrom=kvalo@codeaurora.org From: Kalle Valo To: Arnd Bergmann Cc: Arend Van Spriel , kasan-dev , Andrey Ryabinin , Alexander Potapenko , Dmitry Vyukov , Networking , Linux Kernel Mailing List , linux-media@vger.kernel.org, linux-wireless , kernel-build-reports@lists.linaro.org, "David S . Miller" Subject: Re: [PATCH 08/26] brcmsmac: make some local variables 'static const' to reduce stack size References: <20170302163834.2273519-1-arnd@arndb.de> <20170302163834.2273519-9-arnd@arndb.de> <227c8e5a-fa20-0300-1cb0-1d3ef17deb19@broadcom.com> <87y3wi74y8.fsf@purkki.adurom.net> Date: Tue, 07 Mar 2017 11:44:04 +0200 In-Reply-To: (Arnd Bergmann's message of "Mon, 6 Mar 2017 22:34:12 +0100") Message-ID: <877f414e1n.fsf@codeaurora.org> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Arnd Bergmann writes: > On Mon, Mar 6, 2017 at 5:19 PM, Kalle Valo wrote: >> Arend Van Spriel writes: >> >>> On 2-3-2017 17:38, Arnd Bergmann wrote: >>>> With KASAN and a couple of other patches applied, this driver is one >>>> of the few remaining ones that actually use more than 2048 bytes of >>>> kernel stack: >>>> >>>> broadcom/brcm80211/brcmsmac/phy/phy_n.c: In function >>>> 'wlc_phy_workarounds_nphy_gainctrl': >>>> broadcom/brcm80211/brcmsmac/phy/phy_n.c:16065:1: warning: the >>>> frame size of 3264 bytes is larger than 2048 bytes >>>> [-Wframe-larger-than=] >>>> broadcom/brcm80211/brcmsmac/phy/phy_n.c: In function 'wlc_phy_workarounds_nphy': >>>> broadcom/brcm80211/brcmsmac/phy/phy_n.c:17138:1: warning: the >>>> frame size of 2864 bytes is larger than 2048 bytes >>>> [-Wframe-larger-than=] >>>> >>>> Here, I'm reducing the stack size by marking as many local variables as >>>> 'static const' as I can without changing the actual code. >>> >>> Acked-by: Arend van Spriel >> >> Arnd, via which tree are you planning to submit these? I'm not sure >> what I should do with the wireless drivers patches from this series. > > I'm not quite sure myself yet. I'd probably want the first few patches that > do most of the work get merged through Andrew's linux-mm tree once > we have come to agreement on them. The driver specific patches like > the brcmsmac ones depend on the introduction of noinline_for_kasan > or noinline_if_stackbloat and could either go in along with the first > set, or as a follow-up through the normal maintainer trees. Either way is fine for me. Just mark clearly if you want the wireless drivers patches to go through via my tree, otherwise I'll ignore them. -- Kalle Valo