DPDK patches and discussions
 help / color / mirror / Atom feed
From: Nipun Gupta <nipun.gupta@nxp.com>
To: Jerin Jacob <jerinjacobk@gmail.com>
Cc: Jerin Jacob <jerinj@marvell.com>, dpdk-dev <dev@dpdk.org>,
	Pavan Nikhilesh <pbhagavatula@marvell.com>,
	Sunil Kumar Kori <skori@marvell.com>,
	Hemant Agrawal <hemant.agrawal@nxp.com>,
	"Richardson, Bruce" <bruce.richardson@intel.com>,
	Marko Kovacevic <marko.kovacevic@intel.com>,
	Ori Kam <orika@mellanox.com>,
	Radu Nicolau <radu.nicolau@intel.com>,
	Tomasz Kantecki <tomasz.kantecki@intel.com>,
	"Van Haaren, Harry" <harry.van.haaren@intel.com>,
	"nikhil.rao@intel.com" <nikhil.rao@intel.com>
Subject: Re: [dpdk-dev] [PATCH] eventdev: flag to identify same destined packets enqueue
Date: Tue, 1 Oct 2019 15:06:33 +0000	[thread overview]
Message-ID: <VI1PR04MB4480056DDF916A3872DA6124E69D0@VI1PR04MB4480.eurprd04.prod.outlook.com> (raw)
In-Reply-To: <CALBAE1Nq1K-5mW3Wwoo2z8OgLN4+550C_R=7QL6_EMG1cr=aNg@mail.gmail.com>



