On Thu, 22 Oct 2026 19:35:43 GMT, Serguei Spitsyn <[email protected]> wrote:

>> Yunbo Zhang has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   8392135: Unified logging creates extra log files
>>   
>>   Synchronize file identity comparisons with rotation using the existing 
>> semaphore, and add a concurrent alias-matching regression test.
>
> test/hotspot/jtreg/serviceability/logging/TestEquivalentLogFilePaths.java 
> line 60:
> 
>> 58:                 "-Xlog:gc=info:" + name,
>> 59:                 "-Xlog:class+load=info:" + quote + log.toAbsolutePath() 
>> + quote,
>> 60:                 "-Xlog:safepoint=info:./" + name,
> 
> Nit: There is no check for this log configuration messages below.

Thanks. I removed the safepoint option. `-Xlog:gc=info` and 
`-Xlog:class+load=info` are sufficient for this test.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/33090#discussion_r4356008858

Reply via email to