summaryrefslogtreecommitdiff
path: root/sys/dev/pv/hyperv.c
diff options
context:
space:
mode:
authorMike Belopuhov <mikeb@cvs.openbsd.org>2016-08-05 18:16:05 +0000
committerMike Belopuhov <mikeb@cvs.openbsd.org>2016-08-05 18:16:05 +0000
commitabbee944d54fce28c21e5ff5ae62eff40a24758b (patch)
tree89c41eb57d45d0b42b747e9b7a4d64ffdea196f3 /sys/dev/pv/hyperv.c
parente22103c24c66d3dd321cb18d6559f9c21a3f0d4a (diff)
Switch pending event clearing to an atomic swap operation
Rather than performing an atomic bit clearing for every encountered event bit set we can adjust the code to perform an atomic swap of a single row of the events array and decrease the amount of expensive atomic operations. From FreeBSD.
Diffstat (limited to 'sys/dev/pv/hyperv.c')
-rw-r--r--sys/dev/pv/hyperv.c7
1 files changed, 4 insertions, 3 deletions
diff --git a/sys/dev/pv/hyperv.c b/sys/dev/pv/hyperv.c
index 1aba89795e1..7a8e22cca19 100644
--- a/sys/dev/pv/hyperv.c
+++ b/sys/dev/pv/hyperv.c
@@ -534,7 +534,7 @@ hv_event_intr(struct hv_softc *sc)
int cpu = CPU_INFO_UNIT(ci);
int bit, dword, maxdword, relid;
struct hv_channel *ch;
- uint32_t *revents;
+ uint32_t *revents, pending;
evt = (struct hv_synic_event_flags *)sc->sc_siep[cpu] + HV_MESSAGE_SINT;
if ((sc->sc_proto == HV_VMBUS_VERSION_WS2008) ||
@@ -559,8 +559,9 @@ hv_event_intr(struct hv_softc *sc)
for (dword = 0; dword < maxdword; dword++) {
if (revents[dword] == 0)
continue;
- for (bit = 0; bit < 32; bit++) {
- if (!atomic_clearbit_ptr(&revents[dword], bit))
+ pending = atomic_swap_uint(&revents[dword], 0);
+ for (bit = 0; pending > 0; pending >>= 1, bit++) {
+ if ((pending & 1) == 0)
continue;
relid = (dword << 5) + bit;
/* vmbus channel protocol message */