tree: git://linuxtv.org/mchehab/media-next.git master head: 74ac528a8f2668d34d9a343cae8f30f2235ccbdc commit: 12a9989bf254fb449233a61b104472d4838c44e2 [521/554] media: atomisp: add debug for hmm alloc config: i386-allyesconfig (attached as .config) compiler: gcc-9 (Debian 9.3.0-13) 9.3.0 reproduce (this is a W=1 build): git checkout 12a9989bf254fb449233a61b104472d4838c44e2 # save the attached .config to linux build tree make W=1 ARCH=i386 If you fix the issue, kindly add following tag as appropriate Reported-by: kbuild test robot All warnings (new ones prefixed by >>, old ones prefixed by <<): cc1: warning: drivers/staging/media/atomisp//pci/hive_isp_css_include/memory_access/: No such file or directory [-Wmissing-include-dirs] In file included from include/linux/printk.h:326, from include/linux/kernel.h:15, from drivers/staging/media/atomisp/pci/hmm/hmm.c:22: drivers/staging/media/atomisp/pci/hmm/hmm.c: In function 'hmm_alloc': >> drivers/staging/media/atomisp/pci/hmm/hmm.c:269:6: warning: format '%ld' expects argument of type 'long int', but argument 6 has type 'size_t' {aka 'unsigned int'} [-Wformat=] 269 | "%s: pages: 0x%08x (%ld bytes), type: %d from highmem %d, user ptr %p, cached %dn", | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ include/linux/dynamic_debug.h:125:15: note: in definition of macro '__dynamic_func_call' 125 | func(&id, ##__VA_ARGS__); | ^~~~~~~~~~~ include/linux/dynamic_debug.h:157:2: note: in expansion of macro '_dynamic_func_call' 157 | _dynamic_func_call(fmt,__dynamic_dev_dbg, | ^~~~~~~~~~~~~~~~~~ include/linux/dev_printk.h:114:2: note: in expansion of macro 'dynamic_dev_dbg' 114 | dynamic_dev_dbg(dev, dev_fmt(fmt), ##__VA_ARGS__) | ^~~~~~~~~~~~~~~ include/linux/dev_printk.h:114:23: note: in expansion of macro 'dev_fmt' 114 | dynamic_dev_dbg(dev, dev_fmt(fmt), ##__VA_ARGS__) | ^~~~~~~ >> drivers/staging/media/atomisp/pci/hmm/hmm.c:268:6: note: in expansion of macro 'dev_dbg' 268 | dev_dbg(atomisp_dev, | ^~~~~~~ drivers/staging/media/atomisp/pci/hmm/hmm.c:269:28: note: format string is defined here 269 | "%s: pages: 0x%08x (%ld bytes), type: %d from highmem %d, user ptr %p, cached %dn", | ~~^ | | | long int | %d vim +269 drivers/staging/media/atomisp/pci/hmm/hmm.c 220 221 ia_css_ptr hmm_alloc(size_t bytes, enum hmm_bo_type type, 222 int from_highmem, const void __user *userptr, 223 const uint16_t attrs) 224 { 225 unsigned int pgnr; 226 struct hmm_buffer_object *bo; 227 bool cached = attrs & ATOMISP_MAP_FLAG_CACHED; 228 int ret; 229 230 WARN_ON(attrs & ATOMISP_MAP_FLAG_CONTIGUOUS); 231 232 /* 233 * Check if we are initialized. In the ideal world we wouldn't need 234 * this but we can tackle it once the driver is a lot cleaner 235 */ 236 237 if (!hmm_initialized) 238 hmm_init(); 239 /* Get page number from size */ 240 pgnr = size_to_pgnr_ceil(bytes); 241 242 /* Buffer object structure init */ 243 bo = hmm_bo_alloc(&bo_device, pgnr); 244 if (!bo) { 245 dev_err(atomisp_dev, "hmm_bo_create failed.\n"); 246 goto create_bo_err; 247 } 248 249 /* Allocate pages for memory */ 250 ret = hmm_bo_alloc_pages(bo, type, from_highmem, userptr, cached); 251 if (ret) { 252 dev_err(atomisp_dev, "hmm_bo_alloc_pages failed.\n"); 253 goto alloc_page_err; 254 } 255 256 /* Combine the virtual address and pages together */ 257 ret = hmm_bo_bind(bo); 258 if (ret) { 259 dev_err(atomisp_dev, "hmm_bo_bind failed.\n"); 260 goto bind_err; 261 } 262 263 hmm_mem_stat.tol_cnt += pgnr; 264 265 if (attrs & ATOMISP_MAP_FLAG_CLEARED) 266 hmm_set(bo->start, 0, bytes); 267 > 268 dev_dbg(atomisp_dev, > 269 "%s: pages: 0x%08x (%ld bytes), type: %d from highmem %d, user ptr %p, cached %d\n", 270 __func__, bo->start, bytes, type, from_highmem, userptr, cached); 271 272 return bo->start; 273 274 bind_err: 275 hmm_bo_free_pages(bo); 276 alloc_page_err: 277 hmm_bo_unref(bo); 278 create_bo_err: 279 return 0; 280 } 281 --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org