Re: [dpdk-stable] [PATCH] examples/l2fwd/main.c: free resources in case of error

["Varghese, Vipin" <vipin.varghese@intel.com>]

snipped
> Subject: [PATCH] examples/l2fwd/main.c: free resources in case of error
> 
> Bugzilla ID: 437
> Cc: dev@dpdk.org
> Cc: stable@dpdk.org
> Cc: bruce.richardson@intel.com
> Cc: vipin.varghese@intel.com
> Cc: jgrajcia@cisco.com
> Signed-off-by: Muhammad Bilal <m.bilal@emumba.com>
> ---
>  examples/l2fwd/main.c | 72 ++++++++++++++++++++++++++++++++++---------
>  1 file changed, 57 insertions(+), 15 deletions(-)
> 
> diff --git a/examples/l2fwd/main.c b/examples/l2fwd/main.c index
> 88ddfe589..dbaa457e2 100644
> --- a/examples/l2fwd/main.c
> +++ b/examples/l2fwd/main.c
> @@ -517,6 +517,17 @@ signal_handler(int signum)
>  	}
>  }
> 
> +static void
> +stop_and_close_eth_dev(uint16_t portid) {
> +	RTE_ETH_FOREACH_DEV(portid) {
> +		printf("Closing port %d...", portid);
> +		rte_eth_dev_stop(portid);
> +		rte_eth_dev_close(portid);
> +		printf(" Done\n");
> +	}
> +}
> +
If the solution is to cleanup in application, then all application needs to do cleanup. Hence `+1` for updating this to all examples in DPDK.