CC: kbuild-...@lists.01.org
BCC: l...@intel.com
CC: linux-ker...@vger.kernel.org
TO: Dan Williams <dan.j.willi...@intel.com>
CC: Jonathan Cameron <jonathan.came...@huawei.com>

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git 
master
head:   3123109284176b1532874591f7c81f3837bbdc17
commit: 4faf31b43468c58e2c8c91cc5fa26f08a6b733be cxl/mbox: Move mailbox and 
other non-PCI specific infrastructure to the core
date:   7 months ago
:::::: branch date: 24 hours ago
:::::: commit date: 7 months ago
compiler: ia64-linux-gcc (GCC) 11.2.0

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <l...@intel.com>


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 -- kbuild@lists.01.org
To unsubscribe send an email to kbuild-le...@lists.01.org

Reply via email to