Re: [gem5-dev] Review Request: ISA parser: Simplify operand type handling.

2011-06-12 Thread Gabe Black
On 06/04/11 09:32, Steve Reinhardt wrote: On Sat, Jun 4, 2011 at 1:57 AM, Gabe Black gbl...@eecs.umich.edu wrote: To clarify, is this signed/unsigned issue something we need to deal with before this patch goes in, or can it be dealt with separately later? I'd like to see it handled before the

Re: [gem5-dev] Review Request: LibElf: Build the error management code in libelf.

2011-06-12 Thread Gabe Black
Ping On 06/04/11 11:39, Gabe Black wrote: --- This is an automatically generated e-mail. To reply, visit: http://reviews.m5sim.org/r/735/ --- Review request for Default, Ali

Re: [gem5-dev] Review Request: Loader: Handle bad section names when loading an ELF file.

2011-06-12 Thread Gabe Black
Ping On 06/04/11 11:42, Gabe Black wrote: --- This is an automatically generated e-mail. To reply, visit: http://reviews.m5sim.org/r/736/ --- Review request for Default, Ali

Re: [gem5-dev] Review Request: [mq]: FunctionalAccess.9.patch

2011-06-12 Thread Nilay Vaish
--- This is an automatically generated e-mail. To reply, visit: http://reviews.m5sim.org/r/611/ --- (Updated 2011-06-12 14:55:00.907885) Review request for Default. Summary

Re: [gem5-dev] Review Request: Ruby: Add support for functional accesses

2011-06-12 Thread Nilay Vaish
--- This is an automatically generated e-mail. To reply, visit: http://reviews.m5sim.org/r/611/ --- (Updated 2011-06-12 14:55:53.667339) Review request for Default. Summary

[gem5-dev] Cron m5test@zizzer /z/m5/regression/do-regression --scratch all

2011-06-12 Thread Cron Daemon
* build/SPARC_SE/tests/opt/long/70.twolf/sparc/linux/simple-atomic FAILED! * build/SPARC_SE/tests/opt/long/50.vortex/sparc/linux/simple-timing FAILED! * build/SPARC_SE/tests/opt/long/10.mcf/sparc/linux/simple-timing FAILED! *

Re: [gem5-dev] Cron m5test@zizzer /z/m5/regression/do-regression --scratch all

2011-06-12 Thread Korey Sewell
Looks like I still got some work to do w/the SPARC regressions. I definitely didnt think adding a few control flags would cause pain, but I must've unknowingly hit some other spot in the code since control flags should have nothing to do with how the simple cpus operate. On Sun, Jun 12, 2011 at

Re: [gem5-dev] Review Request: Loader: Handle bad section names when loading an ELF file.

2011-06-12 Thread Steve Reinhardt
--- This is an automatically generated e-mail. To reply, visit: http://reviews.m5sim.org/r/736/#review1325 --- Ship it! - Steve On 2011-06-04 11:42:05, Gabe Black wrote:

Re: [gem5-dev] Review Request: LibElf: Build the error management code in libelf.

2011-06-12 Thread Steve Reinhardt
--- This is an automatically generated e-mail. To reply, visit: http://reviews.m5sim.org/r/735/#review1326 --- Ship it! - Steve On 2011-06-04 11:39:02, Gabe Black wrote:

Re: [gem5-dev] Review Request: inorder/dtb: make sure DTB translate correct address

2011-06-12 Thread Korey Sewell
On 2011-06-11 09:56:39, Steve Reinhardt wrote: Shouldn't you get rid of the cache_unit.cc changes now? I thought that was the point. This is still a hack, in my opinion; note that the comment on the _pc field in mem/request.hh says for tracing/debugging, i.e., it's not intended to

Re: [gem5-dev] Cron m5test@zizzer /z/m5/regression/do-regression --scratch all

2011-06-12 Thread Korey Sewell
ok, looks like these sparc runs are just regression updates that I didnt do for the long runs (phew!). I'll update them shortly. On Sun, Jun 12, 2011 at 6:19 PM, Korey Sewell ksew...@umich.edu wrote: Looks like I still got some work to do w/the SPARC regressions. I definitely didnt think

Re: [gem5-dev] Review Request: ISA parser: Simplify operand type handling.

2011-06-12 Thread Steve Reinhardt
On Sun, Jun 12, 2011 at 1:05 AM, Gabe Black gbl...@eecs.umich.edu wrote: I was thinking about this today, and if we expand the read/write functions to handle signed types too, we're really just expanding the arbitrary set of types they can handle, not removing the limitation that you have to

[gem5-dev] changeset in gem5: sparc: update long regressions

2011-06-12 Thread Korey Sewell
changeset 7a32aa3acd72 in /z/repo/gem5 details: http://repo.gem5.org/gem5?cmd=changeset;node=7a32aa3acd72 description: sparc: update long regressions diffstat: tests/long/00.gzip/ref/sparc/linux/o3-timing/config.ini| 2 +-