On 15 Dec 2022, at 12:45, Ilya Maximets wrote:
> On 12/14/22 17:33, Eelco Chaudron wrote:
>> The commit mentioned below causes the ovs_flow_tbl_lookup() function
>> to be called with the masked key. However, it's supposed to be called
>> with the unmasked key.
>
> Hi, Eelco. Thanks for the fix!
>
> Could you, please, add more information to the commit message on
> why this is a problem, with some examples? This will be useful
> for someone in the future trying to understand why we actually
> have to use an unmasked key here.
ACK will add the same description as in the OVS dpif-netdev commit.
> Also, I suppose, 'Cc: [email protected]' tag is needed in the
> commit message since it's a fix for a bug that is actually impacts
> users and needs to be backported.
Will do in the v3.
> Best regards, Ilya Maximets.
>
>>
>> This change reverses the commit below, but rather than having the key
>> on the stack, it's allocated.
>>
>> Fixes: 190aa3e77880 ("openvswitch: Fix Frame-size larger than 1024 bytes
>> warning.")
>>
>> Signed-off-by: Eelco Chaudron <[email protected]>
>>
>> ---
>> Version history:
>> - v2: Fixed ENOME(N/M) error. Forgot to do a stg refresh.
>>
>> net/openvswitch/datapath.c | 25 ++++++++++++++++---------
>> 1 file changed, 16 insertions(+), 9 deletions(-)
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
