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.

Sign up for GitHub

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

Jump to bottom

[GC] Backport 8204524: Unnecessary memory barriers in G1ParScanThreadState::copy_to_survivor #154

Closed
tanghaoth90 opened this issue Dec 15, 2020 · 0 comments
Closed

[GC] Backport 8204524: Unnecessary memory barriers in G1ParScanThreadState::copy_to_survivor #154

tanghaoth90 opened this issue Dec 15, 2020 · 0 comments

Comments

@tanghaoth90
Copy link
Collaborator

Description
full fences before/after the CAS for an object copy is not necessary

  • Covered ParNew as well. JDK-8205908 supposed to do this but not implemented.

Steps to Reproduce
N/A

Expected behavior
N/A

JDK version
N/A

Execution environment

  • OS and version:
  • CPU model:
  • Number of CPU cores:
  • Size of physical memory:
  • Inside Linux container?
    • Linux container name (docker, pouch, etc):
    • Linux container version:
tanghaoth90 pushed a commit to tanghaoth90/dragonwell8_hotspot that referenced this issue Dec 15, 2020
[GC] Backport 8204524: Unnecessary memory barriers in G1ParScanThread…
1b0f04b 
…State::copy_to_survivor

Summary: full fences before/after the CAS for an object copy is not necessary
  - Covered ParNew as well. JDK-8205908 supposed to do this but not implemented.

Test Plan: hotspot/test/gc/{g1,parNew}/

Reviewed-by: linade, mmyxym

Issue: dragonwell-project/dragonwell8#154
@tanghaoth90 tanghaoth90 mentioned this issue Dec 15, 2020
Merged
tanghaoth90 pushed a commit to tanghaoth90/dragonwell8_hotspot that referenced this issue Dec 15, 2020
[GC] Backport 8204524: Unnecessary memory barriers in G1ParScanThread…
97c7aae 
…State::copy_to_survivor

Summary: full fences before/after the CAS for an object copy is not necessary
  - Covered ParNew as well. JDK-8205908 supposed to do this but not implemented.

Test Plan: hotspot/test/gc/{g1,parNew}/

Reviewed-by: linade, mmyxym

Issue: dragonwell-project/dragonwell8#154
tanghaoth90 pushed a commit to tanghaoth90/dragonwell8_hotspot that referenced this issue Dec 15, 2020
[GC] Backport 8204524: Unnecessary memory barriers in G1ParScanThread…
7f53c53 
…State::copy_to_survivor

Summary: full fences before/after the CAS for an object copy is not necessary
  - Covered ParNew as well. JDK-8205908 supposed to do this but not implemented.

Test Plan: hotspot/test/gc/{g1,parNew}/

Reviewed-by: linade, mmyxym

Issue: dragonwell-project/dragonwell8#154
tanghaoth90 pushed a commit to dragonwell-project/dragonwell8_hotspot that referenced this issue Dec 15, 2020
@tanghaoth90
[GC] Backport 8204524: Unnecessary memory barriers in G1ParScanThread…
ab977e7 
…State::copy_to_survivor

Summary: full fences before/after the CAS for an object copy is not necessary
  - Covered ParNew as well. JDK-8205908 supposed to do this but not implemented.

Test Plan: hotspot/test/gc/{g1,parNew}/

Reviewed-by: linade, mmyxym

Issue: dragonwell-project/dragonwell8#154
@henrysternc henrysternc mentioned this issue Dec 31, 2021
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@tanghaoth90