[11u] RFR 8229020: Failure on CPUs allowing loads reordering: assert(_tasks[t] == 1) failed: What else?

[Hohensee, Paul]

Lgtm. Also, you don't have to request a review if a patch applies cleanly net of file name/location/line number changes. Just note that is the case in your JBS issue comment when you tag the bug with jdk11u-fix-request.

Paul

On 10/11/19, 10:24 AM, "jdk-updates-dev on behalf of Zhengyu Gu" <jdk-updates-dev-bounces at openjdk.java.net on behalf of zgu at redhat.com> wrote:

    Hi,
    
    Please review this one line patch.
    
    It removes a paranoid assertion, that may trigger false alarm on weak 
    memory systems.
    
    The original patch does not apply cleanly, mainly just due to line shifts.
    
    Original patch:http://cr.openjdk.java.net/~jiefu/8229020/webrev.02/
    Original code review thread: 
    https://mail.openjdk.java.net/pipermail/hotspot-gc-dev/2019-August/026643.html
    
    
    Bug: https://bugs.openjdk.java.net/browse/JDK-8229020
    Webrev: http://cr.openjdk.java.net/~zgu/JDK-8229020-11u/webrev.00/
    
    Thanks,
    
    -Zhengyu