[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <aBnKeD-KxlPoAJWM@kernel.org>
Date: Tue, 6 May 2025 11:38:16 +0300
From: Mike Rapoport <rppt@...nel.org>
To: Oscar Salvador <osalvador@...e.de>
Cc: kernel test robot <lkp@...el.com>, oe-kbuild-all@...ts.linux.dev,
linux-kernel@...r.kernel.org,
Andrew Morton <akpm@...ux-foundation.org>,
Linux Memory Management List <linux-mm@...ck.org>
Subject: Re: WARNING: modpost: vmlinux: section mismatch in reference:
set_high_memory+0x8c (section: .text.unlikely) -> zone_movable_pfn (section:
.init.data)
On Tue, May 06, 2025 at 09:29:08AM +0200, Oscar Salvador wrote:
> On Tue, May 06, 2025 at 09:42:54AM +0800, kernel test robot wrote:
> > tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
> > head: 01f95500a162fca88cefab9ed64ceded5afabc12
> > commit: e120d1bc12da5c1bb871c346f741296610fd6fcb arch, mm: set high_memory in free_area_init()
> > date: 7 weeks ago
> > config: arm-randconfig-r062-20250506 (https://download.01.org/0day-ci/archive/20250506/202505060901.Qcs06UoB-lkp@intel.com/config)
> > compiler: arm-linux-gnueabi-gcc (GCC) 10.5.0
> > reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250506/202505060901.Qcs06UoB-lkp@intel.com/reproduce)
> >
> > If you fix the issue in a separate patch/commit (i.e. not just a new version of
> > the same patch/commit), kindly add following tags
> > | Reported-by: kernel test robot <lkp@...el.com>
> > | Closes: https://lore.kernel.org/oe-kbuild-all/202505060901.Qcs06UoB-lkp@intel.com/
> >
> > All warnings (new ones prefixed by >>, old ones prefixed by <<):
> >
> > >> WARNING: modpost: vmlinux: section mismatch in reference: set_high_memory+0x8c (section: .text.unlikely) -> zone_movable_pfn (section: .init.data)
>
> Perhaps?
Looks right, thanks for a quick fix!
Care sending a formal patch?
> diff --git a/mm/mm_init.c b/mm/mm_init.c
> index 80a5370ac6ab..5efec0399e18 100644
> --- a/mm/mm_init.c
> +++ b/mm/mm_init.c
> @@ -1785,7 +1785,7 @@ static bool arch_has_descending_max_zone_pfns(void)
> return IS_ENABLED(CONFIG_ARC) && !IS_ENABLED(CONFIG_ARC_HAS_PAE40);
> }
>
> -static void set_high_memory(void)
> +static void __init set_high_memory(void)
> {
> phys_addr_t highmem = memblock_end_of_DRAM();
>
>
> --
> Oscar Salvador
> SUSE Labs
>
--
Sincerely yours,
Mike.
Powered by blists - more mailing lists