Integrated: 8321269: Require platforms to define DEFAULT_CACHE_LINE_SIZE
Aleksey Shipilev
shade at openjdk.org
Tue Dec 5 10:38:54 UTC 2023
On Mon, 4 Dec 2023 12:42:45 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:
> Found it while doing new code that wants to know the cache line size. Currently, there is a fallback in `globalDefinitions.hpp` that defaults `DEFAULT_CACHE_LINE_SIZE` to `64` if platform does not define it. Instead of relying on default, force platform definitions to tell what is the reasonable default for the platform. This would simplify porting to other architectures, with less surprises for them.
>
> The actual sizes do not change. If any existing platform needs adjustments, those should be handled as separate issues.
This pull request has now been integrated.
Changeset: a56286f7
Author: Aleksey Shipilev <shade at openjdk.org>
URL: https://git.openjdk.org/jdk/commit/a56286f7ad9a8110026f48eb45f1d7a273b2f9fb
Stats: 7 lines in 4 files changed: 6 ins; 0 del; 1 mod
8321269: Require platforms to define DEFAULT_CACHE_LINE_SIZE
Reviewed-by: stefank, stuefe, dholmes
-------------
PR: https://git.openjdk.org/jdk/pull/16948
More information about the hotspot-dev
mailing list