----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3781/ -----------------------------------------------------------
(Updated Feb. 13, 2017, 3 p.m.) Review request for Default. Repository: gem5 Description (updated) ------- Changeset 11808:1c38f8e541ba --------------------------- riscv: Remove ECALL tests from insttest The system calls tested in rv64i.cpp in RISC-V's insttest suite have different behavior depending on the operating system and file system they are run on. This patch ignores the output of those tests and only ensures that the instructions in RV64I complete successfully. [Change deletion of ECALL test to block comment.] [Restore ECALL test but remove test output to test only for completion without error.] [Update patch description and again try to push EMPTY files for rv64i tests.] Diffs (updated) ----- tests/quick/se/02.insttest/ref/riscv/linux-rv64i/minor-timing/simout 63325e5b0a9d tests/quick/se/02.insttest/ref/riscv/linux-rv64i/minor-timing/simerr 63325e5b0a9d tests/quick/se/02.insttest/ref/riscv/linux-rv64i/minor-timing/config.ini 63325e5b0a9d tests/quick/se/02.insttest/ref/riscv/linux-rv64i/minor-timing/config.json 63325e5b0a9d tests/quick/se/02.insttest/ref/riscv/linux-rv64i/simple-timing/config.json 63325e5b0a9d tests/quick/se/02.insttest/ref/riscv/linux-rv64i/simple-timing-ruby/stats.txt 63325e5b0a9d tests/quick/se/02.insttest/ref/riscv/linux-rv64i/simple-timing/config.ini 63325e5b0a9d tests/quick/se/02.insttest/ref/riscv/linux-rv64i/simple-atomic/config.ini 63325e5b0a9d tests/quick/se/02.insttest/ref/riscv/linux-rv64i/simple-atomic/config.json 63325e5b0a9d tests/quick/se/02.insttest/ref/riscv/linux-rv64i/simple-atomic/simerr 63325e5b0a9d tests/quick/se/02.insttest/ref/riscv/linux-rv64i/simple-atomic/simout 63325e5b0a9d tests/quick/se/02.insttest/ref/riscv/linux-rv64i/simple-atomic/stats.txt 63325e5b0a9d tests/quick/se/02.insttest/ref/riscv/linux-rv64i/simple-timing-ruby/config.ini 63325e5b0a9d tests/quick/se/02.insttest/ref/riscv/linux-rv64i/simple-timing-ruby/config.json 63325e5b0a9d tests/quick/se/02.insttest/ref/riscv/linux-rv64i/simple-timing-ruby/simerr 63325e5b0a9d tests/quick/se/02.insttest/ref/riscv/linux-rv64i/simple-timing-ruby/simout 63325e5b0a9d tests/quick/se/02.insttest/ref/riscv/linux-rv64i/minor-timing/stats.txt 63325e5b0a9d tests/quick/se/02.insttest/ref/riscv/linux-rv64i/simple-timing/simerr 63325e5b0a9d tests/quick/se/02.insttest/ref/riscv/linux-rv64i/simple-timing/simout 63325e5b0a9d tests/quick/se/02.insttest/ref/riscv/linux-rv64i/simple-timing/stats.txt 63325e5b0a9d Diff: http://reviews.gem5.org/r/3781/diff/ Testing ------- Thanks, Alec Roelke _______________________________________________ gem5-dev mailing list gem5-dev@gem5.org http://m5sim.org/mailman/listinfo/gem5-dev