RFR: 8267517: async logging for stdout and stderr

Xin Liu xliu at openjdk.java.net
Wed Dec 15 03:29:00 UTC 2021


On Tue, 14 Dec 2021 02:01:09 GMT, David Holmes <dholmes at openjdk.org> wrote:

> Hi Xin,
> 
> I took a look at this but to be honest I couldn't work out what most of the changes actually had to do with the topic of adding async logging for stdout/stderr. IIUC the change from `LogFileOutput` to `LogFileStreamOutput` in the API now enables it to be used for stdout/stderr - right? But most of the changes seemed to relate to log file rotation which doesn't even apply to stdout/stderr! What has force_rotation got to do with things?
> 
> David

Sorry, this is my fault. This patch incorporates with JDK-8275083. They are so intertwined that I can't split them. 
So far, I fail to measure any meaningful performance gain from JDK-8275083, so let me refactor the code and remove JDK-8275083.

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

PR: https://git.openjdk.java.net/jdk/pull/6132


More information about the hotspot-runtime-dev mailing list