RFR(S): 8196487: AIX: os::get_summary_cpu_info does not need to call into libperfstat

[Thomas Stüfe]

On Wed, Jan 31, 2018 at 5:07 PM, Langer, Christoph <christoph.langer at sap.com
> wrote:

> Hi Goetz, Thomas,
>
> thanks for the review. I pushed with added defines for some power
> architectures that might be too new for older AIX OS versions where OpenJDK
> could theoretically be built: http://hg.openjdk.java.net/
> jdk/hs/rev/8a9808a9dd6b
>
>
Good thinking. Thank you.

..Thomas


> Best regards
> Christoph
>
> -----Original Message-----
> From: Lindenmaier, Goetz
> Sent: Mittwoch, 31. Januar 2018 11:13
> To: Langer, Christoph <christoph.langer at sap.com>;
> hotspot-runtime-dev at openjdk.java.net; ppc-aix-port-dev at openjdk.java.net
> Subject: RE: RFR(S): 8196487: AIX: os::get_summary_cpu_info does not need
> to call into libperfstat
>
> Hi Christoph,
>
> change looks good.
>
> Thanks,
>   Goetz.
>
>
> > -----Original Message-----
> > From: hotspot-runtime-dev [mailto:hotspot-runtime-dev-
> > bounces at openjdk.java.net] On Behalf Of Langer, Christoph
> > Sent: Mittwoch, 31. Januar 2018 10:10
> > To: hotspot-runtime-dev at openjdk.java.net; ppc-aix-port-
> > dev at openjdk.java.net
> > Subject: RFR(S): 8196487: AIX: os::get_summary_cpu_info does not need to
> > call into libperfstat
> >
> > Hi,
> >
> > please review an AIX only change:
> >
> > Webrev: http://cr.openjdk.java.net/~clanger/webrevs/8196487.0/
> > Bug: https://bugs.openjdk.java.net/browse/JDK-8196487
> >
> > The implementation of os::get_summary_cpu_info() on AIX currently calls
> > libperfstat::get_cpuinfo() which in turn only fills the version field of
> struct
> > cpuinfo_t for the purpose of os::get_summary_cpu_info. This should be
> > decoupled.
> >
> > Thanks & Best regards
> > Christoph
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/ppc-aix-port-dev/attachments/20180131/0396cb31/attachment.html>