> -----Original Message-----
> From: Jerin Jacob <jerinjacobk@gmail.com>
> Sent: Tuesday, October 1, 2019 7:50 PM
> To: Nipun Gupta <nipun.gupta@nxp.com>
> Cc: Jerin Jacob <jerinj@marvell.com>; dpdk-dev <dev@dpdk.org>; Pavan
> Nikhilesh <pbhagavatula@marvell.com>; Sunil Kumar Kori
> <skori@marvell.com>; Hemant Agrawal <hemant.agrawal@nxp.com>;
> Richardson, Bruce <bruce.richardson@intel.com>; Marko Kovacevic
> <marko.kovacevic@intel.com>; Ori Kam <orika@mellanox.com>; Radu
> Nicolau <radu.nicolau@intel.com>; Tomasz Kantecki
> <tomasz.kantecki@intel.com>; Van Haaren, Harry
> <harry.van.haaren@intel.com>; nikhil.rao@intel.com
> Subject: Re: [dpdk-dev] [PATCH] eventdev: flag to identify same destined
> packets enqueue
> 
> On Tue, Oct 1, 2019 at 7:32 PM Nipun Gupta <nipun.gupta@nxp.com> wrote:
> >
> >
> >
> > > -----Original Message-----
> > > From: Jerin Jacob <jerinjacobk@gmail.com>
> > > Sent: Tuesday, October 1, 2019 6:40 PM
> > > To: Nipun Gupta <nipun.gupta@nxp.com>
> > > Cc: dpdk-dev <dev@dpdk.org>; Jerin Jacob <jerinj@marvell.com>; Pavan
> > > Nikhilesh <pbhagavatula@marvell.com>; Sunil Kumar Kori
> > > <skori@marvell.com>; Hemant Agrawal <hemant.agrawal@nxp.com>;
> > > Richardson, Bruce <bruce.richardson@intel.com>; Marko Kovacevic
> > > <marko.kovacevic@intel.com>; Ori Kam <orika@mellanox.com>; Radu
> > > Nicolau <radu.nicolau@intel.com>; Tomasz Kantecki
> > > <tomasz.kantecki@intel.com>; Van Haaren, Harry
> > > <harry.van.haaren@intel.com>; nikhil.rao@intel.com
> > > Subject: Re: [dpdk-dev] [PATCH] eventdev: flag to identify same
> destined
> > > packets enqueue
> > >
> > > On Tue, Oct 1, 2019 at 5:11 PM Nipun Gupta <nipun.gupta@nxp.com>
> wrote:
> > > >
> > > >
> > > >
> > > > > -----Original Message-----
> > > > > From: Jerin Jacob <jerinjacobk@gmail.com>
> > > > > Sent: Tuesday, October 1, 2019 1:14 PM
> > > > > To: Nipun Gupta <nipun.gupta@nxp.com>
> > > > > Cc: dpdk-dev <dev@dpdk.org>; Jerin Jacob <jerinj@marvell.com>;
> Pavan
> > > > > Nikhilesh <pbhagavatula@marvell.com>; Sunil Kumar Kori
> > > <skori@marvell.com>;
> > > > > Hemant Agrawal <hemant.agrawal@nxp.com>; Richardson, Bruce
> > > > > <bruce.richardson@intel.com>; Marko Kovacevic
> > > > > <marko.kovacevic@intel.com>; Ori Kam <orika@mellanox.com>;
> Radu
> > > Nicolau
> > > > > <radu.nicolau@intel.com>; Tomasz Kantecki
> > > <tomasz.kantecki@intel.com>; Van
> > > > > Haaren, Harry <harry.van.haaren@intel.com>; nikhil.rao@intel.com
> > > > > Subject: Re: [dpdk-dev] [PATCH] eventdev: flag to identify same
> > > destined
> > > > > packets enqueue
> > > > >
> > > > > On Tue, Oct 1, 2019 at 12:32 PM Nipun Gupta <nipun.gupta@nxp.com>
> > > wrote:
> > > > > >
> > > > > > This patch introduces a `flag` in the Eth TX adapter enqueue API.
> > > > > > Some drivers may support burst functionality only with the packets
> > > > > > having same destination device and queue.
> > > > > >
> > > > > > The flag `RTE_EVENT_ETH_TX_ADAPTER_ENQUEUE_SAME_DEST`
> can
> > > be used
> > > > > > to indicate this so the underlying driver, for drivers to utilize
> > > > > > burst functionality appropriately.
> > > > >
> > > > > I understand the cost of aggregating packets based on port and
> queue
> > > > > to make it burst.
> > > > > But, Could you share the use case where how do you want to use this
> > > > > flag? I see two possibilities in eventdev context.
> > > > > (Where dequeue can be from any ethdev port)
> > > > > a) The application does the aggregation. If so, what would the
> > > > > difference be in doing driver or application?
> > > > > b) We may use this flag when the system has only one port and one
> > > queue.
> > > > >
> > > > > Could you share how do you want to use this flag?
> > > >
> > > > I would say both the cases you mentioned. I prefer this to depend on
> the
> > > smartness of
> > > > the application, as it is aware of the number of eth devices/queues it is
> > > using.
> > > > This is the reason to avoid in the driver.
> > > >
> > > > A user can also use separate event ports for separate or a group of
> > > Ethernet
> > > > devices/queues, to create easy segregation.
> > >
> > > If it is specific to _very_ static configuration, you can assume all
> > > the events comes
> > > from a specific eventdev port, comes only from a specific ethdev port.
> >
> > Hi Jerin,
> >
> > If I understand correctly this assumption would be made in the driver?
> > But then applications like l2fwd-event will break.
> 
> Yes. What I meant is a specific static configuration only this scheme can use.
> 
> 
> >
> > >
> > > >
> > > > Also the drawback of implementing in the driver, can be that when the
> > > events to
> > > > Separate devices and queues are so mixed up, that segregation does
> not
> > > make
> > > > sense, it shall be overhead for the driver to scan all the events.
> > >
> > > In the worst case, both, applications need to scan and driver need to
> > > iterate over
> > > all the events.
> >
> > Agree, this is what we want to avoid.
> >
> > >
> > >
> > > In generic l2fwd-eventdev applications etc, q0 will be connected p0 and
> p1
> > > etc.
> > > This flag will be zero.
> >
> > Okay.
> >
> > >
> > > But, If you think, there is a specific use case for it we can add this flag.
> > >
> > >
> > > >
> > > > A flag would help in providing the decision flexibility to the applications.
> > > >
> > > > >
> > > > > And another point is, tx adapter is NOT experimental now, We need
> > > > > depreciation notice for ABI change.
> > > > > If you share the exact use case, then we could think of adding a new
> > > > > symbol instead of breaking ABI and
> > > > > add it for next release.
> > > >
> > > > I have heard the discussion that we may get some exceptions to
> > > deprecation
> > > > process for 19.11 as the APIs will freeze of 1 year post it.
> > > > Anyway, if you have a better way with symbol, please suggest.
> > >
> > >
> > > One option could be (not as bad as changing the enqueue prototype) to
> > > add new field in struct rte_event_eth_tx_adapter_conf.
> > >
> > > Since this scheme can be used ONLY on the static configuration, adding
> > > a few fields
> > > for Tx adapter configuration would help.
> > > If that field is set you can choose dev->txa_enqueue light weight
> > > enqueue function
> > > if not, the driver can aggregate the buffers and send them.
> >
> > Thanks for suggesting this, we also thought of it, but would prefer having
> this as a
> > runtime option rather than static option.
> > We would not like to have one time configuration restriction.
> 
> Not sure how it can be used in runtime. If q0 is connected p0 and p1 and then
> it flag has to be cleared. if q0 is only connected to p0 then we can
> use this scheme.
> So it is pretty much static in nature.
> 
> How do you think, it can be used in runtime.
> 
> If we think, If we are planning to use like below in application, it
> would be really bad
> in the worst case.(mbuf cache misses in-app and driver)
> 
> In app:
> const port = event[0].mbuf.port;
> const queue = event[0].mbuf.queue;
> for (i = i; i < nb_events; i++) {
>           if (port != event[i].mbuf.port || queue != event[i].mbuf.queue)
>                    break;
> }
> if (i == nb_events)
>          flag = 1;
> else
>         flag = 0;
> 

