Re: [PATCH v3] mmc: sdio: check the buffer address for sdio API

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Shawn,

[auto build test WARNING on linus/master]
[also build test WARNING on v4.10-rc7 next-20170203]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

url:    https://github.com/0day-ci/linux/commits/Shawn-Lin/mmc-sdio-check-the-buffer-address-for-sdio-API/20170206-163619
config: x86_64-randconfig-x008-201706 (attached as .config)
compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901
reproduce:
        # save the attached .config to linux build tree
        make ARCH=x86_64 

All warnings (new ones prefixed by >>):

   drivers/mmc/core/sdio_io.c: In function 'sdio_io_addr_sanity_check':
>> drivers/mmc/core/sdio_io.c:313:1: warning: control reaches end of non-void function [-Wreturn-type]
    }
    ^

vim +313 drivers/mmc/core/sdio_io.c

   297		 */
   298		return orig_sz;
   299	}
   300	EXPORT_SYMBOL_GPL(sdio_align_size);
   301	
   302	static int sdio_io_addr_sanity_check(void *buf)
   303	{
   304	#if defined(CONFIG_MODULES) && defined(MODULES_VADDR)
   305		unsigned long addr = (unsigned long)buf;
   306	
   307		if ((addr >= MODULES_VADDR && addr < MODULES_END) ||
   308		    object_is_on_stack(buf))
   309			return 1;
   310	#else
   311		return (is_vmalloc_addr(buf) || object_is_on_stack(buf));
   312	#endif
 > 313	}
   314	
   315	/* Split an arbitrarily sized data transfer into several
   316	 * IO_RW_EXTENDED commands. */
   317	static int sdio_io_rw_ext_helper(struct sdio_func *func, int write,
   318		unsigned addr, int incr_addr, u8 *buf, unsigned size)
   319	{
   320		unsigned remainder = size;
   321		unsigned max_blocks;

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

Attachment: .config.gz
Description: application/gzip


[Index of Archives]     [Linux USB Devel]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux