On 5/31/2023 11:23 PM, Jakub Kicinski wrote:
On Tue, 30 May 2023 16:45:01 -0700 Tony Nguyen wrote:+ 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