drivers/gpu/drm/drm_gem_shmem_helper.c:540:22: error: incompatible types when assigning to type 'pgprot_t' {aka 'struct '} from type 'int'

2020-06-20 Thread kernel test robot
Hi Gerd,

FYI, the error/warning still remains.

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git 
master
head:   4333a9b0b67bb4e8bcd91bdd80da80b0ec151162
commit: 0be895893607fb3447478d6e33dfb60644195a09 drm/shmem: switch shmem helper 
to _gem_object_funcs.mmap
date:   8 months ago
config: m68k-randconfig-r006-20200621 (attached as .config)
compiler: m68k-linux-gcc (GCC) 9.3.0
reproduce (this is a W=1 build):
wget 
https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O 
~/bin/make.cross
chmod +x ~/bin/make.cross
git checkout 0be895893607fb3447478d6e33dfb60644195a09
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross 
ARCH=m68k 

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot 

All errors (new ones prefixed by >>):

   drivers/gpu/drm/drm_gem_shmem_helper.c: In function 
'drm_gem_shmem_vmap_locked':
   drivers/gpu/drm/drm_gem_shmem_helper.c:261:17: error: implicit declaration 
of function 'pgprot_writecombine' [-Werror=implicit-function-declaration]
 261 | VM_MAP, pgprot_writecombine(PAGE_KERNEL));
 | ^~~
   drivers/gpu/drm/drm_gem_shmem_helper.c:261:17: error: incompatible type for 
argument 4 of 'vmap'
 261 | VM_MAP, pgprot_writecombine(PAGE_KERNEL));
 | ^~~~
 | |
 | int
   In file included from include/asm-generic/io.h:887,
