All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] migration/multifd: Remove superfluous semicolons
@ 2020-09-21  4:02 Philippe Mathieu-Daudé
  2020-09-21  4:40 ` Li Qiang
  2020-09-23 17:14 ` Laurent Vivier
  0 siblings, 2 replies; 3+ messages in thread
From: Philippe Mathieu-Daudé @ 2020-09-21  4:02 UTC (permalink / raw)
  To: qemu-devel
  Cc: qemu-trivial, Philippe Mathieu-Daudé,
	Dr. David Alan Gilbert, Juan Quintela

checkpatch.pl report superfluous semicolons since commit
ee0f3c09e01, but this one was missed:

  scripts/checkpatch.pl d32ca5ad798~..d32ca5ad798
  ERROR: superfluous trailing semicolon
  #498: FILE: migration/multifd.c:308:
  +    ram_counters.transferred += transferred;;
  total: 1 errors, 1 warnings, 2073 lines checked

Fixes: d32ca5ad798 ("multifd: Split multifd code into its own file")
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
---
 migration/multifd.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/migration/multifd.c b/migration/multifd.c
index d0441202aae..8aaa2f9ade1 100644
--- a/migration/multifd.c
+++ b/migration/multifd.c
@@ -447,7 +447,7 @@ static int multifd_send_pages(QEMUFile *f)
                 + p->packet_len;
     qemu_file_update_transfer(f, transferred);
     ram_counters.multifd_bytes += transferred;
-    ram_counters.transferred += transferred;;
+    ram_counters.transferred += transferred;
     qemu_mutex_unlock(&p->mutex);
     qemu_sem_post(&p->sem);
 
-- 
2.26.2



^ permalink raw reply related	[flat|nested] 3+ messages in thread
* Re: [PATCH] migration/multifd: Remove superfluous semicolons
  2020-09-21  4:02 [PATCH] migration/multifd: Remove superfluous semicolons Philippe Mathieu-Daudé
@ 2020-09-21  4:40 ` Li Qiang
  2020-09-23 17:14 ` Laurent Vivier
  1 sibling, 0 replies; 3+ messages in thread
From: Li Qiang @ 2020-09-21  4:40 UTC (permalink / raw)
  To: Philippe Mathieu-Daudé
  Cc: qemu-trivial, Juan Quintela, Qemu Developers, Dr. David Alan Gilbert

Philippe Mathieu-Daudé <f4bug@amsat.org> 于2020年9月21日周一 下午12:03写道：
>
> checkpatch.pl report superfluous semicolons since commit
> ee0f3c09e01, but this one was missed:
>
>   scripts/checkpatch.pl d32ca5ad798~..d32ca5ad798
>   ERROR: superfluous trailing semicolon
>   #498: FILE: migration/multifd.c:308:
>   +    ram_counters.transferred += transferred;;
>   total: 1 errors, 1 warnings, 2073 lines checked
>
> Fixes: d32ca5ad798 ("multifd: Split multifd code into its own file")
> Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>

Reviewed-by: Li Qiang <liq3ea@gmail.com>

> ---
>  migration/multifd.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/migration/multifd.c b/migration/multifd.c
> index d0441202aae..8aaa2f9ade1 100644
> --- a/migration/multifd.c
> +++ b/migration/multifd.c
> @@ -447,7 +447,7 @@ static int multifd_send_pages(QEMUFile *f)
>                  + p->packet_len;
>      qemu_file_update_transfer(f, transferred);
>      ram_counters.multifd_bytes += transferred;
> -    ram_counters.transferred += transferred;;
> +    ram_counters.transferred += transferred;
>      qemu_mutex_unlock(&p->mutex);
>      qemu_sem_post(&p->sem);
>
> --
> 2.26.2
>
>


^ permalink raw reply	[flat|nested] 3+ messages in thread
* Re: [PATCH] migration/multifd: Remove superfluous semicolons
  2020-09-21  4:02 [PATCH] migration/multifd: Remove superfluous semicolons Philippe Mathieu-Daudé
  2020-09-21  4:40 ` Li Qiang
@ 2020-09-23 17:14 ` Laurent Vivier
  1 sibling, 0 replies; 3+ messages in thread
From: Laurent Vivier @ 2020-09-23 17:14 UTC (permalink / raw)
  To: Philippe Mathieu-Daudé, qemu-devel
  Cc: qemu-trivial, Dr. David Alan Gilbert, Juan Quintela

Le 21/09/2020 à 06:02, Philippe Mathieu-Daudé a écrit :
> checkpatch.pl report superfluous semicolons since commit
> ee0f3c09e01, but this one was missed:
> 
>   scripts/checkpatch.pl d32ca5ad798~..d32ca5ad798
>   ERROR: superfluous trailing semicolon
>   #498: FILE: migration/multifd.c:308:
>   +    ram_counters.transferred += transferred;;
>   total: 1 errors, 1 warnings, 2073 lines checked
> 
> Fixes: d32ca5ad798 ("multifd: Split multifd code into its own file")
> Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
> ---
>  migration/multifd.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/migration/multifd.c b/migration/multifd.c
> index d0441202aae..8aaa2f9ade1 100644
> --- a/migration/multifd.c
> +++ b/migration/multifd.c
> @@ -447,7 +447,7 @@ static int multifd_send_pages(QEMUFile *f)
>                  + p->packet_len;
>      qemu_file_update_transfer(f, transferred);
>      ram_counters.multifd_bytes += transferred;
> -    ram_counters.transferred += transferred;;
> +    ram_counters.transferred += transferred;
>      qemu_mutex_unlock(&p->mutex);
>      qemu_sem_post(&p->sem);
>  
> 

Applied to my trivial-patches branch.

Thanks,
Laurent



^ permalink raw reply	[flat|nested] 3+ messages in thread
end of thread, other threads:[~2020-09-23 17:15 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-09-21  4:02 [PATCH] migration/multifd: Remove superfluous semicolons Philippe Mathieu-Daudé
2020-09-21  4:40 ` Li Qiang
2020-09-23 17:14 ` Laurent Vivier

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.