Re: [gem5-dev] Review Request: checkpoint: move curTick from globals to eventq

2013-01-24 Thread Ali Saidi
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/1664/#review3897 --- I think the need for this somewhat depends on the threading model. It see

[gem5-dev] Review Request: checkpoint: move curTick from globals to eventq

2013-01-24 Thread Nilay Vaish
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/1664/ --- Review request for Default. Description --- Changeset 9480:c1585edc3d31 --

Re: [gem5-dev] Review Request: ARM: Fix an issue with clang generating wrong code.

2013-01-24 Thread Anthony Gutierrez
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/1662/#review3896 --- Ship it! Ship It! - Anthony Gutierrez On Jan. 22, 2013, 1:50 p.m., Al

Re: [gem5-dev] changeset in gem5: branch predictor: move out of o3 and inorder ...

2013-01-24 Thread Ali Saidi
Thanks for getting this done Nilay. Ali On 24.01.2013 12:25, Nilay Vaish wrote: > changeset d059f8a95a42 in /z/repo/gem5 > details: http://repo.gem5.org/gem5?cmd=changeset;node=d059f8a95a42 [1] > description: > branch predictor: move out of o3 and inorder cpus > This patch moves the branch

[gem5-dev] changeset in gem5: regressions: update stats due to branch predi...

2013-01-24 Thread Nilay Vaish
changeset b0fa6b872f40 in /z/repo/gem5 details: http://repo.gem5.org/gem5?cmd=changeset;node=b0fa6b872f40 description: regressions: update stats due to branch predictor changes The actual statistical values are being updated for only two tests belonging to sparc architectur

[gem5-dev] changeset in gem5: branch predictor: move out of o3 and inorder ...

2013-01-24 Thread Nilay Vaish
changeset d059f8a95a42 in /z/repo/gem5 details: http://repo.gem5.org/gem5?cmd=changeset;node=d059f8a95a42 description: branch predictor: move out of o3 and inorder cpus This patch moves the branch predictor files in the o3 and inorder directories to src/cpu/pred. This allow

Re: [gem5-dev] Review Request: branch predictor: move out of o3 and inorder cpus

2013-01-24 Thread Andreas Hansson
I think the parser issues we've had before and it even depends on how it is executed on zizzer (on command line, through python, and even with different ways of invoking it through python) Andreas On 24/01/2013 18:17, "Nilay" wrote: >On Thu, January 24, 2013 8:50 am, Nilay wrote: >> On Thu, Jan

Re: [gem5-dev] Review Request: branch predictor: move out of o3 and inorder cpus

2013-01-24 Thread Nilay
On Thu, January 24, 2013 8:50 am, Nilay wrote: > On Thu, January 24, 2013 3:08 am, Nilay wrote: > >> >> Seems like I will not be able to keep my promise. Following differences >> were found after running all the regression tests -- >> >> 1. tests/long/se/20.parser/ref/arm/linux/o3-timing/ >> sim_

Re: [gem5-dev] Review Request: cpu: Fix a livelock in the o3 cpu.

2013-01-24 Thread Andreas Sandberg
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/1660/#review3895 --- What's the point of the avoidQuiesceLiveLock flag? Is there anything prev

Re: [gem5-dev] Review Request: cache: remove drainManager because it's not used

2013-01-24 Thread Andreas Sandberg
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/1471/#review3894 --- Ship it! Ship It! - Andreas Sandberg On Jan. 23, 2013, 11:37 a.m., An

Re: [gem5-dev] Review Request: branch predictor: move out of o3 and inorder cpus

2013-01-24 Thread Nilay
On Thu, January 24, 2013 3:08 am, Nilay wrote: > > Seems like I will not be able to keep my promise. Following differences > were found after running all the regression tests -- > > 1. tests/long/se/20.parser/ref/arm/linux/o3-timing/ > sim_insts 505237723 505237723

Re: [gem5-dev] Review Request: branch predictor: move out of o3 and inorder cpus

2013-01-24 Thread Nilay
On Tue, January 15, 2013 6:15 pm, Ali Saidi wrote: > > On Jan 15, 2013, at 6:59 PM, Nilay Vaish wrote: > >> On Tue, 15 Jan 2013, Ali Saidi wrote: >> >>> >>> On Jan. 8, 2013, 6:29 a.m., Ali Saidi wrote: > Hi Nilay, > > Thanks for resurrecting this. In principle I have no issues, I

[gem5-dev] Cron /z/m5/regression/do-regression quick

2013-01-24 Thread Cron Daemon
scons: `build/ALPHA/gem5.debug' is up to date. scons: `build/ALPHA_MOESI_hammer/gem5.debug' is up to date. scons: `build/ALPHA_MESI_CMP_directory/gem5.debug' is up to date. scons: `build/ALPHA_MOESI_CMP_directory/gem5.debug' is up to date. scons: `build/ALPHA_MOESI_CMP_token/gem5.debug' is up to da