Hiya, I've created and tested a new webrev using lparstat and following your guidelines, webrev.01 is here: http://cr.openjdk.java.net/~aleonard/8217896/webrev.01/ If someone is happy to sponsor and review please? Many thanks Andrew
Andrew Leonard Java Runtimes Development IBM Hursley IBM United Kingdom Ltd Phone internal: 245913, external: 01962 815913 internet email: andrew_m_leon...@uk.ibm.com From: Magnus Ihse Bursie <magnus.ihse.bur...@oracle.com> To: Andrew Leonard <andrew_m_leon...@uk.ibm.com>, David Holmes <david.hol...@oracle.com> Cc: build-dev@openjdk.java.net Date: 30/01/2019 17:16 Subject: Re: Proposal: JDK-8217896: Make better use of LCPUs when building on AIX On 2019-01-30 17:37, Andrew Leonard wrote: > Thanks David, > I just found the reference you mention, it is a little vague, but says > maybe as you said. I've reworked and tested the following which seems > better: > NUM_CORES=`lparstat -m | grep -o "lcpu=[[0-9]]*" | cut -d "=" -f 2` > I'll create a new webrev with this. Note that you should use $GREP and $CUT; in theory we should also check for lparstat, but we've been sloppy with that for the other cpu checking methods so maybe we can be that for AIX here as well. /Magnus > Thanks > Andrew > > Andrew Leonard > Java Runtimes Development > IBM Hursley > IBM United Kingdom Ltd > Phone internal: 245913, external: 01962 815913 > internet email: andrew_m_leon...@uk.ibm.com > > > > > From: David Holmes <david.hol...@oracle.com> > To: Erik Joelsson <erik.joels...@oracle.com>, Andrew Leonard > <andrew_m_leon...@uk.ibm.com>, build-dev@openjdk.java.net > Date: 30/01/2019 02:25 > Subject: Re: Proposal: JDK-8217896: Make better use of LCPUs when > building on AIX > > > > On 30/01/2019 3:05 am, Erik Joelsson wrote: >> Hello, >> >> I'm OK with this, but since I can't verify it at all, I would be more >> comfortable if one of the other AIX users would review this as well. > Ditto. I did some googling on this and pmcycles is flagged as maybe > being deprecated in the future and may require privileges to execute. > lparstat -i seems to be more commonly used. > > David > ----- > >> /Erik >> >> On 2019-01-29 05:08, Andrew Leonard wrote: >>> Hi, >>> Please can I get a sponsor for this build performance enhancement on > AIX, >>> by better determining the number of LCPUs on smt AIX machines: >>> > https://urldefense.proofpoint.com/v2/url?u=https-3A__cr.openjdk.java.net_-7Ealeonard_8217896_webrev.00_&d=DwIDaQ&c=jf_iaSHvJObTbx-siA1ZOg&r=NaV8Iy8Ld-vjpXZFDdTbgGlRTghGHnwM75wUPd5_NUQ&m=FpNeQfFJBut0AAVlbC0w7o0urDCArfDb4PY1TtMVW4I&s=8vmPiiNU5P6MnXKdE-eyyBm48kgCssjH5VyWHAt5zcM&e= > >>> My testing shows up to 30% improved build times when leveraging the >>> available LCPUs. >>> Many thanks >>> Andrew >>> >>> Andrew Leonard >>> Java Runtimes Development >>> IBM Hursley >>> IBM United Kingdom Ltd >>> Phone internal: 245913, external: 01962 815913 >>> internet email: andrew_m_leon...@uk.ibm.com >>> >>> >>> Unless stated otherwise above: >>> IBM United Kingdom Limited - Registered in England and Wales with > number >>> 741598. >>> Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire PO6 >>> 3AU > > > > > Unless stated otherwise above: > IBM United Kingdom Limited - Registered in England and Wales with number > 741598. > Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire PO6 3AU Unless stated otherwise above: IBM United Kingdom Limited - Registered in England and Wales with number 741598. Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire PO6 3AU