Conversation
transactions in the memory controller.
shavvn
left a comment
There was a problem hiding this comment.
The delayedInfo class definition seems to be missing. Can you make sure you included all the changes made in order to build the code successfully? Thanks!
Changes to be committed: modified: MultiChannelMemorySystem.h
|
Hi Shawn. |
shavvn
left a comment
There was a problem hiding this comment.
Hi @rommelsv ,
Thanks for the patch.
I was able to build after your commit. But I think there's still some problems that I can see now:
-
There's no default parameter provided for DQ_DEPTH, which I assume will be in sys.ini, right?
-
I tried to add a DQ_DEPTH=20 to sys.ini and ran a few experiments, but it doesn't look like the program took it as an input. I think it should be included in configmap of IniReader.cpp. (I saw you had it but somehow commented it out?)
|
HI Shavvn, |
Implementation of the Delay Queue, an extra delay for all memory transactions in the memory controller.