> -----Original Message-----
> From: Jerin Jacob [mailto:jerin.ja...@caviumnetworks.com]
> Sent: Monday, May 7, 2018 3:39 PM
> To: Gujjar, Abhinandan S <abhinandan.guj...@intel.com>
> Cc: hemant.agra...@nxp.com; akhil.go...@nxp.com; dev@dpdk.org; Vangati,
> Narender <narender.vang...@intel.com>; Rao, Nikhil <nikhil....@intel.com>;
> Eads, Gage <gage.e...@intel.com>
> Subject: Re: [v3,4/5] test: add event crypto adapter auto-test
>
> -----Original Message-----
> > Date: Sun, 6 May 2018 00:17:09 +0530
> > From: Abhinandan Gujjar <abhinandan.guj...@intel.com>
> > To: jerin.ja...@caviumnetworks.com, hemant.agra...@nxp.com,
> > akhil.go...@nxp.com, dev@dpdk.org
> > CC: narender.vang...@intel.com, abhinandan.guj...@intel.com,
> > nikhil....@intel.com, gage.e...@intel.com
> > Subject: [v3,4/5] test: add event crypto adapter auto-test
> > X-Mailer: git-send-email 1.9.1
> >
> > Signed-off-by: Abhinandan Gujjar <abhinandan.guj...@intel.com>
> > ---
> > +struct event_crypto_adapter_test_params {
> > + struct rte_mempool *mbuf_pool;
> > + struct rte_mempool *op_mpool;
> > + struct rte_mempool *session_mpool;
> > + struct rte_cryptodev_config *config;
> > + uint8_t crypto_event_port_id;
> > +};
> > +
> > +struct rte_event response_info = {
> > + .queue_id = TEST_APP_EV_QUEUE_ID,
> > + .sched_type = RTE_SCHED_TYPE_ATOMIC,
> > + .flow_id = 0xAABB,
> > + .sub_event_type = 0,
> > + .event_type = 0,
> > + .priority = 0,
> > + .impl_opaque = 0
> > +};
>
> This may break old compiler.
> see http://dpdk.org/dev/patchwork/patch/39398/
This change was done as Akhil had suggested.
My compiler was not throwing any warning on this.
Anyway, I will revert the change now.
>
> > +
> > +struct rte_event_crypto_request request_info = {
> > + .cdev_id = TEST_CDEV_ID,
> > + .queue_pair_id = TEST_CDEV_QP_ID
> > +};
> > +
> > +static int
> > +test_crypto_adapter_stats(void)
> > +{
> > + struct rte_event_crypto_adapter_stats stats;
> > +
> > + rte_event_crypto_adapter_stats_get(TEST_ADAPTER_ID, &stats);
> > + printf(" +------------------------------------------------------+\n");
> > + printf(" + Crypto adapter stats for instance %u:\n", TEST_ADAPTER_ID);
> > + printf(" + Event port poll count %lu\n",
> > + stats.event_poll_count);
> > + printf(" + Event dequeue count %lu\n",
> > + stats.event_deq_count);
> > + printf(" + Cryptodev enqueue count %lu\n",
> > + stats.crypto_enq_count);
> > + printf(" + Cryptodev enqueue failed count %lu\n",
> > + stats.crypto_enq_fail);
> > + printf(" + Cryptodev dequeue count %lu\n",
> > + stats.crypto_deq_count);
> > + printf(" + Event enqueue count %lu\n",
> > + stats.event_enq_count);
> > + printf(" + Event enqueue retry count %lu\n",
> > + stats.event_enq_retry_count);
> > + printf(" + Event enqueue fail count %lu\n",
>
> Use PRIx64 to fix issue with 32b build target.
Ok
>
> > + stats.event_enq_fail_count);
> > + printf("
> > ++------------------------------------------------------+\n");
> > +
> > + rte_event_crypto_adapter_stats_reset(TEST_ADAPTER_ID);
> > + return TEST_SUCCESS;
> > +}
> > +
> > +static int
> > +configure_eventdev(void)
> > +{
> > + const char *eventdev_name = "event_sw0";
> > + struct rte_event_queue_conf queue_conf;
> > + struct rte_event_dev_config devconf;
> > + struct rte_event_dev_info info;
> > + uint32_t queue_count;
> > + uint32_t port_count;
> > + int ret;
> > + uint8_t qid;
> > +
> > + evdev = rte_event_dev_get_dev_id(eventdev_name);
>
> No need to assume it is "event_sw0" driver. You can select 0th event device
> for
> testing. If none of the event device is available then you can create
> "event_sw0"
> with vdev.
Ok
>
> > + if (evdev < 0) {
> > + if (rte_vdev_init(eventdev_name, NULL) < 0) {
> > + RTE_LOG(DEBUG, USER1, "Error creating eventdev\n");
> > + return TEST_FAILED;
> > + }
> > + evdev = rte_event_dev_get_dev_id(eventdev_name);
> > + if (evdev < 0) {
> > + RTE_LOG(DEBUG, USER1, "Error finding eventdev!\n");
> > + return TEST_FAILED;
> > + }
> > + }
> > +
> > + ret = rte_event_dev_info_get(evdev, &info);
> > + TEST_ASSERT_SUCCESS(ret, "Failed to get event dev info\n");
> > +
> > +static struct unit_test_suite service_tests = {
>
> It is not "service_tests". Choose an appropriate name.
Ok
