Re: [PATCH v1 1/2] drm/vmwgfx: drop use of drmP.h in header files

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

 



Hi Sam,

Thank you for the patch! Yet something to improve:

[auto build test ERROR on linus/master]
[also build test ERROR on v5.2-rc5 next-20190621]
[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/Sam-Ravnborg/drm-vmwgfx-drop-use-of-drmP-h-in-header-files/20190622-234524
config: i386-randconfig-x010-201924 (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=i386 

If you fix the issue, kindly add following tag
Reported-by: kbuild test robot <lkp@xxxxxxxxx>

All error/warnings (new ones prefixed by >>):

   drivers/gpu//drm/vmwgfx/vmwgfx_mob.c: In function 'vmw_mob_build_pt':
>> drivers/gpu//drm/vmwgfx/vmwgfx_mob.c:517:22: error: implicit declaration of function 'kmap_atomic'; did you mean 'in_atomic'? [-Werror=implicit-function-declaration]
      save_addr = addr = kmap_atomic(page);
                         ^~~~~~~~~~~
                         in_atomic
>> drivers/gpu//drm/vmwgfx/vmwgfx_mob.c:517:20: warning: assignment makes pointer from integer without a cast [-Wint-conversion]
      save_addr = addr = kmap_atomic(page);
                       ^
>> drivers/gpu//drm/vmwgfx/vmwgfx_mob.c:526:3: error: implicit declaration of function 'kunmap_atomic'; did you mean 'in_atomic'? [-Werror=implicit-function-declaration]
      kunmap_atomic(save_addr);
      ^~~~~~~~~~~~~
      in_atomic
   cc1: some warnings being treated as errors

vim +517 drivers/gpu//drm/vmwgfx/vmwgfx_mob.c

3530bdc3 Thomas Hellstrom 2012-11-21  491  
3530bdc3 Thomas Hellstrom 2012-11-21  492  /*
3530bdc3 Thomas Hellstrom 2012-11-21  493   * vmw_mob_build_pt - Build a pagetable
3530bdc3 Thomas Hellstrom 2012-11-21  494   *
0fd53cfb Thomas Hellstrom 2013-10-24  495   * @data_addr:      Array of DMA addresses to the underlying buffer
3530bdc3 Thomas Hellstrom 2012-11-21  496   *                  object's data pages.
3530bdc3 Thomas Hellstrom 2012-11-21  497   * @num_data_pages: Number of buffer object data pages.
3530bdc3 Thomas Hellstrom 2012-11-21  498   * @pt_pages:       Array of page pointers to the page table pages.
3530bdc3 Thomas Hellstrom 2012-11-21  499   *
3530bdc3 Thomas Hellstrom 2012-11-21  500   * Returns the number of page table pages actually used.
3530bdc3 Thomas Hellstrom 2012-11-21  501   * Uses atomic kmaps of highmem pages to avoid TLB thrashing.
3530bdc3 Thomas Hellstrom 2012-11-21  502   */
0fd53cfb Thomas Hellstrom 2013-10-24  503  static unsigned long vmw_mob_build_pt(struct vmw_piter *data_iter,
3530bdc3 Thomas Hellstrom 2012-11-21  504  				      unsigned long num_data_pages,
0fd53cfb Thomas Hellstrom 2013-10-24  505  				      struct vmw_piter *pt_iter)
3530bdc3 Thomas Hellstrom 2012-11-21  506  {
3530bdc3 Thomas Hellstrom 2012-11-21  507  	unsigned long pt_size = num_data_pages * VMW_PPN_SIZE;
3530bdc3 Thomas Hellstrom 2012-11-21  508  	unsigned long num_pt_pages = DIV_ROUND_UP(pt_size, PAGE_SIZE);
0fd53cfb Thomas Hellstrom 2013-10-24  509  	unsigned long pt_page;
b9eb1a61 Thomas Hellstrom 2015-04-02  510  	u32 *addr, *save_addr;
3530bdc3 Thomas Hellstrom 2012-11-21  511  	unsigned long i;
0fd53cfb Thomas Hellstrom 2013-10-24  512  	struct page *page;
3530bdc3 Thomas Hellstrom 2012-11-21  513  
3530bdc3 Thomas Hellstrom 2012-11-21  514  	for (pt_page = 0; pt_page < num_pt_pages; ++pt_page) {
0fd53cfb Thomas Hellstrom 2013-10-24  515  		page = vmw_piter_page(pt_iter);
0fd53cfb Thomas Hellstrom 2013-10-24  516  
0fd53cfb Thomas Hellstrom 2013-10-24 @517  		save_addr = addr = kmap_atomic(page);
3530bdc3 Thomas Hellstrom 2012-11-21  518  
3530bdc3 Thomas Hellstrom 2012-11-21  519  		for (i = 0; i < PAGE_SIZE / VMW_PPN_SIZE; ++i) {
f2a0dcb1 Thomas Hellstrom 2014-01-15  520  			vmw_mob_assign_ppn(&addr,
f2a0dcb1 Thomas Hellstrom 2014-01-15  521  					   vmw_piter_dma_addr(data_iter));
0fd53cfb Thomas Hellstrom 2013-10-24  522  			if (unlikely(--num_data_pages == 0))
3530bdc3 Thomas Hellstrom 2012-11-21  523  				break;
0fd53cfb Thomas Hellstrom 2013-10-24  524  			WARN_ON(!vmw_piter_next(data_iter));
3530bdc3 Thomas Hellstrom 2012-11-21  525  		}
3530bdc3 Thomas Hellstrom 2012-11-21 @526  		kunmap_atomic(save_addr);
0fd53cfb Thomas Hellstrom 2013-10-24  527  		vmw_piter_next(pt_iter);
3530bdc3 Thomas Hellstrom 2012-11-21  528  	}
3530bdc3 Thomas Hellstrom 2012-11-21  529  
3530bdc3 Thomas Hellstrom 2012-11-21  530  	return num_pt_pages;
3530bdc3 Thomas Hellstrom 2012-11-21  531  }
3530bdc3 Thomas Hellstrom 2012-11-21  532  

:::::: The code at line 517 was first introduced by commit
:::::: 0fd53cfb09108c33b924b069fe2c62fa4e7b11a0 drm/vmwgfx: Use the linux DMA api also for MOBs

:::::: TO: Thomas Hellstrom <thellstrom@xxxxxxxxxx>
:::::: CC: Thomas Hellstrom <thellstrom@xxxxxxxxxx>

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

Attachment: .config.gz
Description: application/gzip

_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel

[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux