On 27-Jan-18 2:53 PM, Radoslaw Biernacki wrote:
Looks OK.
Following note is aside from the patch.
Might be beneficial (in some rare cases) to add bailout recovery with
goto's in test_memzone_basic()
Just in case one of the rte_memzone_reserve() we should not make return -1,
but instead a goto to below section where we call rte_memzone_free().
This way we would be able to free only the allocated memzones and prevent
leaking out those memzones to other tests.
Thanks, and yep, it's on my todo list :) didn't get around to it yet.
Reviewed-by: Radoslaw Biernacki <r <ferruh.yi...@intel.com>
adoslaw.bierna...@linaro.com>
--
Thanks,
Anatoly