diff options
author | Doug Berger <opendmb@gmail.com> | 2024-04-26 01:27:21 +0300 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2024-05-17 13:15:04 +0300 |
commit | 74022f08e03da2dbf61bd7dd5dcffcc1f0cfbc58 (patch) | |
tree | 8c1cbf0659bee8659cef0fd78e08ad68974e1080 /lib/dynamic_debug.c | |
parent | da108f4d4291d87b20b0c88eb13976cc8ca9e407 (diff) | |
download | linux-74022f08e03da2dbf61bd7dd5dcffcc1f0cfbc58.tar.xz |
net: bcmgenet: synchronize UMAC_CMD access
commit 0d5e2a82232605b337972fb2c7d0cbc46898aca1 upstream.
The UMAC_CMD register is written from different execution
contexts and has insufficient synchronization protections to
prevent possible corruption. Of particular concern are the
acceses from the phy_device delayed work context used by the
adjust_link call and the BH context that may be used by the
ndo_set_rx_mode call.
A spinlock is added to the driver to protect contended register
accesses (i.e. reg_lock) and it is used to synchronize accesses
to UMAC_CMD.
Fixes: 1c1008c793fa ("net: bcmgenet: add main driver file")
Cc: stable@vger.kernel.org
Signed-off-by: Doug Berger <opendmb@gmail.com>
Acked-by: Florian Fainelli <florian.fainelli@broadcom.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'lib/dynamic_debug.c')
0 files changed, 0 insertions, 0 deletions