Re: [PATCH v3 02/17] eventdev: increase size of enq deq conf variables

[Jerin Jacob <jerin.jacob@caviumnetworks.com>]

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 <harry.van.haaren@intel.com>

Acked-by: Jerin Jacob <jerin.jacob@caviumnetworks.com>

> ---
>  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
>