On Thu, 2008-02-07 at 18:12, Simon Riggs wrote:
> I just realised you are using a lookup to get the text for the name of
> the lock. You used the same lookup table for both releases?
Oh, it wasn't quite that bad. :-) The two DTrace scripts had been
revised to correspond with the two different declarations of LWLockId in
lwlock.h (for 8.2.5 and 8.3 respectively). But somehow the value for the
last lock in our lookup table corresponding to
FirstLockMgrLock = FirstBufMappingLock + NUM_BUFFER_PARTITIONS,
in the enum got turned into n+1 instead of n+NUM_BUFFER_PARTITIONS. The
other locks should have been output correctly, however.
But as Tom pointed out, the dynamic locks were not in the equation. So
now we're measuring all lock waits instead of assuming. :-)
Staale
