summaryrefslogtreecommitdiff
path: root/sys/dev/hyperv
diff options
context:
space:
mode:
authornonaka <nonaka@NetBSD.org>2020-05-24 10:31:59 +0000
committernonaka <nonaka@NetBSD.org>2020-05-24 10:31:59 +0000
commitd286de11b5e4cb0c7bc012d0fa621257ec167235 (patch)
tree8bc1e158f809ca22de1c5910d9d3e6a27852c71b /sys/dev/hyperv
parent5760fea4525555a21901e91522263b77d3f92405 (diff)
hvn(4): Prevent from occur panic when acquiring the media status.
hvn(4) sleeps during hvn_media_status(), and thus requires an adaptive mutex for the media lock. Taken from recent iwm(4) change.
Diffstat (limited to 'sys/dev/hyperv')
-rw-r--r--sys/dev/hyperv/if_hvn.c16
1 files changed, 11 insertions, 5 deletions
diff --git a/sys/dev/hyperv/if_hvn.c b/sys/dev/hyperv/if_hvn.c
index 9b0d7b80448..c9aa3573919 100644
--- a/sys/dev/hyperv/if_hvn.c
+++ b/sys/dev/hyperv/if_hvn.c
@@ -1,4 +1,4 @@
-/* $NetBSD: if_hvn.c,v 1.17 2020/02/04 05:25:38 thorpej Exp $ */
+/* $NetBSD: if_hvn.c,v 1.18 2020/05/24 10:31:59 nonaka Exp $ */
/* $OpenBSD: if_hvn.c,v 1.39 2018/03/11 14:31:34 mikeb Exp $ */
/*-
@@ -35,7 +35,7 @@
*/
#include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: if_hvn.c,v 1.17 2020/02/04 05:25:38 thorpej Exp $");
+__KERNEL_RCSID(0, "$NetBSD: if_hvn.c,v 1.18 2020/05/24 10:31:59 nonaka Exp $");
#ifdef _KERNEL_OPT
#include "opt_inet.h"
@@ -133,6 +133,7 @@ struct hvn_softc {
struct ethercom sc_ec;
struct ifmedia sc_media;
+ kmutex_t sc_media_lock; /* XXX */
struct if_percpuq *sc_ipq;
int sc_link_state;
int sc_promisc;
@@ -294,8 +295,10 @@ hvn_attach(device_t parent, device_t self, void *aux)
/* Initialize ifmedia structures. */
sc->sc_ec.ec_ifmedia = &sc->sc_media;
- ifmedia_init(&sc->sc_media, IFM_IMASK, hvn_media_change,
- hvn_media_status);
+ /* XXX media locking needs revisiting */
+ mutex_init(&sc->sc_media_lock, MUTEX_DEFAULT, IPL_SOFTNET);
+ ifmedia_init_with_lock(&sc->sc_media, IFM_IMASK,
+ hvn_media_change, hvn_media_status, &sc->sc_media_lock);
ifmedia_add(&sc->sc_media, IFM_ETHER | IFM_MANUAL, 0, NULL);
ifmedia_set(&sc->sc_media, IFM_ETHER | IFM_MANUAL);
@@ -341,7 +344,9 @@ hvn_attach(device_t parent, device_t self, void *aux)
fail4: hvn_rndis_detach(sc);
if_percpuq_destroy(sc->sc_ipq);
-fail3: hvn_tx_ring_destroy(sc);
+fail3: ifmedia_fini(&sc->sc_media);
+ mutex_destroy(&sc->sc_media_lock);
+ hvn_tx_ring_destroy(sc);
fail2: hvn_rx_ring_destroy(sc);
fail1: hvn_nvs_detach(sc);
}
@@ -363,6 +368,7 @@ hvn_detach(device_t self, int flags)
ether_ifdetach(ifp);
if_detach(ifp);
ifmedia_fini(&sc->sc_media);
+ mutex_destroy(&sc->sc_media_lock);
if_percpuq_destroy(sc->sc_ipq);
hvn_rndis_detach(sc);