Re: [gem5-dev] Review Request 3806: sparc: fix bugs caused by cd7f3a1dbf55

2017-02-13 Thread Steve Reinhardt
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3806/#review9427 --- Ship it! I don't remember much about SPARC address translation, but

[gem5-dev] Review Request 3810: x86: adjust Walker::WalkerState::pageFault to not change mode

2017-02-13 Thread Bjoern A. Zeeb
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3810/ --- Review request for Default. Repository: gem5 Description --- After

[gem5-dev] Open Review Request - 3806

2017-02-13 Thread Potter, Brandon
Hello all, I have a review open to fix a bug for the SPARC TLB and translation code that only affects SE Mode. The fix should solve the SE Mode regression issue where the SPARC tests run forever. I posted it a little over a week ago and plan to push it this coming Thursday (02/16/17) if I do

[gem5-dev] changeset in gem5: riscv: Remove ECALL tests from insttest

2017-02-13 Thread Alec Roelke
changeset 0e73ec98b6bc in /z/repo/gem5 details: http://repo.gem5.org/gem5?cmd=changeset;node=0e73ec98b6bc description: 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

[gem5-dev] changeset in gem5: misc: Clean up and complete the gem5<->System...

2017-02-13 Thread Christian Menard
changeset 4d1f9823013a in /z/repo/gem5 details: http://repo.gem5.org/gem5?cmd=changeset;node=4d1f9823013a description: misc: Clean up and complete the gem5<->SystemC-TLM bridge [6/10] The current TLM bridge only provides a Slave Port that allows the gem5 world to send

Re: [gem5-dev] Review Request 3809: imported patch rr_arbiter_fix

2017-02-13 Thread Jason Lowe-Power
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3809/#review9426 --- Ship it! Ship It! - Jason Lowe-Power On Feb. 12, 2017, 4:36 a.m.,

Re: [gem5-dev] RISC-V: Unknown opcode 0x00

2017-02-13 Thread ar4jc
Hi Boris, Have you had any luck with this? I’ve taken another look at the loader debug output and the exec trace, and I’m stumped. The data from the loader appears correct as far as addresses and sizes of segments go, and as I said the address gem5 tries to load from when it crashes contains

Re: [gem5-dev] Review Request 3781: riscv: Remove ECALL tests from insttest

2017-02-13 Thread Jason Lowe-Power
> On Jan. 25, 2017, 11:28 a.m., Andreas Sandberg wrote: > > Ship It! > > Andreas Hansson wrote: > What was the verdict? Is it just a matter of an updated patch > description/summary and then this should be pushed? > > Andreas Sandberg wrote: > Yes, the description needs to be updated.

Re: [gem5-dev] Review Request 3781: riscv: Remove ECALL tests from insttest

2017-02-13 Thread Alec Roelke
> On Jan. 25, 2017, 11:28 a.m., Andreas Sandberg wrote: > > Ship It! > > Andreas Hansson wrote: > What was the verdict? Is it just a matter of an updated patch > description/summary and then this should be pushed? > > Andreas Sandberg wrote: > Yes, the description needs to be updated.

Re: [gem5-dev] Review Request 3781: riscv: Remove ECALL tests from insttest

2017-02-13 Thread Alec Roelke
--- 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

Re: [gem5-dev] Review Request 3781: riscv: Remove ECALL tests from insttest

2017-02-13 Thread Andreas Sandberg
> On Jan. 25, 2017, 11:28 a.m., Andreas Sandberg wrote: > > Ship It! > > Andreas Hansson wrote: > What was the verdict? Is it just a matter of an updated patch > description/summary and then this should be pushed? Yes, the description needs to be updated. I'm not sure what happened to the