JDK-8141056 : Erroneous assignment in HeapRegionSet.cpp
Type:Bug
Component:hotspot
Sub-Component:gc
Affected Version:8,9
Priority:P2
Status:Closed
Resolution:Fixed
Submitted:2015-10-30
Updated:2020-08-14
Resolved:2015-11-03
The Version table provides details related to the release that this issue/RFE will be addressed.
Unresolved : Release in which this issue/RFE will be addressed. Resolved: Release in which this issue/RFE has been resolved. Fixed : Release in which this issue/RFE has been fixed. The release containing this fix may be available for download as an Early Access Release or a General Availability Release.
THe following should be an equals check, not an assignment:
if (_last = curr) {
_last = NULL;
}
Comments
Fix Request (8u)
Fixes the corner case in G1, and keeps codebases in sync (I see 8u261). Patch applies cleanly to 8u, passes tier1 tests.
24-03-2020
_last is a cache value to speed up additions to the list. NULL means, do not use cache.
So this code at the moment is only a perf problem, no functional one.
Enabling/fixing this might uncover bugs though :)