Skip to content

Commit e92d5d9

Browse files
committed
Add on-target tests
1 parent 0811180 commit e92d5d9

File tree

15 files changed

+677
-3
lines changed

15 files changed

+677
-3
lines changed

.github/bors.toml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,8 @@ status = [
88
"rt-ci-linux (1.42.0)",
99
"rt-ci-other-os (macOS-latest)",
1010
"rt-ci-other-os (windows-latest)",
11+
"hil-qemu",
12+
"hil-rtt-compile",
1113
"rustfmt",
1214
"clippy",
1315
]

.github/workflows/ci.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,6 @@ jobs:
2929
toolchain: ${{ matrix.rust }}
3030
override: true
3131
- name: Run tests
32-
run: cargo test --all --exclude cortex-m-rt
32+
run: cargo test --all --exclude cortex-m-rt --exclude testsuite
3333

3434
# FIXME: test on macOS and Windows

.github/workflows/on-target.yml

Lines changed: 51 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,51 @@
1+
on:
2+
push:
3+
branches: [ staging, trying, master ]
4+
pull_request:
5+
6+
name: cortex-m on-target tests
7+
8+
jobs:
9+
10+
hil-qemu:
11+
runs-on: ubuntu-latest
12+
steps:
13+
- uses: actions/checkout@v2
14+
- uses: actions-rs/toolchain@v1
15+
with:
16+
profile: minimal
17+
toolchain: stable
18+
override: true
19+
target: thumbv7m-none-eabi
20+
- name: Build testsuite
21+
run: RUSTFLAGS='-C link-arg=-Tlink.x -D warnings' cargo build -p testsuite --target thumbv7m-none-eabi --features testsuite/semihosting
22+
- name: Install QEMU
23+
run: sudo apt-get update && sudo apt-get install qemu qemu-system-arm
24+
- name: Run testsuite
25+
run: |
26+
qemu-system-arm \
27+
-cpu cortex-m3 \
28+
-machine lm3s6965evb \
29+
-nographic \
30+
-semihosting-config enable=on,target=native \
31+
-kernel target/thumbv7m-none-eabi/debug/testsuite
32+
33+
hil-compile-rtt:
34+
runs-on: ubuntu-latest
35+
steps:
36+
- uses: actions/checkout@v2
37+
- uses: actions-rs/toolchain@v1
38+
with:
39+
profile: minimal
40+
toolchain: stable
41+
override: true
42+
target: thumbv7em-none-eabi
43+
- name: Build testsuite
44+
run: RUSTFLAGS='-C link-arg=-Tlink.x -D warnings' cargo build -p testsuite --target thumbv7em-none-eabi --features testsuite/rtt
45+
- name: Upload testsuite binaries
46+
uses: actions/upload-artifact@v2
47+
with:
48+
name: testsuite-bin
49+
if-no-files-found: error
50+
retention-days: 1
51+
path: target/thumbv7em-none-eabi/debug/testsuite

Cargo.toml

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -35,11 +35,14 @@ std = []
3535

3636
[workspace]
3737
members = [
38-
"xtask",
3938
"cortex-m-rt",
4039
"cortex-m-semihosting",
40+
"panic-itm",
4141
"panic-semihosting",
42-
"panic-itm"
42+
"testsuite",
43+
"testsuite/minitest",
44+
"testsuite/minitest/macros",
45+
"xtask",
4346
]
4447

4548
[package.metadata.docs.rs]

src/peripheral/dwt.rs

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -155,6 +155,18 @@ impl DWT {
155155
}
156156
}
157157

158+
/// Disables the cycle counter
159+
#[cfg(not(armv6m))]
160+
#[inline]
161+
pub fn disable_cycle_counter(&mut self) {
162+
unsafe {
163+
self.ctrl.modify(|mut r| {
164+
r.set_cyccntena(false);
165+
r
166+
});
167+
}
168+
}
169+
158170
/// Returns `true` if the cycle counter is enabled
159171
#[cfg(not(armv6m))]
160172
#[inline]

testsuite/.cargo/config.toml

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,6 @@
1+
[target.'cfg(all(target_arch = "arm", target_os = "none"))']
2+
rustflags = ["-C", "link-arg=-Tlink.x"]
3+
runner = "qemu-system-arm -cpu cortex-m3 -machine lm3s6965evb -nographic -semihosting-config enable=on,target=native -kernel"
4+
5+
[build]
6+
target = "thumbv7m-none-eabi"

testsuite/Cargo.toml

Lines changed: 23 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,23 @@
1+
[package]
2+
authors = ["The Cortex-M Team <[email protected]>"]
3+
name = "testsuite"
4+
publish = false
5+
edition = "2018"
6+
version = "0.1.0"
7+
8+
[features]
9+
rtt = ["rtt-target", "minitest/rtt"]
10+
semihosting = ["cortex-m-semihosting", "minitest/semihosting"]
11+
12+
[dependencies]
13+
cortex-m-rt.path = "../cortex-m-rt"
14+
cortex-m.path = ".."
15+
minitest.path = "minitest"
16+
17+
[dependencies.rtt-target]
18+
version = "0.3.1"
19+
optional = true
20+
21+
[dependencies.cortex-m-semihosting]
22+
path = "../cortex-m-semihosting"
23+
optional = true

