[Qemu-devel] [PATCH V2] migration/rdma: clang compilation fix

qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed

* [Qemu-devel]  [PATCH V2] migration/rdma: clang compilation fix
@ 2019-03-04 18:49 Marcel Apfelbaum
  2019-03-04 18:53 ` Dr. David Alan Gilbert
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: Marcel Apfelbaum @ 2019-03-04 18:49 UTC (permalink / raw)
  To: qemu-devel; +Cc: dgilbert, eblake

Configuring QEMU with:
        ../configure --cc=clang --enable-rdma

Leads to compilation error:

  CC      migration/rdma.o
  CC      migration/block.o
  qemu/migration/rdma.c:3615:58: error: taking address of packed member 'rkey' of class or structure
      'RDMARegisterResult' may result in an unaligned pointer value [-Werror,-Waddress-of-packed-member]
                            (uintptr_t)host_addr, NULL, &reg_result->rkey,
                                                         ^~~~~~~~~~~~~~~~
Fix it by using a temp local variable.

Signed-off-by: Marcel Apfelbaum <marcel.apfelbaum@gmail.com>
---

v1 -> v2: 
 - Use a temp local variable (Dave)

 migration/rdma.c | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/migration/rdma.c b/migration/rdma.c
index 54a3c11540..e39ee77558 100644
--- a/migration/rdma.c
+++ b/migration/rdma.c
@@ -3611,13 +3611,16 @@ static int qemu_rdma_registration_handle(QEMUFile *f, void *opaque)
                 }
                 chunk_start = ram_chunk_start(block, chunk);
                 chunk_end = ram_chunk_end(block, chunk + reg->chunks);
+                /* avoid "-Waddress-of-packed-member" warning */
+                uint32_t tmp_rkey = 0;
                 if (qemu_rdma_register_and_get_keys(rdma, block,
-                            (uintptr_t)host_addr, NULL, &reg_result->rkey,
+                            (uintptr_t)host_addr, NULL, &tmp_rkey,
                             chunk, chunk_start, chunk_end)) {
                     error_report("cannot get rkey");
                     ret = -EINVAL;
                     goto out;
                 }
+                reg_result->rkey = tmp_rkey;
 
                 reg_result->host_addr = (uintptr_t)block->local_host_addr;
 
-- 
2.17.1

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [Qemu-devel] [PATCH V2] migration/rdma: clang compilation fix
  2019-03-04 18:49 [Qemu-devel] [PATCH V2] migration/rdma: clang compilation fix Marcel Apfelbaum
@ 2019-03-04 18:53 ` Dr. David Alan Gilbert
  2019-03-04 23:25 ` Philippe Mathieu-Daudé
  2019-03-05 13:25 ` Dr. David Alan Gilbert
  2 siblings, 0 replies; 4+ messages in thread
From: Dr. David Alan Gilbert @ 2019-03-04 18:53 UTC (permalink / raw)
  To: Marcel Apfelbaum; +Cc: qemu-devel, eblake

* Marcel Apfelbaum (marcel.apfelbaum@gmail.com) wrote:
> Configuring QEMU with:
>         ../configure --cc=clang --enable-rdma
> 
> Leads to compilation error:
> 
>   CC      migration/rdma.o
>   CC      migration/block.o
>   qemu/migration/rdma.c:3615:58: error: taking address of packed member 'rkey' of class or structure
>       'RDMARegisterResult' may result in an unaligned pointer value [-Werror,-Waddress-of-packed-member]
>                             (uintptr_t)host_addr, NULL, &reg_result->rkey,
>                                                          ^~~~~~~~~~~~~~~~
> Fix it by using a temp local variable.
> 
> Signed-off-by: Marcel Apfelbaum <marcel.apfelbaum@gmail.com>

Reviewed-by: Dr. David Alan Gilbert <dgilbert@redhat.com>

