RFR: 6478546: FileInputStream.read() throws OutOfMemoryError when there is plenty available [v5]

Brian Burkhalter bpb at openjdk.org
Mon Oct 9 19:02:19 UTC 2023


> Limit native memory allocation and move write loop from the native layer into Java. This change should make the OOME reported in the issue much less likely.

Brian Burkhalter 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. The pull request contains seven additional commits since the last revision:

 - Merge
 - 6478546: do-while -> while-do in writeBytes; remove NULL and bounds checks in native
 - 6478546: Move buffer clamping up to Java layer; correct read behavior to match legacy
 - 6478546: Decrease malloc limit to 1.5 MB
 - 6478546: Minor refactoring
 - 6478546: Prevent short read
 - 6478546: FileInputStream.read() throws OutOfMemoryError when there is plenty available

-------------

Changes:
  - all: https://git.openjdk.org/jdk/pull/14981/files
  - new: https://git.openjdk.org/jdk/pull/14981/files/cdb8455c..3a016a72

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=14981&range=04
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=14981&range=03-04

  Stats: 200415 lines in 4979 files changed: 96645 ins; 49069 del; 54701 mod
  Patch: https://git.openjdk.org/jdk/pull/14981.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/14981/head:pull/14981

PR: https://git.openjdk.org/jdk/pull/14981


More information about the core-libs-dev mailing list