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

Scheduling #221

Merged
merged 5 commits into from Sep 11, 2018

Conversation

Projects
None yet
2 participants
@kayceesrk
Member

kayceesrk commented Sep 10, 2018

This PR changes GC scheduling to use number of blocks rather than number of words for sweeping. I've also removed the 40%/60% split for mark/sweep, which I don't fully understand. With this PR, in most cases, the running time is better, fewer major GCs are performed, and space overhead remains within prescribed limits. See http://ocamllabs.io/multicore/ for updated graphs.

@kayceesrk kayceesrk closed this Sep 10, 2018

@kayceesrk kayceesrk reopened this Sep 10, 2018

@stedolan

This comment has been minimized.

Show comment
Hide comment
@stedolan

stedolan Sep 11, 2018

Contributor

Looks good! I still think we need to do more work on this, but this is a clear improvement over what's there.

Contributor

stedolan commented Sep 11, 2018

Looks good! I still think we need to do more work on this, but this is a clear improvement over what's there.

@stedolan stedolan merged commit e2e08f5 into master Sep 11, 2018

0 of 2 checks passed

continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
continuous-integration/travis-ci/push The Travis CI build is in progress
Details

@kayceesrk kayceesrk deleted the scheduling branch Sep 12, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment