From da8d4450553ef09aff64eeecb774cb6abecabdb0 Mon Sep 17 00:00:00 2001 From: Kevin Paul Herbert Date: Tue, 15 Dec 2015 16:33:13 -0800 Subject: dpdk_buffer.c: Ensure that the mbuf reference count is as expected Add some more ASSERTs to track down improper frees. Change-Id: I2bd4b69fb14f522c82e6006131b6ad982f6f7e6b Signed-off-by: Kevin Paul Herbert --- vlib/vlib/dpdk_buffer.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/vlib/vlib/dpdk_buffer.c b/vlib/vlib/dpdk_buffer.c index 68c1fcdb..43612a86 100644 --- a/vlib/vlib/dpdk_buffer.c +++ b/vlib/vlib/dpdk_buffer.c @@ -401,11 +401,13 @@ del_free_list (vlib_main_t * vm, vlib_buffer_free_list_t * f) for (i = 0; i < vec_len (f->unaligned_buffers); i++) { b = vlib_get_buffer (vm, f->unaligned_buffers[i]); mb = ((struct rte_mbuf *)b)-1; + ASSERT(rte_mbuf_refcnt_read(mb) == 1); rte_pktmbuf_free (mb); } for (i = 0; i < vec_len (f->aligned_buffers); i++) { b = vlib_get_buffer (vm, f->aligned_buffers[i]); mb = ((struct rte_mbuf *)b)-1; + ASSERT(rte_mbuf_refcnt_read(mb) == 1); rte_pktmbuf_free (mb); } vec_free (f->name); @@ -723,6 +725,7 @@ vlib_buffer_free_inline (vlib_main_t * vm, else { mb = ((struct rte_mbuf *)b)-1; + ASSERT(rte_mbuf_refcnt_read(mb) == 1); rte_pktmbuf_free (mb); } } -- cgit 1.2.3-korg