On 28/11/2022 13:11, Marcelo Leitner wrote: > Exactly. Then, when this patchset (or similar) come up, it won't > update all actions with the same stats anymore. It will require a set > of stats from hw for the gact with PIPE action here. But if drivers > are ignoring this action, they can't have specific stats for it. Or am > I missing something? > > So it is better for the drivers to reject the whole flow instead of > simply ignoring it, and let vswitchd probe if it should or should not > use this action.
I agree. Drivers should only accept a flow with a 'gact pipe' action if they are able to perform the stats offload it requests. Getting userland to autodiscover whether it can make use of this is a SMOP, better than having it always try to use it and then sometimes get confused that the resulting stats are wrong or meaningless. -ed _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
