Re: [gem5-dev] Review Request 2250: arm: add preliminary ISA splits for ARM arch

2014-05-04 Thread Steve Reinhardt via gem5-dev
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/2250/#review5087 --- Ship it! Ship It! - Steve Reinhardt On April 23, 2014, 5:24 a.m., And

Re: [gem5-dev] Review Request 2249: arch: support dynamic ISA file generation in per-ISA SConscripts

2014-05-04 Thread Steve Reinhardt via gem5-dev
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/2249/#review5086 --- If these changes are necessary for the build system to work with the ISA

Re: [gem5-dev] Review Request 2248: arch: support dynamic ISA file generation in SConscripts

2014-05-04 Thread Steve Reinhardt via gem5-dev
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/2248/#review5085 --- Are all these changes really related to the ISA split? Some look like i

Re: [gem5-dev] Review Request 2247: arch: teach ISA parser how to split code across files

2014-05-04 Thread Steve Reinhardt via gem5-dev
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/2247/#review5083 --- I didn't dig through this in detail, but it doesn't look any more confusi

Re: [gem5-dev] Review Request 2246: config: Add hooks to enable new config sys

2014-05-04 Thread Steve Reinhardt via gem5-dev
> On May 4, 2014, 4:56 p.m., Steve Reinhardt wrote: > > src/python/m5/simulate.py, line 117 > > > > > > for all these 'if hasattr()' changes (here and below, and in > > simulate.py): seems like it would be much cleaner to m

Re: [gem5-dev] micorop error

2014-05-04 Thread He Hu via gem5-dev
Hi Steve, We will try. There would be many conflicts waiting for us. Thanks, Hu -Original Message- From: gem5-dev-boun...@gem5.org [mailto:gem5-dev-boun...@gem5.org] On Behalf Of Steve Reinhardt Sent: Thursday, May 1, 2014 11:25 PM To: gem5 Developer List Subject: Re: [gem5-dev] micorop e

Re: [gem5-dev] Review Request 2246: config: Add hooks to enable new config sys

2014-05-04 Thread Steve Reinhardt via gem5-dev
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/2246/#review5082 --- Overall this looks pretty interesting. Some minor quibbles below, but no

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

2014-05-04 Thread Cron Daemon via gem5-dev
scons: `build/ALPHA_MOESI_hammer/tests/opt/quick/fs' is up to date. scons: `build/ALPHA_MOESI_hammer/tests/opt/long/se' is up to date. scons: `build/ALPHA_MOESI_hammer/tests/opt/long/fs' is up to date. scons: `build/ALPHA_MESI_Two_Level/tests/opt/quick/fs' is up to date. scons: `build/ALPHA_MESI_Tw