All of lore.kernel.org
 help / color / mirror / Atom feed
* [nvdimm:libnvdimm-pending 286/291] drivers/acpi/osl.c:769:14: warning: variable 'status' set but not used
@ 2020-05-19 10:56 kbuild test robot
  0 siblings, 0 replies; 2+ messages in thread
From: kbuild test robot @ 2020-05-19 10:56 UTC (permalink / raw)
  To: kbuild-all

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

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/djbw/nvdimm.git libnvdimm-pending
head:   4e2f899e682d1e6c647651bc36bcdba3b3577485
commit: ff99d8c56c8e8d60122b87333e3b807fd1638431 [286/291] ACPI: Drop rcu usage for MMIO mappings
config: i386-allyesconfig (attached as .config)
compiler: gcc-7 (Ubuntu 7.5.0-6ubuntu2) 7.5.0
reproduce:
        git checkout ff99d8c56c8e8d60122b87333e3b807fd1638431
        # save the attached .config to linux build tree
        make ARCH=i386 

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

All warnings (new ones prefixed by >>, old ones prefixed by <<):

drivers/acpi/osl.c: In function 'acpi_os_vprintf':
drivers/acpi/osl.c:151:2: warning: function 'acpi_os_vprintf' might be a candidate for 'gnu_printf' format attribute [-Wsuggest-attribute=format]
vsprintf(buffer, fmt, args);
^~~~~~~~
drivers/acpi/osl.c: In function 'acpi_os_write_memory':
>> drivers/acpi/osl.c:769:14: warning: variable 'status' set but not used [-Wunused-but-set-variable]
acpi_status status;
^~~~~~

vim +/status +769 drivers/acpi/osl.c

d2b7355684cdcf Dan Williams 2020-05-13  764  
e615bf5b551986 Myron Stowe  2012-01-20  765  acpi_status
653f4b538f66d3 Bob Moore    2012-02-14  766  acpi_os_write_memory(acpi_physical_address phys_addr, u64 value, u32 width)
e615bf5b551986 Myron Stowe  2012-01-20  767  {
e615bf5b551986 Myron Stowe  2012-01-20  768  	unsigned int size = width / 8;
d2b7355684cdcf Dan Williams 2020-05-13 @769  	acpi_status status;
ff99d8c56c8e8d Dan Williams 2020-04-27  770  	bool did_fallback = false;
ff99d8c56c8e8d Dan Williams 2020-04-27  771  	void __iomem *virt_addr;
e615bf5b551986 Myron Stowe  2012-01-20  772  
ff99d8c56c8e8d Dan Williams 2020-04-27  773  	virt_addr = acpi_os_rw_map(phys_addr, size, &did_fallback);
e615bf5b551986 Myron Stowe  2012-01-20  774  	if (!virt_addr)
e615bf5b551986 Myron Stowe  2012-01-20  775  		return AE_BAD_ADDRESS;
e615bf5b551986 Myron Stowe  2012-01-20  776  
d2b7355684cdcf Dan Williams 2020-05-13  777  	status = acpi_os_write_iomem(virt_addr, value, width);
e615bf5b551986 Myron Stowe  2012-01-20  778  
ff99d8c56c8e8d Dan Williams 2020-04-27  779  	acpi_os_rw_unmap(virt_addr, did_fallback);
ff99d8c56c8e8d Dan Williams 2020-04-27  780  	return AE_OK;
e615bf5b551986 Myron Stowe  2012-01-20  781  }
e615bf5b551986 Myron Stowe  2012-01-20  782  

:::::: The code at line 769 was first introduced by commit
:::::: d2b7355684cdcf842a9853915ae1c4927ae9a80a ACPI: Store pre-mapped registers in APEI entries

:::::: TO: Dan Williams <dan.j.williams@intel.com>
:::::: CC: Dan Williams <dan.j.williams@intel.com>

---
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: 72327 bytes --]

^ permalink raw reply	[flat|nested] 2+ messages in thread
* [nvdimm:libnvdimm-pending 286/291] drivers/acpi/osl.c:769:14: warning: variable 'status' set but not used
@ 2020-05-19 14:46 kbuild test robot
  0 siblings, 0 replies; 2+ messages in thread
