[m5-dev] CacheController's wakeup function

2011-02-14 Thread Nilay Vaish
I thought of this a moment ago, so I have not confirmed this empirically. The CacheController's wakeup function includes a while loop, in which all the queues are checked. Consider the Hammer protocol's L1 Cache Controller. It has four incoming queues - trigger, response, forward, mandatory. Th

[m5-dev] Next steps

2011-02-14 Thread Nilay Vaish
Hi Brad, I have checked in the patch dealing with Perfect Switch. I am will clean the patch that removes CacheMsg class soon. What to take up next? I am kind of bored with this optimization stuff right now. -- Nilay ___ m5-dev mailing list m5-dev@m5

Re: [m5-dev] Review Request: Alpha: Import the alpha-system files into the m5 repository.

2011-02-14 Thread nathan binkert
> Actually, I think we want to run HG convert (from hg to hg) first and then do a pull -f after that. I can take a look at this tonight. Ok. I think I've got this right. I'm not going to put it on reviewboard since there are so many changes. I've got the formula below for the hg convert. Afte

[m5-dev] changeset in m5: Info: Clean up some info files.

2011-02-14 Thread Gabe Black
changeset 13692327bb0b in /z/repo/m5 details: http://repo.m5sim.org/m5?cmd=changeset;node=13692327bb0b description: Info: Clean up some info files. Get rid of RELEASE_NOTES since we no longer do releases, update some of the information in README, and update the date in LIC

Re: [m5-dev] Review Request: Info: Clean up some info files.

2011-02-14 Thread Gabe Black
Yeah, that would probably be overkill to do at build time. It would probably be useful for initially extracting licenses to put in a LICENSES directory though. Gabe On 02/14/11 13:12, nathan binkert wrote: > I do have code that can parse source files to extract licenses. We > could write a scrip

Re: [m5-dev] Review Request: configs: set default cache params

2011-02-14 Thread Gabe Black
--- This is an automatically generated e-mail. To reply, visit: http://reviews.m5sim.org/r/485/#review866 --- configs/common/CacheConfig.py

[m5-dev] changeset in m5: Ruby: Improve Change PerfectSwitch's wakeup fun...

2011-02-14 Thread Nilay Vaish
changeset e5550966464a in /z/repo/m5 details: http://repo.m5sim.org/m5?cmd=changeset;node=e5550966464a description: Ruby: Improve Change PerfectSwitch's wakeup function Currently the wakeup function for the PerfectSwitch contains three loops - loop on number of virtual net

Re: [m5-dev] Review Request: ruby: naming for RubyNetwork dprintfs

2011-02-14 Thread Nathan Binkert
--- This is an automatically generated e-mail. To reply, visit: http://reviews.m5sim.org/r/489/#review865 --- Seems like a reasonable idea to me, but a Ruby person has to agree. src/

[m5-dev] Review Request: ruby: naming for RubyNetwork dprintfs

2011-02-14 Thread Korey Sewell
--- This is an automatically generated e-mail. To reply, visit: http://reviews.m5sim.org/r/489/ --- Review request for Default, Ali Saidi, Gabe Black, Steve Reinhardt, and Nathan Binke

Re: [m5-dev] Review Request: Info: Clean up some info files.

2011-02-14 Thread nathan binkert
I do have code that can parse source files to extract licenses. We could write a script that sucks out all of the licenses automatically (collapsing common pieces as necessary). Seems like it could be overkill though. Nate On Mon, Feb 14, 2011 at 8:54 PM, Gabe Black wrote: >This is an a

Re: [m5-dev] Review Request: Info: Clean up some info files.

2011-02-14 Thread Gabe Black
> On 2011-02-14 06:48:03, Nathan Binkert wrote: > > LICENSE, line 1 > > > > > > Should we put everyone in here? (And all licenses?) > > Ali Saidi wrote: > I think we should as most of the license text says in, "source an

