> -----Original Message-----
> From: Jerin Jacob [mailto:jerinjac...@gmail.com]
> Sent: Tuesday, October 22, 2019 2:15 PM
> To: Rao, Nikhil <nikhil....@intel.com>
> Cc: Nipun Gupta <nipun.gu...@nxp.com>; Jerin Jacob <jer...@marvell.com>;
> dpdk-dev <dev@dpdk.org>; Pavan Nikhilesh <pbhagavat...@marvell.com>;
> Sunil Kumar Kori <sk...@marvell.com>; Richardson, Bruce
> <bruce.richard...@intel.com>; Kovacevic, Marko
> <marko.kovace...@intel.com>; Ori Kam <or...@mellanox.com>; Nicolau, Radu
> <radu.nico...@intel.com>; Kantecki, Tomasz <tomasz.kante...@intel.com>;
> Van Haaren, Harry <harry.van.haa...@intel.com>; Hemant Agrawal
> <hemant.agra...@nxp.com>
> Subject: Re: [dpdk-dev] [PATCH] eventdev: flag to identify same destined
> packets enqueue
>
> On Mon, Oct 21, 2019 at 5:05 PM Rao, Nikhil <nikhil....@intel.com> wrote:
> >
> > > -----Original Message-----
> > > From: Jerin Jacob [mailto:jerinjac...@gmail.com]
> > > Sent: Thursday, October 3, 2019 3:57 PM
> > > To: Hemant Agrawal <hemant.agra...@nxp.com>
> > > Cc: Rao, Nikhil <nikhil....@intel.com>; Nipun Gupta
> > > <nipun.gu...@nxp.com>; Jerin Jacob <jer...@marvell.com>; dpdk-dev
> > > <dev@dpdk.org>; Pavan Nikhilesh <pbhagavat...@marvell.com>; Sunil
> > > Kumar Kori <sk...@marvell.com>; Richardson, Bruce
> > > <bruce.richard...@intel.com>; Kovacevic, Marko
> > > <marko.kovace...@intel.com>; Ori Kam <or...@mellanox.com>; Nicolau,
> > > Radu <radu.nico...@intel.com>; Kantecki, Tomasz
> > > <tomasz.kante...@intel.com>; Van Haaren, Harry
> > > <harry.van.haa...@intel.com>
> > > Subject: Re: [dpdk-dev] [PATCH] eventdev: flag to identify same
> > > destined packets enqueue
> > >
> > </snip>
> >
> > > > > > But I am not able to recollect, Why Nikhil would like to use
> > > > > > the separate functions. Nikhil could you remind us why
> > > > > > rte_event_eth_tx_adapter_enqueue() can not be used for sending
> > > > > > the packet for SW Tx adapter.
> > > > > >
> > > > > [Nikhil] The goal was to keep the workers using the loop below.
> > > > >
> > > > > while (1) {
> > > > > rte_event_dequeue_burst(...);
> > > > > (event processing)
> > > > > rte_event_enqueue_burst(...); }
> > >
> > > We do have specialized functions for specific enqueue use case like
> > > rte_event_enqueue_new_burst() or
> > > rte_event_enqueue_forward_burst() to avoid any performance impact.
> > >
> > > Since PMD agruments are same for rte_event_enqueue_burst() and
> > > rte_event_eth_tx_adapter_enqueue()
> > > assigning simple function pointer assignment to
> > > rte_event_eth_tx_adapter_enqueue as dev->txa_enqueue =
> > > dev->enqueue_burst
> > > would have worked to have same Tx function across all platfroms
> > > without peformance overhead.
> > > Offcouse I understand, Slow path direct event enqueue assigment
> > > needs different treatment.
> > >
> > >
> > > ie in fastpath.
> > >
> > > while (1) {
> > > rte_event_dequeue_burst(...);
> > > if (tx_stage)
> > > rte_event_eth_tx_adapter_enqueue()...
> > > }
> > >
> > > What do you say?
> > >
> >
> > Sorry missed this question previously - Unless I have misunderstood your
> email, the event processing stage would have if conditions for each of the
> stages (or minimally the tx stage), no disagreement on that, the only
> difference
> would be set up of the event[] arrays that are sent to
> rte_event_enqueue_burst() and rte_event_eth_tx_adapter_enqueue() resulting
> in an additional call to rte_event_enqueue_burst(). If that’s true, since the
> abstraction has a cost to it, should we be adding it ?
>
> It there is a cost then we should not be adding it.
> I think, the following scheme can avoid the cost by adding the following in a
> _slow path_ as the prototype of the driver API is the same.
>
> dev->txa_enqueue = dev->enqueue_burst;
>
I was thinking of the event loop below which results in 2 calls to
rte_event_enqueue_burst()
while (1) {
rte_event_dequeue_burst(...);
for_all_events {
if (tx_stage)
event_tx[tx_cnt++] = ...
else
event_non_tx[non_tx_cnt++] = ...
}
if (tx_cnt)
rte_event_eth_tx_adapter_enqueue(event_tx, tx_cnt);
if (non_tx_cnt)
rte_event_enqueue_burst(event_non_tx, non_tx_cnt);
}
Thanks,
Nikhil
