[linuxtv-media:master 141/161] drivers/vfio/pci/vfio_pci_rdwr.c:240:11: error: implicit declaration of function 'ioport_map'
11:33 a.m.
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(a)intel.com>
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(a)redhat.com>
:::::: CC: Alex Williamson <alex.williamson(a)redhat.com>
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
887
days inactive
887
days old
0 comments
1 participants
participants (1)
-
kbuild test robot