RFR: 8296158: Refactor the verification of CDS region checksum [v2]

Ioi Lam iklam at openjdk.org
Tue Feb 14 22:12:09 UTC 2023


> Consolidate the two old functions
> 
> - `FileMapInfo::region_crc_check(char* buf, size_t size, int expected_crc)`
> - `FileMapInfo::verify_region_checksum(int i)`
> 
> Into a single function `FileMapRegion::check_region_crc()`.
> 
> Also performed some minor renaming and removed a useless API.

Ioi Lam has updated the pull request incrementally with one additional commit since the last revision:

  @matias9927 suggestions

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/12424/files
  - new: https://git.openjdk.org/jdk/pull/12424/files/3ad8d838..10dcd0b1

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=12424&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=12424&range=00-01

  Stats: 7 lines in 1 file changed: 0 ins; 3 del; 4 mod
  Patch: https://git.openjdk.org/jdk/pull/12424.diff
  Fetch: git fetch https://git.openjdk.org/jdk pull/12424/head:pull/12424

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


More information about the hotspot-runtime-dev mailing list