> ---
> 
> v1 -> v2: 
>  - Use a temp local variable (Dave)
> 
>  migration/rdma.c | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
> 
> diff --git a/migration/rdma.c b/migration/rdma.c
> index 54a3c11540..e39ee77558 100644
> --- a/migration/rdma.c
> +++ b/migration/rdma.c
> @@ -3611,13 +3611,16 @@ static int qemu_rdma_registration_handle(QEMUFile *f, void *opaque)
>                  }
>                  chunk_start = ram_chunk_start(block, chunk);
>                  chunk_end = ram_chunk_end(block, chunk + reg->chunks);
> +                /* avoid "-Waddress-of-packed-member" warning */
> +                uint32_t tmp_rkey = 0;
>                  if (qemu_rdma_register_and_get_keys(rdma, block,
> -                            (uintptr_t)host_addr, NULL, &reg_result->rkey,
> +                            (uintptr_t)host_addr, NULL, &tmp_rkey,
>                              chunk, chunk_start, chunk_end)) {
>                      error_report("cannot get rkey");
>                      ret = -EINVAL;
>                      goto out;
>                  }
> +                reg_result->rkey = tmp_rkey;
>  
>                  reg_result->host_addr = (uintptr_t)block->local_host_addr;
>  
> -- 
> 2.17.1
> 
--
Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [Qemu-devel] [PATCH V2] migration/rdma: clang compilation fix
  2019-03-04 18:49 [Qemu-devel] [PATCH V2] migration/rdma: clang compilation fix Marcel Apfelbaum
  2019-03-04 18:53 ` Dr. David Alan Gilbert
@ 2019-03-04 23:25 ` Philippe Mathieu-Daudé
  2019-03-05 13:25 ` Dr. David Alan Gilbert
  2 siblings, 0 replies; 4+ messages in thread
From: Philippe Mathieu-Daudé @ 2019-03-04 23:25 UTC (permalink / raw)
  To: Marcel Apfelbaum, qemu-devel; +Cc: dgilbert

On 3/4/19 7:49 PM, Marcel Apfelbaum wrote:
> Configuring QEMU with:
>         ../configure --cc=clang --enable-rdma
> 
> Leads to compilation error:
> 
>   CC      migration/rdma.o
>   CC      migration/block.o
>   qemu/migration/rdma.c:3615:58: error: taking address of packed member 'rkey' of class or structure
>       'RDMARegisterResult' may result in an unaligned pointer value [-Werror,-Waddress-of-packed-member]
>                             (uintptr_t)host_addr, NULL, &reg_result->rkey,
>                                                          ^~~~~~~~~~~~~~~~
> Fix it by using a temp local variable.
> 
> Signed-off-by: Marcel Apfelbaum <marcel.apfelbaum@gmail.com>
> ---
> 
> v1 -> v2: 
>  - Use a temp local variable (Dave)

Way cleaner than your v1 :)

> 
>  migration/rdma.c | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
> 
> diff --git a/migration/rdma.c b/migration/rdma.c
> index 54a3c11540..e39ee77558 100644
> --- a/migration/rdma.c
> +++ b/migration/rdma.c
> @@ -3611,13 +3611,16 @@ static int qemu_rdma_registration_handle(QEMUFile *f, void *opaque)
>                  }
>                  chunk_start = ram_chunk_start(block, chunk);
>                  chunk_end = ram_chunk_end(block, chunk + reg->chunks);
> +                /* avoid "-Waddress-of-packed-member" warning */

No sure the comment is helpful.

> +                uint32_t tmp_rkey = 0;

Probably no need to zero-initialize.

Anyway:
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>

>                  if (qemu_rdma_register_and_get_keys(rdma, block,
> -                            (uintptr_t)host_addr, NULL, &reg_result->rkey,
> +                            (uintptr_t)host_addr, NULL, &tmp_rkey,
>                              chunk, chunk_start, chunk_end)) {
>                      error_report("cannot get rkey");
>                      ret = -EINVAL;
>                      goto out;
>                  }
> +                reg_result->rkey = tmp_rkey;
>  
>                  reg_result->host_addr = (uintptr_t)block->local_host_addr;
>  
> 

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [Qemu-devel] [PATCH V2] migration/rdma: clang compilation fix
  2019-03-04 18:49 [Qemu-devel] [PATCH V2] migration/rdma: clang compilation fix Marcel Apfelbaum
  2019-03-04 18:53 ` Dr. David Alan Gilbert
  2019-03-04 23:25 ` Philippe Mathieu-Daudé
@ 2019-03-05 13:25 ` Dr. David Alan Gilbert
  2 siblings, 0 replies; 4+ messages in thread
From: Dr. David Alan Gilbert @ 2019-03-05 13:25 UTC (permalink / raw)
  To: Marcel Apfelbaum; +Cc: qemu-devel, eblake

* Marcel Apfelbaum (marcel.apfelbaum@gmail.com) wrote:
> Configuring QEMU with:
>         ../configure --cc=clang --enable-rdma
> 
> Leads to compilation error:
> 
>   CC      migration/rdma.o
>   CC      migration/block.o
>   qemu/migration/rdma.c:3615:58: error: taking address of packed member 'rkey' of class or structure
>       'RDMARegisterResult' may result in an unaligned pointer value [-Werror,-Waddress-of-packed-member]
>                             (uintptr_t)host_addr, NULL, &reg_result->rkey,
>                                                          ^~~~~~~~~~~~~~~~
> Fix it by using a temp local variable.
> 
> Signed-off-by: Marcel Apfelbaum <marcel.apfelbaum@gmail.com>

Queued

> ---
> 
> v1 -> v2: 
>  - Use a temp local variable (Dave)
> 
>  migration/rdma.c | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
> 
> diff --git a/migration/rdma.c b/migration/rdma.c
> index 54a3c11540..e39ee77558 100644
> --- a/migration/rdma.c
> +++ b/migration/rdma.c
> @@ -3611,13 +3611,16 @@ static int qemu_rdma_registration_handle(QEMUFile *f, void *opaque)
>                  }
>                  chunk_start = ram_chunk_start(block, chunk);
>                  chunk_end = ram_chunk_end(block, chunk + reg->chunks);
> +                /* avoid "-Waddress-of-packed-member" warning */
> +                uint32_t tmp_rkey = 0;
>                  if (qemu_rdma_register_and_get_keys(rdma, block,
> -                            (uintptr_t)host_addr, NULL, &reg_result->rkey,
> +                            (uintptr_t)host_addr, NULL, &tmp_rkey,
>                              chunk, chunk_start, chunk_end)) {
>                      error_report("cannot get rkey");
>                      ret = -EINVAL;
>                      goto out;
>                  }
> +                reg_result->rkey = tmp_rkey;
>  
>                  reg_result->host_addr = (uintptr_t)block->local_host_addr;
>  
> -- 
> 2.17.1
> 
> 
--
Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2019-03-05 13:25 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-03-04 18:49 [Qemu-devel] [PATCH V2] migration/rdma: clang compilation fix Marcel Apfelbaum
2019-03-04 18:53 ` Dr. David Alan Gilbert
2019-03-04 23:25 ` Philippe Mathieu-Daudé
2019-03-05 13:25 ` Dr. David Alan Gilbert

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).