ssnetsim / supersim

A flexible event-driven cycle-accurate network simulator
Apache License 2.0
8 stars 1 forks source link

Application level maximum outstanding transactions is wrong. #34

Closed nicmcd closed 4 years ago

nicmcd commented 4 years ago

Blast, Pulse, and AllToAll applications have a maximum outstanding transaction limiter. If enabled, the terminal will hold off creating transactions until the number outstanding drops below the threshold. This breaks the SuperSim mentality that latency should be measured from the time a transaction is desired to be sent, not the time when it is actually sent.

The proposed change is to remove the limit from the Applications/Terminals. Any transaction level queuing should be done in the Interface model.