[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH for-2.12] rdma: Fix 32-bit compilation
From: |
Marcel Apfelbaum |
Subject: |
Re: [Qemu-devel] [PATCH for-2.12] rdma: Fix 32-bit compilation |
Date: |
Tue, 20 Mar 2018 12:30:31 +0200 |
User-agent: |
Mozilla/5.0 (Macintosh; Intel Mac OS X 10.13; rv:52.0) Gecko/20100101 Thunderbird/52.6.0 |
Hi Eric,
On 19/03/2018 23:53, Eric Blake wrote:
> Use the correct printf formats, so that a 32-bit compile doesn't
> spit out lots of warnings about %lx being incompatible with uint64_t.
> Broken since initial commit ef6d4ccd.
>
> Signed-off-by: Eric Blake <address@hidden>
> ---
>
> I don't know if 'make vm-build-ubuntu.i368' would catch this (it failed
> for me for other reasons); I found it via a 32-bit rawhide VM.
>
I couldn't run 'make vm-build-ubuntu.i368' either. (Stuck on "Booting from Hard
Disk...")
I run make address@hidden to be sure it compiles on 32bit arch,
however I found out the docker configuration results in 'RDMA support no', so
it will not help.
Fam, is there any way the docker image can be updated with the RDMA libraries
so we can check this too?
> hw/rdma/rdma_backend.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/hw/rdma/rdma_backend.c b/hw/rdma/rdma_backend.c
> index e306fba5344..89020fdcf62 100644
> --- a/hw/rdma/rdma_backend.c
> +++ b/hw/rdma/rdma_backend.c
> @@ -656,7 +656,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", \
> + warn_report("%s = 0x%" PRIx64 " is higher than host device " \
> + "capability 0x%" PRIx64, \
> #member, (uint64_t)req->member, (uint64_t)dev.member); \
> req->member = dev.member; \
> } \
>
Reviewed-by: Marcel Apfelbaum <address@hidden>
Thanks,
Marcel