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>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <202102181614.pUVzuo3g-lkp@intel.com>
Date:   Thu, 18 Feb 2021 17:02:48 +0800
From:   kernel test robot <lkp@...el.com>
To:     Lizhi Hou <lizhi.hou@...inx.com>, linux-kernel@...r.kernel.org
Cc:     kbuild-all@...ts.01.org, Lizhi Hou <lizhih@...inx.com>,
        linux-fpga@...r.kernel.org, maxz@...inx.com,
        sonal.santan@...inx.com, michal.simek@...inx.com,
        stefanos@...inx.com, devicetree@...r.kernel.org, trix@...hat.com,
        mdf@...nel.org
Subject: Re: [PATCH V3 XRT Alveo 18/18] fpga: xrt: Kconfig and Makefile
 updates for XRT drivers

Hi Lizhi,

I love your patch! Yet something to improve:

[auto build test ERROR on linux/master]
[also build test ERROR on linus/master v5.11 next-20210217]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]

url:    https://github.com/0day-ci/linux/commits/Lizhi-Hou/XRT-Alveo-driver-overview/20210218-151631
base:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git 2ab38c17aac10bf55ab3efde4c4db3893d8691d2
config: parisc-randconfig-r025-20210218 (attached as .config)
compiler: hppa64-linux-gcc (GCC) 9.3.0
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/0day-ci/linux/commit/9069f42b5fcafe471ceb1047ee53983098b2edba
        git remote add linux-review https://github.com/0day-ci/linux
        git fetch --no-tags linux-review Lizhi-Hou/XRT-Alveo-driver-overview/20210218-151631
        git checkout 9069f42b5fcafe471ceb1047ee53983098b2edba
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=parisc 

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

All error/warnings (new ones prefixed by >>):

   drivers/fpga/xrt/lib/main.c: In function 'xleaf_register_driver':
>> drivers/fpga/xrt/lib/main.c:144:8: error: implicit declaration of function 'vzalloc'; did you mean 'kvzalloc'? [-Werror=implicit-function-declaration]
     144 |  map = vzalloc(sizeof(*map));
         |        ^~~~~~~
         |        kvzalloc
>> drivers/fpga/xrt/lib/main.c:144:6: warning: assignment to 'struct xrt_drv_map *' from 'int' makes pointer from integer without a cast [-Wint-conversion]
     144 |  map = vzalloc(sizeof(*map));
         |      ^
   drivers/fpga/xrt/lib/main.c: In function 'xleaf_unregister_driver':
>> drivers/fpga/xrt/lib/main.c:181:2: error: implicit declaration of function 'vfree'; did you mean 'kvfree'? [-Werror=implicit-function-declaration]
     181 |  vfree(map);
         |  ^~~~~
         |  kvfree
   cc1: some warnings being treated as errors


vim +144 drivers/fpga/xrt/lib/main.c

