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.zh...@intel.com>; dev@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.zh...@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
length.
And you can do as following to reduce the line length:
PMD_INIT_LOG(ERR,
"Failed to configure queue region mapping = %d ",
hw->aq.asq_last_status);
<...>
