RFR: 8264006: Fix AOT library loading on CPUs with 256-byte dcache line
David Holmes
dholmes at openjdk.java.net
Wed Mar 24 13:32:43 UTC 2021
On Wed, 24 Mar 2021 13:26:31 GMT, David Holmes <dholmes at openjdk.org> wrote:
>> I must confess I don't like this solution at all: it sounds very delicate. Couldn't you define a function `VM_Version::get_ContendedPaddingWidth()`and call that?
>
> The compiler folk will need to see if VM_Version::initialize itself has any dependencies on the AOTLoader initialization. Changing the initialization order is always risky.
>
> On a side note please don't modify any copyright line except for Oracle's when modifying files, unless instructed to by the owner of that copyright.
>
> Thanks,
> David
It may be possible to factor out the necessary logic to the VM_Version::early_initialize() function instead.
-------------
PR: https://git.openjdk.java.net/jdk/pull/3169
More information about the hotspot-runtime-dev
mailing list