hw/rdma: Use correct print format in CHK_ATTR macro
authorYuval Shaia <yuval.shaia@oracle.com>
Thu, 22 Mar 2018 09:52:19 +0000 (11:52 +0200)
committerMarcel Apfelbaum <marcel@redhat.com>
Fri, 23 Mar 2018 15:38:55 +0000 (18:38 +0300)
Macro should not cast the given variable to u64 instead it should use
the supplied format argument (fmt).

Reported-by: Eric Blake <eblake@redhat.com>
Signed-off-by: Yuval Shaia <yuval.shaia@oracle.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Tested-by: Eric Blake <eblake@redhat.com>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Message-Id: <20180322095220.9976-3-yuval.shaia@oracle.com>
Signed-off-by: Marcel Apfelbaum <marcel@redhat.com>
hw/rdma/rdma_backend.c

index b1d2913b2c5cee2da555bccb9bf3d8e22d67b6fa..1dbb17bd2e22ba0a1a03f2d5e598e42dbf07f711 100644 (file)
@@ -668,8 +668,8 @@ void rdma_backend_destroy_qp(RdmaBackendQP *qp)
 #define CHK_ATTR(req, dev, member, fmt) ({ \
     pr_dbg("%s="fmt","fmt"\n", #member, dev.member, req->member); \
     if (req->member > dev.member) { \
-        warn_report("%s = 0x%lx is higher than host device capability 0x%lx", \
-                    #member, (uint64_t)req->member, (uint64_t)dev.member); \
+        warn_report("%s = "fmt" is higher than host device capability "fmt, \
+                    #member, req->member, dev.member); \
         req->member = dev.member; \
     } \
     pr_dbg("%s="fmt"\n", #member, req->member); })