On 24 Sep 2025, at 4:44, Ratheesh Kannoth wrote:
>> From: Eelco Chaudron <mailto:[email protected]>
>
>> The revalidator threads do not query individual flows; they will use the
>> dump APIs, so for tc, they will be the netdev_tc_flow_dump_XXX() functions.
>
> Offloaded interfaces are eth0 and sdp1-0 and its stats are incremented at HW
> level (no driver involved), but flow_dump_create() request is for br0.
> Marvell offload driver code is not updating br0 stats (br0 rx and tx are 0).
>
> So, driver code should update br0 stats (rx and tx) ?
Yes, the tc driver should return stats for each rule installed. You can use the
tc command to see the installed rules in the kernel and the stats.
> // below prints are generated by debug prints added by me
>
> Sep 24 02:28:43 localhost.localdomain ovs-vswitchd[408]:
> ovs|00014|netdev_offload_tc(handler33)|ERR|netdev_tc_flow_put:2338 add rule
> eth0
> Sep 24 02:28:43 localhost.localdomain ovs-vswitchd[408]:
> ovs|00009|netdev_offload_tc(handler32)|ERR|netdev_tc_flow_put:2338 add rule
> sdp1-0
> Sep 24 02:28:43 localhost.localdomain ovs-vswitchd[408]:
> ovs|00039|netdev_offload_tc(revalidator35)|ERR|Dropped 15 log messages in
> last 1 seconds (most recently, 0 seconds ago) due to excessive rate
> Sep 24 02:28:43 localhost.localdomain ovs-vswitchd[408]:
> ovs|00040|netdev_offload_tc(revalidator35)|ERR|netdev_tc_flow_dump_create:607
> called for br0
> Sep 24 02:28:44 localhost.localdomain ovs-vswitchd[408]:
> ovs|00041|netdev_offload_tc(revalidator35)|ERR|Dropped 15 log messages in
> last 1 seconds (most recently, 0 seconds ago) due to excessive rate
> Sep 24 02:28:44 localhost.localdomain ovs-vswitchd[408]:
> ovs|00042|netdev_offload_tc(revalidator35)|ERR|netdev_tc_flow_dump_create:607
> called for br0
_______________________________________________
discuss mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-discuss
