On Fri, Oct 07, 2022 at 04:39:25PM +0200, 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). > > Correct me if I'm wrong, but actually act_gact action with "pipe" > control action should already do this counting job.
act_gact is so transparent that I never see it/remembers about it :) Yup, although it's not offloadabe with pipe control actio AFAICT. Marcelo _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
