[dpdk-dev] [PATCH] virtio: fix memory leak of virtqueue memzones

2016-04-28 Thread Tan, Jianfeng
Hi Yuanhan, On 4/28/2016 6:37 AM, Yuanhan Liu wrote: > On Tue, Apr 26, 2016 at 12:32:12PM +, Jianfeng Tan wrote: >> Issue: When virtio was proposed in DPDK, there is no API to free memzones. >> But this has changed since rte_memzone_free() has been implemented by >> commit ff909fe21f. > The m

[dpdk-dev] [PATCH] virtio: fix memory leak of virtqueue memzones

2016-04-27 Thread Yuanhan Liu
On Thu, Apr 28, 2016 at 10:01:06AM +0800, Tan, Jianfeng wrote: > >>+ > >>+ snprintf(vq_name, sizeof(vq_name), "port%d_%s%d", > >>+dev->data->port_id, queue_names[queue_type], queue_idx); > >>+ vq = rte_zmalloc(vq_name, sizeof(struct virtqueue) + > >>+vq_size * si

[dpdk-dev] [PATCH] virtio: fix memory leak of virtqueue memzones

2016-04-27 Thread Yuanhan Liu
On Tue, Apr 26, 2016 at 12:32:12PM +, Jianfeng Tan wrote: > Issue: When virtio was proposed in DPDK, there is no API to free memzones. > But this has changed since rte_memzone_free() has been implemented by > commit ff909fe21f. The more proper way to reference a commit is commit_id ("

[dpdk-dev] [PATCH] virtio: fix memory leak of virtqueue memzones

2016-04-26 Thread Jianfeng Tan
Issue: When virtio was proposed in DPDK, there is no API to free memzones. But this has changed since rte_memzone_free() has been implemented by commit ff909fe21f. This patch is to make sure memzones in struct virtqueue, like mz and virtio_net_hdr_mz, are freed when queue is released or setup fail