[dpdk-dev] [PATCH v4 4/4] eventdev: Add tests for event eth Rx adapter APIs
Rao, Nikhil
nikhil.rao at intel.com
Sun Sep 24 20:24:38 CEST 2017
On 9/22/2017 5:42 PM, Jerin Jacob wrote:
> -----Original Message-----
>> Date: Fri, 22 Sep 2017 02:47:14 +0530
>> From: Nikhil Rao <nikhil.rao at intel.com>
>> To: jerin.jacob at caviumnetworks.com, bruce.richardson at intel.com
>> CC: gage.eads at intel.com, dev at dpdk.org, thomas at monjalon.net,
>> harry.van.haaren at intel.com, hemant.agrawal at nxp.com, nipun.gupta at nxp.com,
>> narender.vangati at intel.com, erik.g.carrillo at intel.com,
>> abhinandan.gujjar at intel.com, santosh.shukla at caviumnetworks.com
>> Subject: [PATCH v4 4/4] eventdev: Add tests for event eth Rx adapter APIs
>> X-Mailer: git-send-email 2.7.4
>>
>> Add unit tests for rte_event_eth_rx_adapter_xxx() APIs
>
<snip>
>> +#include <string.h>
>> +#include <rte_common.h>
>> +#include <rte_mempool.h>
>> +#include <rte_mbuf.h>
>> +#include <rte_ethdev.h>
>> +#include <rte_eventdev.h>
>> +
>> +#include <rte_event_eth_rx_adapter.h>
>> +
>> +#include "test.h"
>> +
>> +/* i40e limits max to 64 */
>
> This comment could be removed.
>
OK, I am documenting why the code doesn't just use
dev_info.max_rx_queues, won't the comment be useful to retain ?
>> +#define MAX_NUM_RX_QUEUE 64
>> +#define NB_MBUFS (8192 * num_ports * MAX_NUM_RX_QUEUE)
>> +#define MBUF_CACHE_SIZE 512
>> +#define MBUF_PRIV_SIZE 0
>> +
>> +struct event_eth_rx_adapter_test_params {
>> + struct rte_mempool *mp;
>> + uint16_t rx_rings, tx_rings;
>> + uint32_t caps;
>> +};
>> +
>> +static struct event_eth_rx_adapter_test_params default_params;
>> +
>> +static int
>> +testsuite_setup(void)
>> +{
>> + int err;
>> + err = init_ports(rte_eth_dev_count());
>> + TEST_ASSERT(err == 0, "Port initialization failed err %d\n", err);
>
> I guess, We check rte_event_dev_count() >= 1 before proceeding.
Yes, I can add the same logic as in test_evendev.
>
>> +
>> + struct rte_event_dev_config config = {
>> + .nb_event_queues = 1,
>> + .nb_event_ports = 1,
>> + .nb_events_limit = 4096,
>> + .nb_event_queue_flows = 1024,
>> + .nb_event_port_dequeue_depth = 16,
>> + .nb_event_port_enqueue_depth = 16
>> + };
>> +
>> + err = rte_event_dev_configure(0, &config);
>> + TEST_ASSERT(err == 0, "Event device initialization failed err %d\n",
>> + err);
>> +
>> + err = rte_event_eth_rx_adapter_caps_get(0, 0, &default_params.caps);
>> + TEST_ASSERT(err == 0, "Failed to get adapter cap err %d\n",
>> + err);
>> +
>> + return err;
>> +}
>> +
>
Thanks for the review,
Nikhil
More information about the dev
mailing list