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

Scheduling #221

Merged
merged 5 commits into from Sep 11, 2018
Merged

Scheduling #221

merged 5 commits into from Sep 11, 2018

Conversation

@kayceesrk
Copy link
Contributor

@kayceesrk 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
Copy link
Contributor

@stedolan 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
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
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.