]> CyberLeo.Net >> Repos - FreeBSD/FreeBSD.git/commit
Fix two bugs that could result in PMC sampling effectively stopping.
authorjhb <jhb@FreeBSD.org>
Tue, 19 May 2015 19:15:19 +0000 (19:15 +0000)
committerjhb <jhb@FreeBSD.org>
Tue, 19 May 2015 19:15:19 +0000 (19:15 +0000)
commit7b63ce61a6e369f2dadc73573be6d07f110bd3d4
tree2bf73c6a4531d7252d19110d105a03b9dfaf0bf5
parent7f7477da161929e6969122c9ddde5e8a73c387a0
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
Differential Revision: https://reviews.freebsd.org/D2557
Reviewed by: emaste
MFC after: 1 week
Sponsored by: Norse Corp, Inc.
sys/dev/hwpmc/hwpmc_core.c
sys/dev/hwpmc/hwpmc_mod.c