diff mbox

[dpdk-dev,v2,1/2] test/memzone: Fix test_memzone_basic issue

Message ID 1515995546-5981-2-git-send-email-phil.yang@arm.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 fail Compilation issues

Commit Message

Phil Yang Jan. 15, 2018, 5:52 a.m. UTC 
  No need to free a NULL memzone. It will cause test
termination.

Signed-off-by: Phil Yang <phil.yang@arm.com>
---
 test/test/test_memzone.c | 4 ----
 1 file changed, 4 deletions(-)

Comments

Anatoly Burakov Jan. 15, 2018, 9:48 a.m. UTC | #1 
On 15-Jan-18 5:52 AM, Phil Yang wrote:
> No need to free a NULL memzone. It will cause test
> termination.
> 
> Signed-off-by: Phil Yang <phil.yang@arm.com>
> ---

Acked-by: Anatoly Burakov <anatoly.burakov@intel.com>
diff mbox

Patch

diff --git a/test/test/test_memzone.c b/test/test/test_memzone.c
index 6e80977..9c20172 100644
--- a/test/test/test_memzone.c
+++ b/test/test/test_memzone.c
@@ -251,10 +251,6 @@  test_memzone_reserve_flags(void)
 				printf("MEMZONE FLAG 2MB\n");
 				return -1;
 			}
-			if (rte_memzone_free(mz)) {
-				printf("Fail memzone free\n");
-				return -1;
-			}
 		}
 
 		if (hugepage_2MB_avail && hugepage_1GB_avail) {