[dpdk-dev] [PATCH v2 1/1] vhost: fix leak of fds and mmaps

2016-02-06 Thread Yuanhan Liu
Hey Rich, Sorry for the long delay; I barely forgot it :( On Tue, Jan 19, 2016 at 10:13:23AM -0800, Rich Lane wrote: > On Sun, Jan 17, 2016 at 11:58 PM, Yuanhan Liu > wrote: > > On Sun, Jan 17, 2016 at 11:57:18AM -0800, Rich Lane wrote: > > +/* > > + * Implementation-specific cleanu

[dpdk-dev] [PATCH v2 1/1] vhost: fix leak of fds and mmaps

2016-01-19 Thread Rich Lane
On Sun, Jan 17, 2016 at 11:58 PM, Yuanhan Liu wrote: > On Sun, Jan 17, 2016 at 11:57:18AM -0800, Rich Lane wrote: > > The common vhost code only supported a single mmap per device. vhost-user > > worked around this by saving the address/length/fd of each mmap after > the end > > of the rte_virtio

[dpdk-dev] [PATCH v2 1/1] vhost: fix leak of fds and mmaps

2016-01-18 Thread Yuanhan Liu
On Sun, Jan 17, 2016 at 11:57:18AM -0800, Rich Lane wrote: > The common vhost code only supported a single mmap per device. vhost-user > worked around this by saving the address/length/fd of each mmap after the end > of the rte_virtio_memory struct. This only works if the vhost-user code frees > de

[dpdk-dev] [PATCH v2 1/1] vhost: fix leak of fds and mmaps

2016-01-17 Thread Rich Lane
The common vhost code only supported a single mmap per device. vhost-user worked around this by saving the address/length/fd of each mmap after the end of the rte_virtio_memory struct. This only works if the vhost-user code frees dev->mem, since the common code is unaware of the extra info. The VHO