Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752068AbdGRSjE (ORCPT ); Tue, 18 Jul 2017 14:39:04 -0400 Received: from conssluserg-05.nifty.com ([210.131.2.90]:41494 "EHLO conssluserg-05.nifty.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752000AbdGRSjC (ORCPT ); Tue, 18 Jul 2017 14:39:02 -0400 DKIM-Filter: OpenDKIM Filter v2.10.3 conssluserg-05.nifty.com v6IIcwp9029043 X-Nifty-SrcIP: [209.85.213.169] MIME-Version: 1.0 In-Reply-To: <20170630225822.25349-1-sboyd@codeaurora.org> References: <20170630225822.25349-1-sboyd@codeaurora.org> From: Masahiro Yamada Date: Wed, 19 Jul 2017 03:38:57 +0900 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [PATCH] kbuild: modpost: Warn about references from rodata to __init text To: Stephen Boyd Cc: Michal Marek , Linux Kernel Mailing List , linux-arm-msm , linux-arm-kernel , Linux Kbuild mailing list , Rob Clark , Bjorn Andersson Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1217 Lines: 31 2017-07-01 7:58 GMT+09:00 Stephen Boyd : > If we have a structure that's marked const it will be placed > into the .rodata section but it could reference an init section > function. Include the read only data section in the check we have > for read/write data sections referencing init sections so we can > find this class of problems. This exposes quite a few places > where const marked structures are referencing __init functions and > __init data that we were previously ignoring. > > Cc: Rob Clark > Cc: Bjorn Andersson > Signed-off-by: Stephen Boyd > --- > > Making this change leads to quite a few other errors even on the > multi_v7_defconfig for ARM[1]. I still need to do a build of the > allmodconfig to see how many other errors there, but it seems to > be quite a few. I suppose those will need to be fixed before we can > merge this? > > [1] https://www.irccloud.com/pastebin/zzqfWGQY/ Applied to linux-kbuild/misc. Thanks! Now people will notice the warnings in linux-next, and will start fixing drivers. We still have plenty of time until the next merge windows. :-) -- Best Regards Masahiro Yamada