Re: RFR: 8166220: Catalog API: JAXP XML Processor Support - add StAX test coverage

2016-09-20 Thread Daniel Fuchs
Hi Joe, test/javax/xml/jaxp/unittest/catalog/CatalogSupportBase.java 603 /** 604 * Returns the text of the first element found by the reader. 605 * @param streamReader the XMLStreamReader 606 * @return the text of the first element 607 * @throws XMLStreamException 6

Re: Reg - Sub Process creation from java takes more time

2016-09-20 Thread Manjunath SV
Hi, Virtual size is always constant - 12.6gb, no longer pauses in GC logs. VM will not fail if we run for loner time. We submit 400+ jobs(3-4 times per day) to thread pool of size 30, each thread has to execute more than 13+ jobs, initially threads starts sub process in milli seconds, some tim

Re: Reg - Sub Process creation from java takes more time

2016-09-20 Thread Peter Levart
Hi Manjunath, There could be a bug somewhere in the process API. Could you try using ProcessBuilder API to execute linux-command directly and use the API to setup stdin/out/err redirects instead of using a shell script. Just to see if you get the same slowdown in that case. Regards, Peter On Sep

RFR(L): 8161211: better inlining support for loop bytecode intrinsics

2016-09-20 Thread Michael Haupt
Dear all, please review this change. Bug: https://bugs.openjdk.java.net/browse/JDK-8161211 Webrev: http://cr.openjdk.java.net/~mhaupt/8161211/webrev.00/ The method handle loop combinators introduced with JEP 274 were originally not intrinsified, leading to poor performance as compared to a pure-

Re: MethodHandle loop body parameters and Stream.reduce accumulator parameters are not in the same order

2016-09-20 Thread Michael Haupt
Hi Rémi, I think you can stay tuned for the fix to JDK-8151179; part of what it addresses is just this. The changes are currently in the CCC loop but should come up for public re-review soon. Best, Michael > Am 14.09.2016 um 15:19 schrieb Remi Forax : > > Hi everybody, > i've just found that

Re: Last Ant Test Failure with JDK9 - JAXP Secure Processing and XSLT Extensions

2016-09-20 Thread Stefan Bodewig
Hi Joe On 2016-09-08, Joe Wang wrote: > It's great to know you've found the solution! Hopefully this is indeed > the last issue for you with JDK 9 :-) > I've checked in a fix for the redirect failure [1] into the dev > repo. It would probably be included in the next week's build > (b136). Please

JDK 9 RFR of JDK-8166248: tools/pack200/Pack200Test.java fails on Win32: Could not reserve enough space

2016-09-20 Thread Amy Lu
tools/pack200/Pack200Test.java This test requires -Xmx1280m to run. Test is not suitable for 32-bit platform. Please review this patch to skip on 32-bit platform. bug: https://bugs.openjdk.java.net/browse/JDK-8166248 webrev: http://cr.openjdk.java.net/~amlu/8166248/webrev.00/ Thanks, Amy --

Re: JDK 9 RFR of JDK-8166248: tools/pack200/Pack200Test.java fails on Win32: Could not reserve enough space

2016-09-20 Thread Kumar Srinivasan
Looks good. Thanks for making the change. Kumar tools/pack200/Pack200Test.java This test requires -Xmx1280m to run. Test is not suitable for 32-bit platform. Please review this patch to skip on 32-bit platform. bug: https://bugs.openjdk.java.net/browse/JDK-8166248 webrev: http://cr.openjd

Re: RFR(L): 8161211: better inlining support for loop bytecode intrinsics

2016-09-20 Thread Vladimir Ivanov
Looks good. src/java.base/share/classes/java/lang/invoke/LambdaFormEditor.java: +LambdaForm bmhArrayForm(MethodType type, BoundMethodHandle.SpeciesData speciesData) { +int size = type.parameterCount(); +Transform key = Transform.of(Transform.BMH_AS_ARRAY, size); +Lam

Re: RFR: 8166220: Catalog API: JAXP XML Processor Support - add StAX test coverage

2016-09-20 Thread Joe Wang
On 9/20/16, 2:20 AM, Daniel Fuchs wrote: Hi Joe, test/javax/xml/jaxp/unittest/catalog/CatalogSupportBase.java 603 /** 604 * Returns the text of the first element found by the reader. 605 * @param streamReader the XMLStreamReader 606 * @return the text of the first eleme

Re: Last Ant Test Failure with JDK9 - JAXP Secure Processing and XSLT Extensions

2016-09-20 Thread Joe Wang
Thanks Stefan for the update! I'm very glad to know you've gotten all of Ant's tests passed! Best regards, Joe On 9/20/16, 6:46 AM, Stefan Bodewig wrote: Hi Joe On 2016-09-08, Joe Wang wrote: It's great to know you've found the solution! Hopefully this is indeed the last issue for you with

Re: RFR(L): 8161211: better inlining support for loop bytecode intrinsics

2016-09-20 Thread John Rose
There should also be an assert in the new LF constructor, which ensures that the two arguments are congruent. Better yet, just supply one argument (the speciesData), and derive the MT. These new LFs are pretty confusing, and it's best to nail down unused degrees of freedom. — John P.S. I wo