[Tarantool-patches] [PATCH v1 luajit 06/41] perf: adjust chameneos in LuaJIT-benches

[Sergey Kaplun]

Hi, Sergey!
Thanks for the review!
See my answers below.

On 13.11.25, Sergey Bronnikov wrote:
> Hi, Sergey!
> 
> thanks for the patch! LGTM
> 
> Sergey
> 
> On 10/24/25 13:50, Sergey Kaplun wrote:
> > 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 @@

Also, added the comment with the benchmark description as we discussed
offline.

===================================================================
diff --git a/perf/LuaJIT-benches/chameneos.lua b/perf/LuaJIT-benches/chameneos.lua
index c1002041..9bd83081 100644
--- a/perf/LuaJIT-benches/chameneos.lua
+++ b/perf/LuaJIT-benches/chameneos.lua
@@ -1,3 +1,9 @@
+-- The benchmark to check the performance of coroutine interaction
+-- using symmetrical rendezvous requests.
+-- For the details see:
+-- https://pybenchmarks.org/u64q/performance.php?test=chameneosredux
+-- https://cedric.cnam.fr/PUBLIS/RC474.pdf
+
 local bench = require("bench").new(arg)
 
 local co = coroutine
===================================================================

> > +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?

It is the default for x86 arch. I've taken the values from PARAMS_x86,
since this is the most important architecture for the Tarantool, see the
commit message.

> > +local N_ATTEMPTS = N

<snipped>

-- 
Best regards,
Sergey Kaplun