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

2011-04-14 Thread Cron Daemon
scons: *** Source `tests/quick/30.eio-mp/ref/alpha/eio/simple-timing-mp/stats.txt' not found, needed by target `build/ALPHA_SE/tests/fast/quick/30.eio-mp/alpha/eio/simple-timing-mp/status'. * build/ALPHA_SE/tests/fast/quick/30.eio-mp/alpha/eio/simple-atomic-mp passed. * build/ALPHA_SE/t

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

2011-04-14 Thread Nilay Vaish
Brad, can you elaborate on implementing functional accesses for the PioPort? -- Nilay On Wed, 13 Apr 2011, Beckmann, Brad wrote: I just reviewed it. Please let me know if you have any questions. Brad -Original Message- From: m5-dev-boun...@m5sim.org [mailto:m5-dev-boun...@m5sim.o

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

2011-04-14 Thread Nilay Vaish
> On 2011-04-13 13:43:26, Gabe Black wrote: > > Please fix these style issues, including the ones in this file I haven't > > explicitly pointed out. You should be using M5 style generally, but > > especially when your in M5 code. Also, please be sure to point this out to > > one of the classic

[m5-dev] Ruby patch touching classic memory system

2011-04-14 Thread Nilay Vaish
Ali, Nate, Steve Can you go through this patch? I have made some changes to src/mem/packet.hh and src/mem/packet.cc files. http://reviews.m5sim.org/r/611/ Thanks Nilay On Thu, 14 Apr 2011, Nilay Vaish wrote: On 2011-04-13 13:43:26, Gabe Black wrote: Please fix these style issues, incl

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

2011-04-14 Thread Nilay Vaish
> On 2011-04-13 10:28:08, Brad Beckmann wrote: > > src/mem/protocol/MESI_CMP_directory-L1cache.sm, line 141 > > > > > > Why are you adding this function? SLICC already generates a similar > > function: getPermission(). > >

Re: [m5-dev] In-Order ALPHA FS Update

2011-04-14 Thread soumyaroop roy
I am looking forward to it VERY MUCH! -Soumyaroop On Wed, Apr 13, 2011 at 3:48 PM, Gabriel Michael Black < gbl...@eecs.umich.edu> wrote: > Nice work :-). Hopefully we'll get rid of those last vestiges of the Alpha > only days (hwrei, etc) one of these days. > > Gabe > > > Quoting Korey Sewell :

Re: [m5-dev] In-Order ALPHA FS Update

2011-04-14 Thread Korey Sewell
Thanks Gabe. I'll take note of the alpha-specific things and see if things like hwrei, isIprAccess, and the like can't be generalized a bit more so that they can be hidden from the specifics of the CPU models. 1st things 1st though...try to get it working! On Wed, Apr 13, 2011 at 4:48 PM, Gabrie

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

2011-04-14 Thread Brad Beckmann
> On 2011-04-13 10:28:08, Brad Beckmann wrote: > > src/mem/protocol/MESI_CMP_directory-L1cache.sm, line 141 > > > > > > 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-14 Thread Beckmann, Brad
Hi Nilay, Let me start off by saying that I'm not sure if I fully understand the complexities of dealing with functional accesses from the PioPort and I could be overlooking a key concern. I think implementing functional accesses for the PioPort should be very similar to cpu functional acces

Re: [m5-dev] Review Request: python: cleanup python code so stuff doesn't automatically happen at startup

2011-04-14 Thread Ali Saidi
--- This is an automatically generated e-mail. To reply, visit: http://reviews.m5sim.org/r/355/#review1120 --- Ship it! Commit message as written exceeds 80 charecters. - Ali On 20

Re: [m5-dev] Review Request: style: add sort_includes to the style hook

2011-04-14 Thread Ali Saidi
--- This is an automatically generated e-mail. To reply, visit: http://reviews.m5sim.org/r/643/#review1121 --- Ship it! - Ali On 2011-04-13 09:49:03, Nathan Binkert wrote: > >

Re: [m5-dev] Review Request: style: add a user interface wrapper class

2011-04-14 Thread Ali Saidi
--- This is an automatically generated e-mail. To reply, visit: http://reviews.m5sim.org/r/641/#review1122 --- util/style.py This is

Re: [m5-dev] Review Request: style: add a user interface wrapper class

2011-04-14 Thread Ali Saidi
--- This is an automatically generated e-mail. To reply, visit: http://reviews.m5sim.org/r/641/#review1123 --- Ship it! - Ali On 2011-04-13 09:48:38, Nathan Binkert wrote: > >

Re: [m5-dev] Review Request: region: add a utility class for keeping track of regions of some range

2011-04-14 Thread Ali Saidi
--- This is an automatically generated e-mail. To reply, visit: http://reviews.m5sim.org/r/640/#review1124 --- Ship it! Could you put what you intend to use this for in the commit des

Re: [m5-dev] Review Request: SortedDict: add functions for getting ranges of keys, values, items

2011-04-14 Thread Ali Saidi
--- This is an automatically generated e-mail. To reply, visit: http://reviews.m5sim.org/r/639/#review1125 --- Ship it! - Ali On 2011-04-13 09:47:39, Nathan Binkert wrote: > >

Re: [m5-dev] Review Request: style: add a user interface wrapper class

2011-04-14 Thread Nathan Binkert
--- This is an automatically generated e-mail. To reply, visit: http://reviews.m5sim.org/r/641/#review1126 --- util/style.py Hmmm.