Re: [dpdk-dev] [PATCH v3 1/2] ethdev: free all common data when releasing port

2018-10-16 Thread Andrew Rybchenko
On 10/16/18 3:52 PM, Thomas Monjalon wrote: 16/10/2018 14:47, Andrew Rybchenko: On 10/16/18 3:22 PM, Thomas Monjalon wrote: 16/10/2018 13:16, Andrew Rybchenko: On 10/15/18 2:20 AM, Thomas Monjalon wrote: This is a clean-up of common ethdev data freeing. All data freeing are moved to rte_eth_d

Re: [dpdk-dev] [PATCH v3 1/2] ethdev: free all common data when releasing port

2018-10-16 Thread Thomas Monjalon
16/10/2018 14:47, Andrew Rybchenko: > On 10/16/18 3:22 PM, Thomas Monjalon wrote: > > 16/10/2018 13:16, Andrew Rybchenko: > >> On 10/15/18 2:20 AM, Thomas Monjalon wrote: > >>> This is a clean-up of common ethdev data freeing. > >>> All data freeing are moved to rte_eth_dev_release_port() > >>> and

Re: [dpdk-dev] [PATCH v3 1/2] ethdev: free all common data when releasing port

2018-10-16 Thread Andrew Rybchenko
On 10/16/18 3:22 PM, Thomas Monjalon wrote: 16/10/2018 13:16, Andrew Rybchenko: On 10/15/18 2:20 AM, Thomas Monjalon wrote: This is a clean-up of common ethdev data freeing. All data freeing are moved to rte_eth_dev_release_port() and done only in case of primary process. It is probably fixing

Re: [dpdk-dev] [PATCH v3 1/2] ethdev: free all common data when releasing port

2018-10-16 Thread Thomas Monjalon
16/10/2018 13:16, Andrew Rybchenko: > On 10/15/18 2:20 AM, Thomas Monjalon wrote: > > This is a clean-up of common ethdev data freeing. > > All data freeing are moved to rte_eth_dev_release_port() > > and done only in case of primary process. > > > > It is probably fixing some memory leaks for PMDs

Re: [dpdk-dev] [PATCH v3 1/2] ethdev: free all common data when releasing port

2018-10-16 Thread Andrew Rybchenko
On 10/15/18 2:20 AM, Thomas Monjalon wrote: This is a clean-up of common ethdev data freeing. All data freeing are moved to rte_eth_dev_release_port() and done only in case of primary process. It is probably fixing some memory leaks for PMDs which were not freeing all data. Signed-off-by: Thoma

[dpdk-dev] [PATCH v3 1/2] ethdev: free all common data when releasing port

2018-10-14 Thread Thomas Monjalon
This is a clean-up of common ethdev data freeing. All data freeing are moved to rte_eth_dev_release_port() and done only in case of primary process. It is probably fixing some memory leaks for PMDs which were not freeing all data. Signed-off-by: Thomas Monjalon --- drivers/net/af_packet/rte_eth