On 2019/8/9 20:17, kbuild test robot wrote:
> tree:
> https://kernel.googlesource.com/pub/scm/linux/kernel/git/herbert/cryptodev-2.6.git
> master
> head: ec9c7d19336ee98ecba8de80128aa405c45feebb
> commit: 62c455ca853e3e352e465d66a6cc39f1f88caa60 [124/144] crypto: hisilicon
> - add HiSilicon ZIP accelerator support
> config: sh-allmodconfig (attached as .config)
> compiler: sh4-linux-gcc (GCC) 7.4.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 62c455ca853e3e352e465d66a6cc39f1f88caa60
> # save the attached .config to linux build tree
> GCC_VERSION=7.4.0 make.cross ARCH=sh
>
> If you fix the issue, kindly add following tag
> Reported-by: kbuild test robot <l...@intel.com>
>
> All error/warnings (new ones prefixed by >>):
>
> In file included from include/linux/printk.h:332:0,
> from include/linux/kernel.h:15,
> from include/asm-generic/bug.h:18,
> from arch/sh/include/asm/bug.h:112,
> from include/linux/bug.h:5,
> from arch/sh/include/asm/uncached.h:5,
> from arch/sh/include/asm/page.h:49,
> from drivers/crypto/hisilicon/qm.c:3:
> drivers/crypto/hisilicon/qm.c: In function 'qm_mb':
>>> drivers/crypto/hisilicon/qm.c:338:26: warning: format '%llx' expects
>>> argument of type 'long long unsigned int', but argument 6 has type
>>> 'dma_addr_t {aka unsigned int}' [-Wformat=]
> dev_dbg(&qm->pdev->dev, "QM mailbox request to q%u: %u-%llx\n", queue,
Should be %pad for dma, I will fix this.
> ^
> include/linux/dynamic_debug.h:125:15: note: in definition of macro
> '__dynamic_func_call'
> func(&id, ##__VA_ARGS__); \
> ^~~~~~~~~~~
> include/linux/dynamic_debug.h:157:2: note: in expansion of macro
> '_dynamic_func_call'
> _dynamic_func_call(fmt,__dynamic_dev_dbg, \
> ^~~~~~~~~~~~~~~~~~
> include/linux/device.h:1503:2: note: in expansion of macro
> 'dynamic_dev_dbg'
> dynamic_dev_dbg(dev, dev_fmt(fmt), ##__VA_ARGS__)
> ^~~~~~~~~~~~~~~
> include/linux/device.h:1503:23: note: in expansion of macro 'dev_fmt'
> dynamic_dev_dbg(dev, dev_fmt(fmt), ##__VA_ARGS__)
> ^~~~~~~
>>> drivers/crypto/hisilicon/qm.c:338:2: note: in expansion of macro 'dev_dbg'
> dev_dbg(&qm->pdev->dev, "QM mailbox request to q%u: %u-%llx\n", queue,
> ^~~~~~~
> drivers/crypto/hisilicon/qm.c: In function 'qm_irq_register':
>>> drivers/crypto/hisilicon/qm.c:583:20: error: implicit declaration of
>>> function 'pci_irq_vector'; did you mean 'rcu_irq_enter'?
>>> [-Werror=implicit-function-declaration]
> ret = request_irq(pci_irq_vector(pdev, QM_EQ_EVENT_IRQ_VECTOR),
> ^~~~~~~~~~~~~~
> rcu_irq_enter
> In file included from include/linux/printk.h:332:0,
> from include/linux/kernel.h:15,
> from include/asm-generic/bug.h:18,
> from arch/sh/include/asm/bug.h:112,
> from include/linux/bug.h:5,
> from arch/sh/include/asm/uncached.h:5,
> from arch/sh/include/asm/page.h:49,
> from drivers/crypto/hisilicon/qm.c:3:
> drivers/crypto/hisilicon/qm.c: In function 'hisi_qm_create_qp':
>>> drivers/crypto/hisilicon/qm.c:879:16: warning: format '%lx' expects
>>> argument of type 'long unsigned int', but argument 6 has type 'size_t {aka
>>> unsigned int}' [-Wformat=]
> dev_dbg(dev, "allocate qp dma buf(va=%pK, dma=%pad, size=%lx)\n",
Should be size=%zx, will fix this.
> ^
> include/linux/dynamic_debug.h:125:15: note: in definition of macro
> '__dynamic_func_call'
> func(&id, ##__VA_ARGS__); \
> ^~~~~~~~~~~
> include/linux/dynamic_debug.h:157:2: note: in expansion of macro
> '_dynamic_func_call'
> _dynamic_func_call(fmt,__dynamic_dev_dbg, \
> ^~~~~~~~~~~~~~~~~~
> include/linux/device.h:1503:2: note: in expansion of macro
> 'dynamic_dev_dbg'
> dynamic_dev_dbg(dev, dev_fmt(fmt), ##__VA_ARGS__)
> ^~~~~~~~~~~~~~~
> include/linux/device.h:1503:23: note: in expansion of macro 'dev_fmt'
> dynamic_dev_dbg(dev, dev_fmt(fmt), ##__VA_ARGS__)
> ^~~~~~~
> drivers/crypto/hisilicon/qm.c:879:3: note: in expansion of macro 'dev_dbg'
> dev_dbg(dev, "allocate qp dma buf(va=%pK, dma=%pad, size=%lx)\n",
> ^~~~~~~
> drivers/crypto/hisilicon/qm.c: In function 'hisi_qm_init':
>>> drivers/crypto/hisilicon/qm.c:1156:8: error: implicit declaration of
>>> function 'pci_enable_device_mem'; did you mean 'pci_enable_device'?
>>> [-Werror=implicit-function-declaration]
> ret = pci_enable_device_mem(pdev);
> ^~~~~~~~~~~~~~~~~~~~~
> pci_enable_device
>>> drivers/crypto/hisilicon/qm.c:1162:8: error: implicit declaration of
>>> function 'pci_request_mem_regions'; did you mean 'pci_request_regions'?
>>> [-Werror=implicit-function-declaration]
> ret = pci_request_mem_regions(pdev, qm->dev_name);
> ^~~~~~~~~~~~~~~~~~~~~~~
> pci_request_regions
>>> drivers/crypto/hisilicon/qm.c:1185:8: error: implicit declaration of
>>> function 'pci_alloc_irq_vectors'; did you mean 'pci_alloc_consistent'?
>>> [-Werror=implicit-function-declaration]
> ret = pci_alloc_irq_vectors(pdev, num_vec, num_vec, PCI_IRQ_MSI);
> ^~~~~~~~~~~~~~~~~~~~~
> pci_alloc_consistent
>>> drivers/crypto/hisilicon/qm.c:1185:54: error: 'PCI_IRQ_MSI' undeclared
>>> (first use in this function); did you mean 'IRQ_MSK'?
> ret = pci_alloc_irq_vectors(pdev, num_vec, num_vec, PCI_IRQ_MSI);
> ^~~~~~~~~~~
> IRQ_MSK
> drivers/crypto/hisilicon/qm.c:1185:54: note: each undeclared identifier is
> reported only once for each function it appears in
>>> drivers/crypto/hisilicon/qm.c:1201:2: error: implicit declaration of
>>> function 'pci_free_irq_vectors'; did you mean 'pci_free_consistent'?
>>> [-Werror=implicit-function-declaration]
> pci_free_irq_vectors(pdev);
> ^~~~~~~~~~~~~~~~~~~~
> pci_free_consistent
>>> drivers/crypto/hisilicon/qm.c:1205:2: error: implicit declaration of
>>> function 'pci_release_mem_regions'; did you mean 'pci_release_regions'?
>>> [-Werror=implicit-function-declaration]
> pci_release_mem_regions(pdev);
> ^~~~~~~~~~~~~~~~~~~~~~~
> pci_release_regions
> In file included from include/linux/printk.h:332:0,
> from include/linux/kernel.h:15,
> from include/asm-generic/bug.h:18,
> from arch/sh/include/asm/bug.h:112,
> from include/linux/bug.h:5,
> from arch/sh/include/asm/uncached.h:5,
> from arch/sh/include/asm/page.h:49,
> from drivers/crypto/hisilicon/qm.c:3:
> drivers/crypto/hisilicon/qm.c: In function 'hisi_qm_start':
> drivers/crypto/hisilicon/qm.c:1425:16: warning: format '%lx' expects
> argument of type 'long unsigned int', but argument 6 has type 'size_t {aka
> unsigned int}' [-Wformat=]
> dev_dbg(dev, "allocate qm dma buf(va=%pK, dma=%pad, size=%lx)\n",
> ^
> include/linux/dynamic_debug.h:125:15: note: in definition of macro
> '__dynamic_func_call'
> func(&id, ##__VA_ARGS__); \
> ^~~~~~~~~~~
> include/linux/dynamic_debug.h:157:2: note: in expansion of macro
> '_dynamic_func_call'
> _dynamic_func_call(fmt,__dynamic_dev_dbg, \
> ^~~~~~~~~~~~~~~~~~
> include/linux/device.h:1503:2: note: in expansion of macro
> 'dynamic_dev_dbg'
> dynamic_dev_dbg(dev, dev_fmt(fmt), ##__VA_ARGS__)
> ^~~~~~~~~~~~~~~
> include/linux/device.h:1503:23: note: in expansion of macro 'dev_fmt'
> dynamic_dev_dbg(dev, dev_fmt(fmt), ##__VA_ARGS__)
> ^~~~~~~
> drivers/crypto/hisilicon/qm.c:1425:3: note: in expansion of macro 'dev_dbg'
> dev_dbg(dev, "allocate qm dma buf(va=%pK, dma=%pad, size=%lx)\n",
should be size=%zx.
> ^~~~~~~
> cc1: some warnings being treated as errors
>
> vim +/dev_dbg +338 drivers/crypto/hisilicon/qm.c
>
> 263c9959c9376e Zhou Wang 2019-08-02 331
> 263c9959c9376e Zhou Wang 2019-08-02 332 static int qm_mb(struct hisi_qm
> *qm, u8 cmd, dma_addr_t dma_addr, u16 queue,
> 263c9959c9376e Zhou Wang 2019-08-02 333 bool op)
> 263c9959c9376e Zhou Wang 2019-08-02 334 {
> 263c9959c9376e Zhou Wang 2019-08-02 335 struct qm_mailbox mailbox;
> 263c9959c9376e Zhou Wang 2019-08-02 336 int ret = 0;
> 263c9959c9376e Zhou Wang 2019-08-02 337
> 263c9959c9376e Zhou Wang 2019-08-02 @338 dev_dbg(&qm->pdev->dev, "QM
> mailbox request to q%u: %u-%llx\n", queue,
> 263c9959c9376e Zhou Wang 2019-08-02 339 cmd, dma_addr);
> 263c9959c9376e Zhou Wang 2019-08-02 340
> 263c9959c9376e Zhou Wang 2019-08-02 341 mailbox.w0 = cmd |
> 263c9959c9376e Zhou Wang 2019-08-02 342 (op ? 0x1 <<
> QM_MB_OP_SHIFT : 0) |
> 263c9959c9376e Zhou Wang 2019-08-02 343 (0x1 <<
> QM_MB_BUSY_SHIFT);
> 263c9959c9376e Zhou Wang 2019-08-02 344 mailbox.queue_num = queue;
> 263c9959c9376e Zhou Wang 2019-08-02 345 mailbox.base_l =
> lower_32_bits(dma_addr);
> 263c9959c9376e Zhou Wang 2019-08-02 346 mailbox.base_h =
> upper_32_bits(dma_addr);
> 263c9959c9376e Zhou Wang 2019-08-02 347 mailbox.rsvd = 0;
> 263c9959c9376e Zhou Wang 2019-08-02 348
> 263c9959c9376e Zhou Wang 2019-08-02 349 mutex_lock(&qm->mailbox_lock);
> 263c9959c9376e Zhou Wang 2019-08-02 350
> 263c9959c9376e Zhou Wang 2019-08-02 351 if
> (unlikely(qm_wait_mb_ready(qm))) {
> 263c9959c9376e Zhou Wang 2019-08-02 352 ret = -EBUSY;
> 263c9959c9376e Zhou Wang 2019-08-02 353 dev_err(&qm->pdev->dev,
> "QM mailbox is busy to start!\n");
> 263c9959c9376e Zhou Wang 2019-08-02 354 goto busy_unlock;
> 263c9959c9376e Zhou Wang 2019-08-02 355 }
> 263c9959c9376e Zhou Wang 2019-08-02 356
> 263c9959c9376e Zhou Wang 2019-08-02 357 qm_mb_write(qm, &mailbox);
> 263c9959c9376e Zhou Wang 2019-08-02 358
> 263c9959c9376e Zhou Wang 2019-08-02 359 if
> (unlikely(qm_wait_mb_ready(qm))) {
> 263c9959c9376e Zhou Wang 2019-08-02 360 ret = -EBUSY;
> 263c9959c9376e Zhou Wang 2019-08-02 361 dev_err(&qm->pdev->dev,
> "QM mailbox operation timeout!\n");
> 263c9959c9376e Zhou Wang 2019-08-02 362 goto busy_unlock;
> 263c9959c9376e Zhou Wang 2019-08-02 363 }
> 263c9959c9376e Zhou Wang 2019-08-02 364
> 263c9959c9376e Zhou Wang 2019-08-02 365 busy_unlock:
> 263c9959c9376e Zhou Wang 2019-08-02 366 mutex_unlock(&qm->mailbox_lock);
> 263c9959c9376e Zhou Wang 2019-08-02 367
> 263c9959c9376e Zhou Wang 2019-08-02 368 return ret;
> 263c9959c9376e Zhou Wang 2019-08-02 369 }
> 263c9959c9376e Zhou Wang 2019-08-02 370
>
> :::::: The code at line 338 was first introduced by commit
> :::::: 263c9959c9376ec0217d6adc61222a53469eed3c crypto: hisilicon - add queue
> management driver for HiSilicon QM module
>
> :::::: TO: Zhou Wang <wangzh...@hisilicon.com>
> :::::: CC: Herbert Xu <herb...@gondor.apana.org.au>
>
> ---
> 0-DAY kernel test infrastructure Open Source Technology Center
> https://lists.01.org/pipermail/kbuild-all Intel Corporation
>
