Re: [PATCH v4 1/3] lib: introduce dispatcher library

2023-10-03 Thread Jerin Jacob
On Wed, Sep 27, 2023 at 1:43 PM Bruce Richardson wrote: > > On Tue, Sep 26, 2023 at 11:58:37PM +0530, Jerin Jacob wrote: > > On Mon, Sep 25, 2023 at 12:41 PM Mattias Rönnblom > > wrote: > > > > > > On 2023-09-22 09:38, Mattias Rönnblom wrote: > > > > > > > > > > > > > +int > > > > +rte_dispatch

Re: [PATCH v4 1/3] lib: introduce dispatcher library

2023-09-28 Thread Mattias Rönnblom
On 2023-09-27 10:13, Bruce Richardson wrote: On Tue, Sep 26, 2023 at 11:58:37PM +0530, Jerin Jacob wrote: On Mon, Sep 25, 2023 at 12:41 PM Mattias Rönnblom wrote: On 2023-09-22 09:38, Mattias Rönnblom wrote: +int +rte_dispatcher_create(uint8_t id, uint8_t event_dev_id) +{ There are two

Re: [PATCH v4 1/3] lib: introduce dispatcher library

2023-09-27 Thread Bruce Richardson
On Tue, Sep 26, 2023 at 11:58:37PM +0530, Jerin Jacob wrote: > On Mon, Sep 25, 2023 at 12:41 PM Mattias Rönnblom > wrote: > > > > On 2023-09-22 09:38, Mattias Rönnblom wrote: > > > > > > > > > +int > > > +rte_dispatcher_create(uint8_t id, uint8_t event_dev_id) > > > +{ > > > > > > There are two

Re: [PATCH v4 1/3] lib: introduce dispatcher library

2023-09-26 Thread Jerin Jacob
On Mon, Sep 25, 2023 at 12:41 PM Mattias Rönnblom wrote: > > On 2023-09-22 09:38, Mattias Rönnblom wrote: > > > > > +int > > +rte_dispatcher_create(uint8_t id, uint8_t event_dev_id) > > +{ > > > There are two changes I'm considering: > > 1) Removing the "id" to identify the dispatcher, replacing

Re: [PATCH v4 1/3] lib: introduce dispatcher library

2023-09-25 Thread Bruce Richardson
On Mon, Sep 25, 2023 at 09:11:38AM +0200, Mattias Rönnblom wrote: > On 2023-09-22 09:38, Mattias Rönnblom wrote: > > > > > +int > > +rte_dispatcher_create(uint8_t id, uint8_t event_dev_id) > > +{ > > > There are two changes I'm considering: > > 1) Removing the "id" to identify the dispatcher,

Re: [PATCH v4 1/3] lib: introduce dispatcher library

2023-09-25 Thread Mattias Rönnblom
On 2023-09-22 09:38, Mattias Rönnblom wrote: +int +rte_dispatcher_create(uint8_t id, uint8_t event_dev_id) +{ There are two changes I'm considering: 1) Removing the "id" to identify the dispatcher, replacing it with an forward-declared rte_dispatcher struct pointer. struct rte_dispatche

[PATCH v4 1/3] lib: introduce dispatcher library

2023-09-22 Thread Mattias Rönnblom
The purpose of the dispatcher library is to help reduce coupling in an Eventdev-based DPDK application. In addition, the dispatcher also provides a convenient and flexible way for the application to use service cores for application-level processing. Signed-off-by: Mattias Rönnblom Tested-by: Pe