examples/multi_process: fix client crash with sparse ports
authorStephen Hemminger <sthemmin@microsoft.com>
Mon, 5 Aug 2019 16:38:17 +0000 (09:38 -0700)
committerThomas Monjalon <thomas@monjalon.net>
Mon, 25 Nov 2019 22:52:00 +0000 (23:52 +0100)
commit6b124806a3181855fed969d0ad8520831ae0d7e2
treed31f05407b6093d0b5830bc072522e4eab3ac35c
parent1f41d98c207aee8982ced709864c96c463d4503a
examples/multi_process: fix client crash with sparse ports

The mp_client crashes if run on Azure or any system where ethdev
ports are owned. In that case, the tx_buffer and tx_stats for the
real port were initialized correctly, but the wrong port was used.

For example if the server has Ports 3 and 5. Then calling
rte_eth_tx_buffer_flush on any other buffer will dereference null
because the tx buffer for that port was not allocated.

Also:
   - the flush code is common enough that it should not be marked
     unlikely
   - combine conditions to reduce indentation
   - avoid unnecessary if() if sent is zero.

Fixes: e2366e74e029 ("examples: use buffered Tx")
Cc: stable@dpdk.org
Signed-off-by: Stephen Hemminger <sthemmin@microsoft.com>
Acked-by: Matan Azrad <matan@mellanox.com>
examples/multi_process/client_server_mp/mp_client/client.c