On Tue, Sep 27, 2022 at 8:36 PM Abdullah Sevincer <abdullah.sevin...@intel.com> wrote: > > This patch addresses an issue of enqueuing more than > max_enq_depth and not able to dequeuing events equal > to max_cq_depth in a single call of rte_event_enqueue_burst > and rte_event_dequeue_burst. > > Apply fix for restricting enqueue of events to max_enq_depth > so that in a single rte_event_enqueue_burst() call at most > max_enq_depth events are enqueued. > > Also set per port and domain history list sizes based on > cq_depth. This results in dequeuing correct number of > events as set by max_cq_depth. > > Signed-off-by: Abdullah Sevincer <abdullah.sevin...@intel.com>
Fix below issues [for-main]dell[dpdk-next-eventdev] $ ./devtools/check-git-log.sh -n 1 Wrong headline format: event/dlb2: fix max cq_depth/enq_depth cli override grep: warning: stray \ before white space Missing 'Fixes' tag: event/dlb2: fix max cq_depth/enq_depth cli override Invalid patch(es) found - checked 1 patch [for-main]dell[dpdk-next-eventdev] $ > --- > drivers/event/dlb2/dlb2.c | 9 +++++---- > 1 file changed, 5 insertions(+), 4 deletions(-) > > diff --git a/drivers/event/dlb2/dlb2.c b/drivers/event/dlb2/dlb2.c > index 759578378f..dbb8284135 100644 > --- a/drivers/event/dlb2/dlb2.c > +++ b/drivers/event/dlb2/dlb2.c > @@ -813,7 +813,7 @@ dlb2_hw_create_sched_domain(struct dlb2_eventdev *dlb2, > cfg->num_ldb_queues; > > cfg->num_hist_list_entries = resources_asked->num_ldb_ports * > - DLB2_NUM_HIST_LIST_ENTRIES_PER_LDB_PORT; > + evdev_dlb2_default_info.max_event_port_dequeue_depth; > > if (device_version == DLB2_HW_V2_5) { > DLB2_LOG_DBG("sched domain create - ldb_qs=%d, ldb_ports=%d, > dir_ports=%d, atomic_inflights=%d, hist_list_entries=%d, credits=%d\n", > @@ -1538,7 +1538,7 @@ dlb2_hw_create_ldb_port(struct dlb2_eventdev *dlb2, > cfg.cq_depth = rte_align32pow2(dequeue_depth); > cfg.cq_depth_threshold = 1; > > - cfg.cq_history_list_size = DLB2_NUM_HIST_LIST_ENTRIES_PER_LDB_PORT; > + cfg.cq_history_list_size = cfg.cq_depth; > > cfg.cos_id = ev_port->cos_id; > cfg.cos_strict = 0;/* best effots */ > @@ -2966,6 +2966,7 @@ __dlb2_event_enqueue_burst(void *event_port, > struct dlb2_port *qm_port = &ev_port->qm_port; > struct process_local_port_data *port_data; > int retries = ev_port->enq_retries; > + int num_tx; > int i; > > RTE_ASSERT(ev_port->enq_configured); > @@ -2974,8 +2975,8 @@ __dlb2_event_enqueue_burst(void *event_port, > i = 0; > > port_data = &dlb2_port[qm_port->id][PORT_TYPE(qm_port)]; > - > - while (i < num) { > + num_tx = RTE_MIN(num, ev_port->conf.enqueue_depth); > + while (i < num_tx) { > uint8_t sched_types[DLB2_NUM_QES_PER_CACHE_LINE]; > uint8_t queue_ids[DLB2_NUM_QES_PER_CACHE_LINE]; > int pop_offs = 0; > -- > 2.25.1 >