On 7 Oct 2022, at 16:39, Davide Caratti wrote:
> On Fri, Oct 7, 2022 at 3:21 PM Marcelo Leitner <[email protected]> wrote:
>>
>> (+TC folks and netdev@)
>>
>> On Fri, Oct 07, 2022 at 02:42:56PM +0200, Ilya Maximets wrote:
>>> On 10/7/22 13:37, Eelco Chaudron wrote:
>
> [...]
>
>> I don't see how we could achieve this without breaking much of the
>> user experience.
>>
>>>
>>> - or create something like act_count - a dummy action that only
>>> counts packets, and put it in every datapath action from OVS.
>>
>> This seems the easiest and best way forward IMHO. It's actually the
>> 3rd option below but "on demand", considering that tc will already use
>> the stats of the first action as the flow stats (in
>> tcf_exts_dump_stats()), then we can patch ovs to add such action if a
>> meter is also being used (or perhaps even always, because other
>> actions may also drop packets, and for OVS we would really be at the
>> 3rd option below).
Guess we have to add this extra action to each datapath flow offloaded due to
the way flows back and forth translations are handled. Maybe we can do it
selectively, but the code might become messier than it already is :(
> Correct me if I'm wrong, but actually act_gact action with "pipe"
> control action should already do this counting job.
I think we could use that, as we only use TC_ACT_GOTO_CHAIN and TC_ACT_SHOT.
And it looks like TC_ACT_SHOT is not decoded correctly :(
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
