> 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) ?

// 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

Reply via email to