Skip to content

Commit

Permalink
The benchmarking code assumed that there would always be a testcase
Browse files Browse the repository at this point in the history
where K_WAY_HEAP held the value 1. Since that is no longer a legal
value, that assumption is flawed. This commit contains a partial
effort to overcome this assumption, but it is not complete. The
genereated PDF is not quite right. Further effort is needed.

Signed-off-by: J. Eric Ivancich <[email protected]>
  • Loading branch information
ivancich committed Jan 9, 2017
1 parent 9d72e7d commit 22798ab
Show file tree
Hide file tree
Showing 4 changed files with 12 additions and 8 deletions.
4 changes: 4 additions & 0 deletions benchmark/README.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,9 @@
# dmclock benchmarking

**IMPORTANT**: now that K_WAY_HEAP is no longer allowed to have the
value 1, the shell and Python scripts that generate the PDFs no longer
work exactly correctly. Some effort to debug is necessary.

This directory contains scripts to evaluate effects of different
branching-factors (k=1 to k=11) in the IndirectIntrusiveHeap
data-structure. IndirectIntrusiveHeap is now a k-way heap, so finding
Expand Down
6 changes: 3 additions & 3 deletions benchmark/data_gen.sh
Original file line number Diff line number Diff line change
Expand Up @@ -23,10 +23,10 @@ if [ "$3" != "" ]; then
repeat="$3"
fi

echo "k-way: $k_way, num_repeat:$repeat"
echo "k-way:$k_way, num_repeat:$repeat"

# create simulators in different directories
k=1
k=2
while [ $k -le $k_way ]
do
mkdir "build_$k"
Expand All @@ -43,7 +43,7 @@ done
echo '' > $output_file
for config in "$config_dir"/*.conf
do
k=1
k=2
while [ $k -le $k_way ]
do
cd "build_$k"
Expand Down
4 changes: 2 additions & 2 deletions benchmark/data_parser.py
Original file line number Diff line number Diff line change
Expand Up @@ -136,10 +136,10 @@ def create_header(num_cols):
header = fields[0]
#write add_req_{1, ...}
for i in range(num_cols):
header = '%s %s_%i'%(header, fields[1], i+1)
header = '%s %s_%i'%(header, fields[1], i+2)
#write complete_req_{1, ...}
for i in range(num_cols):
header = '%s %s_%i'%(header, fields[2], i+1)
header = '%s %s_%i'%(header, fields[2], i+2)
# new-line
header = '%s\n'%(header)
return header
Expand Down
6 changes: 3 additions & 3 deletions benchmark/run.sh
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
#!/bin/bash

# default value
k_way=2 #11
repeat=3 #5
k_way=3 #11
repeat=2 #5

output_file=""
if [ "$1" != "" ]; then
Expand All @@ -21,4 +21,4 @@ python data_parser.py ${output_file}
echo "now generating bar-chart"
#gnuplot -e 'output_file=value' plot_gen.gnuplot
sh plot_gen.sh ${output_file} ${k_way}
echo "done! check ${output_file}.pdf"
echo "done! check ${output_file}.pdf"

0 comments on commit 22798ab

Please sign in to comment.