Re: [Outreachy kernel] [RFC PATCH] staging: unisys: visorhba: Convert module from IDR to XArray

[kernel test robot <lkp@intel.com>]

[-- Attachment #1: Type: text/plain, Size: 2856 bytes --]

Hi "Fabio,

[FYI, it's a private test report for your RFC patch.]
[auto build test WARNING on staging/staging-testing]

url:    https://github.com/0day-ci/linux/commits/Fabio-M-De-Francesco/staging-unisys-visorhba-Convert-module-from-IDR-to-XArray/20210426-175256
base:   https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git c295d3007ff63064181befa734d9705dfc10b396
config: x86_64-randconfig-s021-20210426 (attached as .config)
compiler: gcc-9 (Debian 9.3.0-22) 9.3.0
reproduce:
        # apt-get install sparse
        # sparse version: v0.6.3-341-g8af24329-dirty
        # https://github.com/0day-ci/linux/commit/c5d6871153dc6ee2fad1f82377b8eb967d14fa7f
        git remote add linux-review https://github.com/0day-ci/linux
        git fetch --no-tags linux-review Fabio-M-De-Francesco/staging-unisys-visorhba-Convert-module-from-IDR-to-XArray/20210426-175256
        git checkout c5d6871153dc6ee2fad1f82377b8eb967d14fa7f
        # save the attached .config to linux build tree
        make W=1 C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' W=1 ARCH=x86_64 

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

All warnings (new ones prefixed by >>):

   drivers/staging/unisys/visorhba/visorhba_main.c: In function 'setup_scsitaskmgmt_handles':
>> drivers/staging/unisys/visorhba/visorhba_main.c:208:6: warning: variable 'ret' set but not used [-Wunused-but-set-variable]
     208 |  int ret;
         |      ^~~


vim +/ret +208 drivers/staging/unisys/visorhba/visorhba_main.c

   193	
   194	/*
   195	 * setup_scsitaskmgmt_handles - Stash the necessary handles so that the
   196	 *				completion processing logic for a taskmgmt
   197	 *				cmd will be able to find who to wake up
   198	 *				and where to stash the result
   199	 * @xa_dtstr: The data object maintaining the pointer<-->int mappings
   200	 * @cmdrsp:   Response from the IOVM
   201	 * @event:    The event handle to associate with an id
   202	 * @result:   The location to place the result of the event handle into
   203	 */
   204	static void setup_scsitaskmgmt_handles(struct uiscmdrsp *cmdrsp,
   205					       wait_queue_head_t *event, u32 *result)
   206	{
   207		void *entry;
 > 208		int ret;
   209	
   210		/* specify the event that has to be triggered when this cmd is complete */
   211		entry = &cmdrsp->scsitaskmgmt.notify_handle;
   212		ret = simple_xa_dtstr_get(result, entry);
   213		/* TODO: Check for and manage errors */
   214	
   215		entry = &cmdrsp->scsitaskmgmt.notifyresult_handle;
   216		ret = simple_xa_dtstr_get(result, entry);
   217		/* TODO: Check for and manage errors */
   218	}
   219	

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

[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 40787 bytes --]