lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [day] [month] [year] [list]
Message-ID: <202205172143.m2ngCwCl-lkp@intel.com>
Date:   Tue, 17 May 2022 21:15:18 +0800
From:   kernel test robot <lkp@...el.com>
To:     Linus Walleij <linus.walleij@...aro.org>
Cc:     llvm@...ts.linux.dev, kbuild-all@...ts.01.org,
        linux-kernel@...r.kernel.org
Subject: [arm-integrator:kernel-in-vmalloc-v5.18-rc1 6/9]
 arch/arm/mm/init.c:276:5: warning: format specifies type 'unsigned long' but
 the argument has type 'char *'

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/linusw/linux-integrator.git kernel-in-vmalloc-v5.18-rc1
head:   67a8d6eb3d015908f35d6091c02f17d7c3b55bf3
commit: e916acde63f41816676698a5d750588269d0fd64 [6/9] ARM: Print virtual memory info again
config: arm-randconfig-r021-20220516 (https://download.01.org/0day-ci/archive/20220517/202205172143.m2ngCwCl-lkp@intel.com/config)
compiler: clang version 15.0.0 (https://github.com/llvm/llvm-project 853fa8ee225edf2d0de94b0dcbd31bea916e825e)
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
        # install arm cross compiling tool for clang build
        # apt-get install binutils-arm-linux-gnueabi
        # https://git.kernel.org/pub/scm/linux/kernel/git/linusw/linux-integrator.git/commit/?id=e916acde63f41816676698a5d750588269d0fd64
        git remote add arm-integrator https://git.kernel.org/pub/scm/linux/kernel/git/linusw/linux-integrator.git
        git fetch --no-tags arm-integrator kernel-in-vmalloc-v5.18-rc1
        git checkout e916acde63f41816676698a5d750588269d0fd64
        # save the config file
        mkdir build_dir && cp config build_dir/.config
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=arm SHELL=/bin/bash arch/arm/mm/ lib/

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@...el.com>

All warnings (new ones prefixed by >>):

   arch/arm/mm/init.c:97:13: warning: no previous prototype for function 'setup_dma_zone' [-Wmissing-prototypes]
   void __init setup_dma_zone(const struct machine_desc *mdesc)
               ^
   arch/arm/mm/init.c:97:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
   void __init setup_dma_zone(const struct machine_desc *mdesc)
   ^
   static 
>> arch/arm/mm/init.c:276:5: warning: format specifies type 'unsigned long' but the argument has type 'char *' [-Wformat]
                     "     kernel : 0x%08lx - 0x%08lx   (%4ld MB)\n",
                     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/printk.h:509:36: note: expanded from macro 'pr_notice'
           printk(KERN_NOTICE pr_fmt(fmt), ##__VA_ARGS__)
                                     ~~~     ^~~~~~~~~~~
   include/linux/printk.h:446:60: note: expanded from macro 'printk'
   #define printk(fmt, ...) printk_index_wrap(_printk, fmt, ##__VA_ARGS__)
                                                       ~~~    ^~~~~~~~~~~
   include/linux/printk.h:418:19: note: expanded from macro 'printk_index_wrap'
                   _p_func(_fmt, ##__VA_ARGS__);                           \
                           ~~~~    ^~~~~~~~~~~
>> arch/arm/mm/init.c:281:5: warning: data argument not used by format string [-Wformat-extra-args]
                     MLM(MODULES_VADDR, MODULES_END),
                     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/printk.h:509:36: note: expanded from macro 'pr_notice'
           printk(KERN_NOTICE pr_fmt(fmt), ##__VA_ARGS__)
           ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~
   include/linux/printk.h:446:60: note: expanded from macro 'printk'
   #define printk(fmt, ...) printk_index_wrap(_printk, fmt, ##__VA_ARGS__)
                            ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~
   include/linux/printk.h:418:19: note: expanded from macro 'printk_index_wrap'
                   _p_func(_fmt, ##__VA_ARGS__);                           \
                           ~~~~    ^
   arch/arm/mm/init.c:268:25: note: expanded from macro 'MLM'
   #define MLM(b, t) b, t, ((t) - (b)) >> 20
                           ^
   arch/arm/mm/init.c:266:13: warning: no previous prototype for function 'mem_init_print_arm_info' [-Wmissing-prototypes]
   void __init mem_init_print_arm_info(void)
               ^
   arch/arm/mm/init.c:266:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
   void __init mem_init_print_arm_info(void)
   ^
   static 
   4 warnings generated.


vim +276 arch/arm/mm/init.c

    96	
  > 97	void __init setup_dma_zone(const struct machine_desc *mdesc)
    98	{
    99	#ifdef CONFIG_ZONE_DMA
   100		if (mdesc->dma_zone_size) {
   101			arm_dma_zone_size = mdesc->dma_zone_size;
   102			arm_dma_limit = PHYS_OFFSET + arm_dma_zone_size - 1;
   103		} else
   104			arm_dma_limit = 0xffffffff;
   105		arm_dma_pfn_limit = arm_dma_limit >> PAGE_SHIFT;
   106	#endif
   107	}
   108	
   109	static void __init zone_sizes_init(unsigned long min, unsigned long max_low,
   110		unsigned long max_high)
   111	{
   112		unsigned long max_zone_pfn[MAX_NR_ZONES] = { 0 };
   113	
   114	#ifdef CONFIG_ZONE_DMA
   115		max_zone_pfn[ZONE_DMA] = min(arm_dma_pfn_limit, max_low);
   116	#endif
   117		max_zone_pfn[ZONE_NORMAL] = max_low;
   118	#ifdef CONFIG_HIGHMEM
   119		max_zone_pfn[ZONE_HIGHMEM] = max_high;
   120	#endif
   121		free_area_init(max_zone_pfn);
   122	}
   123	
   124	#ifdef CONFIG_HAVE_ARCH_PFN_VALID
   125	int pfn_valid(unsigned long pfn)
   126	{
   127		phys_addr_t addr = __pfn_to_phys(pfn);
   128		unsigned long pageblock_size = PAGE_SIZE * pageblock_nr_pages;
   129	
   130		if (__phys_to_pfn(addr) != pfn)
   131			return 0;
   132	
   133		/*
   134		 * If address less than pageblock_size bytes away from a present
   135		 * memory chunk there still will be a memory map entry for it
   136		 * because we round freed memory map to the pageblock boundaries.
   137		 */
   138		if (memblock_overlaps_region(&memblock.memory,
   139					     ALIGN_DOWN(addr, pageblock_size),
   140					     pageblock_size))
   141			return 1;
   142	
   143		return 0;
   144	}
   145	EXPORT_SYMBOL(pfn_valid);
   146	#endif
   147	
   148	static bool arm_memblock_steal_permitted = true;
   149	
   150	phys_addr_t __init arm_memblock_steal(phys_addr_t size, phys_addr_t align)
   151	{
   152		phys_addr_t phys;
   153	
   154		BUG_ON(!arm_memblock_steal_permitted);
   155	
   156		phys = memblock_phys_alloc(size, align);
   157		if (!phys)
   158			panic("Failed to steal %pa bytes at %pS\n",
   159			      &size, (void *)_RET_IP_);
   160	
   161		memblock_phys_free(phys, size);
   162		memblock_remove(phys, size);
   163	
   164		return phys;
   165	}
   166	
   167	#ifdef CONFIG_CPU_ICACHE_MISMATCH_WORKAROUND
   168	void check_cpu_icache_size(int cpuid)
   169	{
   170		u32 size, ctr;
   171	
   172		asm("mrc p15, 0, %0, c0, c0, 1" : "=r" (ctr));
   173	
   174		size = 1 << ((ctr & 0xf) + 2);
   175		if (cpuid != 0 && icache_size != size)
   176			pr_info("CPU%u: detected I-Cache line size mismatch, workaround enabled\n",
   177				cpuid);
   178		if (icache_size > size)
   179			icache_size = size;
   180	}
   181	#endif
   182	
   183	void __init arm_memblock_init(const struct machine_desc *mdesc)
   184	{
   185		/* Register the kernel text, kernel data and initrd with memblock. */
   186		memblock_reserve(__pa(KERNEL_START), KERNEL_END - KERNEL_START);
   187	
   188		reserve_initrd_mem();
   189	
   190		arm_mm_memblock_reserve();
   191	
   192		/* reserve any platform specific memblock areas */
   193		if (mdesc->reserve)
   194			mdesc->reserve();
   195	
   196		early_init_fdt_scan_reserved_mem();
   197	
   198		/* reserve memory for DMA contiguous allocations */
   199		dma_contiguous_reserve(arm_dma_limit);
   200	
   201		arm_memblock_steal_permitted = false;
   202		memblock_dump_all();
   203	}
   204	
   205	void __init bootmem_init(void)
   206	{
   207		memblock_allow_resize();
   208	
   209		find_limits(&min_low_pfn, &max_low_pfn, &max_pfn);
   210	
   211		early_memtest((phys_addr_t)min_low_pfn << PAGE_SHIFT,
   212			      (phys_addr_t)max_low_pfn << PAGE_SHIFT);
   213	
   214		/*
   215		 * sparse_init() tries to allocate memory from memblock, so must be
   216		 * done after the fixed reservations
   217		 */
   218		sparse_init();
   219	
   220		/*
   221		 * Now free the memory - free_area_init needs
   222		 * the sparse mem_map arrays initialized by sparse_init()
   223		 * for memmap_init_zone(), otherwise all PFNs are invalid.
   224		 */
   225		zone_sizes_init(min_low_pfn, max_low_pfn, max_pfn);
   226	}
   227	
   228	/*
   229	 * Poison init memory with an undefined instruction (ARM) or a branch to an
   230	 * undefined instruction (Thumb).
   231	 */
   232	static inline void poison_init_mem(void *s, size_t count)
   233	{
   234		u32 *p = (u32 *)s;
   235		for (; count != 0; count -= 4)
   236			*p++ = 0xe7fddef0;
   237	}
   238	
   239	static void __init free_highpages(void)
   240	{
   241	#ifdef CONFIG_HIGHMEM
   242		unsigned long max_low = max_low_pfn;
   243		phys_addr_t range_start, range_end;
   244		u64 i;
   245	
   246		/* set highmem page free */
   247		for_each_free_mem_range(i, NUMA_NO_NODE, MEMBLOCK_NONE,
   248					&range_start, &range_end, NULL) {
   249			unsigned long start = PFN_UP(range_start);
   250			unsigned long end = PFN_DOWN(range_end);
   251	
   252			/* Ignore complete lowmem entries */
   253			if (end <= max_low)
   254				continue;
   255	
   256			/* Truncate partial highmem entries */
   257			if (start < max_low)
   258				start = max_low;
   259	
   260			for (; start < end; start++)
   261				free_highmem_page(pfn_to_page(start));
   262		}
   263	#endif
   264	}
   265	
   266	void __init mem_init_print_arm_info(void)
   267	{
   268	#define MLM(b, t) b, t, ((t) - (b)) >> 20
   269		pr_notice("Virtual kernel memory layout:\n"
   270			  "    fixmap  : 0x%08lx - 0x%08lx   (%4ld MB)\n"
   271			  "    vmalloc : 0x%08lx - 0x%08lx   (%4ld MB)\n"
   272			  "    lowmem  : 0x%08lx - 0x%08lx   (%4ld MB)\n"
   273	#ifdef CONFIG_MODULES
   274			  "    modules : 0x%08lx - 0x%08lx   (%4ld MB)\n",
   275	#endif
 > 276			  "     kernel : 0x%08lx - 0x%08lx   (%4ld MB)\n",
   277			  MLM(FIXADDR_START, FIXADDR_END),
   278			  MLM(VMALLOC_START, VMALLOC_END),
   279			  MLM(PAGE_OFFSET, (unsigned long)high_memory),
   280	#ifdef CONFIG_MODULES
 > 281			  MLM(MODULES_VADDR, MODULES_END),
   282	#endif
   283			  /* From beginning of .text to end of .bss */
   284			  MLM((unsigned long)_text, (unsigned long)__bss_stop));
   285	#undef MLM
   286	}
   287	

-- 
0-DAY CI Kernel Test Service
https://01.org/lkp

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