changeset c52b4c5f46f8 in /z/repo/gem5
details: http://repo.gem5.org/gem5?cmd=changeset;node=c52b4c5f46f8
description:
        Ruby: More descriptive message buffer connection fatal

        When connecting message buffers between Ruby controllers, it is
        easy to mistakenly connect multiple controllers to the same message
        buffer. This patch prints a more descriptive fatal message than the
        previous assert statement in order to facilitate easier debugging.

diffstat:

 src/mem/ruby/buffers/MessageBuffer.hh |  6 +++++-
 1 files changed, 5 insertions(+), 1 deletions(-)

diffs (16 lines):

diff -r 195d92059654 -r c52b4c5f46f8 src/mem/ruby/buffers/MessageBuffer.hh
--- a/src/mem/ruby/buffers/MessageBuffer.hh     Tue Apr 09 16:12:42 2013 -0500
+++ b/src/mem/ruby/buffers/MessageBuffer.hh     Tue Apr 09 16:15:06 2013 -0500
@@ -79,7 +79,11 @@
     void setPriority(int rank) { m_priority_rank = rank; }
     void setConsumer(Consumer* consumer)
     {
-        assert(m_consumer == NULL);
+        if (m_consumer != NULL) {
+            fatal("Trying to connect %s to MessageBuffer %s. \
+                  \n%s already connected. Check the cntrl_id's.\n",
+                  *consumer, *this, *m_consumer);
+        }
         m_consumer = consumer;
     }
 
_______________________________________________
gem5-dev mailing list
[email protected]
http://m5sim.org/mailman/listinfo/gem5-dev

Reply via email to