testsuite/README.md

Lines changed: 69 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,69 @@
1+
# Testsuite
2+
3+
This workspace contains tests that run on physical and simulated Cortex-M CPUs.
4+
5+
## Building
6+
7+
Exactly one of these features are required:
8+
9+
* `semihosting` Use semihosting for logging, this is used for QEMU.
10+
* `rtt` Use RTT for logging, this is used with physical cortex-m CPUs.
11+
12+
Assuming you are at the root of the repository you can build like this:
13+
14+
```console
15+
$ cd testsuite
16+
$ cargo build --features semihosting
17+
Compiling testsuite v0.1.0 (cortex-m/testsuite)
18+
Finished dev [unoptimized + debuginfo] target(s) in 0.08
19+
```
20+
21+
## Running with QEMU
22+
23+
The runner is already configured for QEMU in `testsuite/.cargo/config.toml`.
24+
Use the `semihosting` feature for logging, QEMU does not have native support for RTT.
25+
26+
For more information on QEMU reference the QEMU section in [The Embedded Rust Book].
27+
28+
```console
29+
$ cd testsuite
30+
$ cargo run --features semihosting
31+
Finished dev [unoptimized + debuginfo] target(s) in 0.01s
32+
Running `qemu-system-arm -cpu cortex-m3 -machine lm3s6965evb -nographic -semihosting-config enable=on,target=native -kernel /cortex-m/target/thumbv7m-none-eabi/debug/testsuite`
33+
Timer with period zero, disabling
34+
Hello world!
35+
(1/1) running `double_take`...
36+
all tests passed!
37+
```
38+
39+
## Running with Physical Hardware
40+
41+
No implementation-specific features are tested right now; any physical `thumbv7m` target should work.
42+
43+
Tests are executed with [probe-run](https://github.com/knurling-rs/probe-run).
44+
45+
* Update `memory.x` in the root of the repository to match your target memory layout.
46+
* Change the `probe-run` chip argument to match your chip, supported chips can be found with `probe-run --list-chips`
47+
* Change the target to match your CPU
48+
49+
```console
50+
$ sed -i 's/FLASH : ORIGIN = 0x00000000, LENGTH = 256K/FLASH : ORIGIN = 0x8000000, LENGTH = 256K/g' memory.x
51+
$ cd testsuite
52+
$ cargo build --target thumbv7em-none-eabi --features rtt
53+
Compiling minitest v0.1.0 (/cortex-m/testsuite/minitest)
54+
Compiling testsuite v0.1.0 (/cortex-m/testsuite)
55+
Finished dev [unoptimized + debuginfo] target(s) in 0.16s
56+
$ probe-run --chip STM32WLE5JCIx --connect-under-reset ../target/thumbv7em-none-eabi/debug/testsuite
57+
(HOST) INFO flashing program (19 pages / 19.00 KiB)
58+
(HOST) INFO success!
59+
────────────────────────────────────────────────────────────────────────────────
60+
Hello world!
61+
(1/2) running `double_take`...
62+
(2/2) running `cycle_count`...
63+
all tests passed!
64+
────────────────────────────────────────────────────────────────────────────────
65+
(HOST) INFO device halted without error
66+
```
67+
68+
[The Embedded Rust Book]: https://docs.rust-embedded.org/book/start/qemu.html
69+
[probe-run]: https://github.com/knurling-rs/probe-run

testsuite/minitest/Cargo.toml

Lines changed: 23 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,23 @@
1+
[package]
2+
authors = ["The Cortex-M Team <[email protected]>"]
3+
name = "minitest"
4+
publish = false
5+
edition = "2018"
6+
version = "0.1.0"
7+
8+
[features]
9+
semihosting = ["cortex-m-semihosting", "minitest-macros/semihosting"]
10+
rtt = ["rtt-target", "minitest-macros/rtt"]
11+
12+
[dependencies]
13+
cortex-m.path = "../.."
14+
cortex-m-rt.path = "../../cortex-m-rt"
15+
minitest-macros.path = "macros"
16+
17+
[dependencies.rtt-target]
18+
version = "0.3.1"
19+
optional = true
20+
21+
[dependencies.cortex-m-semihosting]
22+
path = "../../cortex-m-semihosting"
23+
optional = true

testsuite/minitest/README.md

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
# mini-test
2+
3+
This is an embedded test framework forked from knurling's excellent [`defmt-test`] crate.
4+
5+
This even more minimal than [`defmt-test`] to allow for for testing of this crate without dependency cycles.
6+
7+
[`defmt-test`]: https://crates.io/crates/defmt-test/

0 commit comments

Comments
 (0)