All of lore.kernel.org
 help / color / mirror / Atom feed
From: kernel test robot <lkp@intel.com>
To: Johannes Berg <johannes.berg@intel.com>
Cc: kbuild-all@lists.01.org,
	Linux Memory Management List <linux-mm@kvack.org>,
	Richard Weinberger <richard@nod.at>
Subject: [linux-next:master 10919/11253] drivers/nvdimm/../../tools/testing/nvdimm/test/iomap.c:207:17: warning: passing argument 1 of 'iounmap' discards 'volatile' qualifier from pointer target type
Date: Sat, 19 Jun 2021 03:13:45 +0800	[thread overview]
Message-ID: <202106190339.C2eUsszW-lkp@intel.com> (raw)

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

Hi Johannes,

First bad commit (maybe != root cause):

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master
head:   e71e3a48a7e89fa71fb70bf4602367528864d2ff
commit: 68f5d3f3b6543266b29e047cfaf9842333019b4c [10919/11253] um: add PCI over virtio emulation driver
config: um-allmodconfig (attached as .config)
compiler: gcc-9 (Debian 9.3.0-22) 9.3.0
reproduce (this is a W=1 build):
        # https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commit/?id=68f5d3f3b6543266b29e047cfaf9842333019b4c
        git remote add linux-next https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git
        git fetch --no-tags linux-next master
        git checkout 68f5d3f3b6543266b29e047cfaf9842333019b4c
        # save the attached .config to linux build tree
        make W=1 ARCH=um 

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/nvdimm/../../tools/testing/nvdimm/test/iomap.c:76:15: warning: no previous prototype for '__wrap_devm_ioremap' [-Wmissing-prototypes]
      76 | void __iomem *__wrap_devm_ioremap(struct device *dev,
         |               ^~~~~~~~~~~~~~~~~~~
   drivers/nvdimm/../../tools/testing/nvdimm/test/iomap.c:88:7: warning: no previous prototype for '__wrap_devm_memremap' [-Wmissing-prototypes]
      88 | void *__wrap_devm_memremap(struct device *dev, resource_size_t offset,
         |       ^~~~~~~~~~~~~~~~~~~~
   drivers/nvdimm/../../tools/testing/nvdimm/test/iomap.c:126:7: warning: no previous prototype for '__wrap_devm_memremap_pages' [-Wmissing-prototypes]
     126 | void *__wrap_devm_memremap_pages(struct device *dev, struct dev_pagemap *pgmap)
         |       ^~~~~~~~~~~~~~~~~~~~~~~~~~
   drivers/nvdimm/../../tools/testing/nvdimm/test/iomap.c:159:7: warning: no previous prototype for '__wrap_phys_to_pfn_t' [-Wmissing-prototypes]
     159 | pfn_t __wrap_phys_to_pfn_t(phys_addr_t addr, unsigned long flags)
         |       ^~~~~~~~~~~~~~~~~~~~
   drivers/nvdimm/../../tools/testing/nvdimm/test/iomap.c:169:7: warning: no previous prototype for '__wrap_memremap' [-Wmissing-prototypes]
     169 | void *__wrap_memremap(resource_size_t offset, size_t size,
         |       ^~~~~~~~~~~~~~~
   drivers/nvdimm/../../tools/testing/nvdimm/test/iomap.c:180:6: warning: no previous prototype for '__wrap_devm_memunmap' [-Wmissing-prototypes]
     180 | void __wrap_devm_memunmap(struct device *dev, void *addr)
         |      ^~~~~~~~~~~~~~~~~~~~
   drivers/nvdimm/../../tools/testing/nvdimm/test/iomap.c:190:15: warning: no previous prototype for '__wrap_ioremap' [-Wmissing-prototypes]
     190 | void __iomem *__wrap_ioremap(resource_size_t offset, unsigned long size)
         |               ^~~~~~~~~~~~~~
   drivers/nvdimm/../../tools/testing/nvdimm/test/iomap.c:196:15: warning: no previous prototype for '__wrap_ioremap_wc' [-Wmissing-prototypes]
     196 | void __iomem *__wrap_ioremap_wc(resource_size_t offset, unsigned long size)
         |               ^~~~~~~~~~~~~~~~~
   drivers/nvdimm/../../tools/testing/nvdimm/test/iomap.c: In function '__wrap_iounmap':
>> drivers/nvdimm/../../tools/testing/nvdimm/test/iomap.c:207:17: warning: passing argument 1 of 'iounmap' discards 'volatile' qualifier from pointer target type [-Wdiscarded-qualifiers]
     207 |  return iounmap(addr);
         |                 ^~~~
   In file included from arch/um/include/asm/io.h:7,
                    from include/linux/io.h:13,
                    from drivers/nvdimm/../../tools/testing/nvdimm/test/iomap.c:13:
   include/asm-generic/logic_io.h:37:28: note: expected 'void *' but argument is of type 'volatile void *'
      37 | void iounmap(void __iomem *addr);
         |              ~~~~~~~~~~~~~~^~~~
   drivers/nvdimm/../../tools/testing/nvdimm/test/iomap.c: At top level:
   drivers/nvdimm/../../tools/testing/nvdimm/test/iomap.c:211:6: warning: no previous prototype for '__wrap_memunmap' [-Wmissing-prototypes]
     211 | void __wrap_memunmap(void *addr)
         |      ^~~~~~~~~~~~~~~
   drivers/nvdimm/../../tools/testing/nvdimm/test/iomap.c:347:18: warning: no previous prototype for '__wrap___request_region' [-Wmissing-prototypes]
     347 | struct resource *__wrap___request_region(struct resource *parent,
         |                  ^~~~~~~~~~~~~~~~~~~~~~~
   drivers/nvdimm/../../tools/testing/nvdimm/test/iomap.c:355:5: warning: no previous prototype for '__wrap_insert_resource' [-Wmissing-prototypes]
     355 | int __wrap_insert_resource(struct resource *parent, struct resource *res)
         |     ^~~~~~~~~~~~~~~~~~~~~~
   drivers/nvdimm/../../tools/testing/nvdimm/test/iomap.c:363:5: warning: no previous prototype for '__wrap_remove_resource' [-Wmissing-prototypes]
     363 | int __wrap_remove_resource(struct resource *res)
         |     ^~~~~~~~~~~~~~~~~~~~~~
   drivers/nvdimm/../../tools/testing/nvdimm/test/iomap.c:371:18: warning: no previous prototype for '__wrap___devm_request_region' [-Wmissing-prototypes]
     371 | struct resource *__wrap___devm_request_region(struct device *dev,
         |                  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
   drivers/nvdimm/../../tools/testing/nvdimm/test/iomap.c:381:6: warning: no previous prototype for '__wrap___release_region' [-Wmissing-prototypes]
     381 | void __wrap___release_region(struct resource *parent, resource_size_t start,
         |      ^~~~~~~~~~~~~~~~~~~~~~~
   drivers/nvdimm/../../tools/testing/nvdimm/test/iomap.c:389:6: warning: no previous prototype for '__wrap___devm_release_region' [-Wmissing-prototypes]
     389 | void __wrap___devm_release_region(struct device *dev, struct resource *parent,
         |      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
   drivers/nvdimm/../../tools/testing/nvdimm/test/iomap.c:397:13: warning: no previous prototype for '__wrap_acpi_evaluate_object' [-Wmissing-prototypes]
     397 | acpi_status __wrap_acpi_evaluate_object(acpi_handle handle, acpi_string path,
         |             ^~~~~~~~~~~~~~~~~~~~~~~~~~~
   drivers/nvdimm/../../tools/testing/nvdimm/test/iomap.c:413:21: warning: no previous prototype for '__wrap_acpi_evaluate_dsm' [-Wmissing-prototypes]
     413 | union acpi_object * __wrap_acpi_evaluate_dsm(acpi_handle handle, const guid_t *guid,
         |                     ^~~~~~~~~~~~~~~~~~~~~~~~


vim +207 drivers/nvdimm/../../tools/testing/nvdimm/test/iomap.c

9d27a87ec9e131 Dan Williams 2015-07-10  201  
6bc756193ff61b Dan Williams 2015-06-17  202  void __wrap_iounmap(volatile void __iomem *addr)
6bc756193ff61b Dan Williams 2015-06-17  203  {
9bfa84969dd52b Dan Williams 2015-12-11  204  	struct nfit_test_resource *nfit_res = get_nfit_res((long) addr);
6bc756193ff61b Dan Williams 2015-06-17  205  	if (nfit_res)
6bc756193ff61b Dan Williams 2015-06-17  206  		return;
6bc756193ff61b Dan Williams 2015-06-17 @207  	return iounmap(addr);
6bc756193ff61b Dan Williams 2015-06-17  208  }
6bc756193ff61b Dan Williams 2015-06-17  209  EXPORT_SYMBOL(__wrap_iounmap);
6bc756193ff61b Dan Williams 2015-06-17  210  

:::::: The code at line 207 was first introduced by commit
:::::: 6bc756193ff61bf5e7b3cfedfbb0873bf40f8055 tools/testing/nvdimm: libnvdimm unit test infrastructure

:::::: 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@lists.01.org

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

WARNING: multiple messages have this Message-ID (diff)
From: kernel test robot <lkp@intel.com>
To: kbuild-all@lists.01.org
Subject: [linux-next:master 10919/11253] drivers/nvdimm/../../tools/testing/nvdimm/test/iomap.c:207:17: warning: passing argument 1 of 'iounmap' discards 'volatile' qualifier from pointer target type
Date: Sat, 19 Jun 2021 03:13:45 +0800	[thread overview]
Message-ID: <202106190339.C2eUsszW-lkp@intel.com> (raw)

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

Hi Johannes,

First bad commit (maybe != root cause):

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master
head:   e71e3a48a7e89fa71fb70bf4602367528864d2ff
commit: 68f5d3f3b6543266b29e047cfaf9842333019b4c [10919/11253] um: add PCI over virtio emulation driver
config: um-allmodconfig (attached as .config)
compiler: gcc-9 (Debian 9.3.0-22) 9.3.0
reproduce (this is a W=1 build):
        # https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commit/?id=68f5d3f3b6543266b29e047cfaf9842333019b4c
        git remote add linux-next https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git
        git fetch --no-tags linux-next master
        git checkout 68f5d3f3b6543266b29e047cfaf9842333019b4c
        # save the attached .config to linux build tree
        make W=1 ARCH=um 

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/nvdimm/../../tools/testing/nvdimm/test/iomap.c:76:15: warning: no previous prototype for '__wrap_devm_ioremap' [-Wmissing-prototypes]
      76 | void __iomem *__wrap_devm_ioremap(struct device *dev,
         |               ^~~~~~~~~~~~~~~~~~~
   drivers/nvdimm/../../tools/testing/nvdimm/test/iomap.c:88:7: warning: no previous prototype for '__wrap_devm_memremap' [-Wmissing-prototypes]
      88 | void *__wrap_devm_memremap(struct device *dev, resource_size_t offset,
         |       ^~~~~~~~~~~~~~~~~~~~
   drivers/nvdimm/../../tools/testing/nvdimm/test/iomap.c:126:7: warning: no previous prototype for '__wrap_devm_memremap_pages' [-Wmissing-prototypes]
     126 | void *__wrap_devm_memremap_pages(struct device *dev, struct dev_pagemap *pgmap)
         |       ^~~~~~~~~~~~~~~~~~~~~~~~~~
   drivers/nvdimm/../../tools/testing/nvdimm/test/iomap.c:159:7: warning: no previous prototype for '__wrap_phys_to_pfn_t' [-Wmissing-prototypes]
     159 | pfn_t __wrap_phys_to_pfn_t(phys_addr_t addr, unsigned long flags)
         |       ^~~~~~~~~~~~~~~~~~~~
   drivers/nvdimm/../../tools/testing/nvdimm/test/iomap.c:169:7: warning: no previous prototype for '__wrap_memremap' [-Wmissing-prototypes]
     169 | void *__wrap_memremap(resource_size_t offset, size_t size,
         |       ^~~~~~~~~~~~~~~
   drivers/nvdimm/../../tools/testing/nvdimm/test/iomap.c:180:6: warning: no previous prototype for '__wrap_devm_memunmap' [-Wmissing-prototypes]
     180 | void __wrap_devm_memunmap(struct device *dev, void *addr)
         |      ^~~~~~~~~~~~~~~~~~~~
   drivers/nvdimm/../../tools/testing/nvdimm/test/iomap.c:190:15: warning: no previous prototype for '__wrap_ioremap' [-Wmissing-prototypes]
     190 | void __iomem *__wrap_ioremap(resource_size_t offset, unsigned long size)
         |               ^~~~~~~~~~~~~~
   drivers/nvdimm/../../tools/testing/nvdimm/test/iomap.c:196:15: warning: no previous prototype for '__wrap_ioremap_wc' [-Wmissing-prototypes]
     196 | void __iomem *__wrap_ioremap_wc(resource_size_t offset, unsigned long size)
         |               ^~~~~~~~~~~~~~~~~
   drivers/nvdimm/../../tools/testing/nvdimm/test/iomap.c: In function '__wrap_iounmap':
>> drivers/nvdimm/../../tools/testing/nvdimm/test/iomap.c:207:17: warning: passing argument 1 of 'iounmap' discards 'volatile' qualifier from pointer target type [-Wdiscarded-qualifiers]
     207 |  return iounmap(addr);
         |                 ^~~~
   In file included from arch/um/include/asm/io.h:7,
                    from include/linux/io.h:13,
                    from drivers/nvdimm/../../tools/testing/nvdimm/test/iomap.c:13:
   include/asm-generic/logic_io.h:37:28: note: expected 'void *' but argument is of type 'volatile void *'
      37 | void iounmap(void __iomem *addr);
         |              ~~~~~~~~~~~~~~^~~~
   drivers/nvdimm/../../tools/testing/nvdimm/test/iomap.c: At top level:
   drivers/nvdimm/../../tools/testing/nvdimm/test/iomap.c:211:6: warning: no previous prototype for '__wrap_memunmap' [-Wmissing-prototypes]
     211 | void __wrap_memunmap(void *addr)
         |      ^~~~~~~~~~~~~~~
   drivers/nvdimm/../../tools/testing/nvdimm/test/iomap.c:347:18: warning: no previous prototype for '__wrap___request_region' [-Wmissing-prototypes]
     347 | struct resource *__wrap___request_region(struct resource *parent,
         |                  ^~~~~~~~~~~~~~~~~~~~~~~
   drivers/nvdimm/../../tools/testing/nvdimm/test/iomap.c:355:5: warning: no previous prototype for '__wrap_insert_resource' [-Wmissing-prototypes]
     355 | int __wrap_insert_resource(struct resource *parent, struct resource *res)
         |     ^~~~~~~~~~~~~~~~~~~~~~
   drivers/nvdimm/../../tools/testing/nvdimm/test/iomap.c:363:5: warning: no previous prototype for '__wrap_remove_resource' [-Wmissing-prototypes]
     363 | int __wrap_remove_resource(struct resource *res)
         |     ^~~~~~~~~~~~~~~~~~~~~~
   drivers/nvdimm/../../tools/testing/nvdimm/test/iomap.c:371:18: warning: no previous prototype for '__wrap___devm_request_region' [-Wmissing-prototypes]
     371 | struct resource *__wrap___devm_request_region(struct device *dev,
         |                  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
   drivers/nvdimm/../../tools/testing/nvdimm/test/iomap.c:381:6: warning: no previous prototype for '__wrap___release_region' [-Wmissing-prototypes]
     381 | void __wrap___release_region(struct resource *parent, resource_size_t start,
         |      ^~~~~~~~~~~~~~~~~~~~~~~
   drivers/nvdimm/../../tools/testing/nvdimm/test/iomap.c:389:6: warning: no previous prototype for '__wrap___devm_release_region' [-Wmissing-prototypes]
     389 | void __wrap___devm_release_region(struct device *dev, struct resource *parent,
         |      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
   drivers/nvdimm/../../tools/testing/nvdimm/test/iomap.c:397:13: warning: no previous prototype for '__wrap_acpi_evaluate_object' [-Wmissing-prototypes]
     397 | acpi_status __wrap_acpi_evaluate_object(acpi_handle handle, acpi_string path,
         |             ^~~~~~~~~~~~~~~~~~~~~~~~~~~
   drivers/nvdimm/../../tools/testing/nvdimm/test/iomap.c:413:21: warning: no previous prototype for '__wrap_acpi_evaluate_dsm' [-Wmissing-prototypes]
     413 | union acpi_object * __wrap_acpi_evaluate_dsm(acpi_handle handle, const guid_t *guid,
         |                     ^~~~~~~~~~~~~~~~~~~~~~~~


vim +207 drivers/nvdimm/../../tools/testing/nvdimm/test/iomap.c

9d27a87ec9e131 Dan Williams 2015-07-10  201  
6bc756193ff61b Dan Williams 2015-06-17  202  void __wrap_iounmap(volatile void __iomem *addr)
6bc756193ff61b Dan Williams 2015-06-17  203  {
9bfa84969dd52b Dan Williams 2015-12-11  204  	struct nfit_test_resource *nfit_res = get_nfit_res((long) addr);
6bc756193ff61b Dan Williams 2015-06-17  205  	if (nfit_res)
6bc756193ff61b Dan Williams 2015-06-17  206  		return;
6bc756193ff61b Dan Williams 2015-06-17 @207  	return iounmap(addr);
6bc756193ff61b Dan Williams 2015-06-17  208  }
6bc756193ff61b Dan Williams 2015-06-17  209  EXPORT_SYMBOL(__wrap_iounmap);
6bc756193ff61b Dan Williams 2015-06-17  210  

:::::: The code at line 207 was first introduced by commit
:::::: 6bc756193ff61bf5e7b3cfedfbb0873bf40f8055 tools/testing/nvdimm: libnvdimm unit test infrastructure

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

             reply	other threads:[~2021-06-18 19:14 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-06-18 19:13 kernel test robot [this message]
2021-06-18 19:13 ` [linux-next:master 10919/11253] drivers/nvdimm/../../tools/testing/nvdimm/test/iomap.c:207:17: warning: passing argument 1 of 'iounmap' discards 'volatile' qualifier from pointer target type kernel test robot

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=202106190339.C2eUsszW-lkp@intel.com \
    --to=lkp@intel.com \
    --cc=johannes.berg@intel.com \
    --cc=kbuild-all@lists.01.org \
    --cc=linux-mm@kvack.org \
    --cc=richard@nod.at \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.