[m5-dev] Cron m5test@zizzer /z/m5/regression/do-regression quick

2011-04-16 Thread Cron Daemon
* build/ALPHA_SE/tests/fast/quick/00.hello/alpha/linux/simple-timing-ruby FAILED! * build/ALPHA_SE/tests/fast/quick/60.rubytest/alpha/linux/rubytest-ruby FAILED! * build/ALPHA_SE/tests/fast/quick/20.eio-short/alpha/eio/simple-timing FAILED! *

[m5-dev] Testing Ruby Memory Controller

2011-04-16 Thread Nilay Vaish
Is there a tester for testing the functionality of Ruby's memory controller? Thanks Nilay ___ m5-dev mailing list m5-dev@m5sim.org http://m5sim.org/mailman/listinfo/m5-dev

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

2011-04-16 Thread Nilay Vaish
On 2011-04-13 10:28:08, Brad Beckmann wrote: src/mem/protocol/MESI_CMP_directory-L1cache.sm, line 141 http://reviews.m5sim.org/r/611/diff/6/?file=11548#file11548line141 Why are you adding this function? SLICC already generates a similar function: getPermission().

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

2011-04-16 Thread Nilay Vaish
On 2011-04-15 15:57:28, Steve Reinhardt wrote: configs/example/ruby_mem_test.py, line 98 http://reviews.m5sim.org/r/611/diff/8/?file=11625#file11625line98 You probably don't want to commit this value, do you? If you want to hardwire a number, I'd pick something more reasonable

Re: [m5-dev] Cron m5test@zizzer /z/m5/regression/do-regression quick

2011-04-16 Thread nathan binkert
This is likely due to my push last night, though I ran the full regression suite before I pushed. My guess is that the directory needs to be blown away so a fresh build can happen. Can someone do that for me? I'm in the car about to go camping. Nate On Sat, Apr 16, 2011 at 12:56 AM, Cron

[m5-dev] Bug in changeset 8225 or 8227

2011-04-16 Thread Nilay Vaish
Nate, it seems one of your checkins from yesterday has a bug. I receive the following message on executing any merculrial command. *** failed to import extension style from ./util/style.py: invalid syntax (file_types.py, line 143) -- Nilay ___

Re: [m5-dev] Bug in changeset 8225 or 8227

2011-04-16 Thread nathan binkert
What version of python are you using? It could be that that syntax wasn't available until 2.5 and you're using something older. I can't do anything about it right now because I'm about to leave on a hike. Nate Nate, it seems one of your checkins from yesterday has a bug. I receive the

Re: [m5-dev] Bug in changeset 8225 or 8227

2011-04-16 Thread Nilay Vaish
On Sat, 16 Apr 2011, nathan binkert wrote: What version of python are you using? It could be that that syntax wasn't available until 2.5 and you're using something older. I can't do anything about it right now because I'm about to leave on a hike. Nate Nate, it seems one of your checkins

Re: [m5-dev] Bug in changeset 8225 or 8227

2011-04-16 Thread Steve Reinhardt
Yea, if you look at the mentioned line, it's mode = 'r+' if inplace else 'r', and conditional assignments like this (basically python's version of C's ?:) weren't added until 2.5. Since we officially only require python 2.4 (not 2.5), I'm happy to call this Nate's fault. If you want to change

Re: [m5-dev] Cron m5test@zizzer /z/m5/regression/do-regression quick

2011-04-16 Thread Gabe Black
/z/m5/regression/zizzer/m5/build has been deleted. I'm pretty sure that will do it, but let me know if I need to remove anything else. Gabe On 04/16/11 11:13, nathan binkert wrote: This is likely due to my push last night, though I ran the full regression suite before I pushed. My guess is

Re: [m5-dev] Bug in changeset 8225 or 8227

2011-04-16 Thread Gabe Black
Since we seem to have some folks using an older python, my vote would be to make it 2.4 compatible. I don't have a good intuition about how old 2.4 is, though. If it's really ancient we might want to bump up the required version and force Nilay (and anybody else) to upgrade. Gabe On 04/16/11