Re: [gem5-dev] Review Request 3780: riscv: Fix crash when syscall argument reg index is too high

2017-01-19 Thread Brandon Potter
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3780/#review9282 --- Ship it! Ship It! - Brandon Potter On Jan. 18, 2017, 5:39 p.m., Alec

Re: [gem5-dev] Review Request 3780: riscv: Fix crash when syscall argument reg index is too high

2017-01-18 Thread Jason Lowe-Power
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3780/#review9267 --- Ship it! Ship It! - Jason Lowe-Power On Jan. 18, 2017, 5:39 p.m.,

Re: [gem5-dev] Review Request 3780: riscv: Fix crash when syscall argument reg index is too high

2017-01-18 Thread Alec Roelke
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3780/ --- (Updated Jan. 18, 2017, 5:39 p.m.) Review request for Default. Repository: gem5

Re: [gem5-dev] Review Request 3780: riscv: Fix crash when syscall argument reg index is too high

2017-01-18 Thread Jason Lowe-Power
> On Jan. 18, 2017, 4:06 p.m., Jason Lowe-Power wrote: > > src/arch/riscv/process.cc, line 222 > > > > > > Two comments: > > 1. Shouldn't this at least have a warning? If the syscall arg register > > > 3 there is

Re: [gem5-dev] Review Request 3780: riscv: Fix crash when syscall argument reg index is too high

2017-01-18 Thread Alec Roelke
> On Jan. 18, 2017, 4:06 p.m., Jason Lowe-Power wrote: > > src/arch/riscv/process.cc, line 222 > > > > > > Two comments: > > 1. Shouldn't this at least have a warning? If the syscall arg register > > > 3 there is

[gem5-dev] Review Request 3780: riscv: Fix crash when syscall argument reg index is too high

2017-01-12 Thread Alec Roelke
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3780/ --- Review request for Default. Repository: gem5 Description --- Changeset