CC: [email protected] BCC: [email protected] CC: [email protected] TO: Dan Williams <[email protected]> CC: Jonathan Cameron <[email protected]>
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: 38d741cb70b30741c0e802cbed7bd9cf4fd15fa4 commit: 4faf31b43468c58e2c8c91cc5fa26f08a6b733be cxl/mbox: Move mailbox and other non-PCI specific infrastructure to the core date: 7 months ago :::::: branch date: 4 hours ago :::::: commit date: 7 months ago compiler: powerpc-linux-gcc (GCC) 11.3.0 reproduce (cppcheck warning): # apt-get install cppcheck git checkout 4faf31b43468c58e2c8c91cc5fa26f08a6b733be cppcheck --quiet --enable=style,performance,portability --template=gcc FILE If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot <[email protected]> cppcheck possible warnings: (new ones prefixed by >>, may not real problems) >> drivers/cxl/core/mbox.c:478:55: warning: Parameter 'uuid' can be declared >> with const [constParameter] static int cxl_xfer_log(struct cxl_mem *cxlm, uuid_t *uuid, u32 size, u8 *out) ^ vim +/uuid +478 drivers/cxl/core/mbox.c 4faf31b43468c5 Dan Williams 2021-09-08 477 4faf31b43468c5 Dan Williams 2021-09-08 @478 static int cxl_xfer_log(struct cxl_mem *cxlm, uuid_t *uuid, u32 size, u8 *out) 4faf31b43468c5 Dan Williams 2021-09-08 479 { 4faf31b43468c5 Dan Williams 2021-09-08 480 u32 remaining = size; 4faf31b43468c5 Dan Williams 2021-09-08 481 u32 offset = 0; 4faf31b43468c5 Dan Williams 2021-09-08 482 4faf31b43468c5 Dan Williams 2021-09-08 483 while (remaining) { 4faf31b43468c5 Dan Williams 2021-09-08 484 u32 xfer_size = min_t(u32, remaining, cxlm->payload_size); 4faf31b43468c5 Dan Williams 2021-09-08 485 struct cxl_mbox_get_log { 4faf31b43468c5 Dan Williams 2021-09-08 486 uuid_t uuid; 4faf31b43468c5 Dan Williams 2021-09-08 487 __le32 offset; 4faf31b43468c5 Dan Williams 2021-09-08 488 __le32 length; 4faf31b43468c5 Dan Williams 2021-09-08 489 } __packed log = { 4faf31b43468c5 Dan Williams 2021-09-08 490 .uuid = *uuid, 4faf31b43468c5 Dan Williams 2021-09-08 491 .offset = cpu_to_le32(offset), 4faf31b43468c5 Dan Williams 2021-09-08 492 .length = cpu_to_le32(xfer_size) 4faf31b43468c5 Dan Williams 2021-09-08 493 }; 4faf31b43468c5 Dan Williams 2021-09-08 494 int rc; 4faf31b43468c5 Dan Williams 2021-09-08 495 4faf31b43468c5 Dan Williams 2021-09-08 496 rc = cxl_mem_mbox_send_cmd(cxlm, CXL_MBOX_OP_GET_LOG, &log, 4faf31b43468c5 Dan Williams 2021-09-08 497 sizeof(log), out, xfer_size); 4faf31b43468c5 Dan Williams 2021-09-08 498 if (rc < 0) 4faf31b43468c5 Dan Williams 2021-09-08 499 return rc; 4faf31b43468c5 Dan Williams 2021-09-08 500 4faf31b43468c5 Dan Williams 2021-09-08 501 out += xfer_size; 4faf31b43468c5 Dan Williams 2021-09-08 502 remaining -= xfer_size; 4faf31b43468c5 Dan Williams 2021-09-08 503 offset += xfer_size; 4faf31b43468c5 Dan Williams 2021-09-08 504 } 4faf31b43468c5 Dan Williams 2021-09-08 505 4faf31b43468c5 Dan Williams 2021-09-08 506 return 0; 4faf31b43468c5 Dan Williams 2021-09-08 507 } 4faf31b43468c5 Dan Williams 2021-09-08 508 -- 0-DAY CI Kernel Test Service https://01.org/lkp _______________________________________________ kbuild mailing list -- [email protected] To unsubscribe send an email to [email protected]
