Integrated: 8267579: Thread::cooked_allocated_bytes() hits assert(left >= right) failed: avoid underflow

Markus Grönlund mgronlun at openjdk.java.net
Mon Jun 14 10:43:57 UTC 2021


On Fri, 4 Jun 2021 15:30:53 GMT, Markus Grönlund <mgronlun at openjdk.org> wrote:

> This is a workaround to avoid hitting the assert that was recently added to pointer_delta(). 
> 
> The implementation of cooked_allocated_bytes() is perhaps questionable, in that it reads tlab pointers optimistically. However, the functionality has been in place for a long time, and the impact of changing its behavior more substantially is unknown at this time, hence this defensive workaround to reduce noise and problems seen in general testing.
> 
> Testing: tier1, tier2, tier6, tier8
> 
> Thanks
> Markus

This pull request has now been integrated.

Changeset: c4207355
Author:    Markus Grönlund <mgronlun at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/c4207355d3fc19687c72f47edf21611ffed7efaf
Stats:     21 lines in 3 files changed: 20 ins; 0 del; 1 mod

8267579: Thread::cooked_allocated_bytes() hits assert(left >= right) failed: avoid underflow

Reviewed-by: dcubed, stefank, kbarrett

-------------

PR: https://git.openjdk.java.net/jdk/pull/4363


More information about the hotspot-jfr-dev mailing list