The code in storebuffer.cc looks pretty suspect.  Where it was looking for something from libruby, you replaced that by a zero.  Seems that the code won't work anymore in that case.  If this is correct, then the code is very confusing and a little bit of cleanup would be warranted.

src/mem/ruby/storebuffer/storebuffer.cc (Diff revision 1)

Is this right?  My guess is that the entire "if (!m_use_storebuffer) { }" block should be replaced by an "if (!m_use_storebuffer) fatal("must provide a storebuffer in the configuration")" or something like this.  I don't see how this could work without a storebuffer.  

1. Nilay Vaish Jan. 25, 2011, 12:54 p.m. (Jan. 25, 2011, 12:54 p.m.)

   Even I was surprised when I saw this. May be we should drop m_use_storebuffer 
   all together. Since libruby was not performing any function, dropping 
   m_use_storebuffer should not affect the functionality at all.

2. Nathan Binkert Jan. 25, 2011, 2:21 p.m. (Jan. 25, 2011, 2:21 p.m.)

   If it doesn't mean anything anymore, then we should drop it.

src/mem/ruby/storebuffer/storebuffer.cc (Diff revision 1)

fatal() ?
Isn't a storebuffer required?

src/mem/ruby/storebuffer/storebuffer.cc (Diff revision 1)

panic?

src/mem/ruby/storebuffer/storebuffer.cc (Diff revision 1)

panic?