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

Add extra U_64 data field to Indexable Object Headers #11755

Merged
merged 1 commit into from
Jan 26, 2021
Merged

Add extra U_64 data field to Indexable Object Headers #11755

merged 1 commit into from
Jan 26, 2021

Conversation

bragaigor
Copy link
Contributor

@bragaigor bragaigor commented Jan 23, 2021 •
edited
Loading

Same PR/commit as #11507 .
The issue related to concurrent scavenger (#11616) was solved by eclipse-omr/omr#5751

We can now safely re-introduce dataAddr again.

Signed-off-by: Igor Braga higorb1@gmail.com

@bragaigor
Add extra U_64 data field to Indexable Object Headers
e86fa4f 
Same commit as eclipse-openj9#11507
The issue related to concurrent scavenger
(eclipse-openj9#11616) was solved by
eclipse-omr/omr#5751
We can now safely re-introduce dataAddr again.

Signed-off-by: Igor Braga <higorb1@gmail.com>
@bragaigor
Copy link
Contributor Author

cc: @amicic @dmitripivkine @fjeremic

@amicic
Copy link
Contributor

amicic commented Jan 24, 2021

Jenkins test sanity all jdk8

@eclipse-openj9 eclipse-openj9 deleted a comment from amicic16 Jan 24, 2021
@pshipton
Copy link
Member

The OMR acceptance this morning failed, probably an infra issue. Pls don't merge this until after we get the OMR change promoted.

@pshipton
Copy link
Member

This should fix it. https://ci.eclipse.org/openj9/job/Pipeline-OMR-Acceptance/2199/
After this promotes, we can try some sanity.system PR testing since that is the suite which failed before.

@pshipton
Copy link
Member

pshipton commented Jan 25, 2021 •
edited
Loading

Windows 32-bit build hung. Trying again https://ci.eclipse.org/openj9/job/Pipeline-OMR-Acceptance/2200/

@pshipton
Copy link
Member

OMR is promoted now.

Jenkins test sanity,sanity.system all jdk8

@pshipton
Copy link
Member

pshipton commented Jan 25, 2021 •
edited
Loading

The remaining (failed) build https://ci.eclipse.org/openj9/job/Test_openjdk8_j9_sanity.system_x86-64_windows_Personal_testList_0/2/console hung due to a machine issue, and I disabled the machine.

The half that did already run on Windows seems sufficient as it passed theses tests: DaaLoadTest_daa1_ConcurrentScavenge_0, DaaLoadTest_daa2_ConcurrentScavenge_0.
https://ci.eclipse.org/openj9/job/Test_openjdk8_j9_sanity.system_x86-64_windows_Personal_testList_1/2/tapResults/

@amicic amicic merged commit f93331a into eclipse-openj9:master Jan 26, 2021
@pshipton pshipton mentioned this pull request Jan 27, 2021
Open
@bragaigor bragaigor mentioned this pull request Jan 29, 2021
Open
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@amicic amicic amicic approved these changes

@amicic16 amicic16 amicic16 approved these changes

@fjeremic fjeremic fjeremic approved these changes

Assignees
No one assigned
Labels
comp:gc
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@bragaigor @amicic @pshipton @fjeremic @amicic16