Hi Vivek, I love your patch! Perhaps something to improve:
[auto build test WARNING on fuse/for-next] [also build test WARNING on v4.20-rc6] [cannot apply to next-20181210] [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/Vivek-Goyal/virtio-fs-shared-file-system-for-virtual-machines/20181211-103034 base: https://git.kernel.org/pub/scm/linux/kernel/git/mszeredi/fuse.git for-next config: arm-allmodconfig (attached as .config) compiler: arm-linux-gnueabi-gcc (Debian 7.2.0-11) 7.2.0 reproduce: wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross # save the attached .config to linux build tree GCC_VERSION=7.2.0 make.cross ARCH=arm All warnings (new ones prefixed by >>): In file included from include/linux/kernel.h:14:0, from include/linux/list.h:9, from include/linux/wait.h:7, from include/linux/wait_bit.h:8, from include/linux/fs.h:6, from fs/fuse/virtio_fs.c:7: fs/fuse/virtio_fs.c: In function 'virtio_fs_direct_access': >> fs/fuse/virtio_fs.c:454:11: warning: format '%ld' expects argument of type >> 'long int', but argument 4 has type 'size_t {aka unsigned int}' [-Wformat=] pr_debug("virtio_fs_direct_access(): called. nr_pages=%ld max_nr_pages=%ld\n", nr_pages, max_nr_pages); ^ include/linux/printk.h:292:21: note: in definition of macro 'pr_fmt' #define pr_fmt(fmt) fmt ^~~ include/linux/printk.h:340:2: note: in expansion of macro 'dynamic_pr_debug' dynamic_pr_debug(fmt, ##__VA_ARGS__) ^~~~~~~~~~~~~~~~ >> fs/fuse/virtio_fs.c:454:2: note: in expansion of macro 'pr_debug' pr_debug("virtio_fs_direct_access(): called. nr_pages=%ld max_nr_pages=%ld\n", nr_pages, max_nr_pages); ^~~~~~~~ In file included from include/linux/printk.h:336:0, from include/linux/kernel.h:14, from include/linux/list.h:9, from include/linux/wait.h:7, from include/linux/wait_bit.h:8, from include/linux/fs.h:6, from fs/fuse/virtio_fs.c:7: fs/fuse/virtio_fs.c: In function 'virtio_fs_setup_dax': fs/fuse/virtio_fs.c:617:22: warning: format '%llx' expects argument of type 'long long unsigned int', but argument 6 has type 'phys_addr_t {aka unsigned int}' [-Wformat=] dev_dbg(&vdev->dev, "%s: cache kaddr 0x%px phys_addr 0x%llx len %llx\n", ^ include/linux/dynamic_debug.h:135:39: note: in definition of macro 'dynamic_dev_dbg' __dynamic_dev_dbg(&descriptor, dev, fmt, \ ^~~ include/linux/device.h:1463:23: note: in expansion of macro 'dev_fmt' dynamic_dev_dbg(dev, dev_fmt(fmt), ##__VA_ARGS__) ^~~~~~~ fs/fuse/virtio_fs.c:617:2: note: in expansion of macro 'dev_dbg' dev_dbg(&vdev->dev, "%s: cache kaddr 0x%px phys_addr 0x%llx len %llx\n", ^~~~~~~ vim +454 fs/fuse/virtio_fs.c 442 443 /* Map a window offset to a page frame number. The window offset will have 444 * been produced by .iomap_begin(), which maps a file offset to a window 445 * offset. 446 */ 447 static long virtio_fs_direct_access(struct dax_device *dax_dev, pgoff_t pgoff, 448 long nr_pages, void **kaddr, pfn_t *pfn) 449 { 450 struct virtio_fs *fs = dax_get_private(dax_dev); 451 phys_addr_t offset = PFN_PHYS(pgoff); 452 size_t max_nr_pages = fs->window_len/PAGE_SIZE - pgoff; 453 > 454 pr_debug("virtio_fs_direct_access(): called. nr_pages=%ld > max_nr_pages=%ld\n", nr_pages, max_nr_pages); 455 456 if (kaddr) 457 *kaddr = fs->window_kaddr + offset; 458 if (pfn) 459 *pfn = phys_to_pfn_t(fs->window_phys_addr + offset, 460 PFN_DEV | PFN_MAP); 461 return nr_pages > max_nr_pages ? max_nr_pages : nr_pages; 462 } 463 --- 0-DAY kernel test infrastructure Open Source Technology Center https://lists.01.org/pipermail/kbuild-all Intel Corporation
.config.gz
Description: application/gzip
