On 5/22/2018 11:17 PM, Thomas Monjalon wrote:
> 12/12/2017 11:05, Nikhil Agarwal:
>> Currently, if the rte_eth_rx_burst() function returns a value less than
>> *nb_pkts*, the application will assume that no more packets are present.
>>
>> Some of the hw queue based hardware can only support smaller burst for RX
>> and TX and thus break the expectation of the rx_burst API.
>>
>> This patch adds support to provide the maximum burst size that can be
>> supported by a given PMD. The dev_info is being memset to '0' in
>> rte_ethdev library. The value of '0' indicates that any value for burst
>> size can be supported i.e. no change for existing PMDs.
>>
>> The application can now use the lowest available max_burst_size value
>> for rte_eth_rx_burst.
>>
>> Signed-off-by: Nikhil Agarwal <nikhil.agar...@linaro.org>
>> ---
>> @@ -1047,6 +1047,7 @@ struct rte_eth_dev_info {
>> /** Configured number of rx/tx queues */
>> uint16_t nb_rx_queues; /**< Number of RX queues. */
>> uint16_t nb_tx_queues; /**< Number of TX queues. */
>> + uint16_t max_burst_size; /**< MAX burst size, 0 for no limit. */
>> };
>
> What is the status of this proposal?
>
> Recently, the preferred tuning have been added by
> "ethdev: support PMD-tuned Tx/Rx parameters"
> http://dpdk.org/commit/3be82f5cc5
Hi Nikhil, Hemant,
PMD returning preferred 'burst_size' support already added, I guess this
patchset is no more valid. I am updating this as rejected.
If something is missing or there are still some relevant pieces in this patch,
please send as a new version on top of latest head.
Thanks,
ferruh
