From: Marcel Apfelbaum <marcel.apfelbaum@gmail.com>
To: Peter Maydell <peter.maydell@linaro.org>
Cc: "QEMU Developers" <qemu-devel@nongnu.org>,
"Yuval Shaia" <yuval.shaia@oracle.com>,
"kamal heib" <kamalheib1@gmail.com>,
"Philippe Mathieu-Daudé" <philmd@redhat.com>,
"Dr. David Alan Gilbert" <dgilbert@redhat.com>
Subject: Re: [Qemu-devel] [PATCH PULL 00/11] RDMA queue
Date: Tue, 26 Feb 2019 20:40:36 +0200 [thread overview]
Message-ID: <57ceff35-e9d9-616e-0d1b-e6879cfedbf4@gmail.com> (raw)
In-Reply-To: <CAFEAcA__1qZP3M97Wah_=1mCAJ9f3WA=Nx7GcoAf28sZBap_sw@mail.gmail.com>
Hi Peter,
On 2/26/19 6:48 PM, Peter Maydell wrote:
> On Sat, 23 Feb 2019 at 14:16, Marcel Apfelbaum
> <marcel.apfelbaum@gmail.com> wrote:
>> The following changes since commit 8eb29f1bf5a974dc4c11d2d1f5e7c7f7a62be116:
>>
>> Merge remote-tracking branch 'remotes/awilliam/tags/vfio-updates-20190221.0' into staging (2019-02-22 15:48:04 +0000)
>>
>> are available in the Git repository at:
>>
>> https://github.com/marcel-apf/qemu tags/rdma-pull-request
>>
>> for you to fetch changes up to e1038e22f7b3ade2e46cf168fa7486414c424e94:
>>
>> hw/rdma: another clang compilation fix (2019-02-23 15:28:25 +0200)
>>
>> ----------------------------------------------------------------
>> RDMA queue
>>
>> * Another Clang compilation fix
>> * Various fixes for the pvrdma device
>>
> Format string issues for 32-bit hosts:
>
> In file included from hw/rdma/trace.c:4:0:
> hw/rdma/trace.h: In function '_nocheck__trace_rdma_mad_message':
> hw/rdma/trace.h:231:18: error: format '%ld' expects argument of type
> 'long int', but argument 6 has type 'size_t {aka unsigned int}'
> [-Werror=format=]
> qemu_log("%d@%zu.%06zu:rdma_mad_message " "mad %s (%ld): %s" "\n",
> ^
> In file included from hw/rdma/trace.c:4:0:
> hw/rdma/trace.h: In function '_nocheck__trace_rdma_rm_alloc_mr':
> hw/rdma/trace.h:507:18: error: format '%ld' expects argument of type
> 'long int', but argument 8 has type 'size_t {aka unsigned int}'
> [-Werror=format=]
> qemu_log("%d@%zu.%06zu:rdma_rm_alloc_mr " "mr_handle=%d,
> host_virt=%p, guest_start=0x%"PRIx64", length=%ld, access_flags=0x%x"
> "\n",
> ^
Will fix and resend (and add 32-bit compilation to my set of tests).
Thanks,
Marcel
> thanks
> -- PMM
next prev parent reply other threads:[~2019-02-26 18:40 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-02-23 14:16 [Qemu-devel] [PATCH PULL 00/11] RDMA queue Marcel Apfelbaum
2019-02-23 14:16 ` [Qemu-devel] [PATCH PULL 01/11] contrib/rdmacm-mux: Fix out-of-bounds risk Marcel Apfelbaum
2019-02-23 14:16 ` [Qemu-devel] [PATCH PULL 02/11] hw/rdma: Switch to generic error reporting way Marcel Apfelbaum
2019-02-23 14:16 ` [Qemu-devel] [PATCH PULL 03/11] hw/rdma: Introduce protected qlist Marcel Apfelbaum
2019-02-23 14:16 ` [Qemu-devel] [PATCH PULL 04/11] hw/rdma: Protect against concurrent execution of poll_cq Marcel Apfelbaum
2019-02-23 14:16 ` [Qemu-devel] [PATCH PULL 05/11] {monitor, hw/pvrdma}: Expose device internals via monitor interface Marcel Apfelbaum
2019-02-23 14:16 ` [Qemu-devel] [PATCH PULL 06/11] hw/rdma: Free all MAD receive buffers when device is closed Marcel Apfelbaum
2019-02-23 14:16 ` [Qemu-devel] [PATCH PULL 07/11] hw/rdma: Free all receive buffers when QP is destroyed Marcel Apfelbaum
2019-02-23 14:16 ` [Qemu-devel] [PATCH PULL 08/11] hw/pvrdma: Delete unneeded function argument Marcel Apfelbaum
2019-02-23 14:16 ` [Qemu-devel] [PATCH PULL 09/11] hw/pvrdma: Delete pvrdma_exit function Marcel Apfelbaum
2019-02-23 14:16 ` [Qemu-devel] [PATCH PULL 10/11] hw/pvrdma: Unregister from shutdown notifier when device goes down Marcel Apfelbaum
2019-02-23 14:16 ` [Qemu-devel] [PATCH PULL 11/11] hw/rdma: another clang compilation fix Marcel Apfelbaum
2019-02-26 16:48 ` [Qemu-devel] [PATCH PULL 00/11] RDMA queue Peter Maydell
2019-02-26 18:40 ` Marcel Apfelbaum [this message]
2019-02-27 9:46 ` Yuval Shaia
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=57ceff35-e9d9-616e-0d1b-e6879cfedbf4@gmail.com \
--to=marcel.apfelbaum@gmail.com \
--cc=dgilbert@redhat.com \
--cc=kamalheib1@gmail.com \
--cc=peter.maydell@linaro.org \
--cc=philmd@redhat.com \
--cc=qemu-devel@nongnu.org \
--cc=yuval.shaia@oracle.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.