[linuxtv-media:master 141/161] drivers/vfio/pci/vfio_pci_rdwr.c:240:11: error: implicit declaration of function 'ioport_map'

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Mauro,

First bad commit (maybe != root cause):

tree:   git://linuxtv.org/media_tree.git master
head:   63735af86d645a47903f912ae89957507edd917b
commit: df823a8208c434eee6e4e9aa016c956d0968e2e2 [141/161] media: cec: rename CEC platform drivers config options
config: nios2-allmodconfig (attached as .config)
compiler: nios2-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 df823a8208c434eee6e4e9aa016c956d0968e2e2
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day GCC_VERSION=9.3.0 make.cross ARCH=nios2 

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

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

   drivers/vfio/pci/vfio_pci_rdwr.c: In function 'vfio_pci_vga_rw':
>> drivers/vfio/pci/vfio_pci_rdwr.c:240:11: error: implicit declaration of function 'ioport_map' [-Werror=implicit-function-declaration]
     240 |   iomem = ioport_map(0x3b0, 0x3bb - 0x3b0 + 1);
         |           ^~~~~~~~~~
>> drivers/vfio/pci/vfio_pci_rdwr.c:240:9: warning: assignment to 'void *' from 'int' makes pointer from integer without a cast [-Wint-conversion]
     240 |   iomem = ioport_map(0x3b0, 0x3bb - 0x3b0 + 1);
         |         ^
   drivers/vfio/pci/vfio_pci_rdwr.c:247:9: warning: assignment to 'void *' from 'int' makes pointer from integer without a cast [-Wint-conversion]
     247 |   iomem = ioport_map(0x3c0, 0x3df - 0x3c0 + 1);
         |         ^
>> drivers/vfio/pci/vfio_pci_rdwr.c:261:15: error: implicit declaration of function 'ioport_unmap'; did you mean 'iommu_unmap'? [-Werror=implicit-function-declaration]
     261 |   is_ioport ? ioport_unmap(iomem) : iounmap(iomem);
         |               ^~~~~~~~~~~~
         |               iommu_unmap
   cc1: some warnings being treated as errors
--
   drivers/gpio/gpio-amd8111.c: In function 'amd_gpio_init':
>> drivers/gpio/gpio-amd8111.c:215:10: error: implicit declaration of function 'ioport_map'; did you mean 'ioremap'? [-Werror=implicit-function-declaration]
     215 |  gp.pm = ioport_map(gp.pmbase + PMBASE_OFFSET, PMBASE_SIZE);
         |          ^~~~~~~~~~
         |          ioremap
>> drivers/gpio/gpio-amd8111.c:215:8: warning: assignment to 'void *' from 'int' makes pointer from integer without a cast [-Wint-conversion]
     215 |  gp.pm = ioport_map(gp.pmbase + PMBASE_OFFSET, PMBASE_SIZE);
         |        ^
>> drivers/gpio/gpio-amd8111.c:231:3: error: implicit declaration of function 'ioport_unmap' [-Werror=implicit-function-declaration]
     231 |   ioport_unmap(gp.pm);
         |   ^~~~~~~~~~~~
   cc1: some warnings being treated as errors

vim +/ioport_map +240 drivers/vfio/pci/vfio_pci_rdwr.c

