From: kernel test robot <lkp@intel.com>
To: admiyo@os.amperecomputing.com,
Jeremy Kerr <jk@codeconstruct.com.au>,
Matt Johnston <matt@codeconstruct.com.au>,
"David S. Miller" <davem@davemloft.net>,
Eric Dumazet <edumazet@google.com>,
Jakub Kicinski <kuba@kernel.org>, Paolo Abeni <pabeni@redhat.com>
Cc: llvm@lists.linux.dev, oe-kbuild-all@lists.linux.dev,
netdev@vger.kernel.org, linux-kernel@vger.kernel.org,
Sudeep Holla <sudeep.holla@arm.com>,
Jonathan Cameron <Jonathan.Cameron@huawei.com>,
Huisong Li <lihuisong@huawei.com>
Subject: Re: [PATCH v6 2/2] mctp pcc: Implement MCTP over PCC Transport
Date: Thu, 31 Oct 2024 21:09:51 +0800 [thread overview]
Message-ID: <202410312048.W6PV1dIU-lkp@intel.com> (raw)
In-Reply-To: <20241029165414.58746-3-admiyo@os.amperecomputing.com>
Hi,
kernel test robot noticed the following build errors:
[auto build test ERROR on rafael-pm/linux-next]
[also build test ERROR on rafael-pm/bleeding-edge linus/master v6.12-rc5 next-20241031]
[cannot apply to horms-ipvs/master]
[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/admiyo-os-amperecomputing-com/mctp-pcc-Check-before-sending-MCTP-PCC-response-ACK/20241030-005644
base: https://git.kernel.org/pub/scm/linux/kernel/git/rafael/linux-pm.git linux-next
patch link: https://lore.kernel.org/r/20241029165414.58746-3-admiyo%40os.amperecomputing.com
patch subject: [PATCH v6 2/2] mctp pcc: Implement MCTP over PCC Transport
config: hexagon-allmodconfig (https://download.01.org/0day-ci/archive/20241031/202410312048.W6PV1dIU-lkp@intel.com/config)
compiler: clang version 20.0.0git (https://github.com/llvm/llvm-project 639a7ac648f1e50ccd2556e17d401c04f9cce625)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20241031/202410312048.W6PV1dIU-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/202410312048.W6PV1dIU-lkp@intel.com/
All error/warnings (new ones prefixed by >>):
In file included from drivers/net/mctp/mctp-pcc.c:12:
In file included from include/linux/if_arp.h:22:
In file included from include/linux/skbuff.h:17:
In file included from include/linux/bvec.h:10:
In file included from include/linux/highmem.h:10:
In file included from include/linux/mm.h:2213:
include/linux/vmstat.h:518:36: warning: arithmetic between different enumeration types ('enum node_stat_item' and 'enum lru_list') [-Wenum-enum-conversion]
518 | return node_stat_name(NR_LRU_BASE + lru) + 3; // skip "nr_"
| ~~~~~~~~~~~ ^ ~~~
In file included from drivers/net/mctp/mctp-pcc.c:12:
In file included from include/linux/if_arp.h:22:
In file included from include/linux/skbuff.h:17:
In file included from include/linux/bvec.h:10:
In file included from include/linux/highmem.h:12:
In file included from include/linux/hardirq.h:11:
In file included from ./arch/hexagon/include/generated/asm/hardirq.h:1:
In file included from include/asm-generic/hardirq.h:17:
In file included from include/linux/irq.h:20:
In file included from include/linux/io.h:14:
In file included from arch/hexagon/include/asm/io.h:328:
include/asm-generic/io.h:548:31: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
548 | val = __raw_readb(PCI_IOBASE + addr);
| ~~~~~~~~~~ ^
include/asm-generic/io.h:561:61: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
561 | val = __le16_to_cpu((__le16 __force)__raw_readw(PCI_IOBASE + addr));
| ~~~~~~~~~~ ^
include/uapi/linux/byteorder/little_endian.h:37:51: note: expanded from macro '__le16_to_cpu'
37 | #define __le16_to_cpu(x) ((__force __u16)(__le16)(x))
| ^
In file included from drivers/net/mctp/mctp-pcc.c:12:
In file included from include/linux/if_arp.h:22:
In file included from include/linux/skbuff.h:17:
In file included from include/linux/bvec.h:10:
In file included from include/linux/highmem.h:12:
In file included from include/linux/hardirq.h:11:
In file included from ./arch/hexagon/include/generated/asm/hardirq.h:1:
In file included from include/asm-generic/hardirq.h:17:
In file included from include/linux/irq.h:20:
In file included from include/linux/io.h:14:
In file included from arch/hexagon/include/asm/io.h:328:
include/asm-generic/io.h:574:61: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
574 | val = __le32_to_cpu((__le32 __force)__raw_readl(PCI_IOBASE + addr));
| ~~~~~~~~~~ ^
include/uapi/linux/byteorder/little_endian.h:35:51: note: expanded from macro '__le32_to_cpu'
35 | #define __le32_to_cpu(x) ((__force __u32)(__le32)(x))
| ^
In file included from drivers/net/mctp/mctp-pcc.c:12:
In file included from include/linux/if_arp.h:22:
In file included from include/linux/skbuff.h:17:
In file included from include/linux/bvec.h:10:
In file included from include/linux/highmem.h:12:
In file included from include/linux/hardirq.h:11:
In file included from ./arch/hexagon/include/generated/asm/hardirq.h:1:
In file included from include/asm-generic/hardirq.h:17:
In file included from include/linux/irq.h:20:
In file included from include/linux/io.h:14:
In file included from arch/hexagon/include/asm/io.h:328:
include/asm-generic/io.h:585:33: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
585 | __raw_writeb(value, PCI_IOBASE + addr);
| ~~~~~~~~~~ ^
include/asm-generic/io.h:595:59: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
595 | __raw_writew((u16 __force)cpu_to_le16(value), PCI_IOBASE + addr);
| ~~~~~~~~~~ ^
include/asm-generic/io.h:605:59: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
605 | __raw_writel((u32 __force)cpu_to_le32(value), PCI_IOBASE + addr);
| ~~~~~~~~~~ ^
In file included from drivers/net/mctp/mctp-pcc.c:21:
>> include/acpi/acpi_drivers.h:72:43: warning: declaration of 'struct acpi_pci_root' will not be visible outside of this function [-Wvisibility]
72 | struct pci_bus *pci_acpi_scan_root(struct acpi_pci_root *root);
| ^
>> drivers/net/mctp/mctp-pcc.c:237:32: error: incomplete definition of type 'struct acpi_device'
237 | struct device *dev = &acpi_dev->dev;
| ~~~~~~~~^
include/linux/acpi.h:801:8: note: forward declaration of 'struct acpi_device'
801 | struct acpi_device;
| ^
>> drivers/net/mctp/mctp-pcc.c:246:3: error: call to undeclared function 'acpi_device_hid'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
246 | acpi_device_hid(acpi_dev));
| ^
drivers/net/mctp/mctp-pcc.c:246:3: note: did you mean 'acpi_device_dep'?
include/acpi/acpi_bus.h:41:6: note: 'acpi_device_dep' declared here
41 | bool acpi_device_dep(acpi_handle target, acpi_handle match);
| ^
>> drivers/net/mctp/mctp-pcc.c:246:3: warning: format specifies type 'char *' but the argument has type 'int' [-Wformat]
245 | dev_dbg(dev, "Adding mctp_pcc device for HID %s\n",
| ~~
| %d
246 | acpi_device_hid(acpi_dev));
| ^~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/dev_printk.h:165:39: note: expanded from macro 'dev_dbg'
165 | dynamic_dev_dbg(dev, dev_fmt(fmt), ##__VA_ARGS__)
| ~~~ ^~~~~~~~~~~
include/linux/dynamic_debug.h:274:19: note: expanded from macro 'dynamic_dev_dbg'
274 | dev, fmt, ##__VA_ARGS__)
| ~~~ ^~~~~~~~~~~
include/linux/dynamic_debug.h:250:59: note: expanded from macro '_dynamic_func_call'
250 | _dynamic_func_call_cls(_DPRINTK_CLASS_DFLT, fmt, func, ##__VA_ARGS__)
| ^~~~~~~~~~~
include/linux/dynamic_debug.h:248:65: note: expanded from macro '_dynamic_func_call_cls'
248 | __dynamic_func_call_cls(__UNIQUE_ID(ddebug), cls, fmt, func, ##__VA_ARGS__)
| ^~~~~~~~~~~
include/linux/dynamic_debug.h:224:15: note: expanded from macro '__dynamic_func_call_cls'
224 | func(&id, ##__VA_ARGS__); \
| ^~~~~~~~~~~
>> drivers/net/mctp/mctp-pcc.c:247:15: error: call to undeclared function 'acpi_device_handle'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
247 | dev_handle = acpi_device_handle(acpi_dev);
| ^
>> drivers/net/mctp/mctp-pcc.c:247:13: error: incompatible integer to pointer conversion assigning to 'acpi_handle' (aka 'void *') from 'int' [-Wint-conversion]
247 | dev_handle = acpi_device_handle(acpi_dev);
| ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/net/mctp/mctp-pcc.c:290:10: error: incomplete definition of type 'struct acpi_device'
290 | acpi_dev->driver_data = mctp_pcc_ndev;
| ~~~~~~~~^
include/linux/acpi.h:801:8: note: forward declaration of 'struct acpi_device'
801 | struct acpi_device;
| ^
>> drivers/net/mctp/mctp-pcc.c:317:27: error: variable has incomplete type 'struct acpi_driver'
317 | static struct acpi_driver mctp_pcc_driver = {
| ^
drivers/net/mctp/mctp-pcc.c:317:15: note: forward declaration of 'struct acpi_driver'
317 | static struct acpi_driver mctp_pcc_driver = {
| ^
>> drivers/net/mctp/mctp-pcc.c:326:1: error: type specifier missing, defaults to 'int'; ISO C99 and later do not support implicit int [-Wimplicit-int]
326 | module_acpi_driver(mctp_pcc_driver);
| ^
| int
>> drivers/net/mctp/mctp-pcc.c:326:20: error: a parameter list without types is only allowed in a function definition
326 | module_acpi_driver(mctp_pcc_driver);
| ^
9 warnings and 8 errors generated.
Kconfig warnings: (for reference only)
WARNING: unmet direct dependencies detected for GET_FREE_REGION
Depends on [n]: SPARSEMEM [=n]
Selected by [m]:
- RESOURCE_KUNIT_TEST [=m] && RUNTIME_TESTING_MENU [=y] && KUNIT [=m]
vim +237 drivers/net/mctp/mctp-pcc.c
231
232 static int mctp_pcc_driver_add(struct acpi_device *acpi_dev)
233 {
234 struct mctp_pcc_lookup_context context = {0, 0, 0};
235 struct mctp_pcc_hw_addr mctp_pcc_hw_addr;
236 struct mctp_pcc_ndev *mctp_pcc_ndev;
> 237 struct device *dev = &acpi_dev->dev;
238 struct net_device *ndev;
239 acpi_handle dev_handle;
240 acpi_status status;
241 int mctp_pcc_mtu;
242 char name[32];
243 int rc;
244
245 dev_dbg(dev, "Adding mctp_pcc device for HID %s\n",
> 246 acpi_device_hid(acpi_dev));
> 247 dev_handle = acpi_device_handle(acpi_dev);
248 status = acpi_walk_resources(dev_handle, "_CRS", lookup_pcct_indices,
249 &context);
250 if (!ACPI_SUCCESS(status)) {
251 dev_err(dev, "FAILURE to lookup PCC indexes from CRS");
252 return -EINVAL;
253 }
254
255 //inbox initialization
256 snprintf(name, sizeof(name), "mctpipcc%d", context.inbox_index);
257 ndev = alloc_netdev(sizeof(struct mctp_pcc_ndev), name, NET_NAME_ENUM,
258 mctp_pcc_setup);
259 if (!ndev)
260 return -ENOMEM;
261
262 mctp_pcc_ndev = netdev_priv(ndev);
263 rc = devm_add_action_or_reset(dev, mctp_cleanup_netdev, ndev);
264 if (rc)
265 goto cleanup_netdev;
266 spin_lock_init(&mctp_pcc_ndev->lock);
267
268 rc = mctp_pcc_initialize_mailbox(dev, &mctp_pcc_ndev->inbox,
269 context.inbox_index);
270 if (rc)
271 goto cleanup_netdev;
272 mctp_pcc_ndev->inbox.client.rx_callback = mctp_pcc_client_rx_callback;
273
274 //outbox initialization
275 rc = mctp_pcc_initialize_mailbox(dev, &mctp_pcc_ndev->outbox,
276 context.outbox_index);
277 if (rc)
278 goto cleanup_netdev;
279
280 mctp_pcc_hw_addr.parent_id = cpu_to_be32(0);
281 mctp_pcc_hw_addr.inbox_id = cpu_to_be16(context.inbox_index);
282 mctp_pcc_hw_addr.outbox_id = cpu_to_be16(context.outbox_index);
283 ndev->addr_len = sizeof(mctp_pcc_hw_addr);
284 dev_addr_set(ndev, (const u8 *)&mctp_pcc_hw_addr);
285
286 mctp_pcc_ndev->acpi_device = acpi_dev;
287 mctp_pcc_ndev->inbox.client.dev = dev;
288 mctp_pcc_ndev->outbox.client.dev = dev;
289 mctp_pcc_ndev->mdev.dev = ndev;
290 acpi_dev->driver_data = mctp_pcc_ndev;
291
292 /* There is no clean way to pass the MTU to the callback function
293 * used for registration, so set the values ahead of time.
294 */
295 mctp_pcc_mtu = mctp_pcc_ndev->outbox.chan->shmem_size -
296 sizeof(struct mctp_pcc_hdr);
297 ndev->mtu = MCTP_MIN_MTU;
298 ndev->max_mtu = mctp_pcc_mtu;
299 ndev->min_mtu = MCTP_MIN_MTU;
300
301 /* ndev needs to be freed before the iomemory (mapped above) gets
302 * unmapped, devm resources get freed in reverse to the order they
303 * are added.
304 */
305 rc = register_netdev(ndev);
306 return rc;
307 cleanup_netdev:
308 free_netdev(ndev);
309 return rc;
310 }
311
312 static const struct acpi_device_id mctp_pcc_device_ids[] = {
313 { "DMT0001"},
314 {}
315 };
316
> 317 static struct acpi_driver mctp_pcc_driver = {
318 .name = "mctp_pcc",
319 .class = "Unknown",
320 .ids = mctp_pcc_device_ids,
321 .ops = {
322 .add = mctp_pcc_driver_add,
323 },
324 };
325
> 326 module_acpi_driver(mctp_pcc_driver);
327
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
prev parent reply other threads:[~2024-10-31 13:10 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-10-29 16:54 [PATCH v6 0/2] MCTP Over PCC Transport admiyo
2024-10-29 16:54 ` [PATCH v6 1/2] mctp pcc: Check before sending MCTP PCC response ACK admiyo
2024-10-30 9:45 ` lihuisong (C)
2024-11-01 0:16 ` Adam Young
2024-11-01 1:30 ` lihuisong (C)
2024-11-02 15:34 ` Adam Young
2024-11-04 3:17 ` lihuisong (C)
2024-11-04 18:44 ` Adam Young
2024-10-29 16:54 ` [PATCH v6 2/2] mctp pcc: Implement MCTP over PCC Transport admiyo
2024-10-31 1:28 ` Jeremy Kerr
2024-10-31 15:50 ` Adam Young
2024-11-01 8:55 ` Jeremy Kerr
2024-11-01 21:19 ` Adam Young
2024-11-05 14:09 ` Jeremy Kerr
2024-11-06 15:59 ` Adam Young
2024-11-12 1:00 ` Jeremy Kerr
2024-11-13 18:41 ` Adam Young
2024-11-05 20:16 ` Adam Young
2024-11-06 10:47 ` Jeremy Kerr
2024-10-31 11:26 ` kernel test robot
2024-10-31 12:07 ` kernel test robot
2024-10-31 12:38 ` kernel test robot
2024-10-31 13:09 ` kernel test robot [this message]
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=202410312048.W6PV1dIU-lkp@intel.com \
--to=lkp@intel.com \
--cc=Jonathan.Cameron@huawei.com \
--cc=admiyo@os.amperecomputing.com \
--cc=davem@davemloft.net \
--cc=edumazet@google.com \
--cc=jk@codeconstruct.com.au \
--cc=kuba@kernel.org \
--cc=lihuisong@huawei.com \
--cc=linux-kernel@vger.kernel.org \
--cc=llvm@lists.linux.dev \
--cc=matt@codeconstruct.com.au \
--cc=netdev@vger.kernel.org \
--cc=oe-kbuild-all@lists.linux.dev \
--cc=pabeni@redhat.com \
--cc=sudeep.holla@arm.com \
/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