from arch/m68k/include/asm/io.h:11,
from arch/m68k/include/asm/pgtable_no.h:14,
from arch/m68k/include/asm/pgtable.h:3,
from include/linux/mm.h:99,
from include/linux/scatterlist.h:8,
from include/linux/dma-buf.h:18,
from drivers/gpu/drm/drm_gem_shmem_helper.c:6:
   include/linux/vmalloc.h:119:14: note: expected 'pgprot_t' {aka 'struct 
'} but argument is of type 'int'
 119 | extern void *vmap(struct page **pages, unsigned int count,
 |  ^~~~
   drivers/gpu/drm/drm_gem_shmem_helper.c: In function 'drm_gem_shmem_mmap':
>> drivers/gpu/drm/drm_gem_shmem_helper.c:540:22: error: incompatible types 
>> when assigning to type 'pgprot_t' {aka 'struct '} from type 'int'
 540 |  vma->vm_page_prot = 
pgprot_writecombine(vm_get_page_prot(vma->vm_flags));
 |  ^~~
   cc1: some warnings being treated as errors

vim +540 drivers/gpu/drm/drm_gem_shmem_helper.c

   513  
   514  /**
   515   * drm_gem_shmem_mmap - Memory-map a shmem GEM object
   516   * @obj: gem object
   517   * @vma: VMA for the area to be mapped
   518   *
   519   * This function implements an augmented version of the GEM DRM file 
mmap
   520   * operation for shmem objects. Drivers which employ the shmem helpers 
should
   521   * use this function as their _gem_object_funcs.mmap handler.
   522   *
   523   * Returns:
   524   * 0 on success or a negative error code on failure.
   525   */
   526  int drm_gem_shmem_mmap(struct drm_gem_object *obj, struct 
vm_area_struct *vma)
   527  {
   528  struct drm_gem_shmem_object *shmem;
   529  int ret;
   530  
   531  shmem = to_drm_gem_shmem_obj(obj);
   532  
   533  ret = drm_gem_shmem_get_pages(shmem);
   534  if (ret) {
   535  drm_gem_vm_close(vma);
   536  return ret;
   537  }
   538  
   539  vma->vm_flags |= VM_IO | VM_MIXEDMAP | VM_DONTEXPAND | 
VM_DONTDUMP;
 > 540  vma->vm_page_prot = 
 > pgprot_writecombine(vm_get_page_prot(vma->vm_flags));
   541  vma->vm_page_prot = pgprot_decrypted(vma->vm_page_prot);
   542  vma->vm_ops = _gem_shmem_vm_ops;
   543  
   544  /* Remove the fake offset */
   545  vma->vm_pgoff -= drm_vma_node_start(>base.vma_node);
   546  
   547  return 0;
   548  }
   549  EXPORT_SYMBOL_GPL(drm_gem_shmem_mmap);
   550  

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-...@lists.01.org


.config.gz
Description: application/gzip


drivers/gpu/drm/drm_gem_shmem_helper.c:540:22: error: incompatible types when assigning to type 'pgprot_t' {aka 'struct '} from type 'int'

2020-06-19 Thread kernel test robot
Hi Gerd,

FYI, the error/warning still remains.

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git 
master
head:   4333a9b0b67bb4e8bcd91bdd80da80b0ec151162
commit: 0be895893607fb3447478d6e33dfb60644195a09 drm/shmem: switch shmem helper 
to _gem_object_funcs.mmap
date:   8 months ago
config: m68k-randconfig-r016-20200619 (attached as .config)
compiler: m68k-linux-gcc (GCC) 9.3.0
reproduce (this is a W=1 build):
wget 
https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O 
~/bin/make.cross
chmod +x ~/bin/make.cross
git checkout 0be895893607fb3447478d6e33dfb60644195a09
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross 
ARCH=m68k 

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot 

All errors (new ones prefixed by >>):

   In file included from include/linux/file.h:9,
from include/linux/dma-buf.h:16,
from drivers/gpu/drm/drm_gem_shmem_helper.c:6:
   include/linux/scatterlist.h: In function 'sg_set_buf':
   arch/m68k/include/asm/page_no.h:33:50: warning: ordered comparison of 
pointer with null pointer [-Wextra]
  33 | #define virt_addr_valid(kaddr) (((void *)(kaddr) >= (void 
*)PAGE_OFFSET) && \
 |  ^~
   include/linux/compiler.h:78:42: note: in definition of macro 'unlikely'
  78 | # define unlikely(x) __builtin_expect(!!(x), 0)
 |  ^
   include/linux/scatterlist.h:143:2: note: in expansion of macro 'BUG_ON'
 143 |  BUG_ON(!virt_addr_valid(buf));
 |  ^~
   include/linux/scatterlist.h:143:10: note: in expansion of macro 
'virt_addr_valid'
 143 |  BUG_ON(!virt_addr_valid(buf));
 |  ^~~
   drivers/gpu/drm/drm_gem_shmem_helper.c: In function 
'drm_gem_shmem_vmap_locked':
   drivers/gpu/drm/drm_gem_shmem_helper.c:261:17: error: implicit declaration 
of function 'pgprot_writecombine' [-Werror=implicit-function-declaration]
 261 | VM_MAP, pgprot_writecombine(PAGE_KERNEL));
 | ^~~
   drivers/gpu/drm/drm_gem_shmem_helper.c:261:17: error: incompatible type for 
argument 4 of 'vmap'
 261 | VM_MAP, pgprot_writecombine(PAGE_KERNEL));
 | ^~~~
 | |
 | int
   In file included from include/asm-generic/io.h:887,
