[ovs-dev] [PATCH V3 01/12] netdev-offload-dpdk: Remove pre-validate of patterns function

[Eli Britstein]

On 6/22/2020 10:33 AM, Eli Britstein wrote:
>
> On 6/22/2020 10:04 AM, Ilya Maximets wrote:
>> On 6/21/20 1:19 PM, Eli Britstein wrote:
>>> The function of adding patterns by requested matches checks that it
>>> consumed all the required matches, and err if not. This nullify the
>>> purpose of the validation function. Future supported matches will only
>>> change the pattern parsing code.
>> I think that validation function here is to speed up the process in case
>> of unsupported offloading patterns.  It saves time for all the not 
>> needed
>> memory allocations and copies.  With this change we will construct 
>> all the
>> patterns and only in the end will realize that we don't need them as the
>> flow can not be offloaded.
> I know, and agree. However, the "save time" is negligible, comparing 
> to more complicated code.
Another note: looking from the point of view of the "good" flows, this 
validation, as fast as it is, is redundant.
>
>>
>> Best regards, Ilya Maximets.