diff mbox

[dpdk-dev,1/3] test: fix typo in memzone autotest

Message ID 5c1a7f3bfe6bbfc86f536541833de4ab1789a95c.1513866482.git.anatoly.burakov@intel.com (mailing list archive)
State Accepted, archived
Delegated to: Thomas Monjalon
Headers

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/Intel-compilation success Compilation OK

Commit Message

Anatoly Burakov Dec. 21, 2017, 6:19 p.m. UTC 
  Fixes: 71330483a193 ("test/memzone: fix memory leak")
Cc: radoslaw.biernacki@linaro.org
Cc: stable@dpdk.org
Signed-off-by: Anatoly Burakov <anatoly.burakov@intel.com>
---
 test/test/test_memzone.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

Comments

Radoslaw Biernacki Jan. 3, 2018, 12:01 p.m. UTC | #1 
s///g effects ;)
Thank you!

Reviewed-by: Radoslaw Biernacki <radoslaw.biernacki@linaro.org>

On 21 December 2017 at 19:19, Anatoly Burakov <anatoly.burakov@intel.com>
wrote:

> Fixes: 71330483a193 ("test/memzone: fix memory leak")
> Cc: radoslaw.biernacki@linaro.org
> Cc: stable@dpdk.org
> Signed-off-by: Anatoly Burakov <anatoly.burakov@intel.com>
> ---
>  test/test/test_memzone.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/test/test/test_memzone.c b/test/test/test_memzone.c
> index 1cf235a..aaffef9 100644
> --- a/test/test/test_memzone.c
> +++ b/test/test/test_memzone.c
> @@ -948,11 +948,11 @@ test_memzone_basic(void)
>         return 0;
>  }
>
> -static int memzone_calk_called;
> +static int memzone_walk_called;
>  static void memzone_walk_clb(const struct rte_memzone *mz __rte_unused,
>                              void *arg __rte_unused)
>  {
> -       memzone_calk_called = 1;
> +       memzone_walk_called = 1;
>  }
>
>  static int
> @@ -996,7 +996,7 @@ test_memzone(void)
>
>         printf("check memzone cleanup\n");
>         rte_memzone_walk(memzone_walk_clb, NULL);
> -       if (memzone_calk_called) {
> +       if (memzone_walk_called) {
>                 printf("there are some memzones left after test\n");
>                 rte_memzone_dump(stdout);
>                 return -1;
> --
> 2.7.4
>
Thomas Monjalon Jan. 17, 2018, 11:35 p.m. UTC | #2 
03/01/2018 13:01, Radoslaw Biernacki:
> s///g effects ;)
> Thank you!
> 
> Reviewed-by: Radoslaw Biernacki <radoslaw.biernacki@linaro.org>
> 
> On 21 December 2017 at 19:19, Anatoly Burakov <anatoly.burakov@intel.com>
> wrote:
> 
> > Fixes: 71330483a193 ("test/memzone: fix memory leak")
> > Cc: radoslaw.biernacki@linaro.org
> > Cc: stable@dpdk.org
> > Signed-off-by: Anatoly Burakov <anatoly.burakov@intel.com>

Series applied, thanks
diff mbox

Patch

diff --git a/test/test/test_memzone.c b/test/test/test_memzone.c
index 1cf235a..aaffef9 100644
--- a/test/test/test_memzone.c
+++ b/test/test/test_memzone.c
@@ -948,11 +948,11 @@  test_memzone_basic(void)
 	return 0;
 }
 
-static int memzone_calk_called;
+static int memzone_walk_called;
 static void memzone_walk_clb(const struct rte_memzone *mz __rte_unused,
 			     void *arg __rte_unused)
 {
-	memzone_calk_called = 1;
+	memzone_walk_called = 1;
 }
 
 static int
@@ -996,7 +996,7 @@  test_memzone(void)
 
 	printf("check memzone cleanup\n");
 	rte_memzone_walk(memzone_walk_clb, NULL);
-	if (memzone_calk_called) {
+	if (memzone_walk_called) {
 		printf("there are some memzones left after test\n");
 		rte_memzone_dump(stdout);
 		return -1;