]> git.kernelconcepts.de Git - karo-tx-linux.git/commitdiff
target/user: Fix size_t format-spec build warning
authorNicholas Bellinger <nab@linux-iscsi.org>
Sun, 28 Feb 2016 02:25:22 +0000 (18:25 -0800)
committerNicholas Bellinger <nab@linux-iscsi.org>
Fri, 11 Mar 2016 05:49:06 +0000 (21:49 -0800)
Fix the following printk size_t warning as per 0-day build:

All warnings (new ones prefixed by >>):

   drivers/target/target_core_user.c: In function 'is_ring_space_avail':
>> drivers/target/target_core_user.c:385:12: warning: format '%lu'
>> expects argument of type 'long unsigned int', but argument 3 has type
>> 'size_t {aka unsigned int}' [-Wformat=]
      pr_debug("no data space: only %lu available, but ask for %lu\n",
               ^

Reported-by: kbuild test robot <fengguang.wu@intel.com>
Signed-off-by: Nicholas Bellinger <nab@linux-iscsi.org>
drivers/target/target_core_user.c

index d0655913896b305b528f999f8019e5727d000d20..fc4789c6067c15a570add431034156c56fc12760 100644 (file)
@@ -356,9 +356,8 @@ static inline size_t spc_bitmap_free(unsigned long *bitmap)
 static bool is_ring_space_avail(struct tcmu_dev *udev, size_t cmd_size, size_t data_needed)
 {
        struct tcmu_mailbox *mb = udev->mb_addr;
-       size_t space;
+       size_t space, cmd_needed;
        u32 cmd_head;
-       size_t cmd_needed;
 
        tcmu_flush_dcache_range(mb, sizeof(*mb));
 
@@ -382,7 +381,7 @@ static bool is_ring_space_avail(struct tcmu_dev *udev, size_t cmd_size, size_t d
 
        space = spc_bitmap_free(udev->data_bitmap);
        if (space < data_needed) {
-               pr_debug("no data space: only %lu available, but ask for %lu\n",
+               pr_debug("no data space: only %zu available, but ask for %zu\n",
                                space, data_needed);
                return false;
        }