From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id 97FCEC433EF for ; Thu, 31 Mar 2022 17:07:07 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 1AB6D6B0072; Thu, 31 Mar 2022 13:07:07 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 15AA66B0073; Thu, 31 Mar 2022 13:07:07 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id F3E3E6B0074; Thu, 31 Mar 2022 13:07:06 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (relay.hostedemail.com [64.99.140.25]) by kanga.kvack.org (Postfix) with ESMTP id DFEAB6B0072 for ; Thu, 31 Mar 2022 13:07:06 -0400 (EDT) Received: from smtpin11.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay13.hostedemail.com (Postfix) with ESMTP id A8CC160E5C for ; Thu, 31 Mar 2022 17:07:06 +0000 (UTC) X-FDA: 79305311652.11.C23CC13 Received: from mail-lj1-f180.google.com (mail-lj1-f180.google.com [209.85.208.180]) by imf16.hostedemail.com (Postfix) with ESMTP id F1FD1180015 for ; Thu, 31 Mar 2022 17:07:05 +0000 (UTC) Received: by mail-lj1-f180.google.com with SMTP id v12so642218ljd.3 for ; Thu, 31 Mar 2022 10:07:05 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20210112; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=fdSyI71UrcYnS9PD8/kYWAhaxlCUy9rY5QWg9D495Js=; b=UMiK2/ok9F3Y1Vp/549u0DOVZzjViy3UwowNqil5I+8duJSK5VCjRULxz4DWNeqgZF 6LWHeeo8vrJHlW3B/3K4hzUWWJUG743p45kFw9pVv4k4JnBNVekIlq5NErSBvjU4WBCB QBoZrq1eVRM/sDhtQi08Q0aeugFk2iLHm9a89lv0lHSOn/ZUatgl8TbFYqVxQjft6Vzu YZEW0k6VyXfaLqJuQRIA6G5aLzK/iBpbDAc7gKNXoH1s2y4u8bk7OlBJTeCWPfMR3AEM 4Wnii3Vn49DrVPkfReZaI6eDJL4MhyJXiu4LzSZ653tkbAPZ0ct8tRQXLiWVX/YmHneu netw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=fdSyI71UrcYnS9PD8/kYWAhaxlCUy9rY5QWg9D495Js=; b=KYpUeJJR6lSifB/0mPqbfuZ5n5RIqn33dR41r9hXTk7eR88covVLy8au4rkyHvFvJN 4fO3ogvQnEW+Hh64lK+6+UBa4oEA1+nqejzePEICBROCKly9Xumf8EAng3TcwSCks7Bd 7LVaOlMZeGhc6Shh9i3bUSRfBgQHAxTZxaUG8Lv/dEq2evMleAp21irCzKKxO0O2D0dO zkWMwT1KDtyakq27Jv57ljoci2OQxMwzbiGAB1ekeRluEPXiPIP+0OX77NHkK/fLBphL 21DmXkBsdlEDY5D71wAL87gCV1fb+7pEMhiRCSRxA/0AXh8HnnNDQe9Ajci39ID3xLl4 iWrg== X-Gm-Message-State: AOAM5302FKGF7CmNgtHjFrq9sNuinZLcmLMwG3rvR5YkXzzm6+PDd5TN Ch7zuHSkCTtaC/eaLCH4BuL5KjjBrqTY/9/bat39aQ== X-Google-Smtp-Source: ABdhPJw8Q/b4RlaadBwVkJ1YfXzyuHazNx9rPS5OzaHjywsX8MEzE4ECP5Xy/jh7hykLR2YuO0IcIF80exL3DqZ36as= X-Received: by 2002:a2e:818f:0:b0:24a:7c17:7226 with SMTP id e15-20020a2e818f000000b0024a7c177226mr11014901ljg.472.1648746423660; Thu, 31 Mar 2022 10:07:03 -0700 (PDT) MIME-Version: 1.0 References: <202203301412.MZ7wQvQz-lkp@intel.com> In-Reply-To: From: Nick Desaulniers Date: Thu, 31 Mar 2022 10:06:52 -0700 Message-ID: Subject: Re: [ammarfaizi2-block:google/android/kernel/common/android12-trusty-5.10 4036/5872] WARNING: modpost: vmlinux.o(.text+0x4111c4): Section mismatch in reference from the function memblock_bottom_up() to the variable .meminit.data:memblock To: Tri Vo , =?UTF-8?B?QXJ2ZSBIasO4bm5ldsOlZw==?= Cc: Mike Rapoport , Roman Gushchin , kernel test robot , Roman Gushchin , llvm@lists.linux.dev, kbuild-all@lists.01.org, "GNU/Weeb Mailing List" , linux-kernel@vger.kernel.org, Suren Baghdasaryan , Andrew Morton , Linux Memory Management List , Minchan Kim , Nathan Chancellor Content-Type: text/plain; charset="UTF-8" X-Stat-Signature: p8us89sgd9ebhsx5in6fdwtcs1fp94dq Authentication-Results: imf16.hostedemail.com; dkim=pass header.d=google.com header.s=20210112 header.b="UMiK2/ok"; spf=pass (imf16.hostedemail.com: domain of ndesaulniers@google.com designates 209.85.208.180 as permitted sender) smtp.mailfrom=ndesaulniers@google.com; dmarc=pass (policy=reject) header.from=google.com X-Rspam-User: X-Rspamd-Server: rspam02 X-Rspamd-Queue-Id: F1FD1180015 X-HE-Tag: 1648746425-385920 X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: On Thu, Mar 31, 2022 at 9:11 AM Nathan Chancellor wrote: > > On Thu, Mar 31, 2022 at 12:42:04PM +0300, Mike Rapoport wrote: > > (added llvm folks) > > > > Hi, > > > > On Wed, Mar 30, 2022 at 02:47:43PM -0700, Roman Gushchin wrote: > > > On Wed, Mar 30, 2022 at 02:53:14PM +0800, kbuild test robot wrote: > > > > Hi Roman, > > > > > > > > FYI, the error/warning still remains. > > > > > > > > tree: https://github.com/ammarfaizi2/linux-block google/android/kernel/common/android12-trusty-5.10 > > > > head: 07055bfd3d810d41a38354693dfaa55a6f8c0025 > > > > commit: 0e0bfc41fdf4d79d39ebe929844cdee44f97366d [4036/5872] UPSTREAM: mm: cma: allocate cma areas bottom-up > > > > config: x86_64-randconfig-a005 (https://download.01.org/0day-ci/archive/20220330/202203301412.MZ7wQvQz-lkp@intel.com/config) > > > > compiler: clang version 15.0.0 (https://github.com/llvm/llvm-project 0f6d9501cf49ce02937099350d08f20c4af86f3d) > > > > reproduce (this is a W=1 build): > > > > wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross > > > > chmod +x ~/bin/make.cross > > > > # https://github.com/ammarfaizi2/linux-block/commit/0e0bfc41fdf4d79d39ebe929844cdee44f97366d > > > > git remote add ammarfaizi2-block https://github.com/ammarfaizi2/linux-block > > > > git fetch --no-tags ammarfaizi2-block google/android/kernel/common/android12-trusty-5.10 > > > > git checkout 0e0bfc41fdf4d79d39ebe929844cdee44f97366d > > > > # save the config file to linux build tree > > > > mkdir build_dir > > > > COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=x86_64 SHELL=/bin/bash > > > > > > > > If you fix the issue, kindly add following tag as appropriate > > > > Reported-by: kernel test robot > > > > > > > > All warnings (new ones prefixed by >>, old ones prefixed by <<): > > > > > > > > >> WARNING: modpost: vmlinux.o(.text+0x4111c4): Section mismatch in reference from the function memblock_bottom_up() to the variable .meminit.data:memblock > > > > The function memblock_bottom_up() references > > > > the variable __meminitdata memblock. > > > > This is often because memblock_bottom_up lacks a __meminitdata > > > > annotation or the annotation of memblock is wrong. > > > > > > I guess this patch should fix it, however I fail to reproduce the original issue. > > > Maybe it's up to the specific compiler version. > > > > > > -- > > > > > > From b55a8dd19f4156d7e24ec39b18ede06965ce1c4f Mon Sep 17 00:00:00 2001 > > > From: Roman Gushchin > > > Date: Wed, 30 Mar 2022 14:42:12 -0700 > > > Subject: [PATCH] memblock: fix memblock_bottom_up() and > > > memblock_set_bottom_up() annotations > > > > > > memblock_bottom_up() and memblock_set_bottom_up() lack __meminitdata > > > annotations causing compiler warnings like: > > > WARNING: modpost: vmlinux.o(.text+0x4111c4): Section mismatch in reference from the function memblock_bottom_up() to the > > > variable .meminit.data:memblock > > > > > > Fix it by adding the missing annotation and removing the wrong > > > __meminit annotation. > > > > > > Reported-by: kernel test robot > > > Signed-off-by: Roman Gushchin > > > --- > > > include/linux/memblock.h | 4 ++-- > > > 1 file changed, 2 insertions(+), 2 deletions(-) > > > > > > diff --git a/include/linux/memblock.h b/include/linux/memblock.h > > > index 50ad19662a32..536bc2fc31e6 100644 > > > --- a/include/linux/memblock.h > > > +++ b/include/linux/memblock.h > > > @@ -460,7 +460,7 @@ static inline void *memblock_alloc_node(phys_addr_t size, > > > /* > > > * Set the allocation direction to bottom-up or top-down. > > > */ > > > -static inline __init_memblock void memblock_set_bottom_up(bool enable) > > > +static inline __initdata_memblock void memblock_set_bottom_up(bool enable) > > > > I think putting __initdata_memlock won't help here, because there should be > > nothing wrong with __meminit function accessing __meminitdata data. > > > > My guesstimate would be that the compiler decided not to inline this and > > still dropped section attribute because of 'inline'. > > > > If this is the case we I think we should > > > > s/inline __init_memblock/__always_inline/ > > > > > { > > > memblock.bottom_up = enable; > > > } > > > @@ -470,7 +470,7 @@ static inline __init_memblock void memblock_set_bottom_up(bool enable) > > > * if this is true, that said, memblock will allocate memory > > > * in bottom-up direction. > > > */ > > > -static inline __init_memblock bool memblock_bottom_up(void) > > > +static inline __initdata_memblock bool memblock_bottom_up(void) > > > { > > > return memblock.bottom_up; > > > } > > > -- > > > 2.30.2 > > > > > > > For the record, I cannot reproduce this on mainline, which has commits > 34dc2efb39a2 ("memblock: fix section mismatch warning") and a024b7c2850d > ("mm: memblock: fix section mismatch warning again"). That first commit > has the same exact warning as this report, which is against an Android > tree (android12-trusty-5.10). > > While I do not see the commit that 34dc2efb39a2 claims to fix in > android12-trusty-5.10, I do see the three commits in android12-5.10: > > a46e3fa13968 ("UPSTREAM: mm: memblock: drop __init from memblock functions to make it inline") > 5f7ec0f4c383 ("UPSTREAM: memblock: fix section mismatch warning") > 8cf5bb6946a2 ("UPSTREAM: mm: memblock: fix section mismatch warning again") It sounds like trusty just needs to pull down from (or rebase onto) android12-5.10 into android12-trusty-5.10. > > I think we can just discard this report for now, unless someone from > Google's trusty team wants to address it in that branch. > > Cheers, > Nathan -- Thanks, ~Nick Desaulniers