[INFO] cloning repository https://github.com/gotti/rs-riscv-sc [INFO] running `Command { std: "git" "-c" "credential.helper=" "-c" "credential.helper=/workspace/cargo-home/bin/git-credential-null" "clone" "--bare" "https://github.com/gotti/rs-riscv-sc" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fgotti%2Frs-riscv-sc", kill_on_drop: false }` [INFO] [stderr] Cloning into bare repository '/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fgotti%2Frs-riscv-sc'... [INFO] running `Command { std: "git" "rev-parse" "HEAD", kill_on_drop: false }` [INFO] [stdout] a5f970eefa71b51ef6052102ba91b9771729fe21 [INFO] checking gotti/rs-riscv-sc against try#59a256151e578f525a26127f2c3e3a6c7e2982ca for pr-149291-1 [INFO] running `Command { std: "git" "clone" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fgotti%2Frs-riscv-sc" "/workspace/builds/worker-5-tc2/source", kill_on_drop: false }` [INFO] [stderr] Cloning into '/workspace/builds/worker-5-tc2/source'... [INFO] [stderr] done. [INFO] started tweaking git repo https://github.com/gotti/rs-riscv-sc [INFO] finished tweaking git repo https://github.com/gotti/rs-riscv-sc [INFO] tweaked toml for git repo https://github.com/gotti/rs-riscv-sc written to /workspace/builds/worker-5-tc2/source/Cargo.toml [INFO] validating manifest of git repo https://github.com/gotti/rs-riscv-sc on toolchain 59a256151e578f525a26127f2c3e3a6c7e2982ca [INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+59a256151e578f525a26127f2c3e3a6c7e2982ca" "metadata" "--manifest-path" "Cargo.toml" "--no-deps", kill_on_drop: false }` [INFO] crate git repo https://github.com/gotti/rs-riscv-sc already has a lockfile, it will not be regenerated [INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+59a256151e578f525a26127f2c3e3a6c7e2982ca" "fetch" "--manifest-path" "Cargo.toml", kill_on_drop: false }` [INFO] [stderr] Blocking waiting for file lock on package cache [INFO] [stderr] Blocking waiting for file lock on package cache [INFO] [stderr] Downloading crates ... [INFO] [stderr] Downloaded anyhow v1.0.42 [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-5-tc2/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-5-tc2/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/cargo-home:/opt/rustwide/cargo-home:ro,Z" "-v" "/var/lib/crater-agent-workspace/rustup-home:/opt/rustwide/rustup-home:ro,Z" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-w" "/opt/rustwide/workdir" "-m" "1610612736" "--user" "0:0" "--network" "none" "ghcr.io/rust-lang/crates-build-env/linux@sha256:94a0c148923f5b2b52a63ef0eeb1882ad339ab61bce784c8077cbe41c61feb6c" "/opt/rustwide/cargo-home/bin/cargo" "+59a256151e578f525a26127f2c3e3a6c7e2982ca" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }` [INFO] [stdout] 64163d51755ca06a7c1f79807e13737886cb12d81643e704d2b99e1b20790f46 [INFO] running `Command { std: "docker" "start" "-a" "64163d51755ca06a7c1f79807e13737886cb12d81643e704d2b99e1b20790f46", kill_on_drop: false }` [INFO] running `Command { std: "docker" "inspect" "64163d51755ca06a7c1f79807e13737886cb12d81643e704d2b99e1b20790f46", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "64163d51755ca06a7c1f79807e13737886cb12d81643e704d2b99e1b20790f46", kill_on_drop: false }` [INFO] [stdout] 64163d51755ca06a7c1f79807e13737886cb12d81643e704d2b99e1b20790f46 [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-5-tc2/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-5-tc2/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/cargo-home:/opt/rustwide/cargo-home:ro,Z" "-v" "/var/lib/crater-agent-workspace/rustup-home:/opt/rustwide/rustup-home:ro,Z" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-e" "CARGO_INCREMENTAL=0" "-e" "RUST_BACKTRACE=full" "-e" "RUSTFLAGS=--cap-lints=forbid" "-e" "RUSTDOCFLAGS=--cap-lints=forbid" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-w" "/opt/rustwide/workdir" "-m" "1610612736" "--user" "0:0" "--network" "none" "ghcr.io/rust-lang/crates-build-env/linux@sha256:94a0c148923f5b2b52a63ef0eeb1882ad339ab61bce784c8077cbe41c61feb6c" "/opt/rustwide/cargo-home/bin/cargo" "+59a256151e578f525a26127f2c3e3a6c7e2982ca" "check" "--frozen" "--all" "--all-targets" "--message-format=json", kill_on_drop: false }` [INFO] [stdout] d885be245f5a5e8373aa43d90add1d37c8de98d2238797bb66a7a87cd75774ed [INFO] running `Command { std: "docker" "start" "-a" "d885be245f5a5e8373aa43d90add1d37c8de98d2238797bb66a7a87cd75774ed", kill_on_drop: false }` [INFO] [stderr] Compiling libc v0.2.99 [INFO] [stderr] Compiling anyhow v1.0.42 [INFO] [stderr] Checking atty v0.2.14 [INFO] [stderr] Checking clap v2.33.3 [INFO] [stderr] Checking rs-riscv-sc v0.1.0 (/opt/rustwide/workdir) [INFO] [stdout] warning: unused import: `env` [INFO] [stdout] --> src/main.rs:4:11 [INFO] [stdout] | [INFO] [stdout] 4 | use std::{env, io}; [INFO] [stdout] | ^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_imports)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `str::FromStr` [INFO] [stdout] --> src/cpu.rs:10:25 [INFO] [stdout] | [INFO] [stdout] 10 | use std::{io, ops::Add, str::FromStr}; [INFO] [stdout] | ^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `sel` [INFO] [stdout] --> src/cpu.rs:129:12 [INFO] [stdout] | [INFO] [stdout] 129 | exec: |sel, state, reg| { [INFO] [stdout] | ^^^ help: if this is intentional, prefix it with an underscore: `_sel` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_variables)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `sp` [INFO] [stdout] --> src/cpu.rs:238:17 [INFO] [stdout] | [INFO] [stdout] 238 | let sp = match self.register.read(2,self.len) { [INFO] [stdout] | ^^ help: if this is intentional, prefix it with an underscore: `_sp` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `rs` [INFO] [stdout] --> src/cpu.rs:298:13 [INFO] [stdout] | [INFO] [stdout] 298 | let rs = Bits::cut_new(inst, 6, 2); [INFO] [stdout] | ^^ help: if this is intentional, prefix it with an underscore: `_rs` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/main.rs:32:9 [INFO] [stdout] | [INFO] [stdout] 32 | let mut mmu = Mmu::new(buf, test_mode); [INFO] [stdout] | ----^^^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_mut)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/main.rs:33:9 [INFO] [stdout] | [INFO] [stdout] 33 | let mut csr = Csr::new([0; 4096]); [INFO] [stdout] | ----^^^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/main.rs:34:9 [INFO] [stdout] | [INFO] [stdout] 34 | let mut reg = Register::new([0; 32]); [INFO] [stdout] | ----^^^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/main.rs:35:9 [INFO] [stdout] | [INFO] [stdout] 35 | let mut sstack = ShadowStack::new(0,[0;255]); [INFO] [stdout] | ----^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: associated items `new`, `read_rd`, `read_rs1`, `read_rs2`, and `read_rs3` are never used [INFO] [stdout] --> src/cpu.rs:154:12 [INFO] [stdout] | [INFO] [stdout] 153 | impl State { [INFO] [stdout] | ---------- associated items in this implementation [INFO] [stdout] 154 | pub fn new(rd: usize, rs1: usize, rs2: usize, imm: u64) -> Self { [INFO] [stdout] | ^^^ [INFO] [stdout] ... [INFO] [stdout] 157 | pub fn read_rd(&self) -> usize { [INFO] [stdout] | ^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 160 | pub fn read_rs1(&self) -> usize { [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 163 | pub fn read_rs2(&self) -> usize { [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 166 | pub fn read_rs3(&self) -> u64 { [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(dead_code)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `SLL` is never used [INFO] [stdout] --> src/cpu.rs:67:15 [INFO] [stdout] | [INFO] [stdout] 67 | pub const SLL: u32 = 0b001; [INFO] [stdout] | ^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `SLT` is never used [INFO] [stdout] --> src/cpu.rs:68:15 [INFO] [stdout] | [INFO] [stdout] 68 | pub const SLT: u32 = 0b010; [INFO] [stdout] | ^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `SLTU` is never used [INFO] [stdout] --> src/cpu.rs:69:15 [INFO] [stdout] | [INFO] [stdout] 69 | pub const SLTU: u32 = 0b011; [INFO] [stdout] | ^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `XOR` is never used [INFO] [stdout] --> src/cpu.rs:70:15 [INFO] [stdout] | [INFO] [stdout] 70 | pub const XOR: u32 = 0b100; [INFO] [stdout] | ^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `SRL_SRA` is never used [INFO] [stdout] --> src/cpu.rs:71:15 [INFO] [stdout] | [INFO] [stdout] 71 | pub const SRL_SRA: u32 = 0b101; [INFO] [stdout] | ^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `OR` is never used [INFO] [stdout] --> src/cpu.rs:72:15 [INFO] [stdout] | [INFO] [stdout] 72 | pub const OR: u32 = 0b110; [INFO] [stdout] | ^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `AND` is never used [INFO] [stdout] --> src/cpu.rs:73:15 [INFO] [stdout] | [INFO] [stdout] 73 | pub const AND: u32 = 0b111; [INFO] [stdout] | ^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `hypervisor` is never used [INFO] [stdout] --> src/cpu.rs:89:15 [INFO] [stdout] | [INFO] [stdout] 89 | pub const hypervisor: u8 = 0b10; [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `lui` is never used [INFO] [stdout] --> src/cpu.rs:100:15 [INFO] [stdout] | [INFO] [stdout] 100 | pub const lui: u32 = 0b011; [INFO] [stdout] | ^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `swsp` is never used [INFO] [stdout] --> src/cpu.rs:110:15 [INFO] [stdout] | [INFO] [stdout] 110 | pub const swsp: u32 = 0b110; [INFO] [stdout] | ^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `sdsp` is never used [INFO] [stdout] --> src/cpu.rs:111:15 [INFO] [stdout] | [INFO] [stdout] 111 | pub const sdsp: u32 = 0b111; [INFO] [stdout] | ^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: function `get_imm_ld` is never used [INFO] [stdout] --> src/cpu.rs:980:12 [INFO] [stdout] | [INFO] [stdout] 980 | pub fn get_imm_ld(inst: u32) -> u32 { [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: methods `get_sp` and `get_stack` are never used [INFO] [stdout] --> src/shadowstack.rs:23:8 [INFO] [stdout] | [INFO] [stdout] 6 | impl ShadowStack { [INFO] [stdout] | ---------------- methods in this implementation [INFO] [stdout] ... [INFO] [stdout] 23 | fn get_sp(&self) -> usize{ [INFO] [stdout] | ^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 26 | fn get_stack(&self, i: usize) -> u64{ [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: methods `get_len` and `get_data` are never used [INFO] [stdout] --> src/bitutils.rs:50:12 [INFO] [stdout] | [INFO] [stdout] 20 | impl Bits { [INFO] [stdout] | --------- methods in this implementation [INFO] [stdout] ... [INFO] [stdout] 50 | pub fn get_len(&self) -> usize{ [INFO] [stdout] | ^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 53 | fn get_data(&self) -> u64 { [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `env` [INFO] [stdout] --> src/main.rs:4:11 [INFO] [stdout] | [INFO] [stdout] 4 | use std::{env, io}; [INFO] [stdout] | ^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_imports)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `str::FromStr` [INFO] [stdout] --> src/cpu.rs:10:25 [INFO] [stdout] | [INFO] [stdout] 10 | use std::{io, ops::Add, str::FromStr}; [INFO] [stdout] | ^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `add` should have an upper case name [INFO] [stdout] --> src/cpu.rs:128:7 [INFO] [stdout] | [INFO] [stdout] 128 | const add: RegisterInst = RegisterInst { [INFO] [stdout] | ^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(non_upper_case_globals)]` (part of `#[warn(nonstandard_style)]`) on by default [INFO] [stdout] help: convert the identifier to upper case [INFO] [stdout] | [INFO] [stdout] 128 - const add: RegisterInst = RegisterInst { [INFO] [stdout] 128 + const ADD: RegisterInst = RegisterInst { [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `user` should have an upper case name [INFO] [stdout] --> src/cpu.rs:87:15 [INFO] [stdout] | [INFO] [stdout] 87 | pub const user: u8 = 0b00; [INFO] [stdout] | ^^^^ [INFO] [stdout] | [INFO] [stdout] help: convert the identifier to upper case [INFO] [stdout] | [INFO] [stdout] 87 - pub const user: u8 = 0b00; [INFO] [stdout] 87 + pub const USER: u8 = 0b00; [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `supervisor` should have an upper case name [INFO] [stdout] --> src/cpu.rs:88:15 [INFO] [stdout] | [INFO] [stdout] 88 | pub const supervisor: u8 = 0b01; [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] help: convert the identifier to upper case [INFO] [stdout] | [INFO] [stdout] 88 - pub const supervisor: u8 = 0b01; [INFO] [stdout] 88 + pub const SUPERVISOR: u8 = 0b01; [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `hypervisor` should have an upper case name [INFO] [stdout] --> src/cpu.rs:89:15 [INFO] [stdout] | [INFO] [stdout] 89 | pub const hypervisor: u8 = 0b10; [INFO] [stdout] | ^^^^^^^^^^ help: convert the identifier to upper case: `HYPERVISOR` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `machine` should have an upper case name [INFO] [stdout] --> src/cpu.rs:90:15 [INFO] [stdout] | [INFO] [stdout] 90 | pub const machine: u8 = 0b11; [INFO] [stdout] | ^^^^^^^ [INFO] [stdout] | [INFO] [stdout] help: convert the identifier to upper case [INFO] [stdout] | [INFO] [stdout] 90 - pub const machine: u8 = 0b11; [INFO] [stdout] 90 + pub const MACHINE: u8 = 0b11; [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `ecall` should have an upper case name [INFO] [stdout] --> src/cpu.rs:94:15 [INFO] [stdout] | [INFO] [stdout] 94 | pub const ecall: u32 = 0; [INFO] [stdout] | ^^^^^ [INFO] [stdout] | [INFO] [stdout] help: convert the identifier to upper case [INFO] [stdout] | [INFO] [stdout] 94 - pub const ecall: u32 = 0; [INFO] [stdout] 94 + pub const ECALL: u32 = 0; [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `mret` should have an upper case name [INFO] [stdout] --> src/cpu.rs:95:15 [INFO] [stdout] | [INFO] [stdout] 95 | pub const mret: u32 = 0b001100000010; [INFO] [stdout] | ^^^^ [INFO] [stdout] | [INFO] [stdout] help: convert the identifier to upper case [INFO] [stdout] | [INFO] [stdout] 95 - pub const mret: u32 = 0b001100000010; [INFO] [stdout] 95 + pub const MRET: u32 = 0b001100000010; [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `li` should have an upper case name [INFO] [stdout] --> src/cpu.rs:99:15 [INFO] [stdout] | [INFO] [stdout] 99 | pub const li: u32 = 0b010; [INFO] [stdout] | ^^ [INFO] [stdout] | [INFO] [stdout] help: convert the identifier to upper case [INFO] [stdout] | [INFO] [stdout] 99 - pub const li: u32 = 0b010; [INFO] [stdout] 99 + pub const LI: u32 = 0b010; [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `lui` should have an upper case name [INFO] [stdout] --> src/cpu.rs:100:15 [INFO] [stdout] | [INFO] [stdout] 100 | pub const lui: u32 = 0b011; [INFO] [stdout] | ^^^ help: convert the identifier to upper case: `LUI` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `beqz` should have an upper case name [INFO] [stdout] --> src/cpu.rs:101:15 [INFO] [stdout] | [INFO] [stdout] 101 | pub const beqz: u32 = 0b110; [INFO] [stdout] | ^^^^ [INFO] [stdout] | [INFO] [stdout] help: convert the identifier to upper case [INFO] [stdout] | [INFO] [stdout] 101 - pub const beqz: u32 = 0b110; [INFO] [stdout] 101 + pub const BEQZ: u32 = 0b110; [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `slli` should have an upper case name [INFO] [stdout] --> src/cpu.rs:105:15 [INFO] [stdout] | [INFO] [stdout] 105 | pub const slli: u32 = 0b000; [INFO] [stdout] | ^^^^ [INFO] [stdout] | [INFO] [stdout] help: convert the identifier to upper case [INFO] [stdout] | [INFO] [stdout] 105 - pub const slli: u32 = 0b000; [INFO] [stdout] 105 + pub const SLLI: u32 = 0b000; [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `jal` should have an upper case name [INFO] [stdout] --> src/cpu.rs:106:15 [INFO] [stdout] | [INFO] [stdout] 106 | pub const jal: u32 = 0b001; [INFO] [stdout] | ^^^ [INFO] [stdout] | [INFO] [stdout] help: convert the identifier to upper case [INFO] [stdout] | [INFO] [stdout] 106 - pub const jal: u32 = 0b001; [INFO] [stdout] 106 + pub const JAL: u32 = 0b001; [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `lwsp` should have an upper case name [INFO] [stdout] --> src/cpu.rs:107:15 [INFO] [stdout] | [INFO] [stdout] 107 | pub const lwsp: u32 = 0b010; [INFO] [stdout] | ^^^^ [INFO] [stdout] | [INFO] [stdout] help: convert the identifier to upper case [INFO] [stdout] | [INFO] [stdout] 107 - pub const lwsp: u32 = 0b010; [INFO] [stdout] 107 + pub const LWSP: u32 = 0b010; [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `mv_ja` should have an upper case name [INFO] [stdout] --> src/cpu.rs:108:15 [INFO] [stdout] | [INFO] [stdout] 108 | pub const mv_ja: u32 = 0b100; [INFO] [stdout] | ^^^^^ [INFO] [stdout] | [INFO] [stdout] help: convert the identifier to upper case [INFO] [stdout] | [INFO] [stdout] 108 - pub const mv_ja: u32 = 0b100; [INFO] [stdout] 108 + pub const MV_JA: u32 = 0b100; [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `ldsp` should have an upper case name [INFO] [stdout] --> src/cpu.rs:109:15 [INFO] [stdout] | [INFO] [stdout] 109 | pub const ldsp: u32 = 0b011; [INFO] [stdout] | ^^^^ [INFO] [stdout] | [INFO] [stdout] help: convert the identifier to upper case [INFO] [stdout] | [INFO] [stdout] 109 - pub const ldsp: u32 = 0b011; [INFO] [stdout] 109 + pub const LDSP: u32 = 0b011; [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `swsp` should have an upper case name [INFO] [stdout] --> src/cpu.rs:110:15 [INFO] [stdout] | [INFO] [stdout] 110 | pub const swsp: u32 = 0b110; [INFO] [stdout] | ^^^^ help: convert the identifier to upper case: `SWSP` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `sdsp` should have an upper case name [INFO] [stdout] --> src/cpu.rs:111:15 [INFO] [stdout] | [INFO] [stdout] 111 | pub const sdsp: u32 = 0b111; [INFO] [stdout] | ^^^^ help: convert the identifier to upper case: `SDSP` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `sel` [INFO] [stdout] --> src/cpu.rs:129:12 [INFO] [stdout] | [INFO] [stdout] 129 | exec: |sel, state, reg| { [INFO] [stdout] | ^^^ help: if this is intentional, prefix it with an underscore: `_sel` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_variables)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `sp` [INFO] [stdout] --> src/cpu.rs:238:17 [INFO] [stdout] | [INFO] [stdout] 238 | let sp = match self.register.read(2,self.len) { [INFO] [stdout] | ^^ help: if this is intentional, prefix it with an underscore: `_sp` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `rs` [INFO] [stdout] --> src/cpu.rs:298:13 [INFO] [stdout] | [INFO] [stdout] 298 | let rs = Bits::cut_new(inst, 6, 2); [INFO] [stdout] | ^^ help: if this is intentional, prefix it with an underscore: `_rs` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/main.rs:32:9 [INFO] [stdout] | [INFO] [stdout] 32 | let mut mmu = Mmu::new(buf, test_mode); [INFO] [stdout] | ----^^^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_mut)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/main.rs:33:9 [INFO] [stdout] | [INFO] [stdout] 33 | let mut csr = Csr::new([0; 4096]); [INFO] [stdout] | ----^^^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/main.rs:34:9 [INFO] [stdout] | [INFO] [stdout] 34 | let mut reg = Register::new([0; 32]); [INFO] [stdout] | ----^^^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/main.rs:35:9 [INFO] [stdout] | [INFO] [stdout] 35 | let mut sstack = ShadowStack::new(0,[0;255]); [INFO] [stdout] | ----^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: associated items `new`, `read_rd`, `read_rs1`, `read_rs2`, and `read_rs3` are never used [INFO] [stdout] --> src/cpu.rs:154:12 [INFO] [stdout] | [INFO] [stdout] 153 | impl State { [INFO] [stdout] | ---------- associated items in this implementation [INFO] [stdout] 154 | pub fn new(rd: usize, rs1: usize, rs2: usize, imm: u64) -> Self { [INFO] [stdout] | ^^^ [INFO] [stdout] ... [INFO] [stdout] 157 | pub fn read_rd(&self) -> usize { [INFO] [stdout] | ^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 160 | pub fn read_rs1(&self) -> usize { [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 163 | pub fn read_rs2(&self) -> usize { [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 166 | pub fn read_rs3(&self) -> u64 { [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(dead_code)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `SLL` is never used [INFO] [stdout] --> src/cpu.rs:67:15 [INFO] [stdout] | [INFO] [stdout] 67 | pub const SLL: u32 = 0b001; [INFO] [stdout] | ^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `SLT` is never used [INFO] [stdout] --> src/cpu.rs:68:15 [INFO] [stdout] | [INFO] [stdout] 68 | pub const SLT: u32 = 0b010; [INFO] [stdout] | ^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `SLTU` is never used [INFO] [stdout] --> src/cpu.rs:69:15 [INFO] [stdout] | [INFO] [stdout] 69 | pub const SLTU: u32 = 0b011; [INFO] [stdout] | ^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `XOR` is never used [INFO] [stdout] --> src/cpu.rs:70:15 [INFO] [stdout] | [INFO] [stdout] 70 | pub const XOR: u32 = 0b100; [INFO] [stdout] | ^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `SRL_SRA` is never used [INFO] [stdout] --> src/cpu.rs:71:15 [INFO] [stdout] | [INFO] [stdout] 71 | pub const SRL_SRA: u32 = 0b101; [INFO] [stdout] | ^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `OR` is never used [INFO] [stdout] --> src/cpu.rs:72:15 [INFO] [stdout] | [INFO] [stdout] 72 | pub const OR: u32 = 0b110; [INFO] [stdout] | ^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `AND` is never used [INFO] [stdout] --> src/cpu.rs:73:15 [INFO] [stdout] | [INFO] [stdout] 73 | pub const AND: u32 = 0b111; [INFO] [stdout] | ^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `hypervisor` is never used [INFO] [stdout] --> src/cpu.rs:89:15 [INFO] [stdout] | [INFO] [stdout] 89 | pub const hypervisor: u8 = 0b10; [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `lui` is never used [INFO] [stdout] --> src/cpu.rs:100:15 [INFO] [stdout] | [INFO] [stdout] 100 | pub const lui: u32 = 0b011; [INFO] [stdout] | ^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `swsp` is never used [INFO] [stdout] --> src/cpu.rs:110:15 [INFO] [stdout] | [INFO] [stdout] 110 | pub const swsp: u32 = 0b110; [INFO] [stdout] | ^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `sdsp` is never used [INFO] [stdout] --> src/cpu.rs:111:15 [INFO] [stdout] | [INFO] [stdout] 111 | pub const sdsp: u32 = 0b111; [INFO] [stdout] | ^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: function `get_imm_ld` is never used [INFO] [stdout] --> src/cpu.rs:980:12 [INFO] [stdout] | [INFO] [stdout] 980 | pub fn get_imm_ld(inst: u32) -> u32 { [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `add` should have an upper case name [INFO] [stdout] --> src/cpu.rs:128:7 [INFO] [stdout] | [INFO] [stdout] 128 | const add: RegisterInst = RegisterInst { [INFO] [stdout] | ^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(non_upper_case_globals)]` (part of `#[warn(nonstandard_style)]`) on by default [INFO] [stdout] help: convert the identifier to upper case [INFO] [stdout] | [INFO] [stdout] 128 - const add: RegisterInst = RegisterInst { [INFO] [stdout] 128 + const ADD: RegisterInst = RegisterInst { [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `user` should have an upper case name [INFO] [stdout] --> src/cpu.rs:87:15 [INFO] [stdout] | [INFO] [stdout] 87 | pub const user: u8 = 0b00; [INFO] [stdout] | ^^^^ [INFO] [stdout] | [INFO] [stdout] help: convert the identifier to upper case [INFO] [stdout] | [INFO] [stdout] 87 - pub const user: u8 = 0b00; [INFO] [stdout] 87 + pub const USER: u8 = 0b00; [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `supervisor` should have an upper case name [INFO] [stdout] --> src/cpu.rs:88:15 [INFO] [stdout] | [INFO] [stdout] 88 | pub const supervisor: u8 = 0b01; [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] help: convert the identifier to upper case [INFO] [stdout] | [INFO] [stdout] 88 - pub const supervisor: u8 = 0b01; [INFO] [stdout] 88 + pub const SUPERVISOR: u8 = 0b01; [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `hypervisor` should have an upper case name [INFO] [stdout] --> src/cpu.rs:89:15 [INFO] [stdout] | [INFO] [stdout] 89 | pub const hypervisor: u8 = 0b10; [INFO] [stdout] | ^^^^^^^^^^ help: convert the identifier to upper case: `HYPERVISOR` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `machine` should have an upper case name [INFO] [stdout] --> src/cpu.rs:90:15 [INFO] [stdout] | [INFO] [stdout] 90 | pub const machine: u8 = 0b11; [INFO] [stdout] | ^^^^^^^ [INFO] [stdout] | [INFO] [stdout] help: convert the identifier to upper case [INFO] [stdout] | [INFO] [stdout] 90 - pub const machine: u8 = 0b11; [INFO] [stdout] 90 + pub const MACHINE: u8 = 0b11; [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `ecall` should have an upper case name [INFO] [stdout] --> src/cpu.rs:94:15 [INFO] [stdout] | [INFO] [stdout] 94 | pub const ecall: u32 = 0; [INFO] [stdout] | ^^^^^ [INFO] [stdout] | [INFO] [stdout] help: convert the identifier to upper case [INFO] [stdout] | [INFO] [stdout] 94 - pub const ecall: u32 = 0; [INFO] [stdout] 94 + pub const ECALL: u32 = 0; [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `mret` should have an upper case name [INFO] [stdout] --> src/cpu.rs:95:15 [INFO] [stdout] | [INFO] [stdout] 95 | pub const mret: u32 = 0b001100000010; [INFO] [stdout] | ^^^^ [INFO] [stdout] | [INFO] [stdout] help: convert the identifier to upper case [INFO] [stdout] | [INFO] [stdout] 95 - pub const mret: u32 = 0b001100000010; [INFO] [stdout] 95 + pub const MRET: u32 = 0b001100000010; [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `li` should have an upper case name [INFO] [stdout] --> src/cpu.rs:99:15 [INFO] [stdout] | [INFO] [stdout] 99 | pub const li: u32 = 0b010; [INFO] [stdout] | ^^ [INFO] [stdout] | [INFO] [stdout] help: convert the identifier to upper case [INFO] [stdout] | [INFO] [stdout] 99 - pub const li: u32 = 0b010; [INFO] [stdout] 99 + pub const LI: u32 = 0b010; [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `lui` should have an upper case name [INFO] [stdout] --> src/cpu.rs:100:15 [INFO] [stdout] | [INFO] [stdout] 100 | pub const lui: u32 = 0b011; [INFO] [stdout] | ^^^ help: convert the identifier to upper case: `LUI` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `beqz` should have an upper case name [INFO] [stdout] --> src/cpu.rs:101:15 [INFO] [stdout] | [INFO] [stdout] 101 | pub const beqz: u32 = 0b110; [INFO] [stdout] | ^^^^ [INFO] [stdout] | [INFO] [stdout] help: convert the identifier to upper case [INFO] [stdout] | [INFO] [stdout] 101 - pub const beqz: u32 = 0b110; [INFO] [stdout] 101 + pub const BEQZ: u32 = 0b110; [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `slli` should have an upper case name [INFO] [stdout] --> src/cpu.rs:105:15 [INFO] [stdout] | [INFO] [stdout] 105 | pub const slli: u32 = 0b000; [INFO] [stdout] | ^^^^ [INFO] [stdout] | [INFO] [stdout] help: convert the identifier to upper case [INFO] [stdout] | [INFO] [stdout] 105 - pub const slli: u32 = 0b000; [INFO] [stdout] 105 + pub const SLLI: u32 = 0b000; [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `jal` should have an upper case name [INFO] [stdout] --> src/cpu.rs:106:15 [INFO] [stdout] | [INFO] [stdout] 106 | pub const jal: u32 = 0b001; [INFO] [stdout] | ^^^ [INFO] [stdout] | [INFO] [stdout] help: convert the identifier to upper case [INFO] [stdout] | [INFO] [stdout] 106 - pub const jal: u32 = 0b001; [INFO] [stdout] 106 + pub const JAL: u32 = 0b001; [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `lwsp` should have an upper case name [INFO] [stdout] --> src/cpu.rs:107:15 [INFO] [stdout] | [INFO] [stdout] 107 | pub const lwsp: u32 = 0b010; [INFO] [stdout] | ^^^^ [INFO] [stdout] | [INFO] [stdout] help: convert the identifier to upper case [INFO] [stdout] | [INFO] [stdout] 107 - pub const lwsp: u32 = 0b010; [INFO] [stdout] 107 + pub const LWSP: u32 = 0b010; [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `mv_ja` should have an upper case name [INFO] [stdout] --> src/cpu.rs:108:15 [INFO] [stdout] | [INFO] [stdout] 108 | pub const mv_ja: u32 = 0b100; [INFO] [stdout] | ^^^^^ [INFO] [stdout] | [INFO] [stdout] help: convert the identifier to upper case [INFO] [stdout] | [INFO] [stdout] 108 - pub const mv_ja: u32 = 0b100; [INFO] [stdout] 108 + pub const MV_JA: u32 = 0b100; [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `ldsp` should have an upper case name [INFO] [stdout] --> src/cpu.rs:109:15 [INFO] [stdout] | [INFO] [stdout] 109 | pub const ldsp: u32 = 0b011; [INFO] [stdout] | ^^^^ [INFO] [stdout] | [INFO] [stdout] help: convert the identifier to upper case [INFO] [stdout] | [INFO] [stdout] 109 - pub const ldsp: u32 = 0b011; [INFO] [stdout] 109 + pub const LDSP: u32 = 0b011; [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `swsp` should have an upper case name [INFO] [stdout] --> src/cpu.rs:110:15 [INFO] [stdout] | [INFO] [stdout] 110 | pub const swsp: u32 = 0b110; [INFO] [stdout] | ^^^^ help: convert the identifier to upper case: `SWSP` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `sdsp` should have an upper case name [INFO] [stdout] --> src/cpu.rs:111:15 [INFO] [stdout] | [INFO] [stdout] 111 | pub const sdsp: u32 = 0b111; [INFO] [stdout] | ^^^^ help: convert the identifier to upper case: `SDSP` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/shadowstack.rs:35:5 [INFO] [stdout] | [INFO] [stdout] 35 | sstack.push(1); [INFO] [stdout] | ^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stdout] = note: `#[warn(unused_must_use)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 35 | let _ = sstack.push(1); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/shadowstack.rs:44:5 [INFO] [stdout] | [INFO] [stdout] 44 | sstack.push(1); [INFO] [stdout] | ^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 44 | let _ = sstack.push(1); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: panic message is not a string literal [INFO] [stdout] --> src/shadowstack.rs:50:26 [INFO] [stdout] | [INFO] [stdout] 50 | Err(r) => panic!(r), [INFO] [stdout] | ^ [INFO] [stdout] | [INFO] [stdout] = note: this usage of `panic!()` is deprecated; it will be a hard error in Rust 2021 [INFO] [stdout] = note: for more information, see [INFO] [stdout] = note: `#[warn(non_fmt_panics)]` (part of `#[warn(rust_2021_compatibility)]`) on by default [INFO] [stdout] help: add a "{}" format string to `Display` the message [INFO] [stdout] | [INFO] [stdout] 50 | Err(r) => panic!("{}", r), [INFO] [stdout] | +++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stderr] Finished `dev` profile [unoptimized + debuginfo] target(s) in 3.19s [INFO] running `Command { std: "docker" "inspect" "d885be245f5a5e8373aa43d90add1d37c8de98d2238797bb66a7a87cd75774ed", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "d885be245f5a5e8373aa43d90add1d37c8de98d2238797bb66a7a87cd75774ed", kill_on_drop: false }` [INFO] [stdout] d885be245f5a5e8373aa43d90add1d37c8de98d2238797bb66a7a87cd75774ed