8186982: [aix] Garbage output for CPU info in hs-err file

[Lindenmaier, Goetz]

Hi Thomas,

looks good, thanks.

Best regards,
  Goetz.

From: Thomas Stüfe [mailto:thomas.stuefe at gmail.com]
Sent: Thursday, August 31, 2017 9:06 PM
To: ppc-aix-port-dev at openjdk.java.net; hotspot-runtime-dev at openjdk.java.net
Cc: Lindenmaier, Goetz <goetz.lindenmaier at sap.com>; Volker Simonis <volker.simonis at gmail.com>
Subject: Re: 8186982: [aix] Garbage output for CPU info in hs-err file

(Added hotspot-runtime)

Hi all @hotspot-runtime,

may I please have additional reviews for this AIX only patch. I already pushed this change - it was AIX only and I had two reviewers, so I thought I was fine.

But I was not aware that a rule exists requiring me to run changes like these through one of the generic mailing lists in addition to the AIX-only mailing lists. Sorry for that. I usually try to avoid crossposting to keep the noise down.

So, this change. It removes the AIX os::pd_print_cpu_info() implementation, which is not only broken but also superfluous since the generic one (os::print_cpu_info()) does just fine and there is nothing exciting to add at os level.

Thank you!

Thomas






On Thu, Aug 31, 2017 at 2:20 PM, Thomas Stüfe <thomas.stuefe at gmail.com<mailto:thomas.stuefe at gmail.com>> wrote:
Hi,

may I please have reviews for this tiny trivial change:

Bug: https://bugs.openjdk.java.net/browse/JDK-8186982
Webrev: http://cr.openjdk.java.net/~stuefe/webrevs/8186982-aix-cpu-info-garbage/webrev.00/webrev/

Thank you!

..Thomas

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/ppc-aix-port-dev/attachments/20170831/86cf6d37/attachment.html>