Merge #242 242: Add the .sdata section to the app binary. r=hudson-ayers a=jrvanwhy RISC-V has the `.sdata` section, which is supposed to be for commonly-accessed global variables. Previously, we discarded this section in the linker script. Discarding this section resulted in missing global variables. I originally discovered this when I tried to print a string of less than 9 bytes and it was not working. I reported it as a kernel bug at https://github.com/tock/tock/issues/2132, then later realized the issue was in the libtock-rs linker script rather than the kernel. I added a test to libtock_test that catches this issue (I verified by hand that the test fails if I don't fix the linker script). Co-authored-by: Johnathan Van Why <jrvanwhy@google.com>
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index 744ea69..ad0738e 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml
@@ -1,7 +1,15 @@ # This workflow provides automated testing. It builds and runs tests on each PR. name: ci -on: pull_request + +# We want to run CI on all pull requests. Additionally, Bors needs workflows to +# run on the `staging` and `trying` branches. +on: + pull_request: + push: + branches: + - staging + - trying jobs: ci:
diff --git a/.github/workflows/size-diff.yml b/.github/workflows/size-diff.yml index 6f8c7d0..2c3b597 100644 --- a/.github/workflows/size-diff.yml +++ b/.github/workflows/size-diff.yml
@@ -2,7 +2,15 @@ # request is created or modified. name: size-diff -on: pull_request + +# We want to run this on all pull requests. Additionally, Bors needs workflows +# to run on the `staging` and `trying` branches to block merges on them. +on: + pull_request: + push: + branches: + - staging + - trying jobs: size-diff:
diff --git a/bors.toml b/bors.toml index b59819f..4b9b851 100644 --- a/bors.toml +++ b/bors.toml
@@ -1,3 +1,4 @@ status = [ - "continuous-integration/travis-ci/push" -] \ No newline at end of file + "ci", + "size-diff", +]