public inbox for [email protected]  
help / color / mirror / Atom feed
From: Srirama Kucherlapati <[email protected]>
To: Heikki Linnakangas <[email protected]>
To: Heikki Linnakangas <[email protected]>
To: Bruce Momjian <[email protected]>
Cc: Peter Eisentraut <[email protected]>
Cc: Alvaro Herrera <[email protected]>
Cc: [email protected] <[email protected]>
Cc: Laurenz Albe <[email protected]>
Cc: Noah Misch <[email protected]>
Cc: Michael Paquier <[email protected]>
Cc: Andres Freund <[email protected]>
Cc: Thomas Munro <[email protected]>
Cc: [email protected] <[email protected]>
Cc: [email protected] <[email protected]>
Cc: Tom Lane <[email protected]>
Subject: RE: AIX support
Date: Fri, 4 Oct 2024 15:04:57 +0000
Message-ID: <CY8PR15MB5602EB54EFE30D4C21F061BEDB722@CY8PR15MB5602.namprd15.prod.outlook.com> (raw)
In-Reply-To: <CY8PR15MB56023A063EFA06C0BAB89442DB692@CY8PR15MB5602.namprd15.prod.outlook.com>
References: <CY5PR11MB639218D04CFAED0FC09ED9DFFDED2@CY5PR11MB6392.namprd11.prod.outlook.com>
	<[email protected]>
	<CY8PR15MB560282A99569E110AEED3257DBCF2@CY8PR15MB5602.namprd15.prod.outlook.com>
	<CY8PR15MB56029A55A848E9089D29FBE6DBC92@CY8PR15MB5602.namprd15.prod.outlook.com>
	<CY8PR15MB5602C6D2B148615EBD7F1AF5DB872@CY8PR15MB5602.namprd15.prod.outlook.com>
	<[email protected]>
	<CY8PR15MB5602A5D9064A401347684A06DB872@CY8PR15MB5602.namprd15.prod.outlook.com>
	<[email protected]>
	<CY8PR15MB5602B407BBD1B7B7C5FF1D8CDB9B2@CY8PR15MB5602.namprd15.prod.outlook.com>
	<[email protected]>
	<DS0PR15MB5623ECC0499C5087EFD729EADB652@DS0PR15MB5623.namprd15.prod.outlook.com>
	<CY8PR15MB5602561E7BF438A1B6C5371BDB612@CY8PR15MB5602.namprd15.prod.outlook.com>
	<CY8PR15MB5602F4F63D8F8811A5987EFFDB682@CY8PR15MB5602.namprd15.prod.outlook.com>
	<[email protected]>
	<CY8PR15MB56023A063EFA06C0BAB89442DB692@CY8PR15MB5602.namprd15.prod.outlook.com>

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.


Warm regards,
Sriram.



view thread (20+ messages)  latest in thread

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected], [email protected], [email protected], [email protected], [email protected], [email protected], [email protected], [email protected], [email protected], [email protected], [email protected], [email protected], [email protected], [email protected]
  Subject: RE: AIX support
  In-Reply-To: <CY8PR15MB5602EB54EFE30D4C21F061BEDB722@CY8PR15MB5602.namprd15.prod.outlook.com>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox