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