Re: [m5-dev] Review Request: ARM/Alpha/Cpu: Change prefetchs to be more like normal loads.

2010-11-03 Thread Ali Saidi
I can do that, but I think it is a bad habit that we should change. I'm happy to do it for review, but for committing it is the wrong plan because it means that there are changesets in the repo that do not pass regressions. Ali On Nov 3, 2010, at 10:08 PM, Steve Reinhardt wrote: > > -

Re: [m5-dev] Review Request: scons: add a parameter to configure SCons' build cache

2010-11-03 Thread Ali Saidi
That depends. If you share a cache with a couple other people, yes it does. Ali On Nov 3, 2010, at 9:58 PM, Steve Reinhardt wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > http://reviews.m5sim.org/r/287/#rev

Re: [m5-dev] Implementation of findTagInSet

2010-11-03 Thread Steve Reinhardt
Ah, the issue is that you're using the old M5 memory hierarchy and not Ruby. You need to run one of the Ruby versions, and use ruby_fs.py instead of fs.py. Steve On Wed, Nov 3, 2010 at 9:00 PM, Nilay wrote: > I ran m5.prof two times. Here are the top five functions -- > > % cumulative sel

Re: [m5-dev] Implementation of findTagInSet

2010-11-03 Thread Nilay
I ran m5.prof two times. Here are the top five functions -- % cumulative self self total time seconds secondscalls s/call s/call name 8.35 5.05 5.05 58969209 0.00 0.00 BaseSimpleCPU::preExecute() 6.32 8.87 3.82 58975463 0.00

Re: [m5-dev] Review Request: ARM/Alpha/Cpu: Change prefetchs to be more like normal loads.

2010-11-03 Thread Steve Reinhardt
--- This is an automatically generated e-mail. To reply, visit: http://reviews.m5sim.org/r/256/#review431 --- Please split out the code changes and the stats changes into separate cha

Re: [m5-dev] Review Request: sim: Use forward declarations for ports.

2010-11-03 Thread Steve Reinhardt
--- This is an automatically generated e-mail. To reply, visit: http://reviews.m5sim.org/r/285/#review430 --- Ship it! - Steve On 2010-11-03 16:12:10, Ali Saidi wrote: > >

Re: [m5-dev] Review Request: scons: add a parameter to configure SCons' build cache

2010-11-03 Thread Steve Reinhardt
--- This is an automatically generated e-mail. To reply, visit: http://reviews.m5sim.org/r/287/#review429 --- Ship it! Looks fine to me assuming it works. Does it help much? - Steve

Re: [m5-dev] Implementation of findTagInSet

2010-11-03 Thread Steve Reinhardt
What was the gprof output? On Wed, Nov 3, 2010 at 4:45 PM, Nilay Vaish wrote: > I profiled M5 but surprisingly I did not find any mention of the function > findTagInSet() in the output obtained from gprof. Does it matter what > coherence protocol is in use? I carried out the following step - > >

[m5-dev] Status of ARM support

2010-11-03 Thread Timothy M Jones
Hi Ali, Could you tell me roughly what the status of ARM support is for full-system simulation? I'm thinking of using M5 ARM for a future project (which won't start until sometime next year) and I will want a simulator that I can run Linux on. Cheers Tim -- Timothy M. Jones http://homepage

Re: [m5-dev] Review Request: sim: Use forward declarations for ports.

2010-11-03 Thread Gabe Black
--- This is an automatically generated e-mail. To reply, visit: http://reviews.m5sim.org/r/285/#review428 --- Ship it! Seems like a good idea to me. - Gabe On 2010-11-03 16:12:10,

Re: [m5-dev] Implementation of findTagInSet

2010-11-03 Thread Nilay Vaish
I profiled M5 but surprisingly I did not find any mention of the function findTagInSet() in the output obtained from gprof. Does it matter what coherence protocol is in use? I carried out the following step - 1. Compiled m5.prof using scons -j 6 USE_MYSQL=False RUBY=True build/ALPHA_FS/m5.prof

[m5-dev] Review Request: scons: add a parameter to configure SCons' build cache

2010-11-03 Thread Ali Saidi
--- This is an automatically generated e-mail. To reply, visit: http://reviews.m5sim.org/r/287/ --- Review request for Default. Summary --- scons: add a parameter to configure SC

Re: [m5-dev] Review Request: ARM/Alpha/Cpu: Change prefetchs to be more like normal loads.

2010-11-03 Thread Ali Saidi
--- This is an automatically generated e-mail. To reply, visit: http://reviews.m5sim.org/r/256/ --- (Updated 2010-11-03 16:13:11.575399) Review request for Default. Summary (updated

[m5-dev] Review Request: runAddrLast and runDataLast shouldn't be stats

2010-11-03 Thread Ali Saidi
--- This is an automatically generated e-mail. To reply, visit: http://reviews.m5sim.org/r/286/ --- Review request for Default. Summary --- runAddrLast and runDataLast shouldn't

[m5-dev] Review Request: sim: Use forward declarations for ports.

2010-11-03 Thread Ali Saidi
--- This is an automatically generated e-mail. To reply, visit: http://reviews.m5sim.org/r/285/ --- Review request for Default. Summary --- sim: Use forward declarations for port

Re: [m5-dev] "build_dir has been deprecated"

2010-11-03 Thread Gabriel Michael Black
A little research shows that as of scons 0.98 the variant_dir variable is available, older than 0.98.1 which we require. It looks like as of 2.0.1 they started complaining if you use build_dir. I'll put together a patch to move us over in the next day or two. I expect it to be very simple,

Re: [m5-dev] [m5-users] Hope for m5 API for Configuration a system.

2010-11-03 Thread Gabriel Michael Black
The best place to look is in the .py files in the src directory. These are where the parameters are set up, and there isn't really any other documentation. You shouldn't change the values in those files since those just set the defaults, but that will let you know what parameters there are

[m5-dev] Hope for m5 API for Configuration a system.

2010-11-03 Thread Gdansk Amir
Hello: I am a m5 users,and now i want to use m5 to build a new system. But when i want to write a Configuration script, i fond it was very hard to define the attribute of the objects, because i don't know what attribute of the object and the accurat word of this corresponding object. So i ant to

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

2010-11-03 Thread Cron Daemon
* build/ALPHA_SE/tests/fast/quick/00.hello/alpha/linux/o3-timing passed. * build/ALPHA_SE/tests/fast/quick/00.hello/alpha/linux/inorder-timing passed. * build/ALPHA_SE/tests/fast/quick/00.hello/alpha/linux/simple-atomic passed. * build/ALPHA_SE/tests/fast/quick/00.hello/alpha/linux