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

[backport] 8135018: AARCH64: Missing memory barriers for CMS collector #201

Merged
merged 1 commit into from Dec 17, 2019

Conversation

navyxliu
Copy link
Contributor

Thank you for taking the time to help improve OpenJDK and Corretto.

If your pull request concerns a security vulnerability then please do not file it.
Instead, report the problem by email to aws-security@amazon.com.
(You can find more information regarding security issues at https://aws.amazon.com/security/vulnerability-reporting/.)

Otherwise, if your pull request concerns OpenJDK 8
and is not specific to Corretto 8,
then we ask you to redirect your contribution to the OpenJDK project.
See http://openjdk.java.net/contribute/ for details on how to do that.

If your issue is specific to Corretto 8,
then you are in the right place.
Please fill in the following information about your pull request.

Description

fix a bug for CMS on aarch64.
C1 and template interpreter both miss a StoreStore membar between put_field and updateCardTable.

Related issues

Motivation and context

How has this been tested?

Platform information

Works on OS: [e.g. Amazon Linux 2 only]
Applies to version [e.g. "build 1.8.0_192-amazon-corretto-preview-b12" (output from "java -version")]

Additional context

Contribution confirmation

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@navyxliu navyxliu merged commit 4bec425 into corretto:develop Dec 17, 2019
@navyxliu navyxliu deleted the develop branch December 17, 2019 19:59
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

Successfully merging this pull request may close these issues.

None yet

2 participants