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

2010-08-12 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-timing-ruby passed. *

Re: [m5-dev] Review Request: memtest: Memtester support for DMA

2010-08-12 Thread Brad Beckmann
On 2010-08-11 19:43:23, Nathan Binkert wrote: It looks like you removed a whole lot of features in this diff. Why did you do this? Can you explain it in the commit message? Ruby doesn't work in atomic mode. Therefore it doesn't make sense to have atomic mode options in this file.

Re: [m5-dev] Review Request: m5: Fixed periodic interrupts to work with draining

2010-08-12 Thread Brad Beckmann
On 2010-08-11 19:47:25, Nathan Binkert wrote: Not sure what you did here, but I still think you should fix the commit message. I fixed the commit message, but hg postreview did not update the description. Basically this patch is needed to checkpoint in timing mode. Otherwise under

Re: [m5-dev] Review Request: ruby: hammer detail miss latency breakdown

2010-08-12 Thread Brad Beckmann
On 2010-08-08 20:53:56, Nathan Binkert wrote: The commit message is somewhat cryptic. Nathan Binkert wrote: Still think it's cryptic. Again, this is just a problem with hg postreview. Here is what the patch currently says: MOESI_hammer: break down miss latency stalled cycles This

Re: [m5-dev] Review Request: ruby: moved python protocol files

2010-08-12 Thread Beckmann, Brad
Yep...that is the issue. BTW, I did use hg mv for this patch. Brad -Original Message- From: Nathan Binkert [mailto:n...@binkert.org] Sent: Wednesday, August 11, 2010 7:54 PM To: Default; Beckmann, Brad; Nathan Binkert Subject: Re: Review Request: ruby: moved python protocol files

Re: [m5-dev] Review Request: ruby: hammer detail miss latency breakdown

2010-08-12 Thread nathan binkert
Again, this is just a problem with hg postreview.  Here is what the patch currently says: Understood. It took me a few reviews to realize that the system must be messed up. (I thought you might ignore some of my suggestions, but not most of them :) Nate

Re: [m5-dev] Review Request: config: Added the topology name to m5 ruby output

2010-08-12 Thread Steve Reinhardt
--- This is an automatically generated e-mail. To reply, visit: http://reviews.m5sim.org/r/88/#review170 --- This works, so I hate to complain, but it would be a lot more pythonic to

Re: [m5-dev] Review Request: config: Added the topology name to m5 ruby output

2010-08-12 Thread Nathan Binkert
On 2010-08-12 10:53:49, Steve Reinhardt wrote: This works, so I hate to complain, but it would be a lot more pythonic to turn topologies into a Python package (add an __init__.py file) and turn all the makeTopology() methods into __init__ methods on the respective Topology

[m5-dev] changeset in m5: TimingSimpleCPU: fix NO_ACCESS memory op handling

2010-08-12 Thread Joel Hestness
changeset cfbbc9178e7a in /z/repo/m5 details: http://repo.m5sim.org/m5?cmd=changeset;node=cfbbc9178e7a description: TimingSimpleCPU: fix NO_ACCESS memory op handling When a request is NO_ACCESS (x86 CDA microinstruction), the memory op doesn't go to the cache, so

[m5-dev] changeset in m5: util/m5/m5.c: ensure readfile() buffer pages ar...

2010-08-12 Thread Joel Hestness
changeset b69cc0fd934d in /z/repo/m5 details: http://repo.m5sim.org/m5?cmd=changeset;node=b69cc0fd934d description: util/m5/m5.c: ensure readfile() buffer pages are in page table (and marked dirty, in case that matters) by touching them beforehand diffstat: util/m5/m5.c | 5