898da78383bc51 Lizhi Hou 2021-02-17  127  
898da78383bc51 Lizhi Hou 2021-02-17  128  int xleaf_register_driver(enum xrt_subdev_id id,
898da78383bc51 Lizhi Hou 2021-02-17  129  			  struct platform_driver *drv,
898da78383bc51 Lizhi Hou 2021-02-17  130  			  struct xrt_subdev_endpoints *eps)
898da78383bc51 Lizhi Hou 2021-02-17  131  {
898da78383bc51 Lizhi Hou 2021-02-17  132  	struct xrt_drv_map *map;
898da78383bc51 Lizhi Hou 2021-02-17  133  
898da78383bc51 Lizhi Hou 2021-02-17  134  	mutex_lock(&xrt_lib_lock);
898da78383bc51 Lizhi Hou 2021-02-17  135  
898da78383bc51 Lizhi Hou 2021-02-17  136  	map = xrt_drv_find_map_by_id_nolock(id);
898da78383bc51 Lizhi Hou 2021-02-17  137  	if (map) {
898da78383bc51 Lizhi Hou 2021-02-17  138  		mutex_unlock(&xrt_lib_lock);
898da78383bc51 Lizhi Hou 2021-02-17  139  		pr_err("Id %d already has a registered driver, 0x%p\n",
898da78383bc51 Lizhi Hou 2021-02-17  140  		       id, map->drv);
898da78383bc51 Lizhi Hou 2021-02-17  141  		return -EEXIST;
898da78383bc51 Lizhi Hou 2021-02-17  142  	}
898da78383bc51 Lizhi Hou 2021-02-17  143  
898da78383bc51 Lizhi Hou 2021-02-17 @144  	map = vzalloc(sizeof(*map));
898da78383bc51 Lizhi Hou 2021-02-17  145  	if (!map) {
898da78383bc51 Lizhi Hou 2021-02-17  146  		mutex_unlock(&xrt_lib_lock);
898da78383bc51 Lizhi Hou 2021-02-17  147  		return -ENOMEM;
898da78383bc51 Lizhi Hou 2021-02-17  148  	}
898da78383bc51 Lizhi Hou 2021-02-17  149  	map->id = id;
898da78383bc51 Lizhi Hou 2021-02-17  150  	map->drv = drv;
898da78383bc51 Lizhi Hou 2021-02-17  151  	map->eps = eps;
898da78383bc51 Lizhi Hou 2021-02-17  152  
898da78383bc51 Lizhi Hou 2021-02-17  153  	xrt_drv_register_driver(map);
898da78383bc51 Lizhi Hou 2021-02-17  154  
898da78383bc51 Lizhi Hou 2021-02-17  155  	list_add(&map->list, &xrt_drv_maps);
898da78383bc51 Lizhi Hou 2021-02-17  156  
898da78383bc51 Lizhi Hou 2021-02-17  157  	mutex_unlock(&xrt_lib_lock);
898da78383bc51 Lizhi Hou 2021-02-17  158  
898da78383bc51 Lizhi Hou 2021-02-17  159  	return 0;
898da78383bc51 Lizhi Hou 2021-02-17  160  }
898da78383bc51 Lizhi Hou 2021-02-17  161  EXPORT_SYMBOL_GPL(xleaf_register_driver);
898da78383bc51 Lizhi Hou 2021-02-17  162  
898da78383bc51 Lizhi Hou 2021-02-17  163  void xleaf_unregister_driver(enum xrt_subdev_id id)
898da78383bc51 Lizhi Hou 2021-02-17  164  {
898da78383bc51 Lizhi Hou 2021-02-17  165  	struct xrt_drv_map *map;
898da78383bc51 Lizhi Hou 2021-02-17  166  
898da78383bc51 Lizhi Hou 2021-02-17  167  	mutex_lock(&xrt_lib_lock);
898da78383bc51 Lizhi Hou 2021-02-17  168  
898da78383bc51 Lizhi Hou 2021-02-17  169  	map = xrt_drv_find_map_by_id_nolock(id);
898da78383bc51 Lizhi Hou 2021-02-17  170  	if (!map) {
898da78383bc51 Lizhi Hou 2021-02-17  171  		mutex_unlock(&xrt_lib_lock);
898da78383bc51 Lizhi Hou 2021-02-17  172  		pr_err("Id %d has no registered driver\n", id);
898da78383bc51 Lizhi Hou 2021-02-17  173  		return;
898da78383bc51 Lizhi Hou 2021-02-17  174  	}
898da78383bc51 Lizhi Hou 2021-02-17  175  
898da78383bc51 Lizhi Hou 2021-02-17  176  	list_del(&map->list);
898da78383bc51 Lizhi Hou 2021-02-17  177  
898da78383bc51 Lizhi Hou 2021-02-17  178  	mutex_unlock(&xrt_lib_lock);
898da78383bc51 Lizhi Hou 2021-02-17  179  
898da78383bc51 Lizhi Hou 2021-02-17  180  	xrt_drv_unregister_driver(map);
898da78383bc51 Lizhi Hou 2021-02-17 @181  	vfree(map);
898da78383bc51 Lizhi Hou 2021-02-17  182  }
898da78383bc51 Lizhi Hou 2021-02-17  183  EXPORT_SYMBOL_GPL(xleaf_unregister_driver);
898da78383bc51 Lizhi Hou 2021-02-17  184  

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org

Download attachment ".config.gz" of type "application/gzip" (37024 bytes)

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