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?
Best regards, Ilya Maximets.
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
