On 5/31/2023 11:23 PM, Jakub Kicinski wrote:

On Tue, 30 May 2023 16:45:01 -0700 Tony Nguyen wrote:

quoted

+	struct idpf_vport *vport = idpf_netdev_to_vport(netdev);
+
+	if (!vport)
+		return;
+
+	*stats = vport->netstats;

How is this atomic vs the update path.

I see your point. 'vport->netstats' can go out of sync if the statistics 
task updates the 'netstats' concurrently. Will guard the stats update 
with a lock. Thanks for the review.

Regards,
Pavan