In such case if cache misses are observed which is more than what sending burst
of traffic is in the applications use-case, the application shall simply not use the flag.

But again I think this is to be governed by the application which is the user of the
event device, on the basis on the configuration and the memory footprint/utilization;
and more than that its performance benchmarking.

> more over the static scheme does not make any change in other drivers.
> If you have the usecase for the dynamic scheme then let us know then
> we add dynamic flag breaking the ABI.

I understand your concern that this shall not be valid on a general cases.

There are certain use-case (NXP internal) of DPDK event which have separate
event ports/cores dedicated for certain tasks - and can use the burst functionality
for performance according to the event ports which are used. Not having this at
runtime will limit the flexibility for such applications.

Regards,
Nipun

> 
> 
> >
> > >
> > >
> > >
> > > >
> > > > >
> > > > >
> > > > >
> > > > >
> > > > > > Signed-off-by: Nipun Gupta <nipun.gupta@nxp.com>
> > > > > > ---
> > > > > >  app/test-eventdev/test_pipeline_common.h          |  6 +++---
> > > > > >  .../prog_guide/event_ethernet_tx_adapter.rst      |  3 ++-
> > > > > >  drivers/event/octeontx/ssovf_evdev.h              |  2 +-
> > > > > >  drivers/event/octeontx/ssovf_worker.c             |  3 ++-
> > > > > >  drivers/event/octeontx2/otx2_evdev.h              | 12 ++++++++----
> > > > > >  drivers/event/octeontx2/otx2_worker.c             |  8 ++++++--
> > > > > >  drivers/event/octeontx2/otx2_worker_dual.c        |  8 ++++++--
> > > > > >  lib/librte_eventdev/rte_event_eth_tx_adapter.h    | 15
> > > +++++++++++++--
> > > > > >  lib/librte_eventdev/rte_eventdev.c                |  3 ++-
> > > > > >  lib/librte_eventdev/rte_eventdev.h                |  2 +-
> > > > > >  10 files changed, 44 insertions(+), 18 deletions(-)
> > > > > >
> > > > > > diff --git a/app/test-eventdev/test_pipeline_common.h
> b/app/test-
> > > > > eventdev/test_pipeline_common.h
> > > > > > index 0440b9e29..6e73c6ab2 100644
> > > > > > --- a/app/test-eventdev/test_pipeline_common.h
> > > > > > +++ b/app/test-eventdev/test_pipeline_common.h
> > > > > > @@ -106,7 +106,7 @@ pipeline_event_tx(const uint8_t dev, const
> > > uint8_t
> > > > > port,
> > > > > >                 struct rte_event * const ev)
> > > > > >  {
> > > > > >         rte_event_eth_tx_adapter_txq_set(ev->mbuf, 0);
> > > > > > -       while (!rte_event_eth_tx_adapter_enqueue(dev, port, ev, 1))
> > > > > > +       while (!rte_event_eth_tx_adapter_enqueue(dev, port, ev, 1,
> 0))
> > > > > >                 rte_pause();
> > > > > >  }
> > > > > >
> > > > > > @@ -116,10 +116,10 @@ pipeline_event_tx_burst(const uint8_t
> dev,
> > > const
> > > > > uint8_t port,
> > > > > >  {
> > > > > >         uint16_t enq;
> > > > > >
> > > > > > -       enq = rte_event_eth_tx_adapter_enqueue(dev, port, ev,
> nb_rx);
> > > > > > +       enq = rte_event_eth_tx_adapter_enqueue(dev, port, ev,
> nb_rx,
> > > 0);
> > > > > >         while (enq < nb_rx) {
> > > > > >                 enq += rte_event_eth_tx_adapter_enqueue(dev, port,
> > > > > > -                               ev + enq, nb_rx - enq);
> > > > > > +                               ev + enq, nb_rx - enq, 0);
> > > > > >         }
> > > > > >  }
> > > > > >
> > > > > > diff --git a/doc/guides/prog_guide/event_ethernet_tx_adapter.rst
> > > > > b/doc/guides/prog_guide/event_ethernet_tx_adapter.rst
> > > > > > index 192f9e1cf..a8c13e136 100644
> > > > > > --- a/doc/guides/prog_guide/event_ethernet_tx_adapter.rst
> > > > > > +++ b/doc/guides/prog_guide/event_ethernet_tx_adapter.rst
> > > > > > @@ -137,11 +137,12 @@ should use the
> > > ``rte_event_enqueue_burst()``
> > > > > function.
> > > > > >         if (cap & RTE_EVENT_ETH_TX_ADAPTER_CAP_INTERNAL_PORT)
> {
> > > > > >
> > > > > >                 event.mbuf = m;
> > > > > > +               eq_flags = 0;
> > > > > >
> > > > > >                 m->port = tx_port;
> > > > > >                 rte_event_eth_tx_adapter_txq_set(m, tx_queue_id);
> > > > > >
> > > > > > -               rte_event_eth_tx_adapter_enqueue(dev_id, ev_port,
> &event,
> > > 1);
> > > > > > +               rte_event_eth_tx_adapter_enqueue(dev_id, ev_port,
> > > &event, 1,
> > > > > eq_flags);
> > > > > >         } else {
> > > > > >
> > > > > >                 event.queue_id = qid; /* event queue linked to adapter
> port */
> > > > > > diff --git a/drivers/event/octeontx/ssovf_evdev.h
> > > > > b/drivers/event/octeontx/ssovf_evdev.h
> > > > > > index 0e622152c..1b156edab 100644
> > > > > > --- a/drivers/event/octeontx/ssovf_evdev.h
> > > > > > +++ b/drivers/event/octeontx/ssovf_evdev.h
> > > > > > @@ -181,7 +181,7 @@ void ssows_flush_events(struct ssows *ws,
> > > uint8_t
> > > > > queue_id,
> > > > > >                 ssows_handle_event_t fn, void *arg);
> > > > > >  void ssows_reset(struct ssows *ws);
> > > > > >  uint16_t sso_event_tx_adapter_enqueue(void *port,
> > > > > > -               struct rte_event ev[], uint16_t nb_events);
> > > > > > +               struct rte_event ev[], uint16_t nb_events, uint8_t
> eq_flags);
> > > > > >  int ssovf_info(struct ssovf_info *info);
> > > > > >  void *ssovf_bar(enum ssovf_type, uint8_t id, uint8_t bar);
> > > > > >  int test_eventdev_octeontx(void);
> > > > > > diff --git a/drivers/event/octeontx/ssovf_worker.c
> > > > > b/drivers/event/octeontx/ssovf_worker.c
> > > > > > index d940b5dd6..1d0467af3 100644
> > > > > > --- a/drivers/event/octeontx/ssovf_worker.c
> > > > > > +++ b/drivers/event/octeontx/ssovf_worker.c
> > > > > > @@ -264,7 +264,7 @@ ssows_reset(struct ssows *ws)
> > > > > >
> > > > > >  uint16_t
> > > > > >  sso_event_tx_adapter_enqueue(void *port,
> > > > > > -               struct rte_event ev[], uint16_t nb_events)
> > > > > > +               struct rte_event ev[], uint16_t nb_events, uint8_t eq_flags)
> > > > > >  {
> > > > > >         uint16_t port_id;
> > > > > >         uint16_t queue_id;
> > > > > > @@ -275,6 +275,7 @@ sso_event_tx_adapter_enqueue(void *port,
> > > > > >         octeontx_dq_t *dq;
> > > > > >
> > > > > >         RTE_SET_USED(nb_events);
> > > > > > +       RTE_SET_USED(eq_flags);
> > > > > >         switch (ev->sched_type) {
> > > > > >         case SSO_SYNC_ORDERED:
> > > > > >                 ssows_swtag_norm(ws, ev->event, SSO_SYNC_ATOMIC);
> > > > > > diff --git a/drivers/event/octeontx2/otx2_evdev.h
> > > > > b/drivers/event/octeontx2/otx2_evdev.h
> > > > > > index 5cd80e3b2..74b749a15 100644
> > > > > > --- a/drivers/event/octeontx2/otx2_evdev.h
> > > > > > +++ b/drivers/event/octeontx2/otx2_evdev.h
> > > > > > @@ -333,16 +333,20 @@ SSO_RX_ADPTR_ENQ_FASTPATH_FUNC
> > > > > >
> > > > > >  #define T(name, f4, f3, f2, f1, f0, sz, flags)                              \
> > > > > >  uint16_t otx2_ssogws_tx_adptr_enq_ ## name(void *port, struct
> > > rte_event
> > > > > ev[],\
> > > > > > -                                          uint16_t nb_events);              \
> > > > > > +                                          uint16_t nb_events,               \
> > > > > > +                                          uint8_t eq_flags);                \
> > > > > >  uint16_t otx2_ssogws_tx_adptr_enq_seg_ ## name(void *port,
> > > \
> > > > > >                                                struct rte_event ev[],        \
> > > > > > -                                              uint16_t nb_events);          \
> > > > > > +                                              uint16_t nb_events,           \
> > > > > > +                                              uint8_t eq_flags);            \
> > > > > >  uint16_t otx2_ssogws_dual_tx_adptr_enq_ ## name(void *port,
> > > \
> > > > > >                                                 struct rte_event ev[],       \
> > > > > > -                                               uint16_t nb_events);         \
> > > > > > +                                               uint16_t nb_events,          \
> > > > > > +                                               uint8_t eq_flags);           \
> > > > > >  uint16_t otx2_ssogws_dual_tx_adptr_enq_seg_ ## name(void
> *port,
> > > > > \
> > > > > >                                                     struct rte_event ev[],   \
> > > > > > -                                                   uint16_t nb_events);     \
> > > > > > +                                                   uint16_t nb_events,      \
> > > > > > +                                                   uint8_t eq_flags);       \
> > > > > >
> > > > > >  SSO_TX_ADPTR_ENQ_FASTPATH_FUNC
> > > > > >  #undef T
> > > > > > diff --git a/drivers/event/octeontx2/otx2_worker.c
> > > > > b/drivers/event/octeontx2/otx2_worker.c
> > > > > > index cd14cd3d2..100e21669 100644
> > > > > > --- a/drivers/event/octeontx2/otx2_worker.c
> > > > > > +++ b/drivers/event/octeontx2/otx2_worker.c
> > > > > > @@ -270,12 +270,14 @@ otx2_ssogws_enq_fwd_burst(void *port,
> > > const
> > > > > struct rte_event ev[],
> > > > > >  #define T(name, f4, f3, f2, f1, f0, sz, flags)                         \
> > > > > >  uint16_t __hot                                                         \
> > > > > >  otx2_ssogws_tx_adptr_enq_ ## name(void *port, struct rte_event
> > > ev[],   \
> > > > > > -                                 uint16_t nb_events)                   \
> > > > > > +                                 uint16_t nb_events,                   \
> > > > > > +                                 uint8_t eq_flags)                     \
> > > > > >  {                                                                      \
> > > > > >         struct otx2_ssogws *ws = port;                                  \
> > > > > >         uint64_t cmd[sz];                                               \
> > > > > >                                                                         \
> > > > > >         RTE_SET_USED(nb_events);                                        \
> > > > > > +       RTE_SET_USED(eq_flags);                                         \
> > > > > >         return otx2_ssogws_event_tx(ws, ev, cmd, flags);                \
> > > > > >  }
> > > > > >  SSO_TX_ADPTR_ENQ_FASTPATH_FUNC
> > > > > > @@ -284,12 +286,14 @@ SSO_TX_ADPTR_ENQ_FASTPATH_FUNC
> > > > > >  #define T(name, f4, f3, f2, f1, f0, sz, flags)                         \
> > > > > >  uint16_t __hot                                                         \
> > > > > >  otx2_ssogws_tx_adptr_enq_seg_ ## name(void *port, struct
> > > rte_event ev[],\
> > > > > > -                                     uint16_t nb_events)               \
> > > > > > +                                     uint16_t nb_events,               \
> > > > > > +                                     uint8_t eq_flags)                 \
> > > > > >  {                                                                      \
> > > > > >         struct otx2_ssogws *ws = port;                                  \
> > > > > >         uint64_t cmd[(sz) + NIX_TX_MSEG_SG_DWORDS - 2];                 \
> > > > > >                                                                         \
> > > > > >         RTE_SET_USED(nb_events);                                        \
> > > > > > +       RTE_SET_USED(eq_flags);                                         \
> > > > > >         return otx2_ssogws_event_tx(ws, ev, cmd, (flags) |              \
> > > > > >                                     NIX_TX_MULTI_SEG_F);                \
> > > > > >  }
> > > > > > diff --git a/drivers/event/octeontx2/otx2_worker_dual.c
> > > > > b/drivers/event/octeontx2/otx2_worker_dual.c
> > > > > > index 37c274a54..c3e48da42 100644
> > > > > > --- a/drivers/event/octeontx2/otx2_worker_dual.c
> > > > > > +++ b/drivers/event/octeontx2/otx2_worker_dual.c
> > > > > > @@ -310,7 +310,8 @@ SSO_RX_ADPTR_ENQ_FASTPATH_FUNC
> > > > > >  uint16_t __hot                                                         \
> > > > > >  otx2_ssogws_dual_tx_adptr_enq_ ## name(void *port,                     \
> > > > > >                                        struct rte_event ev[],           \
> > > > > > -                                      uint16_t nb_events)              \
> > > > > > +                                      uint16_t nb_events,              \
> > > > > > +                                      uint8_t eq_flags)                \
> > > > > >  {                                                                      \
> > > > > >         struct otx2_ssogws_dual *ws = port;                             \
> > > > > >         struct otx2_ssogws *vws =                                       \
> > > > > > @@ -318,6 +319,7 @@ otx2_ssogws_dual_tx_adptr_enq_ ##
> > > name(void *port,
> > > > > \
> > > > > >         uint64_t cmd[sz];                                               \
> > > > > >                                                                         \
> > > > > >         RTE_SET_USED(nb_events);                                        \
> > > > > > +       RTE_SET_USED(eq_flags);                                         \
> > > > > >         return otx2_ssogws_event_tx(vws, ev, cmd, flags);               \
> > > > > >  }
> > > > > >  SSO_TX_ADPTR_ENQ_FASTPATH_FUNC
> > > > > > @@ -327,7 +329,8 @@ SSO_TX_ADPTR_ENQ_FASTPATH_FUNC
> > > > > >  uint16_t __hot                                                         \
> > > > > >  otx2_ssogws_dual_tx_adptr_enq_seg_ ## name(void *port,
> \
> > > > > >                                            struct rte_event ev[],       \
> > > > > > -                                          uint16_t nb_events)          \
> > > > > > +                                          uint16_t nb_events,          \
> > > > > > +                                          uint8_t eq_flags)            \
> > > > > >  {                                                                      \
> > > > > >         struct otx2_ssogws_dual *ws = port;                             \
> > > > > >         struct otx2_ssogws *vws =                                       \
> > > > > > @@ -335,6 +338,7 @@ otx2_ssogws_dual_tx_adptr_enq_seg_ ##
> > > name(void
> > > > > *port,                      \
> > > > > >         uint64_t cmd[(sz) + NIX_TX_MSEG_SG_DWORDS - 2];                 \
> > > > > >                                                                         \
> > > > > >         RTE_SET_USED(nb_events);                                        \
> > > > > > +       RTE_SET_USED(eq_flags);                                         \
> > > > > >         return otx2_ssogws_event_tx(vws, ev, cmd, (flags) |             \
> > > > > >                                     NIX_TX_MULTI_SEG_F);                \
> > > > > >  }
> > > > > > diff --git a/lib/librte_eventdev/rte_event_eth_tx_adapter.h
> > > > > b/lib/librte_eventdev/rte_event_eth_tx_adapter.h
> > > > > > index c848261c4..98be77568 100644
> > > > > > --- a/lib/librte_eventdev/rte_event_eth_tx_adapter.h
> > > > > > +++ b/lib/librte_eventdev/rte_event_eth_tx_adapter.h
> > > > > > @@ -300,6 +300,11 @@ rte_event_eth_tx_adapter_txq_get(struct
> > > rte_mbuf
> > > > > *pkt)
> > > > > >  int
> > > > > >  rte_event_eth_tx_adapter_event_port_get(uint8_t id, uint8_t
> > > > > *event_port_id);
> > > > > >
> > > > > > +#define RTE_EVENT_ETH_TX_ADAPTER_ENQUEUE_SAME_DEST
> 0x1
> > > > > > +/**< This flag is used when all the packets enqueued in the tx
> adapter
> > > are
> > > > > > + * destined for the same Ethernet device, queue pair.
> > > > > > + */
> > > > > > +
> > > > > >  /**
> > > > > >   * Enqueue a burst of events objects or an event object supplied in
> > > > > *rte_event*
> > > > > >   * structure on an  event device designated by its *dev_id* through
> > > the event
> > > > > > @@ -324,6 +329,10 @@
> > > rte_event_eth_tx_adapter_event_port_get(uint8_t
> > > > > id, uint8_t *event_port_id);
> > > > > >   *  The number of event objects to enqueue, typically number of
> > > > > >   *
> rte_event_port_attr_get(...RTE_EVENT_PORT_ATTR_ENQ_DEPTH...)
> > > > > >   *  available for this port.
> > > > > > + * @param flags
> > > > > > + *  See RTE_EVENT_ETH_TX_ADAPTER_ENQUEUE_ flags.
> > > > > > + *  #RTE_EVENT_ETH_TX_ADAPTER_ENQUEUE_SAME_DEST
> signifies
> > > that all
> > > > > the packets
> > > > > > + *  which are enqueued are destined for the same Ethernet device,
> > > queue pair.
> > > > > >   *
> > > > > >   * @return
> > > > > >   *   The number of event objects actually enqueued on the event
> > > device. The
> > > > > > @@ -343,7 +352,8 @@ static inline uint16_t
> > > > > >  rte_event_eth_tx_adapter_enqueue(uint8_t dev_id,
> > > > > >                                 uint8_t port_id,
> > > > > >                                 struct rte_event ev[],
> > > > > > -                               uint16_t nb_events)
> > > > > > +                               uint16_t nb_events,
> > > > > > +                               uint8_t flags)
> > > > > >  {
> > > > > >         const struct rte_eventdev *dev = &rte_eventdevs[dev_id];
> > > > > >
> > > > > > @@ -359,7 +369,8 @@
> rte_event_eth_tx_adapter_enqueue(uint8_t
> > > dev_id,
> > > > > >                 return 0;
> > > > > >         }
> > > > > >  #endif
> > > > > > -       return dev->txa_enqueue(dev->data->ports[port_id], ev,
> > > nb_events);
> > > > > > +       return dev->txa_enqueue(dev->data->ports[port_id], ev,
> > > > > > +                               nb_events, flags);
> > > > > >  }
> > > > > >
> > > > > >  /**
> > > > > > diff --git a/lib/librte_eventdev/rte_eventdev.c
> > > > > b/lib/librte_eventdev/rte_eventdev.c
> > > > > > index f44c869cb..3bf9d7115 100644
> > > > > > --- a/lib/librte_eventdev/rte_eventdev.c
> > > > > > +++ b/lib/librte_eventdev/rte_eventdev.c
> > > > > > @@ -1324,7 +1324,8 @@
> rte_eventdev_find_free_device_index(void)
> > > > > >  static uint16_t
> > > > > >  rte_event_tx_adapter_enqueue(__rte_unused void *port,
> > > > > >                         __rte_unused struct rte_event ev[],
> > > > > > -                       __rte_unused uint16_t nb_events)
> > > > > > +                       __rte_unused uint16_t nb_events,
> > > > > > +                       __rte_unused uint8_t flags)
> > > > > >  {
> > > > > >         rte_errno = ENOTSUP;
> > > > > >         return 0;
> > > > > > diff --git a/lib/librte_eventdev/rte_eventdev.h
> > > > > b/lib/librte_eventdev/rte_eventdev.h
> > > > > > index 5044a13d0..2a5643da3 100644
> > > > > > --- a/lib/librte_eventdev/rte_eventdev.h
> > > > > > +++ b/lib/librte_eventdev/rte_eventdev.h
> > > > > > @@ -1227,7 +1227,7 @@ typedef uint16_t
> > > (*event_dequeue_burst_t)(void
> > > > > *port, struct rte_event ev[],
> > > > > >  /**< @internal Dequeue burst of events from port of a device */
> > > > > >
> > > > > >  typedef uint16_t (*event_tx_adapter_enqueue)(void *port,
> > > > > > -                               struct rte_event ev[], uint16_t nb_events);
> > > > > > +               struct rte_event ev[], uint16_t nb_events, uint8_t flags);
> > > > > >  /**< @internal Enqueue burst of events on port of a device */
> > > > > >
> > > > > >  #define RTE_EVENTDEV_NAME_MAX_LEN      (64)
> > > > > > --
> > > > > > 2.17.1
> > > > > >

  reply	other threads:[~2019-10-01 15:06 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-10-01  6:46 Nipun Gupta
2019-10-01  7:44 ` Jerin Jacob
2019-10-01 11:41   ` Nipun Gupta
2019-10-01 13:09     ` Jerin Jacob
2019-10-01 14:02       ` Nipun Gupta
2019-10-01 14:20         ` Jerin Jacob
2019-10-01 15:06           ` Nipun Gupta [this message]
2019-10-01 15:35             ` Jerin Jacob
2019-10-02  3:08               ` Hemant Agrawal
2019-10-02  7:54                 ` Jerin Jacob
2019-10-03  6:12                   ` Rao, Nikhil
2019-10-03  6:45                     ` Hemant Agrawal
2019-10-03 10:26                       ` Jerin Jacob
2019-10-21 11:35                         ` Rao, Nikhil
2019-10-22  8:45                           ` Jerin Jacob
2019-10-22  9:42                             ` Rao, Nikhil
2019-10-23  4:53                               ` Jerin Jacob
2019-10-01 14:42 ` Aaron Conole
2019-10-01 15:15   ` Nipun Gupta
2019-10-04  6:55 ` [dpdk-dev] [PATCH v2] " Nipun Gupta
2019-10-04  8:21   ` Jerin Jacob
2019-10-04 10:47 ` [dpdk-dev] [PATCH v3] " Nipun Gupta
2019-10-05 16:23   ` Jerin Jacob
2019-10-09  7:32 ` [dpdk-dev] [PATCH v4] " Nipun Gupta
2019-10-10 10:06   ` Rao, Nikhil
2019-10-10 12:43     ` Nipun Gupta
2019-10-11  6:14       ` Jerin Jacob
2019-10-11 13:03 ` [dpdk-dev] [PATCH v5] " Nipun Gupta
2019-10-16  6:41   ` Hemant Agrawal
2019-10-16  7:19   ` Jerin Jacob

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=VI1PR04MB4480056DDF916A3872DA6124E69D0@VI1PR04MB4480.eurprd04.prod.outlook.com \
    --to=nipun.gupta@nxp.com \
    --cc=bruce.richardson@intel.com \
    --cc=dev@dpdk.org \
    --cc=harry.van.haaren@intel.com \
    --cc=hemant.agrawal@nxp.com \
    --cc=jerinj@marvell.com \
    --cc=jerinjacobk@gmail.com \
    --cc=marko.kovacevic@intel.com \
    --cc=nikhil.rao@intel.com \
    --cc=orika@mellanox.com \
    --cc=pbhagavatula@marvell.com \
    --cc=radu.nicolau@intel.com \
    --cc=skori@marvell.com \
    --cc=tomasz.kantecki@intel.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).