Re: [dpdk-dev] [PATCH v2] eventdev: fix event port setup in tx adapter

["Jayatheerthan, Jay" <jay.jayatheerthan@intel.com>]

> -----Original Message-----
> From: Jerin Jacob <jerinjacobk@gmail.com>
> Sent: Wednesday, July 28, 2021 3:01 PM
> To: Jayatheerthan, Jay <jay.jayatheerthan@intel.com>
> Cc: Naga Harish K, S V <s.v.naga.harish.k@intel.com>; dev@dpdk.org
> Subject: Re: [PATCH v2] eventdev: fix event port setup in tx adapter
> 
> On Wed, Jul 28, 2021 at 12:14 PM Jayatheerthan, Jay
> <jay.jayatheerthan@intel.com> wrote:
> >
> > > -----Original Message-----
> > > From: Naga Harish K, S V <s.v.naga.harish.k@intel.com>
> > > Sent: Saturday, July 24, 2021 7:41 PM
> > > To: Jayatheerthan, Jay <jay.jayatheerthan@intel.com>; jerinjacobk@gmail.com
> > > Cc: dev@dpdk.org
> > > Subject: [PATCH v2] eventdev: fix event port setup in tx adapter
> > >
> > > The event port config set by application in
> > > rte_event_eth_tx_adapter_create API is modified in
> > > default configuration callback function. This patch removes
> > > this hardcode to use application provided event port
> > > config value.
> > >
> > > Fixes: ("eventdev: fix event port config override in tx adapter")
> >
> > @Jerin, does this look good to you ?
> 
> Yes. I will merge this. See below.

Thanks!

> 
> >
> > >
> > > Signed-off-by: Naga Harish K, S V <s.v.naga.harish.k@intel.com>
> 
> 
> @Naga Harish K, S V Can we remove "," from the Signoff name.
> 
> Some suggestions for consideration in preferred order. Let me know
> your preferred Signoff name, I will change it accordingly.
> 
> Naga Harish <s.v.naga.harish.k@intel.com>
> 
> or
> 
> Naga Harish K <s.v.naga.harish.k@intel.com>
> 
> or
> 
> Naga Harish K S V <s.v.naga.harish.k@intel.com>

This option is good. (Harish is out and hence I am responding)

> 
> > > ---
> > >  lib/eventdev/rte_event_eth_tx_adapter.c | 1 -
> > >  1 file changed, 1 deletion(-)
> > >
> > > diff --git a/lib/eventdev/rte_event_eth_tx_adapter.c b/lib/eventdev/rte_event_eth_tx_adapter.c
> > > index db260bfb68..18c0359db7 100644
> > > --- a/lib/eventdev/rte_event_eth_tx_adapter.c
> > > +++ b/lib/eventdev/rte_event_eth_tx_adapter.c
> > > @@ -286,7 +286,6 @@ txa_service_conf_cb(uint8_t __rte_unused id, uint8_t dev_id,
> > >               return ret;
> > >       }
> > >
> > > -     pc->event_port_cfg = 0;
> > >       ret = rte_event_port_setup(dev_id, port_id, pc);
> > >       if (ret) {
> > >               RTE_EDEV_LOG_ERR("failed to setup event port %u\n",
> > > --
> > > 2.25.1