]> CyberLeo.Net >> Repos - FreeBSD/FreeBSD.git/commit
MFC 283123:
authorjhb <jhb@FreeBSD.org>
Mon, 1 Jun 2015 18:08:56 +0000 (18:08 +0000)
committerjhb <jhb@FreeBSD.org>
Mon, 1 Jun 2015 18:08:56 +0000 (18:08 +0000)
commit34dd7e04e0d09812f05591fc803ce58d0708ead1
treefd15c2b27333272616e613a027aa2a6a95e1b350
parented437ec23ce9e663b9b89d839cade53697620378
MFC 283123:
Fix two bugs that could result in PMC sampling effectively stopping.
In both cases, the the effect of the bug was that a very small positive
number was written to the counter. This means that a large number of
events needed to occur before the next sampling interrupt would trigger.
Even with very frequently occurring events like clock cycles wrapping all
the way around could take a long time. Both bugs occurred when updating
the saved reload count for an outgoing thread on a context switch.

First, the counter-independent code compares the current reload count
against the count set when the thread switched in and generates a delta
to apply to the saved count. If this delta causes the reload counter
to go negative, it would add a full reload interval to wrap it around to
a positive value. The fix is to add the full reload interval if the
resulting counter is zero.

Second, occasionally the raw counter value read during a context switch
has actually wrapped, but an interrupt has not yet triggered. In this
case the existing logic would return a very large reload count (e.g.
2^48 - 2 if the counter had overflowed by a count of 2). This was seen
both for fixed-function and programmable counters on an E5-2643.
Workaround this case by returning a reload count of zero.

PR: 198149
Sponsored by: Norse Corp, Inc.
sys/dev/hwpmc/hwpmc_core.c
sys/dev/hwpmc/hwpmc_mod.c