Re: [PATCH v3 02/17] eventdev: increase size of enq deq conf variables
From: Jerin Jacob <hidden>
Date: 2017-02-19 12:05:24
On Fri, Feb 17, 2017 at 02:53:57PM +0000, Harry van Haaren wrote:
Large port enqueue sizes were not supported as the value it was stored in was a uint8_t. Using uint8_ts to save space in config apis makes no sense - increasing the 3 instances of uint8_t enqueue / dequeue depths to more appropriate values (based on the context around them). Signed-off-by: Harry van Haaren <redacted>
Acked-by: Jerin Jacob <redacted>
quoted hunk ↗ jump to hunk
--- lib/librte_eventdev/rte_eventdev.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-)diff --git a/lib/librte_eventdev/rte_eventdev.h b/lib/librte_eventdev/rte_eventdev.h index b0c7f9c..8b6cb7a 100644 --- a/lib/librte_eventdev/rte_eventdev.h +++ b/lib/librte_eventdev/rte_eventdev.h@@ -426,7 +426,7 @@ struct rte_event_dev_config { * This value cannot exceed the *max_event_queue_flows* which previously * provided in rte_event_dev_info_get() */ - uint8_t nb_event_port_dequeue_depth; + uint32_t nb_event_port_dequeue_depth; /**< Maximum number of events can be dequeued at a time from an * event port by this device. * This value cannot exceed the *max_event_port_dequeue_depth*@@ -637,12 +637,12 @@ struct rte_event_port_conf { * which was previously supplied to rte_event_dev_configure(). * This should be set to '-1' for *open system*. */ - uint8_t dequeue_depth; + uint16_t dequeue_depth; /**< Configure number of bulk dequeues for this event port. * This value cannot exceed the *nb_event_port_dequeue_depth* * which previously supplied to rte_event_dev_configure() */ - uint8_t enqueue_depth; + uint16_t enqueue_depth; /**< Configure number of bulk enqueues for this event port. * This value cannot exceed the *nb_event_port_enqueue_depth* * which previously supplied to rte_event_dev_configure()-- 2.7.4