Skip to content

Commit

Permalink
Update rev, remove whitelist and change default plan
Browse files Browse the repository at this point in the history
Updated the revision of the `mmtk-core` and the `ruby` repos.

Removed the whitelist.  Now we rely on the exclusion list in
`test/.excludes-mmtk` in the `ruby` repo.

Changed the default plan to StickyImmix since it has been stable so far.
  • Loading branch information
wks committed Oct 11, 2024
1 parent 6986a0f commit 7164655
Show file tree
Hide file tree
Showing 6 changed files with 47 additions and 176 deletions.
2 changes: 0 additions & 2 deletions .github/scripts/ci-test-all.sh
Original file line number Diff line number Diff line change
Expand Up @@ -12,9 +12,7 @@ case $DEBUG_LEVEL in
echo "Skipping test-all for $DEBUG_LEVEL..."
;;
release)
TEST_CASES=$(cat $BINDING_PATH/ruby-test-cases.txt | grep -v '#' | ruby -ne 'puts "../#{$_}"' | xargs)
make test-all \
TESTS="$TEST_CASES" \
RUN_OPTS="--mmtk-plan=$CHOSEN_PLAN" \
TESTOPTS="-v --excludes-dir=../test/.excludes-mmtk -j${CI_JOBS}"
;;
Expand Down
2 changes: 1 addition & 1 deletion .github/scripts/common.sh
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ RUBY_PATH=$BINDING_PATH/../ruby
RUBY_BUILD_PATH=$RUBY_PATH/build
RUBY_INSTALL_PATH=$RUBY_BUILD_PATH/install
RUSTUP_TOOLCHAIN=`cat $BINDING_PATH/mmtk/rust-toolchain`
DEFAULT_PLAN=Immix
DEFAULT_PLAN=StickyImmix

CI_JOBS=4

Expand Down
25 changes: 14 additions & 11 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -251,23 +251,26 @@ make btest RUN_OPTS="--mmtk-plan=StickyImmix"

### All tests

Note that currently it is not our priority to support Ractor. Some tests
involving Ractors are not enabled. You can get a list of enabled tests in the
file `ruby-test-cases.txt`.
We excluded some tests when testing against MMTk. Those test cases are listed
in `test/.excludes-mmtk` in the [`mmtk/ruby`](https://github.com/mmtk/ruby.git)
repository.

To run the tests
- Test cases involving Ractors are excluded because it is currently not a
priority to support Ractors.
- Some tests involve implementation details of CRuby's default GC, such as
compaction and memsize. Those test cases are excluded, too.
- Other excluded tests involve things that are not implemented properly in the
MMTk binding.

```bash
TEST_CASES=$(grep -v '#' ../../mmtk-ruby/ruby-test-cases.txt | awk '{print("../"$1)}' | xargs)
make test-all TESTS="$TEST_CASES" RUN_OPTS="--mmtk-plan=StickyImmix"
```

Or in one line:
To run the tests, run the following command.

```bash
make test-all TESTS="$(grep -v '#' ../../mmtk-ruby/ruby-test-cases.txt | awk '{print("../"$1)}' | xargs)" RUN_OPTS="--mmtk-plan=StickyImmix"
make test-all RUN_OPTS="--mmtk-plan=StickyImmix" TESTOPTS="-v --excludes-dir=../test/.excludes-mmtk"
```

That assumes you are in the `build-debug` or `build-release` directory. Adjust
the path `../test/.excludes-mmtk` if you run it in a different directory.

## Current status

Known working:
Expand Down
58 changes: 29 additions & 29 deletions mmtk/Cargo.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

6 changes: 3 additions & 3 deletions mmtk/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ edition = "2021"
# Metadata for the Ruby repository
[package.metadata.ci-repos.ruby]
repo = "mmtk/ruby" # This is used by actions/checkout, so the format is "owner/repo", not URL.
rev = "c300b86a5260508a46348bd5d3a023a8271c6487"
rev = "459b61a81e9808a257e6f2878fd24a99b4c579c3"

[lib]
name = "mmtk_ruby"
Expand All @@ -36,8 +36,8 @@ probe = "0.5"
features = ["is_mmtk_object", "object_pinning", "sticky_immix_non_moving_nursery"]

# Uncomment the following lines to use mmtk-core from the official repository.
git = "https://github.com/mmtk/mmtk-core.git"
rev = "c4fdce02274a4b3fee49318cce9f6bf3eb378ae0"
git = "https://github.com/wks/mmtk-core.git"
rev = "3b04648bf6510c823f486ecde861ded2ba0a6841"

# Uncomment the following line to use mmtk-core from a local repository.
#path = "../../mmtk-core"
Expand Down
130 changes: 0 additions & 130 deletions ruby-test-cases.txt

This file was deleted.

0 comments on commit 7164655

Please sign in to comment.