On 2017-03-30 03:04, David Holmes wrote:
Hi Alexey,

Can't really review the details of this but wanted to ensure that this gets tested on all platforms before being pushed. Internally use JPRT and "-testset hotspot", but you should also reach out to the AIX-port folk to ensure it works as expected there too.

David,

What are your worries here? There are no platform-dependent changes, apart from the actual logging script which might not work properly if your local copy of time/flock is non-compliant. (We try to test for this, at least for "time".) Unless you're running with LOG=profile, this patch affects nothing.

If the profiling does not work properly on e.g. AIX, then some of the AIX folks would need to supply an additional patch to update shell-profiler.sh.

/Magnus

Thanks,
David

On 30/03/2017 2:21 AM, alexey semenyuk wrote:
Please review and approve enhancement for JDK10.

Bug: https://bugs.openjdk.java.net/browse/JDK-8177770
Webrev: http://cr.openjdk.java.net/~asemenyuk/8177770/webrev.00/

Thanks,
Alexey


Reply via email to