Tuning and subprocess spawn for thomaswue #533
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Check List:
./test.sh <username>
shows no differences between expected and actual outputs)calculate_average_<username>.sh
(make sure to match casing of your GH user name) and is executablecalculate_average_baseline.sh
Apart from some minor tunings (which should amount to ~3%), this PR now creates a subprocess to do the actual work. The main process returns immediately once the output is calculated. The OS clean up unmapping the pages that were mapped in continues in the background process. The code for this is just a few lines of Java Process API in the spawnWorker method. The subprocess is started with the exact same command as the original process, but with a "--worker" argument.
I have also updated the Graal version to the latest CPU release (21.0.2).