On 12 Dec 2024, at 15:46, Eelco Chaudron wrote:
> On 12 Dec 2024, at 15:13, Ilya Maximets wrote:
>
>> On 12/12/24 14:53, Eelco Chaudron wrote:
>>>
>>>
>>> On 11 Dec 2024, at 11:32, Eelco Chaudron wrote:
>>>
>>>> On 5 Dec 2024, at 15:50, Ales Musil wrote:
>>>>
>>>>> The system and netdev datapath have different default pmd_id, which
>>>>> resulted in empty output of ofproto/detrace with kernel datapath.
>>>>> Also indicate that the UFID or cache wasn't available.
>>>>>
>>>>> Make sure we use the correct default pmd_id when it's not specified
>>>>> as an argument. At the same time move slightly adjusted test into
>>>>> system tests so it is tested with both datapaths.
>>>>>
>>>>> Fixes: 600125b2c380 ("ofproto: Add ofproto/detrace command to map UFIDs
>>>>> to OpenFlow.")
>>>>> Signed-off-by: Ales Musil <[email protected]>
>>>>
>>>> The changes look good to me. One small comment from Ilya you missed, which
>>>> we can apply on commit.
>>>>
>>>> Acked-by: Eelco Chaudron <[email protected]>
>>>
>>> Thanks Ales,
>>>
>>> With the nit fixed below applied to main.
>>
>> Thanks! Do you plan to also backport this?
>
> The fix fixed tag only occurs on main, so I was not planning it (or did I
> miss something)?
>
> $ git branch --contains 600125b2c380
> * main
Ignore it, it was my branch history :) I will backport to 3.4.
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