from arch/m68k/include/asm/io.h:11,
from arch/m68k/include/asm/pgtable_no.h:14,
from arch/m68k/include/asm/pgtable.h:3,
from include/linux/mm.h:99,
from include/linux/scatterlist.h:8,
from include/linux/dma-buf.h:18,
from drivers/gpu/drm/drm_gem_shmem_helper.c:6:
   include/linux/vmalloc.h:119:14: note: expected 'pgprot_t' {aka 'struct 
'} but argument is of type 'int'
 119 | extern void *vmap(struct page **pages, unsigned int count,
 |  ^~~~
   drivers/gpu/drm/drm_gem_shmem_helper.c: In function 'drm_gem_shmem_mmap':
>> drivers/gpu/drm/drm_gem_shmem_helper.c:540:22: error: incompatible types 
>> when assigning to type 'pgprot_t' {aka 'struct '} from type 'int'
 540 |  vma->vm_page_prot = 
pgprot_writecombine(vm_get_page_prot(vma->vm_flags));
 |  ^~~
   cc1: some warnings being treated as errors

vim +540 drivers/gpu/drm/drm_gem_shmem_helper.c

   513  
   514  /**
   515   * drm_gem_shmem_mmap - Memory-map a shmem GEM object
   516   * @obj: gem object
   517   * @vma: VMA for the area to be mapped
   518   *
   519   * This function implements an augmented version of the GEM DRM file 
mmap
   520   * operation for shmem objects. Drivers which employ the shmem helpers 
should
   521   * use this function as their _gem_object_funcs.mmap handler.
   522   *
   523   * Returns:
   524   * 0 on success or a negative error code on failure.
   525   */
   526  int drm_gem_shmem_mmap(struct drm_gem_object *obj, struct 
vm_area_struct *vma)
   527  {
   528  struct drm_gem_shmem_object *shmem;
   529  int ret;
   530  
   531  shmem = to_drm_gem_shmem_obj(obj);
   532  
   533  ret = drm_gem_shmem_get_pages(shmem);
   534  if (ret) {
   535  drm_gem_vm_close(vma);
   536  return ret;
   537  }
   538  
   539  vma->vm_flags |= VM_IO | VM_MIXEDMAP | VM_DONTEXPAND | 
VM_DONTDUMP;
 > 540  vma->vm_page_prot = 
 > pgprot_writecombine(vm_get_page_prot(vma->vm_flags));
   541  vma->vm_page_prot = pgprot_decrypted(vma->vm_page_prot);
   542  vma->vm_ops = _gem_shmem_vm_ops;
   543  
   544 

drivers/gpu/drm/drm_gem_shmem_helper.c:540:22: error: incompatible types when assigning to type 'pgprot_t' {aka 'struct '} from type 'int'

2020-06-01 Thread kbuild test robot
Hi Gerd,

FYI, the error/warning still remains.

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git 
master
head:   3d77e6a8804abcc0504c904bd6e5cdf3a5cf8162
commit: 0be895893607fb3447478d6e33dfb60644195a09 drm/shmem: switch shmem helper 
to _gem_object_funcs.mmap
config: m68k-randconfig-r033-20200601 (attached as .config)
compiler: m68k-linux-gcc (GCC) 9.3.0
reproduce (this is a W=1 build):
wget 
https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O 
~/bin/make.cross
chmod +x ~/bin/make.cross
git checkout 0be895893607fb3447478d6e33dfb60644195a09
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross 
ARCH=m68k 

If you fix the issue, kindly add following tag as appropriate
Reported-by: kbuild test robot 

All errors (new ones prefixed by >>, old ones prefixed by <<):

drivers/gpu/drm/drm_gem_shmem_helper.c: In function 'drm_gem_shmem_vmap_locked':
drivers/gpu/drm/drm_gem_shmem_helper.c:261:17: error: implicit declaration of 
function 'pgprot_writecombine' [-Werror=implicit-function-declaration]
261 | VM_MAP, pgprot_writecombine(PAGE_KERNEL));
| ^~~
drivers/gpu/drm/drm_gem_shmem_helper.c:261:17: error: incompatible type for 
argument 4 of 'vmap'
261 | VM_MAP, pgprot_writecombine(PAGE_KERNEL));
| ^~~~
| |
| int
In file included from include/asm-generic/io.h:887,
from arch/m68k/include/asm/io.h:11,
from arch/m68k/include/asm/pgtable_no.h:14,
from arch/m68k/include/asm/pgtable.h:3,
from include/linux/mm.h:99,
from include/linux/scatterlist.h:8,
from include/linux/dma-buf.h:18,
from drivers/gpu/drm/drm_gem_shmem_helper.c:6:
include/linux/vmalloc.h:119:14: note: expected 'pgprot_t' {aka 'struct 
'} but argument is of type 'int'
119 | extern void *vmap(struct page **pages, unsigned int count,
|  ^~~~
drivers/gpu/drm/drm_gem_shmem_helper.c: In function 'drm_gem_shmem_mmap':
<<  from drivers/gpu/drm/drm_gem_shmem_helper.c:6:
>> drivers/gpu/drm/drm_gem_shmem_helper.c:540:22: error: incompatible types 
>> when assigning to type 'pgprot_t' {aka 'struct '} from type 'int'
540 |  vma->vm_page_prot = pgprot_writecombine(vm_get_page_prot(vma->vm_flags));
|  ^~~
cc1: some warnings being treated as errors

# 
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=0be895893607fb3447478d6e33dfb60644195a09
git remote add linus 
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
git remote update linus
git checkout 0be895893607fb3447478d6e33dfb60644195a09
vim +540 drivers/gpu/drm/drm_gem_shmem_helper.c

2194a63a818db7 Noralf Trønnes 2019-03-12  513  
2194a63a818db7 Noralf Trønnes 2019-03-12  514  /**
2194a63a818db7 Noralf Trønnes 2019-03-12  515   * drm_gem_shmem_mmap - 
Memory-map a shmem GEM object
0be895893607fb Gerd Hoffmann  2019-10-16  516   * @obj: gem object
2194a63a818db7 Noralf Trønnes 2019-03-12  517   * @vma: VMA for the area to be 
mapped
2194a63a818db7 Noralf Trønnes 2019-03-12  518   *
2194a63a818db7 Noralf Trønnes 2019-03-12  519   * This function implements an 
augmented version of the GEM DRM file mmap
2194a63a818db7 Noralf Trønnes 2019-03-12  520   * operation for shmem objects. 
Drivers which employ the shmem helpers should
0be895893607fb Gerd Hoffmann  2019-10-16  521   * use this function as their 
_gem_object_funcs.mmap handler.
2194a63a818db7 Noralf Trønnes 2019-03-12  522   *
2194a63a818db7 Noralf Trønnes 2019-03-12  523   * Returns:
2194a63a818db7 Noralf Trønnes 2019-03-12  524   * 0 on success or a negative 
error code on failure.
2194a63a818db7 Noralf Trønnes 2019-03-12  525   */
0be895893607fb Gerd Hoffmann  2019-10-16  526  int drm_gem_shmem_mmap(struct 
drm_gem_object *obj, struct vm_area_struct *vma)
2194a63a818db7 Noralf Trønnes 2019-03-12  527  {
2194a63a818db7 Noralf Trønnes 2019-03-12  528   struct drm_gem_shmem_object 
*shmem;
2194a63a818db7 Noralf Trønnes 2019-03-12  529   int ret;
2194a63a818db7 Noralf Trønnes 2019-03-12  530  
0be895893607fb Gerd Hoffmann  2019-10-16  531   shmem = 
to_drm_gem_shmem_obj(obj);
2194a63a818db7 Noralf Trønnes 2019-03-12  532  
2194a63a818db7 Noralf Trønnes 2019-03-12  533   ret = 
drm_gem_shmem_get_pages(shmem);
2194a63a818db7 Noralf Trønnes 2019-03-12  534   if (ret) {
2194a63a818db7 Noralf Trønnes 2019-03-12  535   drm_gem_vm_close(vma);
2194a63a818db7 Noralf Trønnes 2019-03-12  536   return ret;
2194a63a818db7 Noralf Trønnes 2019-03-12  537   }
2194a63a818db7 Noralf Trønnes 2019-03-12  538  
0be895893607fb Gerd Hoffmann  2019-10-16  539   vma->vm_flags |= VM_IO | 
VM_MIXEDMAP | VM_DONTEXPAND | VM_DONTDUMP;
0be895893607fb Gerd Hoffmann  2019-10-16 @540   vma->vm_page_prot = 
pgprot_writecombine(vm_get_page_prot(vma->vm_flags));
0be895