Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[aarch64] crash due to lack of memory barrier #457

Closed
sandlerwang opened this issue Jan 17, 2023 · 0 comments
Closed

[aarch64] crash due to lack of memory barrier #457

sandlerwang opened this issue Jan 17, 2023 · 0 comments

Comments

@sandlerwang
Copy link
Collaborator

Dragonwell 8 may crash due to lack of memory barriers.
Specific patches are:
8154750: Add missing OrderAccess operations to ClassLoaderData lock-free data structures
https://bugs.openjdk.org/browse/JDK-8154750
8233073: Make BitMap accessors more memory ordering friendly
https://bugs.openjdk.org/browse/JDK-8233073
8221584: SIGSEGV in os::PlatformEvent::unpark() in JvmtiRawMonitor::raw_exit while posting
method exit event
https://bugs.openjdk.org/browse/JDK-8221584
8166197: assert(RelaxAssert || w != Thread::current()->_MutexEvent) failed: invariant
8164207: Checking missing load-acquire in relation to _pd_set in dictionary.cpp
https://bugs.openjdk.org/browse/JDK-8164207

sandlerwang added a commit to sandlerwang/dragonwell8 that referenced this issue Jan 17, 2023
    Summary:  this patch addes mem barriers in JVM.
    Including:
    8154750: Add missing OrderAccess operations to ClassLoaderData
lock-free data structures
    https://bugs.openjdk.org/browse/JDK-8154750
    8233073: Make BitMap accessors more memory ordering friendly
    https://bugs.openjdk.org/browse/JDK-8233073
    8221584: SIGSEGV in os::PlatformEvent::unpark() in
JvmtiRawMonitor::raw_exit while posting
    method exit event
    https://bugs.openjdk.org/browse/JDK-8221584
    8166197: assert(RelaxAssert || w != Thread::current()->_MutexEvent)
failed: invariant
    8164207: Checking missing load-acquire in relation to _pd_set in
dictionary.cpp
    https://bugs.openjdk.org/browse/JDK-8164207
    and adding a loadload barrier in CMS

    Test Plan: e-commerce online tests

    Reviewers: maoliang.ml, lxw263044

    Issue: dragonwell-project#457

    Reviewed-by: mmyxym, kuaiwei

    Issue: dragonwell-project#330
sandlerwang added a commit to sandlerwang/dragonwell8 that referenced this issue Jan 17, 2023
    Summary:  this patch addes mem barriers in JVM.
    Including:
    8154750: Add missing OrderAccess operations to ClassLoaderData
lock-free data structures
    https://bugs.openjdk.org/browse/JDK-8154750
    8233073: Make BitMap accessors more memory ordering friendly
    https://bugs.openjdk.org/browse/JDK-8233073
    8221584: SIGSEGV in os::PlatformEvent::unpark() in
JvmtiRawMonitor::raw_exit while posting
    method exit event
    https://bugs.openjdk.org/browse/JDK-8221584
    8166197: assert(RelaxAssert || w != Thread::current()->_MutexEvent)
failed: invariant
    8164207: Checking missing load-acquire in relation to _pd_set in
dictionary.cpp
    https://bugs.openjdk.org/browse/JDK-8164207
    and adding a loadload barrier in CMS

    Test Plan: Alibaba e-commerce online tests

    Reviewers: maoliang.ml, lxw263044

    Issue: dragonwell-project#457

    Reviewed-by: mmyxym, kuaiwei

    Issue: dragonwell-project#330
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant