Integrated: 8329134: Reconsider TLAB zapping
Aleksey Shipilev
shade at openjdk.org
Mon Apr 1 17:29:45 UTC 2024
On Tue, 26 Mar 2024 21:08:16 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:
> We zap the entire TLAB on initial allocation (`MemAllocator::mem_allocate_inside_tlab_slow`), and then also rezap the object contents when object is allocated from the TLAB (`ThreadLocalAllocBuffer::allocate`). The second part seems excessive, given the TLAB is already fully zapped.
>
> There is also no way to disable this zapping, like you would in other places with the relevant Zap* flags.
>
> Fixing both these issues allows to improve fastdebug tests performance, e.g. in jcstress.
>
> It also allows to remove the related Zero kludge.
>
> Additional testing:
> - [x] Linux AArch64 server fastdebug, `all` tests
> - [x] MacOS AArch64 Zero fastdebug, `bootcycle-images` pass
This pull request has now been integrated.
Changeset: 5698f7ad
Author: Aleksey Shipilev <shade at openjdk.org>
URL: https://git.openjdk.org/jdk/commit/5698f7ad29c939b7e52882ace575dd7113bf41de
Stats: 30 lines in 5 files changed: 5 ins; 16 del; 9 mod
8329134: Reconsider TLAB zapping
Reviewed-by: stefank, rkennke
-------------
PR: https://git.openjdk.org/jdk/pull/18500
More information about the shenandoah-dev
mailing list