Skip to content

Commit

Permalink
target/user: Fix size_t format-spec build warning
Browse files Browse the repository at this point in the history
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>
  • Loading branch information
Nicholas Bellinger committed Mar 11, 2016
1 parent b25c786 commit 0241fd3
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions drivers/target/target_core_user.c
Original file line number Diff line number Diff line change
Expand Up @@ -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));

Expand All @@ -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;
}
Expand Down

0 comments on commit 0241fd3

Please sign in to comment.