RFR: 8293873: Centralize the initialization of UL [v2]

Robbin Ehn rehn at openjdk.org
Fri Oct 21 10:25:58 UTC 2022


On Fri, 23 Sep 2022 10:38:21 GMT, Johan Sjölen <jsjolen at openjdk.org> wrote:

>> Hi,
>> 
>> May I please have a review of this change? It moves around some of the initialization code of UL to one place (namely `LogConfiguration::initialize`). It does not change how UL works after VM start up, and it should not change how UL works after VM shut down, but it might change how UL works before VM start up. I don't believe that this causes any breakage.
>> 
>> Tier 2 and 3 tests are currently running, all green so far.
>> 
>> Some more context is available at https://bugs.openjdk.org/browse/JDK-8293873
>
> Johan Sjölen has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Fix copyright years

Looks like a good improvement and Thomas is happy, so I'm happy.

-------------

Marked as reviewed by rehn (Reviewer).

PR: https://git.openjdk.org/jdk/pull/10307


More information about the hotspot-runtime-dev mailing list