RFR: 8362893: Improve performance for MemorySegment::getString [v2]
Philippe Marschall
duke at openjdk.org
Mon Jul 28 17:10:13 UTC 2025
> Use `JavaLangAccess::uncheckedNewStringNoRepl` in `MemorySegment::getString` to avoid byte[] allocation in the String constructor.
>
> Fall back to the old code in the case of malformed input to get replacement characters as per Javadoc API specification. The existing tests in `TestStringEncoding` seem sufficient to me.
>
> I ran the tier1 test suite and it passes.
>
> For performance analysis I ran.
>
> make test TEST="micro:org.openjdk.bench.java.lang.foreign.ToJavaStringTest" MICRO="OPTIONS=-prof gc"
>
> on an AMD Ryzen 7 PRO 5750GE.
>
> These are the formatted results, the current master is the line on top, this feature branch is the line below. We can see an improvement in throughput driven by a reduction in allocation.
>
>
> Benchmark (size) Mode Cnt Score Error Units
>
> ToJavaStringTest.panama_readString 5 avgt 30 18.996 ± 0.044 ns/op
> ToJavaStringTest.panama_readString 5 avgt 30 13.851 ± 0.028 ns/op
>
> ToJavaStringTest.panama_readString 20 avgt 30 23.570 ± 0.050 ns/op
> ToJavaStringTest.panama_readString 20 avgt 30 18.401 ± 0.069 ns/op
>
> ToJavaStringTest.panama_readString 100 avgt 30 32.094 ± 0.207 ns/op
> ToJavaStringTest.panama_readString 100 avgt 30 24.427 ± 0.112 ns/op
>
> ToJavaStringTest.panama_readString 200 avgt 30 43.029 ± 0.185 ns/op
> ToJavaStringTest.panama_readString 200 avgt 30 31.914 ± 0.064 ns/op
>
> ToJavaStringTest.panama_readString 451 avgt 30 81.145 ± 0.403 ns/op
> ToJavaStringTest.panama_readString 451 avgt 30 58.975 ± 0.233 ns/op
>
> ToJavaStringTest.panama_readString:gc.alloc.rate.norm 5 avgt 30 72.000 ± 0.001 B/op
> ToJavaStringTest.panama_readString:gc.alloc.rate.norm 5 avgt 30 48.000 ± 0.001 B/op
>
> ToJavaStringTest.panama_readString:gc.alloc.rate.norm 20 avgt 30 104.000 ± 0.001 B/op
> ToJavaStringTest.panama_readString:gc.alloc.rate.norm 20 avgt 30 64.000 ± 0.001 B/op
>
> ToJavaStringTest.panama_readString:gc.alloc.rate.norm 100 avgt 30 264.000 ± 0.001 B/op
> ToJavaStringTest.panama_readString:gc.alloc.rate.norm 100 avgt 30 144.000 ± 0.001 B/op
>
> ToJavaStringTest.panama_readString:gc.alloc.rate.norm 200 avgt ...
Philippe Marschall has updated the pull request incrementally with one additional commit since the last revision:
Fix copyright year
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/26493/files
- new: https://git.openjdk.org/jdk/pull/26493/files/24a83a5b..130ad859
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=26493&range=01
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=26493&range=00-01
Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod
Patch: https://git.openjdk.org/jdk/pull/26493.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/26493/head:pull/26493
PR: https://git.openjdk.org/jdk/pull/26493
More information about the core-libs-dev
mailing list