From: kernel test robot <lkp@intel.com>
To: Koichiro Den <den@valinux.co.jp>,
vkoul@kernel.org, mani@kernel.org, Frank.Li@nxp.com,
jingoohan1@gmail.com, lpieralisi@kernel.org,
kwilczynski@kernel.org, robh@kernel.org, bhelgaas@google.com
Cc: oe-kbuild-all@lists.linux.dev, dmaengine@vger.kernel.org,
linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH v3 09/11] PCI: endpoint: pci-epf-test: Add smoke test for EPC remote resource API
Date: Thu, 5 Feb 2026 08:01:43 +0800 [thread overview]
Message-ID: <202602050741.nyI2oa7X-lkp@intel.com> (raw)
In-Reply-To: <20260204145440.950609-10-den@valinux.co.jp>
Hi Koichiro,
kernel test robot noticed the following build warnings:
[auto build test WARNING on pci/next]
[also build test WARNING on next-20260204]
[cannot apply to vkoul-dmaengine/next pci/for-linus linus/master v6.19-rc8]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch#_base_tree_information]
url: https://github.com/intel-lab-lkp/linux/commits/Koichiro-Den/dmaengine-Add-hw_id-to-dma_slave_caps/20260204-230604
base: https://git.kernel.org/pub/scm/linux/kernel/git/pci/pci.git next
patch link: https://lore.kernel.org/r/20260204145440.950609-10-den%40valinux.co.jp
patch subject: [PATCH v3 09/11] PCI: endpoint: pci-epf-test: Add smoke test for EPC remote resource API
config: i386-randconfig-003-20260205 (https://download.01.org/0day-ci/archive/20260205/202602050741.nyI2oa7X-lkp@intel.com/config)
compiler: gcc-13 (Debian 13.3.0-16) 13.3.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20260205/202602050741.nyI2oa7X-lkp@intel.com/reproduce)
If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202602050741.nyI2oa7X-lkp@intel.com/
All warnings (new ones prefixed by >>):
In file included from include/linux/device.h:15,
from include/linux/dmaengine.h:8,
from drivers/pci/endpoint/functions/pci-epf-test.c:11:
drivers/pci/endpoint/functions/pci-epf-test.c: In function 'pci_epf_test_epc_api':
>> drivers/pci/endpoint/functions/pci-epf-test.c:1012:33: warning: format '%llu' expects argument of type 'long long unsigned int', but argument 6 has type 'resource_size_t' {aka 'unsigned int'} [-Wformat=]
1012 | "Invalid remote resource[%d] (type=%d phys=%pa size=%llu)\n",
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/dev_printk.h:110:30: note: in definition of macro 'dev_printk_index_wrap'
110 | _p_func(dev, fmt, ##__VA_ARGS__); \
| ^~~
include/linux/dev_printk.h:154:56: note: in expansion of macro 'dev_fmt'
154 | dev_printk_index_wrap(_dev_err, KERN_ERR, dev, dev_fmt(fmt), ##__VA_ARGS__)
| ^~~~~~~
drivers/pci/endpoint/functions/pci-epf-test.c:1011:25: note: in expansion of macro 'dev_err'
1011 | dev_err(dev,
| ^~~~~~~
drivers/pci/endpoint/functions/pci-epf-test.c:1012:88: note: format string is defined here
1012 | "Invalid remote resource[%d] (type=%d phys=%pa size=%llu)\n",
| ~~~^
| |
| long long unsigned int
| %u
drivers/pci/endpoint/functions/pci-epf-test.c:1020:33: warning: format '%llu' expects argument of type 'long long unsigned int', but argument 5 has type 'resource_size_t' {aka 'unsigned int'} [-Wformat=]
1020 | "Remote resource[%d] overflow (phys=%pa size=%llu)\n",
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/dev_printk.h:110:30: note: in definition of macro 'dev_printk_index_wrap'
110 | _p_func(dev, fmt, ##__VA_ARGS__); \
| ^~~
include/linux/dev_printk.h:154:56: note: in expansion of macro 'dev_fmt'
154 | dev_printk_index_wrap(_dev_err, KERN_ERR, dev, dev_fmt(fmt), ##__VA_ARGS__)
| ^~~~~~~
drivers/pci/endpoint/functions/pci-epf-test.c:1019:25: note: in expansion of macro 'dev_err'
1019 | dev_err(dev,
| ^~~~~~~
drivers/pci/endpoint/functions/pci-epf-test.c:1020:81: note: format string is defined here
1020 | "Remote resource[%d] overflow (phys=%pa size=%llu)\n",
| ~~~^
| |
| long long unsigned int
| %u
vim +1012 drivers/pci/endpoint/functions/pci-epf-test.c
972
973 static void pci_epf_test_epc_api(struct pci_epf_test *epf_test,
974 struct pci_epf_test_reg *reg)
975 {
976 struct pci_epc_remote_resource *resources = NULL;
977 u32 status = le32_to_cpu(reg->status);
978 struct pci_epf *epf = epf_test->epf;
979 struct device *dev = &epf->dev;
980 struct pci_epc *epc = epf->epc;
981 int num_resources;
982 int ret, i;
983
984 num_resources = pci_epc_get_remote_resources(epc, epf->func_no,
985 epf->vfunc_no, NULL, 0);
986 if (num_resources == -EOPNOTSUPP || num_resources == 0)
987 goto out_success;
988 if (num_resources < 0)
989 goto err;
990
991 resources = kcalloc(num_resources, sizeof(*resources), GFP_KERNEL);
992 if (!resources)
993 goto err;
994
995 ret = pci_epc_get_remote_resources(epc, epf->func_no, epf->vfunc_no,
996 resources, num_resources);
997 if (ret < 0) {
998 dev_err(dev, "EPC remote resource query failed: %d\n", ret);
999 goto err_free;
1000 }
1001 if (ret > num_resources) {
1002 dev_err(dev, "EPC API returned %d resources (max %d)\n",
1003 ret, num_resources);
1004 goto err_free;
1005 }
1006
1007 for (i = 0; i < ret; i++) {
1008 struct pci_epc_remote_resource *res = &resources[i];
1009
1010 if (!res->phys_addr || !res->size) {
1011 dev_err(dev,
> 1012 "Invalid remote resource[%d] (type=%d phys=%pa size=%llu)\n",
1013 i, res->type, &res->phys_addr, res->size);
1014 goto err_free;
1015 }
1016
1017 /* Guard against address overflow */
1018 if (res->phys_addr + res->size < res->phys_addr) {
1019 dev_err(dev,
1020 "Remote resource[%d] overflow (phys=%pa size=%llu)\n",
1021 i, &res->phys_addr, res->size);
1022 goto err_free;
1023 }
1024
1025 switch (res->type) {
1026 case PCI_EPC_RR_DMA_CTRL_MMIO:
1027 /* Generic checks above are sufficient. */
1028 break;
1029 case PCI_EPC_RR_DMA_CHAN_DESC:
1030 /*
1031 * hw_chan_id and ep2rc are informational. No extra validation
1032 * beyond the generic checks above is needed.
1033 */
1034 break;
1035 default:
1036 dev_err(dev, "Unknown remote resource type %d\n", res->type);
1037 goto err_free;
1038 }
1039 }
1040
1041 out_success:
1042 kfree(resources);
1043 status |= STATUS_EPC_API_SUCCESS;
1044 reg->status = cpu_to_le32(status);
1045 return;
1046
1047 err_free:
1048 kfree(resources);
1049 err:
1050 status |= STATUS_EPC_API_FAIL;
1051 reg->status = cpu_to_le32(status);
1052 }
1053
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
next prev parent reply other threads:[~2026-02-05 0:02 UTC|newest]
Thread overview: 32+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-02-04 14:54 [PATCH v3 00/11] dmaengine, PCI: endpoint: Enable remote use of integrated DesignWare eDMA Koichiro Den
2026-02-04 14:54 ` [PATCH v3 01/11] dmaengine: Add hw_id to dma_slave_caps Koichiro Den
2026-02-04 19:39 ` Frank Li
2026-02-05 6:46 ` Koichiro Den
2026-02-05 16:04 ` Frank Li
2026-02-04 14:54 ` [PATCH v3 02/11] dmaengine: dw-edma: Report channel hw_id in dma_slave_caps Koichiro Den
2026-02-04 14:54 ` [PATCH v3 03/11] dmaengine: dw-edma: Add per-channel interrupt routing control Koichiro Den
2026-02-04 17:42 ` Frank Li
2026-02-05 6:48 ` Koichiro Den
2026-02-05 16:05 ` Frank Li
2026-02-04 14:54 ` [PATCH v3 04/11] dmaengine: Add selfirq callback registration API Koichiro Den
2026-02-04 17:46 ` Frank Li
2026-02-05 6:50 ` Koichiro Den
2026-02-05 16:07 ` Frank Li
2026-02-04 14:54 ` [PATCH v3 05/11] dmaengine: dw-edma: Implement dmaengine selfirq callbacks using interrupt emulation Koichiro Den
2026-02-04 14:54 ` [PATCH v3 06/11] PCI: endpoint: Add remote resource query API Koichiro Den
2026-02-04 17:55 ` Frank Li
2026-02-05 6:53 ` Koichiro Den
2026-02-05 16:10 ` Frank Li
2026-02-04 14:54 ` [PATCH v3 07/11] PCI: dwc: Record integrated eDMA register window Koichiro Den
2026-02-04 17:57 ` Frank Li
2026-02-04 14:54 ` [PATCH v3 08/11] PCI: dwc: ep: Report integrated DWC eDMA remote resources Koichiro Den
2026-02-04 18:06 ` Frank Li
2026-02-05 6:58 ` Koichiro Den
2026-02-05 16:11 ` Frank Li
2026-02-04 14:54 ` [PATCH v3 09/11] PCI: endpoint: pci-epf-test: Add smoke test for EPC remote resource API Koichiro Den
2026-02-04 19:37 ` Frank Li
2026-02-05 7:01 ` Koichiro Den
2026-02-05 0:01 ` kernel test robot [this message]
2026-02-05 2:37 ` kernel test robot
2026-02-04 14:54 ` [PATCH v3 10/11] misc: pci_endpoint_test: Add EPC remote resource API test ioctl Koichiro Den
2026-02-04 14:54 ` [PATCH v3 11/11] selftests: pci_endpoint: Add EPC remote resource API test Koichiro Den
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=202602050741.nyI2oa7X-lkp@intel.com \
--to=lkp@intel.com \
--cc=Frank.Li@nxp.com \
--cc=bhelgaas@google.com \
--cc=den@valinux.co.jp \
--cc=dmaengine@vger.kernel.org \
--cc=jingoohan1@gmail.com \
--cc=kwilczynski@kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pci@vger.kernel.org \
--cc=lpieralisi@kernel.org \
--cc=mani@kernel.org \
--cc=oe-kbuild-all@lists.linux.dev \
--cc=robh@kernel.org \
--cc=vkoul@kernel.org \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox