runtime/libia2: enable MTE #334
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ayrtonm
force-pushed
the
am/mte_fault
branch
2 times, most recently
from
82bb469
to
2070e23
Compare
|
Rebased this on #353 which adds support for running tests in QEMU. |
ayrtonm
force-pushed
the
am/qemu_tests
branch
20 times, most recently
from
f620a28
to
ff75a18
Compare
fw-immunant
force-pushed
the
am/mte_fault
branch
3 times, most recently
from
6a64337
to
b414dbc
Compare
fw-immunant
force-pushed
the
am/mte_fault
branch
2 times, most recently
from
42fd2b1
to
50ba9c5
Compare
|
This appears to work correctly: we see MTE violations in test logs here. We can wait to merge this until we fix most (if not all) of these with call-gates. |
Without this CTest only prints tests that fail so MTE check failures logged by QEMU won't show up unless the test fails for another reason.
fw-immunant
force-pushed
the
am/mte_fault
branch
from
50ba9c5
to
61e9197
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
This PR actually enables MTE after #311 added support for cross-compiling for ARM. I wrote some basic tests that protects static data but we'll need to test a different way as explained in this comment.