> Small patch fixing a missing memory barrier in FlatArrayKlass::copy_array. > The unit test covers more scenario where a flat value has to be buffered > before to be written to another heap variable. So far, these tests haven't > shown any other issue (they were run repeatedly 50 times using Mach5). > > However, the unit test is quite heavy, testing 5 different scenarios, and > running each for 3 minutes, using all detected cores of the machine each > time. This might qualify this unit test as a "resourcehog" test, it then > should be moved to test/hotspot/jtreg/resourcehogs/ > > Fred
Frederic Parain 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 six additional commits since the last revision: - Merge remote-tracking branch 'upstream/lworld' into arraycopy_membar - Move barrier to buffering code - Fix jtreg tags - Move test to the resourcehogs category - Add new line at the end - Add missing memory barrier in FlatArrayKlass::copy_array ------------- Changes: - all: https://git.openjdk.org/valhalla/pull/2183/files - new: https://git.openjdk.org/valhalla/pull/2183/files/46e44c0a..e29b6580 Webrevs: - full: https://webrevs.openjdk.org/?repo=valhalla&pr=2183&range=04 - incr: https://webrevs.openjdk.org/?repo=valhalla&pr=2183&range=03-04 Stats: 47461 lines in 1374 files changed: 24397 ins; 12286 del; 10778 mod Patch: https://git.openjdk.org/valhalla/pull/2183.diff Fetch: git fetch https://git.openjdk.org/valhalla.git pull/2183/head:pull/2183 PR: https://git.openjdk.org/valhalla/pull/2183
