RFR: Merge jdk17

2021-06-14 Thread Jesper Wilhelmsson
Forwardport JDK 17 -> JDK 18

-

Commit messages:
 - 8267579: Thread::cooked_allocated_bytes() hits assert(left >= right) failed: 
avoid underflow
 - 8268342: java/foreign/channels/TestAsyncSocketChannels.java fails with 
"IllegalStateException: This segment is already closed"
 - 8268630: ProblemList serviceability/jvmti/CompiledMethodLoad/Zombie.java on 
linux-aarch64
 - 8268470: CDS dynamic dump asserts with JFR RecordingStream
 - 8268093: Manual Testcase: 
"sun/security/krb5/config/native/TestDynamicStore.java" Fails with NPE
 - 8268602: a couple runtime/os tests don't check exit code
 - 8268555: Update HttpClient tests that use ITestContext to jtreg 6+1
 - 8268580: runtime/memory/LargePages/TestLargePagesFlags.java should be run in 
driver mode
 - 8268565: runtime/records/RedefineRecord.java should be run in driver mode
 - 8268576: jdk/jfr/event/gc/collection/TestSystemGc.java fails
 - ... and 3 more: https://git.openjdk.java.net/jdk/compare/74007890...b3185354

The merge commit only contains trivial merges, so no merge-specific webrevs 
have been generated.

Changes: https://git.openjdk.java.net/jdk/pull/4484/files
  Stats: 786 lines in 57 files changed: 593 ins; 73 del; 120 mod
  Patch: https://git.openjdk.java.net/jdk/pull/4484.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/4484/head:pull/4484

PR: https://git.openjdk.java.net/jdk/pull/4484


Re: RFR: Merge jdk17

2021-06-14 Thread Daniel D . Daugherty
On Mon, 14 Jun 2021 14:28:33 GMT, Jesper Wilhelmsson  
wrote:

> Forwardport JDK 17 -> JDK 18

Thumbs up!

Thanks for doing this sync forward.

-

Marked as reviewed by dcubed (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/4484


Re: RFR: Merge jdk17 [v2]

2021-06-14 Thread Jesper Wilhelmsson
> Forwardport JDK 17 -> JDK 18

Jesper Wilhelmsson has updated the pull request with a new target base due to a 
merge or a rebase. The incremental webrev excludes the unrelated changes 
brought in by the merge/rebase.

-

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/4484/files
  - new: https://git.openjdk.java.net/jdk/pull/4484/files/b3185354..b3185354

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=4484&range=01
 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=4484&range=00-01

  Stats: 0 lines in 0 files changed: 0 ins; 0 del; 0 mod
  Patch: https://git.openjdk.java.net/jdk/pull/4484.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/4484/head:pull/4484

PR: https://git.openjdk.java.net/jdk/pull/4484


Re: RFR: Merge jdk17 [v2]

2021-06-14 Thread Daniel Fuchs
On Mon, 14 Jun 2021 15:58:15 GMT, Jesper Wilhelmsson  
wrote:

>> Forwardport JDK 17 -> JDK 18
>
> Jesper Wilhelmsson has updated the pull request with a new target base due to 
> a merge or a rebase. The incremental webrev excludes the unrelated changes 
> brought in by the merge/rebase.

Looked at this two changesets and they were fine.
 - 8268342: java/foreign/channels/TestAsyncSocketChannels.java fails with 
"IllegalStateException: This segment is already
 - 8268555: Update HttpClient tests that use ITestContext to jtreg 6+1

-

Marked as reviewed by dfuchs (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/4484


Integrated: Merge jdk17

2021-06-14 Thread Jesper Wilhelmsson
On Mon, 14 Jun 2021 14:28:33 GMT, Jesper Wilhelmsson  
wrote:

> Forwardport JDK 17 -> JDK 18

This pull request has now been integrated.

Changeset: 17295b1b
Author:Jesper Wilhelmsson 
URL:   
https://git.openjdk.java.net/jdk/commit/17295b1bb02b2121978f1459b2e75c5e1031e7ea
Stats: 721 lines in 30 files changed: 573 ins; 73 del; 75 mod

Merge

Reviewed-by: dcubed

-

PR: https://git.openjdk.java.net/jdk/pull/4484


[jdk17] RFR: JDK-8265369 [macos-aarch64] java/net/MulticastSocket/Promiscuous.java failed with "SocketException: Cannot allocate memory"

2021-06-14 Thread Mark Sheppard
… failed with "SocketException: Cannot allocate memory

The test java/net/MulticastSocket/Promiscuous.java has been observed to fail on 
a regular basis on macosx-aarch.
This is typically under heavy test load on a test machine. Analysis of the 
problem have
shown that the setsockopt for joining a multicast group will intermittently 
fail with ENOMEM.

While analysis of test environment shows significant memory usage and some 
memory pressure, it is
not excessive and as such it is deemed transition or temporary condition, such 
that a retry of the
setsockopt system call, has been seen to mitigate the issue. This adds to the 
stability of the
Promiscuous.java test and reduces test failure noise.

The proposed fix is in 
open/src/java.base/unix/native/libnet/PlainDatagramSocketImpl.c
in the mcast_join_leave function.  That is, if setsockopt to join an mcast 
group fails, and the errno == ENOMEM, 
then re-invoke the setsockopt system call for joining a mcast group.
The change has been applied as a conditional compilation.
Additionally this change result in the Promiscuous.java test being removed from 
the
ProblemList.txt.

Please oblige and review the changes for a fix of the issue JDK-8265369

-

Commit messages:
 - JDK-8265369 [macos-aarch64] java/net/MulticastSocket/Promiscuous.java failed 
with "SocketException: Cannot allocate memory"
 - JDK-8265369 [macos-aarch64] java/net/MulticastSocket/Promiscuous.java failed 
with "SocketException: Cannot allocate memory"
 - JDK-8265369 [macos-aarch64] java/net/MulticastSocket/Promiscuous.java failed 
with "SocketException: Cannot allocate memory

Changes: https://git.openjdk.java.net/jdk17/pull/44/files
 Webrev: https://webrevs.openjdk.java.net/?repo=jdk17&pr=44&range=00
  Issue: https://bugs.openjdk.java.net/browse/JDK-8265369
  Stats: 51 lines in 2 files changed: 43 ins; 1 del; 7 mod
  Patch: https://git.openjdk.java.net/jdk17/pull/44.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk17 pull/44/head:pull/44

PR: https://git.openjdk.java.net/jdk17/pull/44


Re: RFR: 8263364: sun/net/www/http/KeepAliveStream/KeepAliveStreamCloseWithWrongContentLength.java wedged in getInputStream [v4]

2021-06-14 Thread Ivan Šipka
> @dfuch  could you please review, thank you.

Ivan Šipka has updated the pull request incrementally with two additional 
commits since the last revision:

 - 8263364: removing surplus imports
 - 8263364: refactor

-

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/4472/files
  - new: https://git.openjdk.java.net/jdk/pull/4472/files/4f530cea..5a5dc151

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=4472&range=03
 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=4472&range=02-03

  Stats: 3 lines in 1 file changed: 1 ins; 2 del; 0 mod
  Patch: https://git.openjdk.java.net/jdk/pull/4472.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/4472/head:pull/4472

PR: https://git.openjdk.java.net/jdk/pull/4472