This patch does not compile when ivshmem is disabled: compile error: CC malloc_heap.o
/home/patchWorkOrg/compilation/lib/librte_eal/common/eal_common_memzone.c(353): error #177: label "error" was declared but never referenced error: ^ I'll send a v2 solving this issue On Thu, Apr 14, 2016 at 4:48 PM, Sergio Gonzalez Monroy < sergio.gonzalez.monroy at intel.com> wrote: > On 14/04/2016 14:48, Mauricio Vasquez B wrote: > >> although previous implementation returned an error when trying to release >> a >> memzone assigned to an ivshmem device, it stills freed it. >> >> Fixes: cd10c42eb5bc ("mem: fix ivshmem freeing") >> >> Signed-off-by: Mauricio Vasquez B < >> mauricio.vasquezbernal at studenti.polito.it> >> --- >> lib/librte_eal/common/eal_common_memzone.c | 12 ++++++++++-- >> 1 file changed, 10 insertions(+), 2 deletions(-) >> >> >> > Thanks for the fix (not sure what I was thinking at the time). > > Acked-by: Sergio Gonzalez Monroy <sergio.gonzalez.monroy at intel.com> > >