Hi Heikki and team,
A few updates…
> > Ok, if we don't need the assembler code at all, that's good. A patch to
> > introduce AIX support should not change it for non-AIX powerpc systems
> > though. That might be a good change, but would need to be justified
> > separately, e.g. by some performance testing, and should be a separate
> > patch.
We ran pgbench for both the patches on powerpc Linux and below are the test
results and both results are close.
PPCLE sync: is the patch with the spin lock using gcc __sync* routines.
PPCLE asm: is the patch with the assemble code.
>> pgbench -c 100 -p 5432 -d postgres -T 180 -r -P 10 -L 10 -j 20
OS/type : PPCLE sync PPCLE asm
---------------------------------------------------------
latency average : 136.257 138.552
---------------------------------------------------------
latency stddev : 234.74 238.603
---------------------------------------------------------
Initial
connection time : 101.791 88.411
---------------------------------------------------------
TPS(without initial
connection time) : 733.633924 721.440648
---------------------------------------------------------
No. of transactions
actually processed : 132080 129893
---------------------------------------------------------
No of transactions
above the 10.0 ms
latency limit : 124235/132080 122183/129893
(94.060%) (94.064%)
---------------------------------------------------------
Please let us know your comments.
BTW we are working on the other review comments as well.
