On Mon, 28 Sep 2026 15:02:33 GMT, Yunbo Zhang <[email protected]> wrote:
>> Please review this small fix. >> >> Configuring the same log file as `vm.log`, `$PWD/vm.log`, and `./vm.log` >> creates separate outputs, splitting the messages across multiple files. >> >> **Fix:** >> >> Let each output match its own name through `matches_name()`. For file >> outputs, also check `os::same_files()` so different paths to the same file >> reuse the existing output. >> >> **Testing:** >> >> - Added `TestEquivalentLogFilePaths.java`. >> - `gtest:LogConfiguration/server`: 21 passed, 0 failed. >> - `test/hotspot/jtreg/serviceability/logging`: 7 passed, 0 failed. >> - `test/hotspot/jtreg/runtime/logging`: 33 passed, 0 failed. >> >> --------- >> - [x] I confirm that I make this contribution in accordance with the >> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai). > > Yunbo Zhang has updated the pull request incrementally with one additional > commit since the last revision: > > 8392135: Unified logging creates extra log files > > Use virtual output name matching instead of identifying output types by > name prefixes and downcasting. Add coverage for file and standard output > matching. Q1: Does this name comparison have a race with the log rotation mechanism? Is there a need in synchronization? ------------- PR Comment: https://git.openjdk.org/jdk/pull/33090#issuecomment-6278694231
