]> git.baikalelectronics.ru Git - kernel.git/commit
Merge branch 'u64_stats-fixups'
authorDavid S. Miller <davem@davemloft.net>
Mon, 29 Aug 2022 12:02:27 +0000 (13:02 +0100)
committerDavid S. Miller <davem@davemloft.net>
Mon, 29 Aug 2022 12:02:27 +0000 (13:02 +0100)
commit6f5d6e7876323ccc7335f4b6fdbef5f01f952026
treefd0bbea44ebff1928f93194f3cd6eb40d3bab4a1
parent12483d0470abdf70b074dd3b29f030cf07deba8d
parent3919fc06a41b547e0a73c6f881920a98d62fb039
Merge branch 'u64_stats-fixups'

Sebastian Andrzej Siewior says:

====================
net: u64_stats fixups for 32bit.

while looking at the u64-stats patch
https://lore.kernel.org/all/20220817162703.728679-10-bigeasy@linutronix.de

I noticed that u64_stats_fetch_begin() is used. That suspicious thing
about it is that network processing, including stats update, is
performed in NAPI and so I would expect to see
u64_stats_fetch_begin_irq() in order to avoid updates from NAPI during
the read. This is only needed on 32bit-UP where the seqcount is not
used. This is address in 2/2. The remaining user take some kind of
precaution and may use u64_stats_fetch_begin().

I updated the previously mentioned patch to get rid of
u64_stats_fetch_begin_irq(). If this is not considered stable patch
worthy then it can be ignored and considred fixed by the other series
which removes the special 32bit cases.

The xrs700x driver reads and writes the counter from preemptible context
so the only missing piece here is at least disable preemption on the
writer side to avoid preemption while the writer is in progress. The
possible reader would spin then until the writer completes its write
critical section which is considered bad. This is addressed in 1/2 by
using u64_stats_update_begin_irqsave() and so disable interrupts during
the write critical section.
The other closet resemblance I found is mdio_bus.c::mdiobus_stats_acct()
where preemtion is disabled unconditionally. This is something I want to
avoid since it also affects 64bit.
====================

Signed-off-by: David S. Miller <davem@davemloft.net>