Skip to content

Enable Gencon/Balanced/OptAvgPause GC policy in testing - #16069

Merged
pshipton merged 1 commit into
eclipse-openj9:masterfrom
fengxue-IS:cont_gc2
Oct 13, 2022
Merged

Enable Gencon/Balanced/OptAvgPause GC policy in testing#16069
pshipton merged 1 commit into
eclipse-openj9:masterfrom
fengxue-IS:cont_gc2

Conversation

@fengxue-IS

Copy link
Copy Markdown
Contributor

Signed-off-by: Jack Lu Jack.S.Lu@ibm.com

Signed-off-by: Jack Lu <Jack.S.Lu@ibm.com>
@fengxue-IS

Copy link
Copy Markdown
Contributor Author

Per discussion with Lin, GC policy have been updated to support VirtualThread/Continuation
@LinHu2016 can you please review this

@fengxue-IS fengxue-IS added comp:test comp:gc project:loom Used to track Project Loom related work labels Oct 6, 2022
@LinHu2016

Copy link
Copy Markdown
Contributor

LGTM

@pshipton

pshipton commented Oct 7, 2022

Copy link
Copy Markdown
Member

Trying the test on a number of platforms
https://openj9-jenkins.osuosl.org/view/Test/job/Grinder/1330/

@pshipton

pshipton commented Oct 7, 2022

Copy link
Copy Markdown
Member

@pshipton
pshipton marked this pull request as draft October 7, 2022 18:46
@pshipton

pshipton commented Oct 7, 2022

Copy link
Copy Markdown
Member

A crash running balanced on most platforms, see the previous comment for details.

@dmitripivkine

Copy link
Copy Markdown
Contributor

A crash running balanced on most platforms, see the previous comment for details.

Crash in Balanced and incomplete handling in Gencon
Fixing the problem in #16087

@babsingh

Copy link
Copy Markdown
Contributor

Reopening, accidentally closed by #16087.

@babsingh babsingh reopened this Oct 12, 2022
@fengxue-IS

Copy link
Copy Markdown
Contributor Author

@LinHu2016 can you take another look please

@pshipton

pshipton commented Oct 12, 2022

Copy link
Copy Markdown
Member

@pshipton

Copy link
Copy Markdown
Member

I guess we need to wait until tomorrow, as the fix was just merged today and I'm using nightly builds for the testing.

@pshipton

pshipton commented Oct 13, 2022

Copy link
Copy Markdown
Member

https://openj9-jenkins.osuosl.org/view/Test/job/Grinder/1383
Rebuild for xlinux to avoid centos6 - https://openj9-jenkins.osuosl.org/job/Grinder/1392/
Previous grinder passed. Run more iterations, 5x x 5 machines.
https://openj9-jenkins.osuosl.org/view/Test/job/Grinder/1393/ - infra issue on zlinux 2/5 machines, but otherwise it all passed.

@LinHu2016

Copy link
Copy Markdown
Contributor

LGTM

@pshipton
pshipton marked this pull request as ready for review October 13, 2022 17:30
@pshipton
pshipton merged commit e8abb75 into eclipse-openj9:master Oct 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp:gc comp:test project:loom Used to track Project Loom related work

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants