Re: [dpdk-dev] [EXT] [PATCH v1 4/4] regexdev: implement regex rte level functions

2020-04-06 Thread Ori Kam
vid.march...@redhat.com > Subject: Re: [dpdk-dev] [EXT] [PATCH v1 4/4] regexdev: implement regex rte > level functions > > 06/04/2020 15:50, Pavan Nikhilesh Bhagavatula: > > From: Thomas Monjalon > > >> >06/04/2020 14:33, Pavan Nikhiles

Re: [dpdk-dev] [EXT] [PATCH v1 4/4] regexdev: implement regex rte level functions

2020-04-06 Thread Thomas Monjalon
06/04/2020 15:50, Pavan Nikhilesh Bhagavatula: > From: Thomas Monjalon > >> >06/04/2020 14:33, Pavan Nikhilesh Bhagavatula: > >> >> >> From: Pavan Nikhilesh Bhagavatula > >> >> >> > >+uint16_t > >> >> >> > >+rte_regexdev_enqueue_burst(uint8_t dev_id, uint16_t > >> >qp_id, > >> >> >> > >+

Re: [dpdk-dev] [EXT] [PATCH v1 4/4] regexdev: implement regex rte level functions

2020-04-06 Thread Pavan Nikhilesh Bhagavatula
>-Original Message- >From: Thomas Monjalon >Sent: Monday, April 6, 2020 7:07 PM >To: Pavan Nikhilesh Bhagavatula >Cc: Ori Kam ; Jerin Jacob Kollanukkaran >; xiang.w.w...@intel.com; dev@dpdk.org; >Shahaf Shuler ; hemant.agra...@nxp.com; >Opher Reviv ; Alex Rosenbaum >; Dovrat Zifroni ;

Re: [dpdk-dev] [EXT] [PATCH v1 4/4] regexdev: implement regex rte level functions

2020-04-06 Thread Thomas Monjalon
06/04/2020 15:22, Pavan Nikhilesh Bhagavatula: > >06/04/2020 14:33, Pavan Nikhilesh Bhagavatula: > >> >> From: Pavan Nikhilesh Bhagavatula > >> >> > >+uint16_t > >> >> > >+rte_regexdev_enqueue_burst(uint8_t dev_id, uint16_t > >qp_id, > >> >> > >+ struct rte_regex_ops **ops, ui

Re: [dpdk-dev] [EXT] [PATCH v1 4/4] regexdev: implement regex rte level functions

2020-04-06 Thread Pavan Nikhilesh Bhagavatula
>06/04/2020 14:33, Pavan Nikhilesh Bhagavatula: >> >> From: Pavan Nikhilesh Bhagavatula >> >> > >+uint16_t >> >> > >+rte_regexdev_enqueue_burst(uint8_t dev_id, uint16_t >qp_id, >> >> > >+ struct rte_regex_ops **ops, uint16_t >nb_ops) >> >> > >+{ >> >> > >+ return regex_

Re: [dpdk-dev] [EXT] [PATCH v1 4/4] regexdev: implement regex rte level functions

2020-04-06 Thread Jerin Jacob
On Mon, Apr 6, 2020 at 6:44 PM Thomas Monjalon wrote: > > 06/04/2020 14:33, Pavan Nikhilesh Bhagavatula: > > >> From: Pavan Nikhilesh Bhagavatula > > >> > >+uint16_t > > >> > >+rte_regexdev_enqueue_burst(uint8_t dev_id, uint16_t qp_id, > > >> > >+ struct rte_regex_ops **op

Re: [dpdk-dev] [EXT] [PATCH v1 4/4] regexdev: implement regex rte level functions

2020-04-06 Thread Thomas Monjalon
06/04/2020 14:33, Pavan Nikhilesh Bhagavatula: > >> From: Pavan Nikhilesh Bhagavatula > >> > >+uint16_t > >> > >+rte_regexdev_enqueue_burst(uint8_t dev_id, uint16_t qp_id, > >> > >+ struct rte_regex_ops **ops, uint16_t nb_ops) > >> > >+{ > >> > >+ return regex_devices

Re: [dpdk-dev] [EXT] [PATCH v1 4/4] regexdev: implement regex rte level functions

2020-04-06 Thread Pavan Nikhilesh Bhagavatula
>> From: Pavan Nikhilesh Bhagavatula >> > >+uint16_t >> > >+rte_regexdev_enqueue_burst(uint8_t dev_id, uint16_t qp_id, >> > >+struct rte_regex_ops **ops, uint16_t nb_ops) >> > >+{ >> > >+ return regex_devices[dev_id]- >> > >>enqueue(regex_devices[dev_id], qp_id, >> > >+

Re: [dpdk-dev] [EXT] [PATCH v1 4/4] regexdev: implement regex rte level functions

2020-04-06 Thread Thomas Monjalon
05/04/2020 17:04, Ori Kam: > From: Pavan Nikhilesh Bhagavatula > > >+uint16_t > > >+rte_regexdev_enqueue_burst(uint8_t dev_id, uint16_t qp_id, > > >+ struct rte_regex_ops **ops, uint16_t nb_ops) > > >+{ > > >+ return regex_devices[dev_id]- > > >>enqueue(regex_devices[dev_id], q

Re: [dpdk-dev] [EXT] [PATCH v1 4/4] regexdev: implement regex rte level functions

2020-04-05 Thread Ori Kam
Hi Pavan, > -Original Message- > From: dev On Behalf Of Pavan Nikhilesh Bhagavatula > > >> >+ > >> >+int > >> >+rte_regexdev_configure(uint8_t dev_id, const struct > >> >rte_regexdev_config *cfg) > >> >+{ > >> >+ if (dev_id >= RTE_MAX_REGEXDEV_DEVS) > >> >+ return -EINVAL; > >> >

Re: [dpdk-dev] [EXT] [PATCH v1 4/4] regexdev: implement regex rte level functions

2020-04-05 Thread Pavan Nikhilesh Bhagavatula
>> >+ >> >+int >> >+rte_regexdev_configure(uint8_t dev_id, const struct >> >rte_regexdev_config *cfg) >> >+{ >> >+ if (dev_id >= RTE_MAX_REGEXDEV_DEVS) >> >+ return -EINVAL; >> >+ if (regex_devices[dev_id] == NULL) >> >+ return -EINVAL; >> >+ if (cfg == NULL) >> >+

Re: [dpdk-dev] [EXT] [PATCH v1 4/4] regexdev: implement regex rte level functions

2020-04-05 Thread Ori Kam
Hi Pavan, Thanks for the review, PSB Thanks, Ori > -Original Message- > From: dev On Behalf Of Pavan Nikhilesh Bhagavatula > > >This commit implements all the RegEx public API. > > > >Signed-off-by: Ori Kam > >--- > > lib/librte_regexdev/rte_regexdev.c | 298 > >+++

Re: [dpdk-dev] [EXT] [PATCH v1 4/4] regexdev: implement regex rte level functions

2020-04-04 Thread Pavan Nikhilesh Bhagavatula
>This commit implements all the RegEx public API. > >Signed-off-by: Ori Kam >--- > lib/librte_regexdev/rte_regexdev.c | 298 >+ > 1 file changed, 298 insertions(+) > >diff --git a/lib/librte_regexdev/rte_regexdev.c >b/lib/librte_regexdev/rte_regexdev.c >index 439