sanchayanmaity/gem5
1
0
Fork 0
Code Issues Pull requests Packages Projects Releases Wiki Activity

cpu: Fix the traffic gen read percentage

Browse source 
This patch fixes the computation that determines whether to perform a
read or a write such that the two corner cases (0 and 100) are both
more efficient and handled correctly.
This commit is contained in:
Andreas Hansson 2013-01-07 13:05:35 -05:00
parent 852a7bcf92
commit f22d3bb9c3
1 changed files with 8 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
src/cpu/testers/traffic_gen/traffic_gen.cc
View file

@ -370,10 +370,11 @@ void
TrafficGen::StateGraph::LinearGen::execute()
{
// choose if we generate a read or a write here
bool isRead = random_mt.random<uint8_t>(0, 100) < readPercent;
bool isRead = readPercent != 0 &&
(readPercent == 100 || random_mt.random<uint8_t>(0, 100) < readPercent);
if (readPercent == 0)
assert(!isRead);
assert((readPercent == 0 && !isRead) || (readPercent == 100 && isRead) ||
readPercent != 100);
DPRINTF(TrafficGen, "LinearGen::execute: %c to addr %x, size %d\n",
isRead ? 'r' : 'w', nextAddr, blocksize);
@ -442,10 +443,11 @@ void
TrafficGen::StateGraph::RandomGen::execute()
{
// choose if we generate a read or a write here
bool isRead = random_mt.random<uint8_t>(0, 100) < readPercent;
bool isRead = readPercent != 0 &&
(readPercent == 100 || random_mt.random<uint8_t>(0, 100) < readPercent);
if (readPercent == 0)
assert(!isRead);
assert((readPercent == 0 && !isRead) || (readPercent == 100 && isRead) ||
readPercent != 100);
// address of the request
Addr addr = random_mt.random<Addr>(startAddr, endAddr - 1);