Re: [dpdk-dev] [PATCH v2] test/eventdev: add multi port test to suite

["Rao, Nikhil" <nikhil.rao@intel.com>]

Hi Vipin,


> -----Original Message-----
> From: Varghese, Vipin
> Sent: Monday, May 21, 2018 4:27 PM
> To: dev@dpdk.org; Rao, Nikhil <nikhil.rao@intel.com>; Jain, Deepak K
> <deepak.k.jain@intel.com>
> Cc: Varghese, Vipin <vipin.varghese@intel.com>
> Subject: [PATCH v2] test/eventdev: add multi port test to suite
> 
> Add a new test to enhance the test suite, to allow multiple eth ports rx queue
> to be added to rx bridge adapter. Update the test function to reflect change
> of port index from 8 to 16 bit
> 
> Signed-off-by: Vipin Varghese <vipin.varghese@intel.com>
> Acked-by: Nikhil Rao <nikhil.rao@intel.com>
> ---
> 
> Changes in V2:
> - reprhased the variable names for dtiver - Nikhil Rao
> - Initialization removed for driver_name - Nikhil Rao
> - removed adapter capability instance - Nikhil Rao
> - reprhased comment - Nikhil Rao
> ---
>  test/test/test_event_eth_rx_adapter.c | 78
> ++++++++++++++++++++++++++++++++++-
>  1 file changed, 77 insertions(+), 1 deletion(-)
> 
> diff --git a/test/test/test_event_eth_rx_adapter.c
> b/test/test/test_event_eth_rx_adapter.c
> 
</sniped>

> +
> +	err = rte_event_eth_rx_adapter_caps_get(TEST_DEV_ID,
> TEST_ETHDEV_ID,
> +			&cap);
> +	TEST_ASSERT(err == 0, "Expected 0 got %d", err);
> +
You can remove the call to caps_get, cap isn't referenced.

Nikhil