Re: [m5-dev] Review Request: Ruby: Change PerfectSwitch's wakeup function

2011-02-14 Thread Nilay Vaish
I realized the reason for this. Actually, the wakeup function performs a round-robin scheduling between the incoming links. This code had moved inside the a new if condition that has been introduced in the code. I have moved that code before the if condition so that the timing remains same as b

Re: [m5-dev] Review Request: Ruby: Change PerfectSwitch's wakeup function

2011-02-14 Thread Nilay Vaish
Brad, this patch to affects the number of ticks required for performing a particular number of loads. I don't expect such a thing to happen. Do you? -- Nilay On Wed, 9 Feb 2011, Brad Beckmann wrote: --- This is an automatically generat

Re: [m5-dev] Review Request: Alpha: Import the alpha-system files into the m5 repository.

2011-02-14 Thread nathan binkert
Actually, I think we want to run HG convert (from hg to hg) first and then do a pull -f after that. I can take a look at this tonight. Nate On Mon, Feb 14, 2011 at 4:03 PM, Ali Saidi wrote: >This is an automatically generated e-mail. To reply, visit: > http://reviews.m5sim.org/r/487/ > >

Re: [m5-dev] Review Request: Alpha: Import the alpha-system files into the m5 repository.

2011-02-14 Thread Ali Saidi
--- This is an automatically generated e-mail. To reply, visit: http://reviews.m5sim.org/r/487/#review863 --- I think you can preserve the history by doing a hg pull -f path to alpha-

Re: [m5-dev] Review Request: Info: Clean up some info files.

2011-02-14 Thread Ali Saidi
> On 2011-02-14 06:48:03, Nathan Binkert wrote: > > LICENSE, line 1 > > > > > > Should we put everyone in here? (And all licenses?) I think we should as most of the license text says in, "source and binary forms." The LICEN

Re: [m5-dev] Review Request: Util: Get rid of the make_release.py script.

2011-02-14 Thread Nathan Binkert
--- This is an automatically generated e-mail. To reply, visit: http://reviews.m5sim.org/r/488/#review861 --- Fine with me, though I'd like a second opinion. (I would like a second op

Re: [m5-dev] Review Request: Info: Clean up some info files.

2011-02-14 Thread Nathan Binkert
--- This is an automatically generated e-mail. To reply, visit: http://reviews.m5sim.org/r/486/#review860 --- Ship it! Fine with me. LICENSE

Re: [m5-dev] Restricting W->W reordering in O3

2011-02-14 Thread Korey Sewell
> > > But I need to find the point to log order of completion of loads and stores > in the LSQ (between LSQ and L1 cache), to check the effect of that change. > I was observing inside LSQUnit::completeDataAccess(PacketPtr pkt). > But the order observed doesn't look like the expected. > Looks like y

[m5-dev] Review Request: Util: Get rid of the make_release.py script.

2011-02-14 Thread Gabe Black
--- This is an automatically generated e-mail. To reply, visit: http://reviews.m5sim.org/r/488/ --- Review request for Default, Ali Saidi, Gabe Black, Steve Reinhardt, and Nathan Binke

Re: [m5-dev] files in the root of the tree

2011-02-14 Thread Gabe Black
I decided not to mention it before, but we also have an AUTHORS file which is a little out of date. For instance, I don't think it includes the contribution of Power support. I like being listed in that file and I'm sure other people like the recognition as well, but the decision of what and who to

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

2011-02-14 Thread Cron Daemon
scons: *** [build/X86_SE/m5.fast.unstripped] Error 1 * build/ALPHA_SE/tests/fast/quick/60.rubytest/alpha/linux/rubytest-ruby passed. * build/ALPHA_SE/tests/fast/quick/00.hello/alpha/tru64/simple-atomic passed. * build/ALPHA_SE/tests/fast/quick/20.eio-short/alpha/eio/simple-atomic pass