----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3753/ -----------------------------------------------------------
(Updated Dec. 12, 2016, 11:09 p.m.) Review request for Default and Tushar Krishna. Changes ------- change logic so that network links are not modeled as a queue. Repository: gem5 Description (updated) ------- Changeset 11761:3b97fe03ed24 --------------------------- ruby: Check MessageBuffer space in garnet NetworkInterface Garnet's NetworkInterface does not consider the size of MessageBuffers when ejecting a Message from the network. Add a size check for the MessageBuffer and only enqueue if space is available. If space is not available, the message if placed in a queue and the credit is held. A callback from the MessageBuffer is implemented to wake the NetworkInterface. If there are messages in the stalled queue, they are processed first, in a FIFO manner and if succesfully ejected, the credit is finally sent back upstream. The maximum size of the stall queue is equal to the number of valid VNETs with MessageBuffers attached. Diffs (updated) ----- src/mem/ruby/network/garnet2.0/flitBuffer.hh 78ef8daecd81de0c392034809b3bc155396bf983 src/mem/ruby/network/MessageBuffer.hh 78ef8daecd81de0c392034809b3bc155396bf983 src/mem/ruby/network/MessageBuffer.cc 78ef8daecd81de0c392034809b3bc155396bf983 src/mem/ruby/network/garnet2.0/NetworkInterface.hh 78ef8daecd81de0c392034809b3bc155396bf983 src/mem/ruby/network/garnet2.0/NetworkInterface.cc 78ef8daecd81de0c392034809b3bc155396bf983 src/mem/ruby/network/garnet2.0/flit.hh 78ef8daecd81de0c392034809b3bc155396bf983 src/mem/ruby/network/garnet2.0/flit.cc 78ef8daecd81de0c392034809b3bc155396bf983 Diff: http://reviews.gem5.org/r/3753/diff/ Testing ------- Thanks, Matthew Poremba _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
