1. Patchew
  2. QEMU
  3. View series

[PATCH] tests/qtest/fuzz: Add missing spaces in description

Thomas Huth posted 1 patch 5 years, 7 months ago
Test checkpatch passed
Test docker-mingw@fedora passed
Test FreeBSD passed
Test docker-quick@centos7 passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20200709083719.22221-1-thuth@redhat.com
Maintainers: Laurent Vivier <lvivier@redhat.com>, Thomas Huth <thuth@redhat.com>, Alexander Bulekov <alxndr@bu.edu>, Paolo Bonzini <pbonzini@redhat.com>, Stefan Hajnoczi <stefanha@redhat.com>, Bandan Das <bsd@redhat.com>
tests/qtest/fuzz/virtio_scsi_fuzz.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
[PATCH] tests/qtest/fuzz: Add missing spaces in description
Posted by Thomas Huth 5 years, 7 months ago 
There should be a space between "forking" and "for".

Signed-off-by: Thomas Huth <thuth@redhat.com>
---
 tests/qtest/fuzz/virtio_scsi_fuzz.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/tests/qtest/fuzz/virtio_scsi_fuzz.c b/tests/qtest/fuzz/virtio_scsi_fuzz.c
index 51dce491ab..3a9ea13736 100644
--- a/tests/qtest/fuzz/virtio_scsi_fuzz.c
+++ b/tests/qtest/fuzz/virtio_scsi_fuzz.c
@@ -191,7 +191,7 @@ static void register_virtio_scsi_fuzz_targets(void)
 {
     fuzz_add_qos_target(&(FuzzTarget){
                 .name = "virtio-scsi-fuzz",
-                .description = "Fuzz the virtio-scsi virtual queues, forking"
+                .description = "Fuzz the virtio-scsi virtual queues, forking "
                                 "for each fuzz run",
                 .pre_vm_init = &counter_shm_init,
                 .pre_fuzz = &virtio_scsi_pre_fuzz,
@@ -202,7 +202,7 @@ static void register_virtio_scsi_fuzz_targets(void)
 
     fuzz_add_qos_target(&(FuzzTarget){
                 .name = "virtio-scsi-flags-fuzz",
-                .description = "Fuzz the virtio-scsi virtual queues, forking"
+                .description = "Fuzz the virtio-scsi virtual queues, forking "
                 "for each fuzz run (also fuzzes the virtio flags)",
                 .pre_vm_init = &counter_shm_init,
                 .pre_fuzz = &virtio_scsi_pre_fuzz,
-- 
2.18.1
Re: [PATCH] tests/qtest/fuzz: Add missing spaces in description
Posted by Philippe Mathieu-Daudé 5 years, 7 months ago 
On 7/9/20 10:37 AM, Thomas Huth wrote:
> There should be a space between "forking" and "for".
> 
> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
>  tests/qtest/fuzz/virtio_scsi_fuzz.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/tests/qtest/fuzz/virtio_scsi_fuzz.c b/tests/qtest/fuzz/virtio_scsi_fuzz.c
> index 51dce491ab..3a9ea13736 100644
> --- a/tests/qtest/fuzz/virtio_scsi_fuzz.c
> +++ b/tests/qtest/fuzz/virtio_scsi_fuzz.c
> @@ -191,7 +191,7 @@ static void register_virtio_scsi_fuzz_targets(void)
>  {
>      fuzz_add_qos_target(&(FuzzTarget){
>                  .name = "virtio-scsi-fuzz",
> -                .description = "Fuzz the virtio-scsi virtual queues, forking"
> +                .description = "Fuzz the virtio-scsi virtual queues, forking "
>                                  "for each fuzz run",
>                  .pre_vm_init = &counter_shm_init,
>                  .pre_fuzz = &virtio_scsi_pre_fuzz,
> @@ -202,7 +202,7 @@ static void register_virtio_scsi_fuzz_targets(void)
>  
>      fuzz_add_qos_target(&(FuzzTarget){
>                  .name = "virtio-scsi-flags-fuzz",
> -                .description = "Fuzz the virtio-scsi virtual queues, forking"
> +                .description = "Fuzz the virtio-scsi virtual queues, forking "
>                  "for each fuzz run (also fuzzes the virtio flags)",
>                  .pre_vm_init = &counter_shm_init,
>                  .pre_fuzz = &virtio_scsi_pre_fuzz,
> 

Uh I thought we had fixed these already :/

Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Re: [PATCH] tests/qtest/fuzz: Add missing spaces in description
Posted by Alexander Bulekov 5 years, 7 months ago 
On 200709 1228, Philippe Mathieu-Daudé wrote:
> On 7/9/20 10:37 AM, Thomas Huth wrote:
> > There should be a space between "forking" and "for".
> > 
> > Signed-off-by: Thomas Huth <thuth@redhat.com>
> > ---
> >  tests/qtest/fuzz/virtio_scsi_fuzz.c | 4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> > 
> > diff --git a/tests/qtest/fuzz/virtio_scsi_fuzz.c b/tests/qtest/fuzz/virtio_scsi_fuzz.c
> > index 51dce491ab..3a9ea13736 100644
> > --- a/tests/qtest/fuzz/virtio_scsi_fuzz.c
> > +++ b/tests/qtest/fuzz/virtio_scsi_fuzz.c
> > @@ -191,7 +191,7 @@ static void register_virtio_scsi_fuzz_targets(void)
> >  {
> >      fuzz_add_qos_target(&(FuzzTarget){
> >                  .name = "virtio-scsi-fuzz",
> > -                .description = "Fuzz the virtio-scsi virtual queues, forking"
> > +                .description = "Fuzz the virtio-scsi virtual queues, forking "
> >                                  "for each fuzz run",
> >                  .pre_vm_init = &counter_shm_init,
> >                  .pre_fuzz = &virtio_scsi_pre_fuzz,
> > @@ -202,7 +202,7 @@ static void register_virtio_scsi_fuzz_targets(void)
> >  
> >      fuzz_add_qos_target(&(FuzzTarget){
> >                  .name = "virtio-scsi-flags-fuzz",
> > -                .description = "Fuzz the virtio-scsi virtual queues, forking"
> > +                .description = "Fuzz the virtio-scsi virtual queues, forking "
> >                  "for each fuzz run (also fuzzes the virtio flags)",
> >                  .pre_vm_init = &counter_shm_init,
> >                  .pre_fuzz = &virtio_scsi_pre_fuzz,
> > 
> 
> Uh I thought we had fixed these already :/
> 
> Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>

Same..
Reviewed-by: Alexander Bulekov <alxndr@bu.edu>

Patchew 2.1-devel-b67e4c2

© 2016 - 2026 Red Hat, Inc.