84237a826b261d Alex Williamson   2013-02-18  213  
84237a826b261d Alex Williamson   2013-02-18  214  ssize_t vfio_pci_vga_rw(struct vfio_pci_device *vdev, char __user *buf,
84237a826b261d Alex Williamson   2013-02-18  215  			       size_t count, loff_t *ppos, bool iswrite)
84237a826b261d Alex Williamson   2013-02-18  216  {
84237a826b261d Alex Williamson   2013-02-18  217  	int ret;
84237a826b261d Alex Williamson   2013-02-18  218  	loff_t off, pos = *ppos & VFIO_PCI_OFFSET_MASK;
84237a826b261d Alex Williamson   2013-02-18  219  	void __iomem *iomem = NULL;
84237a826b261d Alex Williamson   2013-02-18  220  	unsigned int rsrc;
84237a826b261d Alex Williamson   2013-02-18  221  	bool is_ioport;
84237a826b261d Alex Williamson   2013-02-18  222  	ssize_t done;
84237a826b261d Alex Williamson   2013-02-18  223  
84237a826b261d Alex Williamson   2013-02-18  224  	if (!vdev->has_vga)
84237a826b261d Alex Williamson   2013-02-18  225  		return -EINVAL;
84237a826b261d Alex Williamson   2013-02-18  226  
45e86971448943 Arnd Bergmann     2016-12-30  227  	if (pos > 0xbfffful)
45e86971448943 Arnd Bergmann     2016-12-30  228  		return -EINVAL;
45e86971448943 Arnd Bergmann     2016-12-30  229  
45e86971448943 Arnd Bergmann     2016-12-30  230  	switch ((u32)pos) {
84237a826b261d Alex Williamson   2013-02-18  231  	case 0xa0000 ... 0xbffff:
84237a826b261d Alex Williamson   2013-02-18  232  		count = min(count, (size_t)(0xc0000 - pos));
4bdc0d676a6431 Christoph Hellwig 2020-01-06  233  		iomem = ioremap(0xa0000, 0xbffff - 0xa0000 + 1);
84237a826b261d Alex Williamson   2013-02-18  234  		off = pos - 0xa0000;
84237a826b261d Alex Williamson   2013-02-18  235  		rsrc = VGA_RSRC_LEGACY_MEM;
84237a826b261d Alex Williamson   2013-02-18  236  		is_ioport = false;
84237a826b261d Alex Williamson   2013-02-18  237  		break;
84237a826b261d Alex Williamson   2013-02-18  238  	case 0x3b0 ... 0x3bb:
84237a826b261d Alex Williamson   2013-02-18  239  		count = min(count, (size_t)(0x3bc - pos));
84237a826b261d Alex Williamson   2013-02-18 @240  		iomem = ioport_map(0x3b0, 0x3bb - 0x3b0 + 1);
84237a826b261d Alex Williamson   2013-02-18  241  		off = pos - 0x3b0;
84237a826b261d Alex Williamson   2013-02-18  242  		rsrc = VGA_RSRC_LEGACY_IO;
84237a826b261d Alex Williamson   2013-02-18  243  		is_ioport = true;
84237a826b261d Alex Williamson   2013-02-18  244  		break;
84237a826b261d Alex Williamson   2013-02-18  245  	case 0x3c0 ... 0x3df:
84237a826b261d Alex Williamson   2013-02-18  246  		count = min(count, (size_t)(0x3e0 - pos));
84237a826b261d Alex Williamson   2013-02-18  247  		iomem = ioport_map(0x3c0, 0x3df - 0x3c0 + 1);
84237a826b261d Alex Williamson   2013-02-18  248  		off = pos - 0x3c0;
84237a826b261d Alex Williamson   2013-02-18  249  		rsrc = VGA_RSRC_LEGACY_IO;
84237a826b261d Alex Williamson   2013-02-18  250  		is_ioport = true;
84237a826b261d Alex Williamson   2013-02-18  251  		break;
84237a826b261d Alex Williamson   2013-02-18  252  	default:
84237a826b261d Alex Williamson   2013-02-18  253  		return -EINVAL;
84237a826b261d Alex Williamson   2013-02-18  254  	}
84237a826b261d Alex Williamson   2013-02-18  255  
84237a826b261d Alex Williamson   2013-02-18  256  	if (!iomem)
84237a826b261d Alex Williamson   2013-02-18  257  		return -ENOMEM;
84237a826b261d Alex Williamson   2013-02-18  258  
84237a826b261d Alex Williamson   2013-02-18  259  	ret = vga_get_interruptible(vdev->pdev, rsrc);
84237a826b261d Alex Williamson   2013-02-18  260  	if (ret) {
84237a826b261d Alex Williamson   2013-02-18 @261  		is_ioport ? ioport_unmap(iomem) : iounmap(iomem);
84237a826b261d Alex Williamson   2013-02-18  262  		return ret;
84237a826b261d Alex Williamson   2013-02-18  263  	}
84237a826b261d Alex Williamson   2013-02-18  264  
84237a826b261d Alex Williamson   2013-02-18  265  	done = do_io_rw(iomem, buf, off, count, 0, 0, iswrite);
84237a826b261d Alex Williamson   2013-02-18  266  
84237a826b261d Alex Williamson   2013-02-18  267  	vga_put(vdev->pdev, rsrc);
84237a826b261d Alex Williamson   2013-02-18  268  
84237a826b261d Alex Williamson   2013-02-18  269  	is_ioport ? ioport_unmap(iomem) : iounmap(iomem);
84237a826b261d Alex Williamson   2013-02-18  270  
84237a826b261d Alex Williamson   2013-02-18  271  	if (done >= 0)
84237a826b261d Alex Williamson   2013-02-18  272  		*ppos += done;
84237a826b261d Alex Williamson   2013-02-18  273  
84237a826b261d Alex Williamson   2013-02-18  274  	return done;
84237a826b261d Alex Williamson   2013-02-18  275  }
30656177c40804 Alex Williamson   2018-03-21  276  

:::::: The code at line 240 was first introduced by commit
:::::: 84237a826b261de7ddd3d09ee53ee68cb4138937 vfio-pci: Add support for VGA region access

:::::: TO: Alex Williamson <alex.williamson@xxxxxxxxxx>
:::::: CC: Alex Williamson <alex.williamson@xxxxxxxxxx>

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

Attachment: .config.gz
Description: application/gzip


[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]

  Powered by Linux