Skip to content

Commit eaa97b2

Browse files
committed
Add on-target tests
1 parent 8bb2a61 commit eaa97b2

File tree

16 files changed

+733
-3
lines changed

16 files changed

+733
-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-compile-rtt",
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: 81 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,81 @@
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+
env:
22+
RUSTFLAGS: -C link-arg=-Tlink.x -D warnings
23+
run: cargo build -p testsuite --target thumbv7m-none-eabi --features testsuite/semihosting
24+
- name: Install QEMU
25+
run: sudo apt-get update && sudo apt-get install qemu qemu-system-arm
26+
- name: Run testsuite
27+
run: |
28+
qemu-system-arm \
29+
-cpu cortex-m3 \
30+
-machine lm3s6965evb \
31+
-nographic \
32+
-semihosting-config enable=on,target=native \
33+
-kernel target/thumbv7m-none-eabi/debug/testsuite
34+
35+
hil-compile-rtt:
36+
runs-on: ubuntu-latest
37+
steps:
38+
- uses: actions/checkout@v2
39+
- uses: actions-rs/toolchain@v1
40+
with:
41+
profile: minimal
42+
toolchain: stable
43+
override: true
44+
target: thumbv6m-none-eabi
45+
- name: Modify linkerfile
46+
run: |
47+
sed -i 's/FLASH : ORIGIN = 0x00000000, LENGTH = 256K/FLASH : ORIGIN = 0x8000000, LENGTH = 128K/g' memory.x
48+
sed -i 's/RAM : ORIGIN = 0x20000000, LENGTH = 64K/RAM : ORIGIN = 0x20000000, LENGTH = 16K/g' memory.x
49+
- name: Build testsuite
50+
env:
51+
RUSTFLAGS: -C link-arg=-Tlink.x -D warnings
52+
run: cargo build -p testsuite --target thumbv6m-none-eabi --features testsuite/rtt
53+
- name: Upload testsuite binaries
54+
uses: actions/upload-artifact@v2
55+
with:
56+
name: testsuite-bin
57+
if-no-files-found: error
58+
retention-days: 1
59+
path: target/thumbv6m-none-eabi/debug/testsuite
60+
61+
hil-stm32:
62+
runs-on: self-hosted
63+
needs:
64+
- hil-compile-rtt
65+
steps:
66+
- uses: actions/checkout@v2
67+
- name: Display probe-run version
68+
run: probe-run --version
69+
- name: List probes
70+
run: probe-run --list-probes
71+
- uses: actions/download-artifact@v2
72+
with:
73+
name: testsuite-bin
74+
path: testsuite-bin
75+
- name: Run on-target tests
76+
timeout-minutes: 5
77+
run: |
78+
probe-run \
79+
--chip STM32F070RBTx \
80+
--connect-under-reset \
81+
testsuite-bin/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/build.rs

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,18 @@
1+
fn main() {
2+
let target = std::env::var("TARGET").unwrap();
3+
4+
if target.starts_with("thumbv6m-") {
5+
println!("cargo:rustc-cfg=armv6m");
6+
} else if target.starts_with("thumbv7m-") {
7+
println!("cargo:rustc-cfg=armv7m");
8+
} else if target.starts_with("thumbv7em-") {
9+
println!("cargo:rustc-cfg=armv7m");
10+
println!("cargo:rustc-cfg=armv7em"); // (not currently used)
11+
} else if target.starts_with("thumbv8m.base") {
12+
println!("cargo:rustc-cfg=armv8m");
13+
println!("cargo:rustc-cfg=armv8m_base");
14+
} else if target.starts_with("thumbv8m.main") {
15+
println!("cargo:rustc-cfg=armv8m");
16+
println!("cargo:rustc-cfg=armv8m_main");
17+
}
18+
}

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/

testsuite/minitest/macros/Cargo.toml

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,18 @@
1+
[package]
2+
authors = ["The Cortex-M Team <[email protected]>"]
3+
name = "minitest-macros"
4+
publish = false
5+
edition = "2018"
6+
version = "0.1.0"
7+
8+
[lib]
9+
proc-macro = true
10+
11+
[features]
12+
semihosting = []
13+
rtt = []
14+
15+
[dependencies]
16+
proc-macro2 = "1.0.29"
17+
quote = "1.0.10"
18+
syn = { version = "1.0.80", features = ["extra-traits", "full"] }

0 commit comments

Comments
 (0)