Ah...this was the other half of the changes I was looking for in my previous review.  I have a few comments below:

src/mem/protocol/MESI_CMP_directory-L1cache.sm (Diff revision 1)

Minor comment, but can you change this name to "enable_prefetch"

Show all issues

1. Nilay Vaish Oct. 25, 2012, 1:34 a.m. (Oct. 25, 2012, 1:34 a.m.)

   Done!

src/mem/protocol/MESI_CMP_directory-L1cache.sm (Diff revision 1)

As I mentioned in my previous review, for historical reasons you may want to call this the "optionalQueue".

Show all issues

1. Nilay Vaish Oct. 25, 2012, 1:34 a.m. (Oct. 25, 2012, 1:34 a.m.)

   Done!

src/mem/protocol/MESI_CMP_directory-L1cache.sm (Diff revision 1)

Below the L1 cache, do you treat prefetches and non-prefetches the same?  Have you looked into the implications of this?

Show all issues

1. Nilay Vaish Oct. 25, 2012, 1:34 a.m. (Oct. 25, 2012, 1:34 a.m.)

   As of know the prefetch and demand requests are treated equally after 
   the L1 cache. I have not done any experimentation on how the prefetch 
   traffic affects demand traffic. While we can treat them differently, 
   I think we should keep things simple to begin with. Over time, I expect 
   protocols would change to adapt better.