On 12/22/2016 9:19 AM, Zhao1, Wei wrote:
> Hi, Yigit
>
>> -----Original Message-----
>> From: Yigit, Ferruh
>> Sent: Wednesday, December 21, 2016 1:01 AM
>> To: Zhao1, Wei <wei.zh...@intel.com>; dev@dpdk.org
>> Cc: Lu, Wenzhuo <wenzhuo...@intel.com>
>> Subject: Re: [dpdk-dev] [PATCH 15/18] net/ixgbe: parse flow director filter
>>
>> On 12/2/2016 10:43 AM, Wei Zhao wrote:
>>> From: wei zhao1 <wei.zh...@intel.com>
>>>
>>> check if the rule is a flow director rule, and get the flow director info.
>>>
>>> Signed-off-by: wei zhao1 <wei.zh...@intel.com>
>>> Signed-off-by: Wenzhuo Lu <wenzhuo...@intel.com>
>>> ---
>>
>> <...>
>>
>>> + PATTERN_SKIP_VOID(rule, struct ixgbe_fdir_rule,
>>> + RTE_FLOW_ERROR_TYPE_ITEM_NUM);
>>> + if (item->type != RTE_FLOW_ITEM_TYPE_ETH &&
>>> + item->type != RTE_FLOW_ITEM_TYPE_IPV4 &&
>>> + item->type != RTE_FLOW_ITEM_TYPE_IPV6 &&
>>> + item->type != RTE_FLOW_ITEM_TYPE_UDP &&
>>> + item->type != RTE_FLOW_ITEM_TYPE_VXLAN &&
>>> + item->type != RTE_FLOW_ITEM_TYPE_NVGRE) {
>>
>> This gives build error [1], there are a few more same usage:
>>
>> .../drivers/net/ixgbe/ixgbe_ethdev.c:9238:17: error: comparison of constant
>> 241 with expression of type 'const enum rte_flow_item_type' is always true
>> [-Werror,-Wtautological-constant-out-of-range-compare]
>> item->type != RTE_FLOW_ITEM_TYPE_NVGRE) {
>>
>>
>>
>
> Ok, I will add two type definition RTE_FLOW_ITEM_TYPE_NVGRE and
> RTE_FLOW_ITEM_TYPE_E_TAG into const enum rte_flow_item_type to eliminate
> this problem.
> Thank you.
>
CC: Adrien Mazarguil <adrien.mazarg...@6wind.com>
Yes, that is what right thing to do, since rte_flow patchset not merged
yet, perhaps Adrien may want to include this as next version of his
patchset?
What do you think Adrien?
Thanks,
ferruh
