[dpdk-dev] [PATCH v3 1/2] net/i40e: queue region set and flush

Ferruh Yigit ferruh.yigit at intel.com
Thu Sep 21 09:10:29 CEST 2017

On 9/21/2017 7:48 AM, Zhao1, Wei wrote:
> Hi,  Ferruh
>> -----Original Message-----
>> From: Yigit, Ferruh
>> Sent: Wednesday, September 20, 2017 6:36 PM
>> To: Zhao1, Wei <wei.zhao1 at intel.com>; dev at dpdk.org
>> Subject: Re: [dpdk-dev] [PATCH v3 1/2] net/i40e: queue region set and flush
>> On 9/15/2017 4:13 AM, Wei Zhao wrote:
>>> This feature enable queue regions configuration for RSS in PF/VF, so
>>> that different traffic classes or different packet classification
>>> types can be separated to different queues in different queue
>>> regions.This patch can set queue region range, it include queue number
>>> in a region and the index of first queue.
>> Is following correct:
>> So instead of distributing packets to the multiple queues, this will distribute
>> packets into queue reqions which may consists of multiple queues.
>> If so, is there a way to control how packets distributed within same queue
>> region to multiple queues?
>> And is this feature only supported with RSS? Can it be part of RSS
>> configuration instead of PMD specific API?
>>> This patch enable mapping between different priorities (UP) and
>> User priorities (UP)
>>> different traffic classes.It also enable mapping between a region
>>> index and a sepcific flowtype(PCTYPE).It also provide the solution of
>>> flush all configuration about queue region the above described.
>>> Signed-off-by: Wei Zhao <wei.zhao1 at intel.com>


>>> +	ret = i40e_aq_update_vsi_params(hw, &ctxt, NULL);
>>> +	if (ret) {
>>> +		PMD_INIT_LOG(ERR, "Failed to configure queue region "
>>> +			"mapping = %d ", hw->aq.asq_last_status);
>> Please don't split log message.
> If I do not spli it , the line will has more 80 character, there will be warning in patch check log!
> So, which way do you think will be better? Split or warning log?

Checkpatch detects log lines and shouldn't complain about them.

Even it does I thinks checkpatch warning is better, assuming reasonable

And you can do as following to reduce the line length:

	"Failed to configure queue region mapping = %d ", 	


More information about the dev mailing list