Hi, Sergey!
thanks for the patch! LGTM
Sergey
This patch adjusts the aforementioned test to use the benchmark
framework introduced before. The default arguments are adjusted
according to the <PARAM_x86.txt> file. The arguments to the script still
can be provided in the command line run.
---
perf/LuaJIT-benches/chameneos.lua | 32 ++++++++++++++++++++++---------
1 file changed, 23 insertions(+), 9 deletions(-)
diff --git a/perf/LuaJIT-benches/chameneos.lua b/perf/LuaJIT-benches/chameneos.lua
index 78b64c3f..c1002041 100644
--- a/perf/LuaJIT-benches/chameneos.lua
+++ b/perf/LuaJIT-benches/chameneos.lua
@@ -1,8 +1,10 @@
+local bench = require("bench").new(arg)
local co = coroutine
local create, resume, yield = co.create, co.resume, co.yield
-local N = tonumber(arg and arg[1]) or 10
+local N = tonumber(arg and arg[1]) or 1e7
Why 1e7?
+local N_ATTEMPTS = N
local first, second
-- Meet another creature.
@@ -57,12 +59,24 @@ local function schedule(threads)
until false
end
--- A bunch of colorful creatures.
-local threads = {
- creature("blue"),
- creature("red"),
- creature("yellow"),
- creature("blue"),
-}
+bench:add({
+ name = "chameneos",
+ items = N_ATTEMPTS,
+ checker = function(meetings) return meetings == N_ATTEMPTS * 2 end,
+ payload = function()
+ -- A bunch of colorful creatures.
+ local threads = {
+ creature("blue"),
+ creature("red"),
+ creature("yellow"),
+ creature("blue"),
+ }
-io.write(schedule(threads), "\n")
+ local meetings = schedule(threads)
+ -- XXX: Restore meetings for the next iteration.
+ N = N_ATTEMPTS
+ return meetings
+ end,
+})
+
+bench:run_and_report()