Re: [PATCH v4 3/3] infiniband/mm: convert put_page() to put_user_page*()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi John,

Thank you for the patch! Perhaps something to improve:

[auto build test WARNING on linus/master]
[also build test WARNING on v4.19-rc7 next-20181008]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

url:    https://github.com/0day-ci/linux/commits/john-hubbard-gmail-com/get_user_pages-and-RDMA-first-steps/20181009-152159
config: x86_64-randconfig-s4-10091707 (attached as .config)
compiler: gcc-7 (Debian 7.3.0-1) 7.3.0
reproduce:
        # save the attached .config to linux build tree
        make ARCH=x86_64 

All warnings (new ones prefixed by >>):

   drivers/infiniband/hw/qib/qib_user_sdma.c: In function 'qib_user_sdma_init_frag':
>> drivers/infiniband/hw/qib/qib_user_sdma.c:269:24: warning: assignment makes integer from pointer without a cast [-Wint-conversion]
     pkt->addr[i].put_page = put_user_page;
                           ^

vim +269 drivers/infiniband/hw/qib/qib_user_sdma.c

   257	
   258	static void qib_user_sdma_init_frag(struct qib_user_sdma_pkt *pkt,
   259					    int i, u16 offset, u16 len,
   260					    u16 first_desc, u16 last_desc,
   261					    u16 put_page, u16 dma_mapped,
   262					    struct page *page, void *kvaddr,
   263					    dma_addr_t dma_addr, u16 dma_length)
   264	{
   265		pkt->addr[i].offset = offset;
   266		pkt->addr[i].length = len;
   267		pkt->addr[i].first_desc = first_desc;
   268		pkt->addr[i].last_desc = last_desc;
 > 269		pkt->addr[i].put_page = put_user_page;
   270		pkt->addr[i].dma_mapped = dma_mapped;
   271		pkt->addr[i].page = page;
   272		pkt->addr[i].kvaddr = kvaddr;
   273		pkt->addr[i].addr = dma_addr;
   274		pkt->addr[i].dma_length = dma_length;
   275	}
   276	

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

Attachment: .config.gz
Description: application/gzip


[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Photo]     [Yosemite News]     [Yosemite Photos]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux