Re: [dpdk-dev] [PATCH] net/ice: remove bulk alloc compile option

2020-03-07 Thread Ye Xiaolong
On 03/06, Ye Xiaolong wrote: >Hi, Qi > >Thanks for the cleanup. > >On 03/04, Qi Zhang wrote: >>Remove CONFIG_RTE_LIBRTE_ICE_RX_ALLOW_BULK_ALLOC with below >>consideration: >> >>1. a default Rx path can always be selected by setting a proper >> rx_free_thresh value at runtime, see >> ice_check_r

Re: [dpdk-dev] [PATCH] net/ice: remove bulk alloc compile option

2020-03-06 Thread Zhang, Qi Z
> -Original Message- > From: Ye, Xiaolong > Sent: Friday, March 6, 2020 4:46 PM > To: Zhang, Qi Z > Cc: Xing, Beilei ; dev@dpdk.org; sta...@dpdk.org > Subject: Re: [PATCH] net/ice: remove bulk alloc compile option > > Hi, Qi > > Thanks for the cleanup. > > On 03/04, Qi Zhang wrote:

Re: [dpdk-dev] [PATCH] net/ice: remove bulk alloc compile option

2020-03-06 Thread Ye Xiaolong
Hi, Qi Thanks for the cleanup. On 03/04, Qi Zhang wrote: >Remove CONFIG_RTE_LIBRTE_ICE_RX_ALLOW_BULK_ALLOC with below >consideration: > >1. a default Rx path can always be selected by setting a proper > rx_free_thresh value at runtime, see > ice_check_rx_burst_bulk_alloc_preconditions. > >2.

[dpdk-dev] [PATCH] net/ice: remove bulk alloc compile option

2020-03-04 Thread Qi Zhang
Remove CONFIG_RTE_LIBRTE_ICE_RX_ALLOW_BULK_ALLOC with below consideration: 1. a default Rx path can always be selected by setting a proper rx_free_thresh value at runtime, see ice_check_rx_burst_bulk_alloc_preconditions. 2. its not a big deal to always reserve more space for desc ring. "

[dpdk-dev] [PATCH] net/ice: remove bulk alloc compile option

2020-03-04 Thread Qi Zhang
Remove CONFIG_RTE_LIBRTE_ICE_RX_ALLOW_BULK_ALLOC with below consideration: 1. a default Rx path can always be selected by setting a proper rx_free_thresh value at runtime, see ice_check_rx_burst_bulk_alloc_preconditions. 2. its not a big deal to always reserve more space for desc ring. "