Hi Julian,
This is a serviceability issue - cc'd.
On 14/09/2023 12:48 pm, Julian Waters wrote:
Hi all,
In the HotSpot
test /test/hotspot/jtreg/vmTestbase/nsk/jvmti/IterateThroughHeap/filter-tagged/HeapFilter.cpp, the method occurance_expected, which returns a jboolean, returns JNI_ERR in an e
On Wed, 13 Sep 2023 15:30:22 GMT, Joachim Kern wrote:
>> After push of [JDK-8307478](https://bugs.openjdk.org/browse/JDK-8307478) ,
>> the following test started to fail on AIX :
>> com/sun/tools/attach/warnings/DynamicLoadWarningTest.java;
>> The problem was described in
>> [JDK-8309549](https
On Tue, 12 Sep 2023 19:20:16 GMT, Aleksey Shipilev wrote:
> and consume the usual amount of memory.
And how much is that? And at what concurrency level will we not be able to run
these tests in parallel without potentially impacting the way they run i.e.
running out of memory sooner than expe
On Wed, 13 Sep 2023 23:57:43 GMT, Leonid Mesnik wrote:
> The fix 8314828 removed dependency to /vmTestbase and tests started failing.
> Verified by running tests and submitting tier1.
This pull request has now been integrated.
Changeset: 11d431b2
Author:Leonid Mesnik
URL:
https://gi
On Wed, 13 Sep 2023 23:57:43 GMT, Leonid Mesnik wrote:
> The fix 8314828 removed dependency to /vmTestbase and tests started failing.
> Verified by running tests and submitting tier1.
Looks good. Thanks
-
Marked as reviewed by dholmes (Reviewer).
PR Review: https://git.openjdk.org
The fix 8314828 removed dependency to /vmTestbase and tests started failing.
Verified by running tests and submitting tier1.
-
Commit messages:
- 8316228: jcmd tests are broken by 8314828
Changes: https://git.openjdk.org/jdk/pull/15731/files
Webrev: https://webrevs.openjdk.org/?rep
On Wed, 23 Aug 2023 22:58:05 GMT, Leonid Mesnik wrote:
> Mark 3 tests as flagless to don't run them if not needed. Also, small clean
> up.
This pull request has now been integrated.
Changeset: 23fab118
Author:Leonid Mesnik
URL:
https://git.openjdk.org/jdk/commit/23fab1181b6a053efc4
On Wed, 23 Aug 2023 22:58:05 GMT, Leonid Mesnik wrote:
> Mark 3 tests as flagless to don't run them if not needed. Also, small clean
> up.
Marked as reviewed by cjplummer (Reviewer).
-
PR Review: https://git.openjdk.org/jdk/pull/15407#pullrequestreview-1625546191
> 8315149: Add hsperf counters for CPU time of internal GC threads
Jonathan Joo has updated the pull request incrementally with one additional
commit since the last revision:
Update total gc cpu implementation (still not finished)
-
Changes:
- all: https://git.openjdk.org/jdk/p
On Wed, 13 Sep 2023 14:34:42 GMT, Kevin Walls wrote:
> Original failure in 8081569 is a long time ago, and not reproducing.
> Other failures recorded there may be different problems, some of them are
> port related which are helped by other changes since these tests were
> problemlisted.
>
> T
On Tue, 12 Sep 2023 19:20:16 GMT, Aleksey Shipilev wrote:
> Similar to [JDK-8315437](https://bugs.openjdk.org/browse/JDK-8315437),
> current vmTestbase/nsk/monitoring/stress/lowmem tests contains 36 tests, each
> running exclusively. This drags the tier4 test times up. There seem to be no
> re
On Wed, 13 Sep 2023 18:12:15 GMT, Naoto Sato wrote:
> Looks good to me, although I did not look at each l10n file, but sampled
> some. Thanks for tackling this conversion.
Thanks for the review; (In addition to testing), I ran a script to verify only
white space escape sequences exist in JDK .
On Wed, 13 Sep 2023 17:38:28 GMT, Justin Lu wrote:
>> JDK .properties files still use ISO-8859-1 encoding with escape sequences.
>> It would improve readability to see the native characters instead of escape
>> sequences (especially for the L10n process). The majority of files changed
>> are l
> 8315149: Add hsperf counters for CPU time of internal GC threads
Jonathan Joo has updated the pull request incrementally with one additional
commit since the last revision:
Partial commit attempting to add total cpu tracker
-
Changes:
- all: https://git.openjdk.org/jdk/pull/1
> JDK .properties files still use ISO-8859-1 encoding with escape sequences. It
> would improve readability to see the native characters instead of escape
> sequences (especially for the L10n process). The majority of files changed
> are localized resource files.
>
> This change converts the Un
On Tue, 12 Sep 2023 19:20:16 GMT, Aleksey Shipilev wrote:
> Similar to [JDK-8315437](https://bugs.openjdk.org/browse/JDK-8315437),
> current vmTestbase/nsk/monitoring/stress/lowmem tests contains 36 tests, each
> running exclusively. This drags the tier4 test times up. There seem to be no
> re
On Wed, 13 Sep 2023 13:01:32 GMT, Alan Bateman wrote:
>> Of course as soon as I posted that I now realize the issue. There are more
>> things that could "deadlock" virtual threads when all other threads are
>> suspended - such as calling sleep, because it requires a "system" thread to
>> be ac
On Wed, 13 Sep 2023 12:47:48 GMT, Alan Bateman wrote:
>> This PR includes a couple of clarifications of the JDWP and JDI invoke
>> method support. The first is that deadlocks can occur due to a needed
>> "resource" being held. The spec previously just mentioned monitors being
>> held, but this
On Wed, 13 Sep 2023 14:34:42 GMT, Kevin Walls wrote:
> Original failure in 8081569 is a long time ago, and not reproducing.
> Other failures recorded there may be different problems, some of them are
> port related which are helped by other changes since these tests were
> problemlisted.
>
> T
On Tue, 12 Sep 2023 19:20:16 GMT, Aleksey Shipilev wrote:
> Similar to [JDK-8315437](https://bugs.openjdk.org/browse/JDK-8315437),
> current vmTestbase/nsk/monitoring/stress/lowmem tests contains 36 tests, each
> running exclusively. This drags the tier4 test times up. There seem to be no
> re
On Wed, 13 Sep 2023 15:30:22 GMT, Joachim Kern wrote:
>> After push of [JDK-8307478](https://bugs.openjdk.org/browse/JDK-8307478) ,
>> the following test started to fail on AIX :
>> com/sun/tools/attach/warnings/DynamicLoadWarningTest.java;
>> The problem was described in
>> [JDK-8309549](https
> After push of [JDK-8307478](https://bugs.openjdk.org/browse/JDK-8307478) ,
> the following test started to fail on AIX :
> com/sun/tools/attach/warnings/DynamicLoadWarningTest.java;
> The problem was described in
> [JDK-8309549](https://bugs.openjdk.org/browse/JDK-8309549) with a first try
> o
On Tue, 12 Sep 2023 04:59:13 GMT, David Holmes wrote:
>> Joachim Kern has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> try to improve code following Davids suggestions and do some cosmetic
>> changes
>
> src/hotspot/share/prims/jvmtiAgen
Original failure in 8081569 is a long time ago, and not reproducing.
Other failures recorded there may be different problems, some of them are port
related which are helped by other changes since these tests were problemlisted.
This problemlist entry should be removed.
-
Commit mess
On Wed, 13 Sep 2023 01:07:58 GMT, David Holmes wrote:
>> src/java.se/share/data/jdwp/jdwp.spec line 1178:
>>
>>> 1176: "support the timer mechanism for virtual threads, and thus
>>> methods such as "
>>> 1177: ">> href=../../api/java.base/java/lang/Thread.html#sleep(long)>Thread
On Tue, 12 Sep 2023 22:24:18 GMT, Chris Plummer wrote:
> This PR includes a couple of clarifications of the JDWP and JDI invoke method
> support. The first is that deadlocks can occur due to a needed "resource"
> being held. The spec previously just mentioned monitors being held, but this
> is
Hi, please consider.
Inspired by [JDK-8313800](https://bugs.openjdk.org/browse/JDK-8313800). RISC-V
also treats x8/fp as a callee-saved scratch register for some time, and it is
freely used by C2-generated code. Therefore, any code in SA that uses getFP()
in a compiled frame is very likely to co
On Wed, 13 Sep 2023 06:07:06 GMT, Leonid Mesnik wrote:
> I start testing of fix in CI and approve once it is completed.
Thank you!
-
PR Comment: https://git.openjdk.org/jdk/pull/15689#issuecomment-1717099563
28 matches
Mail list logo