From: Jiri Pirko <j...@resnulli.us> Date: Thu, 16 Jun 2016 10:37:13 +0200
> Until now we had stats functions return SW statistics. However, it makes > a lot of sense to return HW stats as default. The existing apps count with > having the defaults stats complete, but that is not true now as the offloaded > forward traffic is not visible there. > > If user wants to know real SW stats, this patchset provides way to get > it as well. I think we haven't been very good about defining good rules nor guidelines for how to handle HW vs SW stats, and I'm talking strictly about what we publish via rtnl_link_stats64. However, if I were working from scratch on a new driver what I would be inclined to do is use HW stats for everything that the chip provides direct and accurate support for, and fill in the gaps with SW stats. Because to me, stats are stats, the user wants to know (for example) how many broadcast packets have gone through the port and doesn't care how you obtain that number. If the problem being addressed is that drivers aren't reporting information on all the packets going through the device, then that's a bug. But it seems to me like mlxsw is already maintaining the software statistic counters, so I can't see a performance reason for not properly providing all of the statistics using HW vs. SW as is appropriate for each and every value to fix this problem. Why create an entire new facility just for that? It doesn't seem to be needed. Maybe you just need to describe things a bit more completely in this header posting.
