MD5 speed test not working on Mac about embench-iot HOT 6 OPEN

The change to pylon/run_*.y suggested does allow the benchmark to run.

We still need to remove the print statements though.

from embench-iot.

The problem is that the benchmark is writing to std out in benchmark_body. The Mac scripts assumes the only info on standard out is timing info. I think the print statements should be removed, or at least put under a debugging option.

from embench-iot.

Hi,

I took a look at the code.
The following patch should fix this issue.
I don't have Mac and cannot test it by myself. Could you try this?

--- a/pylib/run_mac.py
+++ b/pylib/run_mac.py
@@ -64,7 +64,7 @@ def decode_results(stdout_str, stderr_str):
         return 0.0
 
     # Match "Real time: dd.ddd"
-    time = re.search('^Real time: (\d+)[.](\d+)', stdout_str, re.S)
+    time = re.search('^Real time: (\d+)[.](\d+)', stdout_str, re.M)
     if time:
         ms_elapsed = float(time.group(1) + '.' + time.group(2))
         # Return value cannot be zero (will be interpreted as error)

I think the print statements should be removed, or at least put under a debugging option.

I agree with you. The current md5sum test is measuring the performance of printf() :-)

from embench-iot.

from embench-iot.

I think the fix proposed by #152 addresses the use of external library functions.

from embench-iot.

This is what Roger and I wrote.

I think the print statements should be removed, or at least put under a debugging option.
I agree with you. The current md5sum test is measuring the performance of printf() :-)

I still recommend you to replace re.S with re.M in pylib/run_*.py.

https://docs.python.org/3/library/re.html#re.S
https://docs.python.org/3/library/re.html#re.M

from embench-iot.