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]
Date:   Fri, 2 Apr 2021 19:11:00 +0800
From:   kernel test robot <lkp@...el.com>
To:     Yang Li <yang.lee@...ux.alibaba.com>, jaharkes@...cmu.edu
Cc:     kbuild-all@...ts.01.org, coda@...cmu.edu, codalist@...a.cs.cmu.edu,
        linux-kernel@...r.kernel.org, Yang Li <yang.lee@...ux.alibaba.com>
Subject: Re: [PATCH] coda: Remove various instances of an unused variable
 'outp'

Hi Yang,

Thank you for the patch! Yet something to improve:

[auto build test ERROR on linus/master]
[also build test ERROR on v5.12-rc5 next-20210401]
[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/Yang-Li/coda-Remove-various-instances-of-an-unused-variable-outp/20210402-173111
base:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git 1678e493d530e7977cce34e59a86bb86f3c5631e
config: arc-randconfig-r014-20210402 (attached as .config)
compiler: arceb-elf-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/b6484bc8a589df437829010ab82b49c48d56ee46
        git remote add linux-review https://github.com/0day-ci/linux
        git fetch --no-tags linux-review Yang-Li/coda-Remove-various-instances-of-an-unused-variable-outp/20210402-173111
        git checkout b6484bc8a589df437829010ab82b49c48d56ee46
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=arc 

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

All errors (new ones prefixed by >>):

   fs/coda/upcall.c: In function 'venus_setattr':
>> fs/coda/upcall.c:65:9: error: 'outp' undeclared (first use in this function); did you mean 'outl'?
      65 |         outp = (union outputArgs *)(inp); \
         |         ^~~~
   fs/coda/upcall.c:118:2: note: in expansion of macro 'UPARG'
     118 |  UPARG(CODA_SETATTR);
         |  ^~~~~
   fs/coda/upcall.c:65:9: note: each undeclared identifier is reported only once for each function it appears in
      65 |         outp = (union outputArgs *)(inp); \
         |         ^~~~
   fs/coda/upcall.c:118:2: note: in expansion of macro 'UPARG'
     118 |  UPARG(CODA_SETATTR);
         |  ^~~~~
   fs/coda/upcall.c: In function 'venus_close':
>> fs/coda/upcall.c:65:9: error: 'outp' undeclared (first use in this function); did you mean 'outl'?
      65 |         outp = (union outputArgs *)(inp); \
         |         ^~~~
   fs/coda/upcall.c:166:2: note: in expansion of macro 'UPARG'
     166 |  UPARG(CODA_CLOSE);
         |  ^~~~~
   fs/coda/upcall.c: In function 'venus_rename':
>> fs/coda/upcall.c:65:9: error: 'outp' undeclared (first use in this function); did you mean 'outl'?
      65 |         outp = (union outputArgs *)(inp); \
         |         ^~~~
   fs/coda/upcall.c:242:3: note: in expansion of macro 'UPARG'
     242 |   UPARG(CODA_RENAME);
         |   ^~~~~
   fs/coda/upcall.c: In function 'venus_rmdir':
>> fs/coda/upcall.c:65:9: error: 'outp' undeclared (first use in this function); did you mean 'outl'?
      65 |         outp = (union outputArgs *)(inp); \
         |         ^~~~
   fs/coda/upcall.c:308:2: note: in expansion of macro 'UPARG'
     308 |  UPARG(CODA_RMDIR);
         |  ^~~~~
   fs/coda/upcall.c: In function 'venus_remove':
>> fs/coda/upcall.c:65:9: error: 'outp' undeclared (first use in this function); did you mean 'outl'?
      65 |         outp = (union outputArgs *)(inp); \
         |         ^~~~
   fs/coda/upcall.c:329:2: note: in expansion of macro 'UPARG'
     329 |  UPARG(CODA_REMOVE);
         |  ^~~~~
   fs/coda/upcall.c: In function 'venus_link':
>> fs/coda/upcall.c:65:9: error: 'outp' undeclared (first use in this function); did you mean 'outl'?
      65 |         outp = (union outputArgs *)(inp); \
         |         ^~~~
   fs/coda/upcall.c:383:9: note: in expansion of macro 'UPARG'
     383 |         UPARG(CODA_LINK);
         |         ^~~~~
   fs/coda/upcall.c: In function 'venus_symlink':
>> fs/coda/upcall.c:65:9: error: 'outp' undeclared (first use in this function); did you mean 'outl'?
      65 |         outp = (union outputArgs *)(inp); \
         |         ^~~~
   fs/coda/upcall.c:409:2: note: in expansion of macro 'UPARG'
     409 |  UPARG(CODA_SYMLINK);
         |  ^~~~~
   fs/coda/upcall.c: In function 'venus_fsync':
>> fs/coda/upcall.c:65:9: error: 'outp' undeclared (first use in this function); did you mean 'outl'?
      65 |         outp = (union outputArgs *)(inp); \
         |         ^~~~
   fs/coda/upcall.c:439:2: note: in expansion of macro 'UPARG'
     439 |  UPARG(CODA_FSYNC);
         |  ^~~~~
   fs/coda/upcall.c: In function 'venus_access':
>> fs/coda/upcall.c:65:9: error: 'outp' undeclared (first use in this function); did you mean 'outl'?
      65 |         outp = (union outputArgs *)(inp); \
         |         ^~~~
   fs/coda/upcall.c:454:2: note: in expansion of macro 'UPARG'
     454 |  UPARG(CODA_ACCESS);
         |  ^~~~~
   fs/coda/upcall.c: In function 'venus_access_intent':
>> fs/coda/upcall.c:65:9: error: 'outp' undeclared (first use in this function); did you mean 'outl'?
      65 |         outp = (union outputArgs *)(inp); \
         |         ^~~~
   fs/coda/upcall.c:577:2: note: in expansion of macro 'UPARG'
     577 |  UPARG(CODA_ACCESS_INTENT);
         |  ^~~~~


vim +65 fs/coda/upcall.c

^1da177e4c3f41 Linus Torvalds 2005-04-16  60  
^1da177e4c3f41 Linus Torvalds 2005-04-16  61  #define UPARG(op)\
^1da177e4c3f41 Linus Torvalds 2005-04-16  62  do {\
^1da177e4c3f41 Linus Torvalds 2005-04-16  63  	inp = (union inputArgs *)alloc_upcall(op, insize); \
^1da177e4c3f41 Linus Torvalds 2005-04-16  64          if (IS_ERR(inp)) { return PTR_ERR(inp); }\
^1da177e4c3f41 Linus Torvalds 2005-04-16 @65          outp = (union outputArgs *)(inp); \
^1da177e4c3f41 Linus Torvalds 2005-04-16  66          outsize = insize; \
^1da177e4c3f41 Linus Torvalds 2005-04-16  67  } while (0)
^1da177e4c3f41 Linus Torvalds 2005-04-16  68  

---
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" (29974 bytes)

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