On 2022-07-12 20:11, Pavan Nikhilesh Bhagavatula wrote: > +Cc > [email protected]; > [email protected]; > [email protected]; > [email protected]; > [email protected]; > [email protected]; > [email protected]; > [email protected]; > [email protected]; > [email protected]; > [email protected]; > [email protected]; > [email protected]; > > >> -----Original Message----- >> From: Thomas Monjalon <[email protected]> >> Sent: Tuesday, July 12, 2022 8:31 PM >> To: Pavan Nikhilesh Bhagavatula <[email protected]> >> Cc: Jerin Jacob Kollanukkaran <[email protected]>; Ray Kinsella >> <[email protected]>; [email protected]; Pavan Nikhilesh Bhagavatula >> <[email protected]> >> Subject: [EXT] Re: [PATCH 1/2] doc: add enqueue depth for new event type >> >> External Email >> >> ---------------------------------------------------------------------- >> I'm not your teacher, but please consider Cc'ing people outside of your >> company. >> > > I generally add --to-cmd=./devtools/get-maintainer.sh but looks like it's > useless for > sending deprecation notices. > > Maybe we should update it to include lib/driver maintainers when diff sees > deprecation.rst > >> >> 27/06/2022 11:57, [email protected]: >>> From: Pavan Nikhilesh <[email protected]> >>> >>> A new field ``max_event_port_enqueue_new_burst`` will be added to the >>> structure ``rte_event_dev_info``. The field defines the max enqueue >>> burst size of new events (OP_NEW) supported by the underlying event >>> device. >>> >>> Signed-off-by: Pavan Nikhilesh <[email protected]> >>> --- >>> doc/guides/rel_notes/deprecation.rst | 5 +++++ >>> 1 file changed, 5 insertions(+) >>> >>> diff --git a/doc/guides/rel_notes/deprecation.rst >> b/doc/guides/rel_notes/deprecation.rst >>> index 4e5b23c53d..071317e8e3 100644 >>> --- a/doc/guides/rel_notes/deprecation.rst >>> +++ b/doc/guides/rel_notes/deprecation.rst >>> @@ -125,3 +125,8 @@ Deprecation Notices >>> applications should be updated to use the ``dmadev`` library instead, >>> with the underlying HW-functionality being provided by the ``ioat`` or >>> ``idxd`` dma drivers >>> + >>> +* eventdev: The structure ``rte_event_dev_info`` will be extended to >> include the >>> + max enqueue burst size of new events supported by the underlying >> event device. >>> + A new field ``max_event_port_enqueue_new_burst`` will be added to >> the structure >>> + ``rte_event_dev_info`` in DPDK 22.11. >>> >> >> >> >>
Why is this needed, or useful? Why isn't called something with "enqueue_depth" in it, like the already-present field? I think I would rather remove all fields related to the max enqueue/dequeue burst sizes. They serve no purpose, as far as I see. If you have some HW limit on the maximum number of new events it can accept, have the driver retry until backpressure occurs.
