On Thu, 7 Nov 2024 18:30:22 GMT, Vladimir Ivanov <vaiva...@openjdk.org> wrote:
>> The synchronization block may be substituted by the 'volatile' variable >> smaller synchronization block. >> It reduce the total blocking time for the specjvm2008::xml.validation >> workload and improve the reported score. >> Scores for the 112vCPU on the with 28GB heap increased from 17915.83 to >> 22943.2. >> Unit tests was not affected: >> Test summary >> ============================== >> TEST TOTAL PASS FAIL ERROR >> jtreg:test/jaxp 516 516 0 0 >> jtreg:test/jdk/javax/xml 70 70 0 0 >> ============================== >> TEST SUCCESS >> >> The tier1 is OK too. > > Vladimir Ivanov has updated the pull request incrementally with one > additional commit since the last revision: > > 8317542: Specjvm::xml have scalability issue for high vCPU numbers Thanks for the great work! Nice improvement on systems with high vCPUs. There are other areas worth checking. But this is a good step forward. ------------- Marked as reviewed by joehw (Reviewer). PR Review: https://git.openjdk.org/jdk/pull/21815#pullrequestreview-2424531410
