test/red: fix typo in comment

Message ID 20211112135645.23942-1-david.marchand@redhat.com (mailing list archive)
State Accepted, archived
Delegated to: David Marchand
Headers
Series test/red: fix typo in comment |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/iol-broadcom-Functional success Functional Testing PASS
ci/Intel-compilation success Compilation OK
ci/iol-mellanox-Performance success Performance Testing PASS
ci/intel-Testing success Testing PASS
ci/iol-broadcom-Performance success Performance Testing PASS
ci/iol-intel-Performance success Performance Testing PASS
ci/github-robot: build success github build: passed
ci/iol-x86_64-compile-testing success Testing PASS
ci/iol-x86_64-unit-testing success Testing PASS
ci/iol-intel-Functional success Functional Testing PASS
ci/iol-aarch64-compile-testing success Testing PASS
ci/iol-aarch64-unit-testing success Testing PASS

Commit Message

David Marchand Nov. 12, 2021, 1:56 p.m. UTC
  This is probably due a s/red/rte_red/ a long time ago.

Fixes: de3cfa2c9823 ("sched: initial import")

Signed-off-by: David Marchand <david.marchand@redhat.com>
---
 app/test/test_red.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
  

Comments

Cristian Dumitrescu Nov. 15, 2021, 11:52 a.m. UTC | #1
> -----Original Message-----
> From: David Marchand <david.marchand@redhat.com>
> Sent: Friday, November 12, 2021 1:57 PM
> To: dev@dpdk.org
> Cc: Dumitrescu, Cristian <cristian.dumitrescu@intel.com>; Singh, Jasvinder
> <jasvinder.singh@intel.com>
> Subject: [PATCH] test/red: fix typo in comment
> 
> This is probably due a s/red/rte_red/ a long time ago.
> 
> Fixes: de3cfa2c9823 ("sched: initial import")
> 
> Signed-off-by: David Marchand <david.marchand@redhat.com>
> ---
>  app/test/test_red.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/app/test/test_red.c b/app/test/test_red.c
> index e973f3131e..05936cfee8 100644
> --- a/app/test/test_red.c
> +++ b/app/test/test_red.c
> @@ -1049,7 +1049,7 @@ static struct test_queue ft6_tqueue = {
>  static struct test_config func_test6_config = {
>  	.ifname = "functional test 6 interface",
>  	.msg = "functional test 6 : use several queues (each with its own run-
> time data),\n"
> -	"		    use several RED configurations (such that each
> configuration is sharte_red by multiple queues),\n"
> +	"		    use several RED configurations (such that each
> configuration is shared by multiple queues),\n"
>  	"		    increase average queue size to target level,\n"
>  	"		    dequeue all packets until queue is empty,\n"
>  	"		    confirm that average queue size is computed
> correctly while queue is empty\n"
> --
> 2.23.0

Acked-by: Cristian Dumitrescu <cristian.dumitrescu@yahoo.com>
  
David Marchand Nov. 16, 2021, 10:21 a.m. UTC | #2
On Mon, Nov 15, 2021 at 12:52 PM Dumitrescu, Cristian
<cristian.dumitrescu@intel.com> wrote:
> > This is probably due a s/red/rte_red/ a long time ago.
> >
> > Fixes: de3cfa2c9823 ("sched: initial import")
> >
> > Signed-off-by: David Marchand <david.marchand@redhat.com>
> Acked-by: Cristian Dumitrescu <cristian.dumitrescu@yahoo.com>

Applied, thanks.
  

Patch

diff --git a/app/test/test_red.c b/app/test/test_red.c
index e973f3131e..05936cfee8 100644
--- a/app/test/test_red.c
+++ b/app/test/test_red.c
@@ -1049,7 +1049,7 @@  static struct test_queue ft6_tqueue = {
 static struct test_config func_test6_config = {
 	.ifname = "functional test 6 interface",
 	.msg = "functional test 6 : use several queues (each with its own run-time data),\n"
-	"		    use several RED configurations (such that each configuration is sharte_red by multiple queues),\n"
+	"		    use several RED configurations (such that each configuration is shared by multiple queues),\n"
 	"		    increase average queue size to target level,\n"
 	"		    dequeue all packets until queue is empty,\n"
 	"		    confirm that average queue size is computed correctly while queue is empty\n"