From: Volodymyr Fialko <vfialko@marvell.com>
To: "Gujjar, Abhinandan S" <abhinandan.gujjar@intel.com>,
"dev@dpdk.org" <dev@dpdk.org>,
Jerin Jacob Kollanukkaran <jerinj@marvell.com>,
Pavan Nikhilesh Bhagavatula <pbhagavatula@marvell.com>,
Shijith Thotton <sthotton@marvell.com>,
Hemant Agrawal <hemant.agrawal@nxp.com>,
Sachin Saxena <sachin.saxena@oss.nxp.com>,
"Jayatheerthan, Jay" <jay.jayatheerthan@intel.com>
Cc: Akhil Goyal <gakhil@marvell.com>, Anoob Joseph <anoobj@marvell.com>
Subject: RE: [PATCH 1/3] eventdev: introduce event cryptodev vector type
Date: Tue, 27 Sep 2022 09:24:45 +0000 [thread overview]
Message-ID: <MW2PR18MB22824B84DC4E018FFD34604CA8559@MW2PR18MB2282.namprd18.prod.outlook.com> (raw)
In-Reply-To: <PH0PR11MB58321EBAF643F0A30844ECACE8559@PH0PR11MB5832.namprd11.prod.outlook.com>
> -----Original Message-----
> From: Gujjar, Abhinandan S <abhinandan.gujjar@intel.com>
> Sent: Tuesday, September 27, 2022 11:05 AM
> To: Volodymyr Fialko <vfialko@marvell.com>; dev@dpdk.org; Jerin Jacob Kollanukkaran
> <jerinj@marvell.com>; Pavan Nikhilesh Bhagavatula <pbhagavatula@marvell.com>; Shijith Thotton
> <sthotton@marvell.com>; Hemant Agrawal <hemant.agrawal@nxp.com>; Sachin Saxena
> <sachin.saxena@oss.nxp.com>; Jayatheerthan, Jay <jay.jayatheerthan@intel.com>
> Cc: Akhil Goyal <gakhil@marvell.com>; Anoob Joseph <anoobj@marvell.com>
> Subject: [EXT] RE: [PATCH 1/3] eventdev: introduce event cryptodev vector type
>
> External Email
>
> ----------------------------------------------------------------------
>
>
> > -----Original Message-----
> > From: Volodymyr Fialko <vfialko@marvell.com>
> > Sent: Monday, September 26, 2022 4:32 PM
> > To: Gujjar, Abhinandan S <abhinandan.gujjar@intel.com>; dev@dpdk.org;
> > Jerin Jacob Kollanukkaran <jerinj@marvell.com>; Pavan Nikhilesh
> > Bhagavatula <pbhagavatula@marvell.com>; Shijith Thotton
> > <sthotton@marvell.com>; Hemant Agrawal <hemant.agrawal@nxp.com>;
> > Sachin Saxena <sachin.saxena@oss.nxp.com>; Jayatheerthan, Jay
> > <jay.jayatheerthan@intel.com>
> > Cc: Akhil Goyal <gakhil@marvell.com>; Anoob Joseph
> > <anoobj@marvell.com>
> > Subject: RE: [PATCH 1/3] eventdev: introduce event cryptodev vector
> > type
> >
> >
> >
> > > -----Original Message-----
> > > From: Gujjar, Abhinandan S <abhinandan.gujjar@intel.com>
> > > Sent: Saturday, September 24, 2022 10:44 AM
> > > To: Volodymyr Fialko <vfialko@marvell.com>; dev@dpdk.org; Jerin
> > > Jacob Kollanukkaran <jerinj@marvell.com>; Pavan Nikhilesh
> > > Bhagavatula <pbhagavatula@marvell.com>; Shijith Thotton
> > > <sthotton@marvell.com>; Hemant Agrawal <hemant.agrawal@nxp.com>;
> > > Sachin Saxena <sachin.saxena@oss.nxp.com>; Jayatheerthan, Jay
> > > <jay.jayatheerthan@intel.com>
> > > Cc: Akhil Goyal <gakhil@marvell.com>; Anoob Joseph
> > > <anoobj@marvell.com>
> > > Subject: [EXT] RE: [PATCH 1/3] eventdev: introduce event cryptodev
> > > vector type
> > >
> > > External Email
> > >
> > > --------------------------------------------------------------------
> > > --
> > >
> > >
> > > > -----Original Message-----
> > > > From: Volodymyr Fialko <vfialko@marvell.com>
> > > > Sent: Thursday, August 4, 2022 3:29 PM
> > > > To: dev@dpdk.org; Jerin Jacob <jerinj@marvell.com>; Gujjar,
> > > > Abhinandan S <abhinandan.gujjar@intel.com>; Pavan Nikhilesh
> > > > <pbhagavatula@marvell.com>; Shijith Thotton
> > > > <sthotton@marvell.com>; Hemant Agrawal <hemant.agrawal@nxp.com>;
> > > > Sachin Saxena <sachin.saxena@oss.nxp.com>; Jayatheerthan, Jay
> > > > <jay.jayatheerthan@intel.com>
> > > > Cc: gakhil@marvell.com; anoobj@marvell.com; Volodymyr Fialko
> > > > <vfialko@marvell.com>
> > > > Subject: [PATCH 1/3] eventdev: introduce event cryptodev vector
> > > > type
> > > >
> > > > Introduce ability to aggregate crypto operations processed by
> > > > event crypto adapter into single event containing rte_event_vector
> > > > whose event type is RTE_EVENT_TYPE_CRYPTODEV_VECTOR.
> > > >
> > > > Application should set RTE_EVENT_CRYPTO_ADAPTER_EVENT_VECTOR in
> > > > rte_event_crypto_adapter_queue_conf::flag and provide vector
> > > > configuration with respect of
> > > > rte_event_crypto_adapter_vector_limits,
> > > > which could be obtained by calling
> > > > rte_event_crypto_adapter_vector_limits_get, to enable vectorization.
> > > >
> > > > The event crypto adapter would be responsible for vectorizing the
> > > > crypto operations based on provided response information in
> > > > rte_event_crypto_metadata::response_info.
> > > >
> > > > Updated drivers and tests accordingly to new API.
> > > >
> > > > Signed-off-by: Volodymyr Fialko <vfialko@marvell.com>
> > > > ---
> > > > app/test-eventdev/test_perf_common.c | 10 +-
> > > > app/test/test_event_crypto_adapter.c | 12 ++-
> > > > .../prog_guide/event_crypto_adapter.rst | 23 +++-
> > > > drivers/event/cnxk/cn10k_eventdev.c | 4 +-
> > > > drivers/event/cnxk/cn9k_eventdev.c | 5 +-
> > > > drivers/event/dpaa/dpaa_eventdev.c | 9 +-
> > > > drivers/event/dpaa2/dpaa2_eventdev.c | 9 +-
> > > > drivers/event/octeontx/ssovf_evdev.c | 4 +-
> > > > lib/eventdev/eventdev_pmd.h | 35 +++++-
> > > > lib/eventdev/eventdev_trace.h | 6 +-
> > > > lib/eventdev/rte_event_crypto_adapter.c | 90 ++++++++++++++--
> > > > lib/eventdev/rte_event_crypto_adapter.h | 101 +++++++++++++++++-
> > > > lib/eventdev/rte_event_eth_rx_adapter.h | 3 +-
> > > > lib/eventdev/rte_eventdev.h | 8 ++
> > > > 14 files changed, 276 insertions(+), 43 deletions(-)
> > > >
> > >
> > > I don't see dataplane implementation of vectorization in the crypto adapter!
> > > Is it missed out in the patch?
> > > comments inline.
> > >
> > Right now we are targeting crypto_cn10k PMD and ipsec-secgw event mode
> > to support vectorization.
> Is there a way to test this? When can be dataplane changes expected?
>
If the spec looks okay, support in s/w crypto adapter and other h/w PMDs can be added by respective maintainers. Currently, we are adding library change, support for one PMD and an application to test the feature. Feature is exposed with capability flag to not break existing functionality.
> >
> > > > diff --git a/app/test-eventdev/test_perf_common.c b/app/test-
> > > > eventdev/test_perf_common.c index 81420be73a..c770bc93f6 100644
> > > > --- a/app/test-eventdev/test_perf_common.c
> > > > +++ b/app/test-eventdev/test_perf_common.c
> > > > @@ -837,14 +837,14 @@ perf_event_crypto_adapter_setup(struct
> > > > test_perf *t, struct prod_data *p)
> > > > }
> > > >
> > > > if (cap &
> > > > RTE_EVENT_CRYPTO_ADAPTER_CAP_INTERNAL_PORT_QP_EV_BIND) {
> > > > - struct rte_event response_info;
> > > > + struct rte_event_crypto_adapter_queue_conf conf;
> > > >
> > > > - response_info.event = 0;
> > > > - response_info.sched_type = RTE_SCHED_TYPE_ATOMIC;
> > > > - response_info.queue_id = p->queue_id;
> > > > + memset(&conf, 0, sizeof(conf));
> > > > + conf.ev.sched_type = RTE_SCHED_TYPE_ATOMIC;
> > > > + conf.ev.queue_id = p->queue_id;
> > > > ret = rte_event_crypto_adapter_queue_pair_add(
> > > > TEST_PERF_CA_ID, p->ca.cdev_id, p->ca.cdev_qp_id,
> > > > - &response_info);
> > > > + &conf);
> > > > } else {
> > > > ret = rte_event_crypto_adapter_queue_pair_add(
> > > > TEST_PERF_CA_ID, p->ca.cdev_id, p->ca.cdev_qp_id,
> > NULL); diff
> > > > --git a/app/test/test_event_crypto_adapter.c
> > > > b/app/test/test_event_crypto_adapter.c
> > > > index 2ecc7e2cea..bb617c1042 100644
> > > > --- a/app/test/test_event_crypto_adapter.c
> > > > +++ b/app/test/test_event_crypto_adapter.c
> > > > @@ -1175,6 +1175,10 @@ test_crypto_adapter_create(void) static
> > > > int
> > > > test_crypto_adapter_qp_add_del(void)
> > > > {
> > > > + struct rte_event_crypto_adapter_queue_conf queue_conf = {
> > > > + .ev = response_info,
> > > > + };
> > > > +
> > > > uint32_t cap;
> > > > int ret;
> > > >
> > > > @@ -1183,7 +1187,7 @@ test_crypto_adapter_qp_add_del(void)
> > > >
> > > > if (cap &
> > > > RTE_EVENT_CRYPTO_ADAPTER_CAP_INTERNAL_PORT_QP_EV_BIND) {
> > > > ret =
> > > > rte_event_crypto_adapter_queue_pair_add(TEST_ADAPTER_ID,
> > > > - TEST_CDEV_ID, TEST_CDEV_QP_ID,
> > > > &response_info);
> > > > + TEST_CDEV_ID, TEST_CDEV_QP_ID,
> > > > &queue_conf);
> > > > } else
> > > > ret =
> > > > rte_event_crypto_adapter_queue_pair_add(TEST_ADAPTER_ID,
> > > > TEST_CDEV_ID, TEST_CDEV_QP_ID,
> > NULL); @@ -1206,6 +1210,10 @@
> > > > configure_event_crypto_adapter(enum
> > > > rte_event_crypto_adapter_mode mode)
> > > > .new_event_threshold = 1200,
> > > > };
> > > >
> > > > + struct rte_event_crypto_adapter_queue_conf queue_conf = {
> > > > + .ev = response_info,
> > > > + };
> > > > +
> > > > uint32_t cap;
> > > > int ret;
> > > >
> > > > @@ -1238,7 +1246,7 @@ configure_event_crypto_adapter(enum
> > > > rte_event_crypto_adapter_mode mode)
> > > >
> > > > if (cap &
> > > > RTE_EVENT_CRYPTO_ADAPTER_CAP_INTERNAL_PORT_QP_EV_BIND) {
> > > > ret =
> > > > rte_event_crypto_adapter_queue_pair_add(TEST_ADAPTER_ID,
> > > > - TEST_CDEV_ID, TEST_CDEV_QP_ID,
> > > > &response_info);
> > > > + TEST_CDEV_ID, TEST_CDEV_QP_ID,
> > > > &queue_conf);
> > > > } else
> > > > ret =
> > > > rte_event_crypto_adapter_queue_pair_add(TEST_ADAPTER_ID,
> > > > TEST_CDEV_ID, TEST_CDEV_QP_ID, NULL); diff
> > --git
> > > > a/doc/guides/prog_guide/event_crypto_adapter.rst
> > > > b/doc/guides/prog_guide/event_crypto_adapter.rst
> > > > index 4fb5c688e0..554df7e358 100644
> > > > --- a/doc/guides/prog_guide/event_crypto_adapter.rst
> > > > +++ b/doc/guides/prog_guide/event_crypto_adapter.rst
> > > > @@ -201,10 +201,10 @@ capability, event information must be passed
> > > > to the add API.
> > > >
> > > > ret = rte_event_crypto_adapter_caps_get(id, evdev, &cap);
> > > > if (cap &
> > > > RTE_EVENT_CRYPTO_ADAPTER_CAP_INTERNAL_PORT_QP_EV_BIND) {
> > > > - struct rte_event event;
> > > > + struct rte_event_crypto_adapter_queue_conf conf;
> > > >
> > > > - // Fill in event information & pass it to add API
> > > > - rte_event_crypto_adapter_queue_pair_add(id, cdev_id, qp_id,
> > > > &event);
> > > > + // Fill in conf.event information & pass it to add API
> > > > + rte_event_crypto_adapter_queue_pair_add(id,
> > > > + cdev_id, qp_id, &conf);
> > > > } else
> > > > rte_event_crypto_adapter_queue_pair_add(id,
> > > > cdev_id, qp_id, NULL);
> > > >
> > > > @@ -291,6 +291,23 @@ the ``rte_crypto_op``.
> > > > rte_memcpy(op + len, &m_data, sizeof(m_data));
> > > > }
> > > >
> > > > +Enable event vectorization
> > > > +~~~~~~~~~~~~~~~~~~~~~~~~~~
> > > > +
> > > > +The event crypto adapter can aggregate outcoming crypto
> > > > +operations based on provided response information of
> > > > +``rte_event_crypto_metadata::response_info``
> > > > +and generate a ``rte_event`` containing ``rte_event_vector``
> > > > +whose event type is ``RTE_EVENT_TYPE_CRYPTODEV_VECTOR``.
> > > > +To enable vectorization application should set
> > > > +RTE_EVENT_CRYPTO_ADAPTER_EVENT_VECTOR in
> > > > +``rte_event_crypto_adapter_queue_conf::flag`` and provide vector
> > > > +configuration(size, mempool, etc.) with respect of
> > > > +``rte_event_crypto_adapter_vector_limits``, which could be
> > > > +obtained by calling ``rte_event_crypto_adapter_vector_limits_get()``.
> > > > +
> > > > +The RTE_EVENT_CRYPTO_ADAPTER_CAP_EVENT_VECTOR capability
> > indicates
> > > > +whether PMD supports this feature.
> > > > +
> > > > Start the adapter instance
> > > > ~~~~~~~~~~~~~~~~~~~~~~~~~~
> > > >
> > > > diff --git a/drivers/event/cnxk/cn10k_eventdev.c
> > > > b/drivers/event/cnxk/cn10k_eventdev.c
> > > > index 5a0cab40a9..e74ec57382 100644
> > > > --- a/drivers/event/cnxk/cn10k_eventdev.c
> > > > +++ b/drivers/event/cnxk/cn10k_eventdev.c
> > > > @@ -889,11 +889,11 @@ static int
> > > > cn10k_crypto_adapter_qp_add(const struct rte_eventdev *event_dev,
> > > > const struct rte_cryptodev *cdev,
> > > > int32_t queue_pair_id,
> > > > - const struct rte_event *event)
> > > > + const struct rte_event_crypto_adapter_queue_conf
> > > > *conf)
> > > > {
> > > > struct cnxk_sso_evdev *dev = cnxk_sso_pmd_priv(event_dev);
> > > >
> > > > - RTE_SET_USED(event);
> > > > + RTE_SET_USED(conf);
> > > >
> > > > CNXK_VALID_DEV_OR_ERR_RET(event_dev->dev, "event_cn10k");
> > > > CNXK_VALID_DEV_OR_ERR_RET(cdev->device, "crypto_cn10k"); diff --
> > > > git a/drivers/event/cnxk/cn9k_eventdev.c
> > > > b/drivers/event/cnxk/cn9k_eventdev.c
> > > > index 2e27030049..45ed547cb0 100644
> > > > --- a/drivers/event/cnxk/cn9k_eventdev.c
> > > > +++ b/drivers/event/cnxk/cn9k_eventdev.c
> > > > @@ -1120,11 +1120,12 @@ cn9k_crypto_adapter_caps_get(const struct
> > > > rte_eventdev *event_dev, static int
> > > > cn9k_crypto_adapter_qp_add(const struct rte_eventdev *event_dev,
> > > > const struct rte_cryptodev *cdev,
> > > > - int32_t queue_pair_id, const struct rte_event *event)
> > > > + int32_t queue_pair_id,
> > > > + const struct rte_event_crypto_adapter_queue_conf
> > > > *conf)
> > > > {
> > > > struct cnxk_sso_evdev *dev = cnxk_sso_pmd_priv(event_dev);
> > > >
> > > > - RTE_SET_USED(event);
> > > > + RTE_SET_USED(conf);
> > > >
> > > > CNXK_VALID_DEV_OR_ERR_RET(event_dev->dev, "event_cn9k");
> > > > CNXK_VALID_DEV_OR_ERR_RET(cdev->device, "crypto_cn9k"); diff
> > > > --git a/drivers/event/dpaa/dpaa_eventdev.c
> > > > b/drivers/event/dpaa/dpaa_eventdev.c
> > > > index ff6cc0be18..2b9ecd9fbf 100644
> > > > --- a/drivers/event/dpaa/dpaa_eventdev.c
> > > > +++ b/drivers/event/dpaa/dpaa_eventdev.c
> > > > @@ -26,6 +26,7 @@
> > > > #include <rte_eventdev.h>
> > > > #include <eventdev_pmd_vdev.h>
> > > > #include <rte_ethdev.h>
> > > > +#include <rte_event_crypto_adapter.h>
> > > > #include <rte_event_eth_rx_adapter.h> #include
> > > > <rte_event_eth_tx_adapter.h> #include <cryptodev_pmd.h> @@
> > > > -775,10
> > > > +776,10 @@ static int dpaa_eventdev_crypto_queue_add(const struct
> > > > rte_eventdev *dev,
> > > > const struct rte_cryptodev *cryptodev,
> > > > int32_t rx_queue_id,
> > > > - const struct rte_event *ev)
> > > > + const struct rte_event_crypto_adapter_queue_conf *conf)
> > > > {
> > > > struct dpaa_eventdev *priv = dev->data->dev_private;
> > > > - uint8_t ev_qid = ev->queue_id;
> > > > + uint8_t ev_qid = conf->ev.queue_id;
> > > > u16 ch_id = priv->evq_info[ev_qid].ch_id;
> > > > int ret;
> > > >
> > > > @@ -786,10 +787,10 @@ dpaa_eventdev_crypto_queue_add(const struct
> > > > rte_eventdev *dev,
> > > >
> > > > if (rx_queue_id == -1)
> > > > return dpaa_eventdev_crypto_queue_add_all(dev,
> > > > - cryptodev, ev);
> > > > + cryptodev, &conf->ev);
> > > >
> > > > ret = dpaa_sec_eventq_attach(cryptodev, rx_queue_id,
> > > > - ch_id, ev);
> > > > + ch_id, &conf->ev);
> > > > if (ret) {
> > > > DPAA_EVENTDEV_ERR(
> > > > "dpaa_sec_eventq_attach failed: ret: %d\n", ret); diff -
> > -git
> > > > a/drivers/event/dpaa2/dpaa2_eventdev.c
> > > > b/drivers/event/dpaa2/dpaa2_eventdev.c
> > > > index ffc7b8b073..0137736794 100644
> > > > --- a/drivers/event/dpaa2/dpaa2_eventdev.c
> > > > +++ b/drivers/event/dpaa2/dpaa2_eventdev.c
> > > > @@ -26,6 +26,7 @@
> > > > #include <rte_bus_vdev.h>
> > > > #include <ethdev_driver.h>
> > > > #include <cryptodev_pmd.h>
> > > > +#include <rte_event_crypto_adapter.h>
> > > > #include <rte_event_eth_rx_adapter.h> #include
> > > > <rte_event_eth_tx_adapter.h>
> > > >
> > > > @@ -865,10 +866,10 @@ static int
> > > > dpaa2_eventdev_crypto_queue_add(const struct rte_eventdev *dev,
> > > > const struct rte_cryptodev *cryptodev,
> > > > int32_t rx_queue_id,
> > > > - const struct rte_event *ev)
> > > > + const struct rte_event_crypto_adapter_queue_conf *conf)
> > > > {
> > > > struct dpaa2_eventdev *priv = dev->data->dev_private;
> > > > - uint8_t ev_qid = ev->queue_id;
> > > > + uint8_t ev_qid = conf->ev.queue_id;
> > > > struct dpaa2_dpcon_dev *dpcon = priv->evq_info[ev_qid].dpcon;
> > > > int ret;
> > > >
> > > > @@ -876,10 +877,10 @@ dpaa2_eventdev_crypto_queue_add(const struct
> > > > rte_eventdev *dev,
> > > >
> > > > if (rx_queue_id == -1)
> > > > return dpaa2_eventdev_crypto_queue_add_all(dev,
> > > > - cryptodev, ev);
> > > > + cryptodev, &conf->ev);
> > > >
> > > > ret = dpaa2_sec_eventq_attach(cryptodev, rx_queue_id,
> > > > - dpcon, ev);
> > > > + dpcon, &conf->ev);
> > > > if (ret) {
> > > > DPAA2_EVENTDEV_ERR(
> > > > "dpaa2_sec_eventq_attach failed: ret: %d\n", ret); diff
> > --git
> > > > a/drivers/event/octeontx/ssovf_evdev.c
> > > > b/drivers/event/octeontx/ssovf_evdev.c
> > > > index 9e14e35d10..17acd8ef64 100644
> > > > --- a/drivers/event/octeontx/ssovf_evdev.c
> > > > +++ b/drivers/event/octeontx/ssovf_evdev.c
> > > > @@ -745,12 +745,12 @@ static int
> > > > ssovf_crypto_adapter_qp_add(const struct rte_eventdev *dev,
> > > > const struct rte_cryptodev *cdev,
> > > > int32_t queue_pair_id,
> > > > - const struct rte_event *event)
> > > > + const struct rte_event_crypto_adapter_queue_conf
> > > > *conf)
> > > > {
> > > > struct cpt_instance *qp;
> > > > uint8_t qp_id;
> > > >
> > > > - RTE_SET_USED(event);
> > > > + RTE_SET_USED(conf);
> > > >
> > > > if (queue_pair_id == -1) {
> > > > for (qp_id = 0; qp_id < cdev->data->nb_queue_pairs; qp_id++) {
> > > > diff --git a/lib/eventdev/eventdev_pmd.h
> > > > b/lib/eventdev/eventdev_pmd.h index 69402668d8..bcfc9cbcb2 100644
> > > > --- a/lib/eventdev/eventdev_pmd.h
> > > > +++ b/lib/eventdev/eventdev_pmd.h
> > > > @@ -907,6 +907,7 @@ rte_event_pmd_selftest_seqn(struct rte_mbuf
> > > > *mbuf) }
> > > >
> > > > struct rte_cryptodev;
> > > > +struct rte_event_crypto_adapter_queue_conf;
> > > >
> > > > /**
> > > > * This API may change without prior notice @@ -961,11 +962,11 @@
> > > > typedef int (*eventdev_crypto_adapter_caps_get_t)
> > > > * - <0: Error code returned by the driver function.
> > > > *
> > > > */
> > > > -typedef int (*eventdev_crypto_adapter_queue_pair_add_t)
> > > > - (const struct rte_eventdev *dev,
> > > > - const struct rte_cryptodev *cdev,
> > > > - int32_t queue_pair_id,
> > > > - const struct rte_event *event);
> > > > +typedef int (*eventdev_crypto_adapter_queue_pair_add_t)(
> > > > + const struct rte_eventdev *dev,
> > > > + const struct rte_cryptodev *cdev,
> > > > + int32_t queue_pair_id,
> > > > + const struct rte_event_crypto_adapter_queue_conf
> > > > *queue_conf);
> > > >
> > > >
> > > > /**
> > > > @@ -1074,6 +1075,27 @@ typedef int
> > > > (*eventdev_crypto_adapter_stats_reset)
> > > > (const struct rte_eventdev *dev,
> > > > const struct rte_cryptodev *cdev);
> > > >
> > > > +struct rte_event_crypto_adapter_vector_limits;
> > > > +/**
> > > > + * Get event vector limits for a given event, crypto device pair.
> > > > + *
> > > > + * @param dev
> > > > + * Event device pointer
> > > > + *
> > > > + * @param cdev
> > > > + * Crypto device pointer
> > > > + *
> > > > + * @param[out] limits
> > > > + * Pointer to the limits structure to be filled.
> > > > + *
> > > > + * @return
> > > > + * - 0: Success.
> > > > + * - <0: Error code returned by the driver function.
> > > > + */
> > > > +typedef int (*eventdev_crypto_adapter_vector_limits_get_t)(
> > > > + const struct rte_eventdev *dev, const struct rte_cryptodev *cdev,
> > > > + struct rte_event_crypto_adapter_vector_limits *limits);
> > > > +
> > > > /**
> > > > * Retrieve the event device's eth Tx adapter capabilities.
> > > > *
> > > > @@ -1339,6 +1361,9 @@ struct eventdev_ops {
> > > > /**< Get crypto stats */
> > > > eventdev_crypto_adapter_stats_reset crypto_adapter_stats_reset;
> > > > /**< Reset crypto stats */
> > > > + eventdev_crypto_adapter_vector_limits_get_t
> > > > + crypto_adapter_vector_limits_get;
> > > > + /**< Get event vector limits for the crypto adapter */
> > > >
> > > > eventdev_eth_rx_adapter_q_stats_get
> > > > eth_rx_adapter_queue_stats_get;
> > > > /**< Get ethernet Rx queue stats */ diff --git
> > > > a/lib/eventdev/eventdev_trace.h b/lib/eventdev/eventdev_trace.h
> > > > index 5ec43d80ee..d48cd58850 100644
> > > > --- a/lib/eventdev/eventdev_trace.h
> > > > +++ b/lib/eventdev/eventdev_trace.h
> > > > @@ -18,6 +18,7 @@ extern "C" {
> > > > #include <rte_trace_point.h>
> > > >
> > > > #include "rte_eventdev.h"
> > > > +#include "rte_event_crypto_adapter.h"
> > > > #include "rte_event_eth_rx_adapter.h"
> > > > #include "rte_event_timer_adapter.h"
> > > >
> > > > @@ -271,11 +272,12 @@ RTE_TRACE_POINT( RTE_TRACE_POINT(
> > > > rte_eventdev_trace_crypto_adapter_queue_pair_add,
> > > > RTE_TRACE_POINT_ARGS(uint8_t adptr_id, uint8_t cdev_id,
> > > > - const void *event, int32_t queue_pair_id),
> > > > + int32_t queue_pair_id,
> > > > + const struct rte_event_crypto_adapter_queue_conf *conf),
> > > > rte_trace_point_emit_u8(adptr_id);
> > > > rte_trace_point_emit_u8(cdev_id);
> > > > rte_trace_point_emit_i32(queue_pair_id);
> > > > - rte_trace_point_emit_ptr(event);
> > > > + rte_trace_point_emit_ptr(conf);
> > > > )
> > > >
> > > > RTE_TRACE_POINT(
> > > > diff --git a/lib/eventdev/rte_event_crypto_adapter.c
> > > > b/lib/eventdev/rte_event_crypto_adapter.c
> > > > index 7c695176f4..73a4f231e2 100644
> > > > --- a/lib/eventdev/rte_event_crypto_adapter.c
> > > > +++ b/lib/eventdev/rte_event_crypto_adapter.c
> > > > @@ -921,11 +921,12 @@ int
> > > > rte_event_crypto_adapter_queue_pair_add(uint8_t id,
> > > > uint8_t cdev_id,
> > > > int32_t queue_pair_id,
> > > > - const struct rte_event *event)
> > > > + const struct rte_event_crypto_adapter_queue_conf
> > > > *conf)
> > > > {
> > > > + struct rte_event_crypto_adapter_vector_limits limits;
> > > > struct event_crypto_adapter *adapter;
> > > > - struct rte_eventdev *dev;
> > > > struct crypto_device_info *dev_info;
> > > > + struct rte_eventdev *dev;
> > > > uint32_t cap;
> > > > int ret;
> > > >
> > > > @@ -951,11 +952,47 @@
> > > > rte_event_crypto_adapter_queue_pair_add(uint8_t
> > > > id,
> > > > }
> > > >
> > > > if ((cap &
> > > > RTE_EVENT_CRYPTO_ADAPTER_CAP_INTERNAL_PORT_QP_EV_BIND) &&
> > > > - (event == NULL)) {
> > > > + (conf == NULL)) {
> > > > RTE_EDEV_LOG_ERR("Conf value can not be NULL for
> > dev_id=%u",
> > > > cdev_id);
> > > > return -EINVAL;
> > > > }
> > > Newline?
> > >
> > Ack
> >
> > > > + if ((conf != NULL) &&
> > > Checking conf twice?
> > Will rewrite as if conf == NULL/else, to avoid double checking.
> >
> > > > + (conf->flags & RTE_EVENT_CRYPTO_ADAPTER_EVENT_VECTOR)) {
> > > Else condition if the flags is not set?
> > There's no additional handing for case when flag is no set.
> >
> > > > + if ((cap &
> > > > RTE_EVENT_CRYPTO_ADAPTER_CAP_EVENT_VECTOR) == 0) {
> > > > + RTE_EDEV_LOG_ERR("Event vectorization is not
> > > > supported,"
> > > > + "dev %" PRIu8 " cdev %" PRIu8, id,
> > > > + cdev_id);
> > > > + return -ENOTSUP;
> > > > + }
> > > > +
> > > > + ret = rte_event_crypto_adapter_vector_limits_get(
> > > > + adapter->eventdev_id, cdev_id, &limits);
> > > > + if (ret < 0) {
> > > > + RTE_EDEV_LOG_ERR("Failed to get event device vector
> > > > "
> > > > + "limits, dev %" PRIu8 " cdev %" PRIu8,
> > > > + id, cdev_id);
> > > > + return -EINVAL;
> > > > + }
> > > New line? Please check other cases.
> > >
> > Ack
> >
> > > > + if (conf->vector_sz < limits.min_sz ||
> > > > + conf->vector_sz > limits.max_sz ||
> > > > + conf->vector_timeout_ns < limits.min_timeout_ns ||
> > > > + conf->vector_timeout_ns > limits.max_timeout_ns ||
> > > > + conf->vector_mp == NULL) {
> > > > + RTE_EDEV_LOG_ERR("Invalid event vector
> > > > configuration,"
> > > > + " dev %" PRIu8 " cdev %" PRIu8,
> > > > + id, cdev_id);
> > > > + return -EINVAL;
> > > > + }
> > > > + if (conf->vector_mp->elt_size <
> > > > + (sizeof(struct rte_event_vector) +
> > > > + (sizeof(uintptr_t) * conf->vector_sz))) {
> > > > + RTE_EDEV_LOG_ERR("Invalid event vector
> > > > configuration,"
> > > > + " dev %" PRIu8 " cdev %" PRIu8,
> > > > + id, cdev_id);
> > > > + return -EINVAL;
> > > > + }
> > > > + }
> > > >
> > > > dev_info = &adapter->cdevs[cdev_id];
> > > >
> > > > @@ -990,7 +1027,7 @@
> > rte_event_crypto_adapter_queue_pair_add(uint8_t
> > > > id,
> > > > ret = (*dev->dev_ops->crypto_adapter_queue_pair_add)(dev,
> > > > dev_info->dev,
> > > > queue_pair_id,
> > > > - event);
> > > > + conf);
> > > > if (ret)
> > > > return ret;
> > > >
> > > > @@ -1030,8 +1067,8 @@
> > > > rte_event_crypto_adapter_queue_pair_add(uint8_t
> > > > id,
> > > > rte_service_component_runstate_set(adapter->service_id, 1);
> > > > }
> > > >
> > > > - rte_eventdev_trace_crypto_adapter_queue_pair_add(id, cdev_id,
> > > > event,
> > > > - queue_pair_id);
> > > > + rte_eventdev_trace_crypto_adapter_queue_pair_add(id, cdev_id,
> > > > + queue_pair_id, conf);
> > > > return 0;
> > > > }
> > > >
> > > > @@ -1290,3 +1327,44 @@
> > > > rte_event_crypto_adapter_event_port_get(uint8_t
> > > > id, uint8_t *event_port_id)
> > > >
> > > > return 0;
> > > > }
> > > > +
> > > > +int
> > > > +rte_event_crypto_adapter_vector_limits_get(
> > > > + uint8_t dev_id, uint16_t cdev_id,
> > > > + struct rte_event_crypto_adapter_vector_limits *limits) {
> > > > + struct rte_cryptodev *cdev;
> > > > + struct rte_eventdev *dev;
> > > > + uint32_t cap;
> > > > + int ret;
> > > > +
> > > > + RTE_EVENTDEV_VALID_DEVID_OR_ERR_RET(dev_id, -EINVAL);
> > > > +
> > > > + if (!rte_cryptodev_is_valid_dev(cdev_id)) {
> > > > + RTE_EDEV_LOG_ERR("Invalid dev_id=%" PRIu8, cdev_id);
> > > > + return -EINVAL;
> > > > + }
> > > > +
> > > > + if (limits == NULL)
> > > > + return -EINVAL;
> > > Add appropriate error message like above?
> > Ack, will add.
> >
> > > > +
> > > > + dev = &rte_eventdevs[dev_id];
> > > > + cdev = rte_cryptodev_pmd_get_dev(cdev_id);
> > > > +
> > > > + ret = rte_event_crypto_adapter_caps_get(dev_id, cdev_id, &cap);
> > > > + if (ret) {
> > > > + RTE_EDEV_LOG_ERR("Failed to get adapter caps edev %" PRIu8
> > > > + "cdev %" PRIu16, dev_id, cdev_id);
> > > > + return ret;
> > > > + }
> > > > +
> > > > + if (!(cap & RTE_EVENT_CRYPTO_ADAPTER_CAP_EVENT_VECTOR))
> > > > + return -ENOTSUP;
> > > Same here.
> > Ack, will add.
> >
> > > > +
> > > > + RTE_FUNC_PTR_OR_ERR_RET(
> > > > + *dev->dev_ops->crypto_adapter_vector_limits_get,
> > > > + -ENOTSUP);
> > > > +
> > > > + return dev->dev_ops->crypto_adapter_vector_limits_get(
> > > > + dev, cdev, limits);
> > > > +}
> > > > diff --git a/lib/eventdev/rte_event_crypto_adapter.h
> > > > b/lib/eventdev/rte_event_crypto_adapter.h
> > > > index d90a19e72c..7dd6171b9b 100644
> > > > --- a/lib/eventdev/rte_event_crypto_adapter.h
> > > > +++ b/lib/eventdev/rte_event_crypto_adapter.h
> > > > @@ -253,6 +253,78 @@ struct rte_event_crypto_adapter_conf {
> > > > */
> > > > };
> > > >
> > > > +#define RTE_EVENT_CRYPTO_ADAPTER_EVENT_VECTOR 0x1
> > > > +/**< This flag indicates that crypto operations processed on the
> > > > +crypto
> > > > + * adapter need to be vectorized
> > > > + * @see rte_event_crypto_adapter_queue_conf::flags
> > > > + */
> > > > +
> > > > +/**
> > > > + * Adapter queue configuration structure */ struct
> > > > +rte_event_crypto_adapter_queue_conf {
> > > > + uint32_t flags;
> > > > + /**< Flags for handling crypto operations
> > > > + * @see RTE_EVENT_CRYPTO_ADAPTER_EVENT_VECTOR
> > > > + */
> > > > + struct rte_event ev;
> > > > + /**< If HW supports cryptodev queue pair to event queue binding,
> > > > + * application is expected to fill in event information.
> > > > + * @see
> > > > RTE_EVENT_CRYPTO_ADAPTER_CAP_INTERNAL_PORT_QP_EV_BIND
> > > > + */
> > > > + uint16_t vector_sz;
> > > > + /**< Indicates the maximum number for crypto operations to
> > > > +combine
> > > > and
> > > > + * form a vector.
> > > > + * @see rte_event_crypto_adapter_vector_limits::min_sz
> > > > + * @see rte_event_crypto_adapter_vector_limits::max_sz
> > > > + * Valid when RTE_EVENT_CRYPTO_ADAPTER_EVENT_VECTOR flag is
> > > > set in
> > > > + * @see rte_event_crypto_adapter_queue_conf::rx_queue_flags
> > > rx_queue_flags??
> > Typo, should be conf::flags.
> >
> > > > + */
> > > > + uint64_t vector_timeout_ns;
> > > > + /**<
> > > > + * Indicates the maximum number of nanoseconds to wait for
> > > > aggregating
> > > > + * crypto operations. Should be within vectorization limits of the
> > > > + * adapter
> > > > + * @see rte_event_crypto_adapter_vector_limits::min_timeout_ns
> > > > + * @see rte_event_crypto_adapter_vector_limits::max_timeout_ns
> > > > + * Valid when RTE_EVENT_CRYPTO_ADAPTER_EVENT_VECTOR flag is
> > > > set in
> > > > + * @see rte_event_crypto_adapter_queue_conf::flags
> > > > + */
> > > > + struct rte_mempool *vector_mp;
> > > > + /**< Indicates the mempool that should be used for allocating
> > > > + * rte_event_vector container.
> > > > + * Should be created by using `rte_event_vector_pool_create`.
> > > > + * Valid when RTE_EVENT_CRYPTO_ADAPTER_EVENT_VECTOR flag is
> > > > set in
> > > > + * @see rte_event_crypto_adapter_queue_conf::flags.
> > > > + */
> > > > +};
> > > > +
> > > > +/**
> > > > + * A structure used to retrieve event crypto adapter vector limits.
> > > > + */
> > > > +struct rte_event_crypto_adapter_vector_limits {
> > > > + uint16_t min_sz;
> > > > + /**< Minimum vector limit configurable.
> > > > + * @see rte_event_crypto_adapter_queue_conf::vector_sz
> > > > + */
> > > > + uint16_t max_sz;
> > > > + /**< Maximum vector limit configurable.
> > > > + * @see rte_event_crypto_adapter_queue_conf::vector_sz
> > > > + */
> > > > + uint8_t log2_sz;
> > > > + /**< True if the size configured should be in log2.
> > > > + * @see rte_event_crypto_adapter_queue_conf::vector_sz
> > > > + */
> > > > + uint64_t min_timeout_ns;
> > > > + /**< Minimum vector timeout configurable.
> > > > + * @see rte_event_crypto_adapter_queue_conf::vector_timeout_ns
> > > > + */
> > > > + uint64_t max_timeout_ns;
> > > > + /**< Maximum vector timeout configurable.
> > > > + * @see rte_event_crypto_adapter_queue_conf::vector_timeout_ns
> > > > + */
> > > > +};
> > > > +
> > > > /**
> > > > * Function type used for adapter configuration callback. The callback is
> > > > * used to fill in members of the struct
> > > > rte_event_crypto_adapter_conf, this @@ -392,10 +464,9 @@
> > > rte_event_crypto_adapter_free(uint8_t id);
> > > > * Cryptodev queue pair identifier. If queue_pair_id is set -1,
> > > > * adapter adds all the pre configured queue pairs to the instance.
> > > > *
> > > > - * @param event
> > > > - * if HW supports cryptodev queue pair to event queue binding,
> > > > application is
> > > > - * expected to fill in event information, else it will be NULL.
> > > > - * @see
> > RTE_EVENT_CRYPTO_ADAPTER_CAP_INTERNAL_PORT_QP_EV_BIND
> > > > + * @param conf
> > > > + * Additional configuration structure of type
> > > > + * *rte_event_crypto_adapter_queue_conf*
> > > > *
> > > > * @return
> > > > * - 0: Success, queue pair added correctly.
> > > > @@ -405,7 +476,7 @@ int
> > > > rte_event_crypto_adapter_queue_pair_add(uint8_t id,
> > > > uint8_t cdev_id,
> > > > int32_t queue_pair_id,
> > > > - const struct rte_event *event);
> > > > + const struct rte_event_crypto_adapter_queue_conf
> > > > *conf);
> > > >
> > > > /**
> > > > * Delete a queue pair from an event crypto adapter.
> > > > @@ -523,6 +594,26 @@
> > > > rte_event_crypto_adapter_service_id_get(uint8_t
> > > > id, uint32_t *service_id); int
> > > > rte_event_crypto_adapter_event_port_get(uint8_t
> > > > id, uint8_t *event_port_id);
> > > >
> > > > +/**
> > > > + * Retrieve vector limits for a given event dev and crypto dev pair.
> > > > + * @see rte_event_crypto_adapter_vector_limits
> > > > + *
> > > > + * @param dev_id
> > > > + * Event device identifier.
> > > > + * @param cdev_id
> > > > + * Crypto device identifier.
> > > > + * @param [out] limits
> > > > + * A pointer to rte_event_crypto_adapter_vector_limits structure
> > > > +that has to
> > > > + * be filled.
> > > Space missing before "be filled"
> > Ack
> >
> > > > + *
> > > > + * @return
> > > > + * - 0: Success.
> > > > + * - <0: Error code on failure.
> > > > + */
> > > > +int rte_event_crypto_adapter_vector_limits_get(
> > > > + uint8_t dev_id, uint16_t cdev_id,
> > > > + struct rte_event_crypto_adapter_vector_limits *limits);
> > > > +
> > > > /**
> > > > * Enqueue a burst of crypto operations as event objects supplied
> > > > in
> > > > *rte_event*
> > > > * structure on an event crypto adapter designated by its event
> > > > *dev_id* through diff --git
> > > > a/lib/eventdev/rte_event_eth_rx_adapter.h
> > > > b/lib/eventdev/rte_event_eth_rx_adapter.h
> > > > index 3608a7b2cf..c8f2936866 100644
> > > > --- a/lib/eventdev/rte_event_eth_rx_adapter.h
> > > > +++ b/lib/eventdev/rte_event_eth_rx_adapter.h
> > > > @@ -457,7 +457,8 @@ int rte_event_eth_rx_adapter_free(uint8_t id);
> > > > * @see RTE_EVENT_ETH_RX_ADAPTER_CAP_MULTI_EVENTQ
> > > > *
> > > > * @param conf
> > > > - * Additional configuration structure of type
> > > > *rte_event_eth_rx_adapter_conf*
> > > > + * Additional configuration structure of type
> > > > + * *rte_event_eth_rx_adapter_queue_conf*
> > > These changes are not relevant. Please consider sending separate patch.
> > >
> > Ack, Will send this change as a separate patch.
> >
> > > > *
> > > > * @return
> > > > * - 0: Success, Receive queue added correctly.
> > > > diff --git a/lib/eventdev/rte_eventdev.h
> > > > b/lib/eventdev/rte_eventdev.h index
> > > > 6a6f6ea4c1..1a737bf851 100644
> > > > --- a/lib/eventdev/rte_eventdev.h
> > > > +++ b/lib/eventdev/rte_eventdev.h
> > > > @@ -1203,6 +1203,9 @@ struct rte_event_vector { #define
> > > > RTE_EVENT_TYPE_ETH_RX_ADAPTER_VECTOR
> > \
> > > > (RTE_EVENT_TYPE_VECTOR | RTE_EVENT_TYPE_ETH_RX_ADAPTER)
> > /**< The
> > > > event vector generated from eth Rx adapter. */
> > > > +#define RTE_EVENT_TYPE_CRYPTODEV_VECTOR \
> > > > + (RTE_EVENT_TYPE_VECTOR | RTE_EVENT_TYPE_CRYPTODEV) /**< The
> > > > event
> > > > +vector generated from cryptodev adapter. */
> > > >
> > > > #define RTE_EVENT_TYPE_MAX 0x10
> > > > /**< Maximum number of event types */ @@ -1420,6 +1423,11 @@
> > > > rte_event_timer_adapter_caps_get(uint8_t dev_id, uint32_t *caps);
> > > > * the private data information along with the crypto session.
> > > > */
> > > >
> > > > +#define RTE_EVENT_CRYPTO_ADAPTER_CAP_EVENT_VECTOR 0x10
> > > > +/**< Flag indicates HW is capable of aggregating processed
> > > > + * crypto operations into rte_event_vector.
> > > > + */
> > > > +
> > > > /**
> > > > * Retrieve the event device's crypto adapter capabilities for the
> > > > * specified cryptodev device
> > > > --
> > > > 2.25.1
next prev parent reply other threads:[~2022-09-27 9:24 UTC|newest]
Thread overview: 36+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-06-22 1:38 [PATCH v2] app/test: add event inline security tests Volodymyr Fialko
2022-06-22 11:33 ` [PATCH v3] " Volodymyr Fialko
2022-06-22 16:32 ` Anoob Joseph
2022-06-28 8:29 ` Akhil Goyal
2022-06-28 12:09 ` [PATCH] doc: announce change in crypto adapter queue add Volodymyr Fialko
2022-06-28 12:40 ` Akhil Goyal
2022-07-11 14:56 ` Jerin Jacob
2022-07-12 5:31 ` [EXT] " Akhil Goyal
2022-07-13 6:49 ` Gujjar, Abhinandan S
2022-07-14 9:04 ` Hemant Agrawal
2022-07-17 11:32 ` Thomas Monjalon
2022-08-04 9:59 ` [PATCH 0/3] Vector support for event crypto adapter Volodymyr Fialko
2022-08-04 9:59 ` [PATCH 1/3] eventdev: introduce event cryptodev vector type Volodymyr Fialko
2022-09-21 18:32 ` Akhil Goyal
2022-09-22 4:53 ` Gujjar, Abhinandan S
2022-09-24 8:43 ` Gujjar, Abhinandan S
2022-09-26 11:02 ` Volodymyr Fialko
2022-09-27 9:05 ` Gujjar, Abhinandan S
2022-09-27 9:24 ` Volodymyr Fialko [this message]
2022-09-27 9:38 ` Gujjar, Abhinandan S
2022-09-27 13:26 ` Jerin Jacob
2022-09-28 14:43 ` Gujjar, Abhinandan S
2022-09-28 16:13 ` Jerin Jacob
2022-08-04 9:59 ` [PATCH 2/3] crypto/cnxk: add burst enqueue for event crypto Volodymyr Fialko
2022-08-04 9:59 ` [PATCH 3/3] crypto/cnxk: add vectorization " Volodymyr Fialko
2022-09-26 11:36 ` [PATCH v2 0/3] Vector support for event crypto adapter Volodymyr Fialko
2022-09-26 11:36 ` [PATCH v2 1/3] eventdev: introduce event cryptodev vector type Volodymyr Fialko
2022-09-26 11:36 ` [PATCH v2 2/3] crypto/cnxk: add burst enqueue for event crypto Volodymyr Fialko
2022-09-26 11:36 ` [PATCH v2 3/3] crypto/cnxk: add vectorization " Volodymyr Fialko
2022-10-01 0:42 ` [PATCH v3 0/2] Vector support for event crypto adapter Volodymyr Fialko
2022-10-01 0:42 ` [PATCH v3 1/2] eventdev: introduce event cryptodev vector type Volodymyr Fialko
2022-10-01 0:42 ` [PATCH v3 2/2] crypto/cnxk: add vectorization for event crypto Volodymyr Fialko
2022-10-01 3:42 ` [PATCH v3 0/2] Vector support for event crypto adapter Akhil Goyal
2022-10-01 8:00 ` Gujjar, Abhinandan S
2022-10-01 8:47 ` Akhil Goyal
2022-10-02 1:56 ` Gujjar, Abhinandan S
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=MW2PR18MB22824B84DC4E018FFD34604CA8559@MW2PR18MB2282.namprd18.prod.outlook.com \
--to=vfialko@marvell.com \
--cc=abhinandan.gujjar@intel.com \
--cc=anoobj@marvell.com \
--cc=dev@dpdk.org \
--cc=gakhil@marvell.com \
--cc=hemant.agrawal@nxp.com \
--cc=jay.jayatheerthan@intel.com \
--cc=jerinj@marvell.com \
--cc=pbhagavatula@marvell.com \
--cc=sachin.saxena@oss.nxp.com \
--cc=sthotton@marvell.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).