Re: [PATCH net v2 7/9] net: aquantia: Reset driver level statistics to zero on initialization
flat view
From: Andrew Lunn <andrew@lunn.ch>
Date: 2017-12-11 15:51:59
quoted hunk ↗ jump to hunk
--- a/drivers/net/ethernet/aquantia/atlantic/hw_atl/hw_atl_a0.c +++ b/drivers/net/ethernet/aquantia/atlantic/hw_atl/hw_atl_a0.c@@ -344,6 +344,13 @@ static int hw_atl_a0_hw_init(struct aq_hw_s *self, hw_atl_a0_hw_rss_set(self, &aq_nic_cfg->aq_rss); hw_atl_a0_hw_rss_hash_set(self, &aq_nic_cfg->aq_rss); + /* Read initial hardware counters + * and reset current in-driver statistics + */ + hw_atl_utils_update_stats(self); + memset(&PHAL_ATLANTIC_A0->curr_stats, 0, + sizeof(PHAL_ATLANTIC_A0->curr_stats)); + err = aq_hw_err_from_flags(self); if (err < 0) goto err_exit;
Hi Igor
It looks like this is called on pm state change? Does this mean that
resume will zero the statistics? That is probably not what you want.
You should probably only clear the statics in the probe function.
Andrew