From: kbuild test robot @ 2020-05-19 14:46 UTC (permalink / raw)
  To: kbuild-all

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

Hi Dan,

FYI, the error/warning still remains.

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/djbw/nvdimm.git libnvdimm-pending
head:   4e2f899e682d1e6c647651bc36bcdba3b3577485
commit: ff99d8c56c8e8d60122b87333e3b807fd1638431 [286/291] ACPI: Drop rcu usage for MMIO mappings
config: ia64-randconfig-r025-20200519 (attached as .config)
compiler: ia64-linux-gcc (GCC) 9.3.0
reproduce:
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        git checkout ff99d8c56c8e8d60122b87333e3b807fd1638431
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=ia64 

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

All warnings (new ones prefixed by >>, old ones prefixed by <<):

drivers/acpi/osl.c: In function 'acpi_os_vprintf':
drivers/acpi/osl.c:151:2: warning: function 'acpi_os_vprintf' might be a candidate for 'gnu_printf' format attribute [-Wsuggest-attribute=format]
151 |  vsprintf(buffer, fmt, args);
|  ^~~~~~~~
drivers/acpi/osl.c: In function 'acpi_os_write_memory':
>> drivers/acpi/osl.c:769:14: warning: variable 'status' set but not used [-Wunused-but-set-variable]
769 |  acpi_status status;
|              ^~~~~~

vim +/status +769 drivers/acpi/osl.c

d2b7355684cdcf Dan Williams 2020-05-13  764  
e615bf5b551986 Myron Stowe  2012-01-20  765  acpi_status
653f4b538f66d3 Bob Moore    2012-02-14  766  acpi_os_write_memory(acpi_physical_address phys_addr, u64 value, u32 width)
e615bf5b551986 Myron Stowe  2012-01-20  767  {
e615bf5b551986 Myron Stowe  2012-01-20  768  	unsigned int size = width / 8;
d2b7355684cdcf Dan Williams 2020-05-13 @769  	acpi_status status;
ff99d8c56c8e8d Dan Williams 2020-04-27  770  	bool did_fallback = false;
ff99d8c56c8e8d Dan Williams 2020-04-27  771  	void __iomem *virt_addr;
e615bf5b551986 Myron Stowe  2012-01-20  772  
ff99d8c56c8e8d Dan Williams 2020-04-27  773  	virt_addr = acpi_os_rw_map(phys_addr, size, &did_fallback);
e615bf5b551986 Myron Stowe  2012-01-20  774  	if (!virt_addr)
e615bf5b551986 Myron Stowe  2012-01-20  775  		return AE_BAD_ADDRESS;
e615bf5b551986 Myron Stowe  2012-01-20  776  
d2b7355684cdcf Dan Williams 2020-05-13  777  	status = acpi_os_write_iomem(virt_addr, value, width);
e615bf5b551986 Myron Stowe  2012-01-20  778  
ff99d8c56c8e8d Dan Williams 2020-04-27  779  	acpi_os_rw_unmap(virt_addr, did_fallback);
ff99d8c56c8e8d Dan Williams 2020-04-27  780  	return AE_OK;
e615bf5b551986 Myron Stowe  2012-01-20  781  }
e615bf5b551986 Myron Stowe  2012-01-20  782  

:::::: The code at line 769 was first introduced by commit
:::::: d2b7355684cdcf842a9853915ae1c4927ae9a80a ACPI: Store pre-mapped registers in APEI entries

:::::: TO: Dan Williams <dan.j.williams@intel.com>
:::::: CC: Dan Williams <dan.j.williams@intel.com>

---
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: 35171 bytes --]

^ permalink raw reply	[flat|nested] 2+ messages in thread
end of thread, other threads:[~2020-05-19 14:46 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-05-19 10:56 [nvdimm:libnvdimm-pending 286/291] drivers/acpi/osl.c:769:14: warning: variable 'status' set but not used kbuild test robot
2020-05-19 14:46 kbuild test robot

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.