Thread (30 messages) flat view 30 messages, 3 authors, 2021-01-18

Re: [PATCH v6 net-next 03/15] net: procfs: hold netif_lists_lock when retrieving device statistics

From: Saeed Mahameed <saeed@kernel.org>
Date: 2021-01-12 21:48:24

On Tue, 2021-01-12 at 15:44 +0200, Vladimir Oltean wrote:
On Mon, Jan 11, 2021 at 03:46:32PM -0800, Saeed Mahameed wrote:
quoted
This can be very costly, holding a mutex while traversing the whole
netedv lists and reading their stats, we need to at least allow
multiple readers to enter as it was before, so maybe you want to
use
rw_semaphore instead of the mutex.

or just have a unified approach of rcu+refcnt/dev_hold as you did
for
bonding and failover patches #13..#14, I used the same approach to
achieve the same for sysfs and procfs more than 2 years ago, you
are
welcome to use my patches:
https://lore.kernel.org/netdev/4cc44e85-cb5e-502c-30f3-c6ea564fe9ac@gmail.com/ (local)
Ok, what mail address do you want me to keep for your sign off?
Either is fine. Just make sure author and signed-off are the same :).
Thanks!

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help