Integrated: 8373557: Remove stale comments after JDK-8372584
Jonas Norlinder
jnorlinder at openjdk.org
Mon Dec 15 06:16:09 UTC 2025
On Thu, 11 Dec 2025 22:03:54 GMT, Jonas Norlinder <jnorlinder at openjdk.org> wrote:
> Hi all,
>
> This is a small patch to fix stale comments after removing reading /proc to get user CPU time.
>
> A note on the proposed removal of:
>
> // This is the fastest way to get thread cpu time on Linux.
> // Returns cpu time (user+sys) for any thread, not only for current.
> // POSIX compliant clocks are implemented in the kernels 2.6.16+.
> // It might work on 2.6.10+ with a special kernel/glibc patch.
> // For reference, please, see IEEE Std 1003.1-2004:
> // http://www.unix.org/single_unix_specification
>
> jlong os::Linux::thread_cpu_time(clockid_t clockid) {
>
>
> Since we may call this method with a clockid_t that will only resolve user time this comment may be confusing. Moreover, I don't think we need to keep the comment here when POSIX compliant clock are implemented.
This pull request has now been integrated.
Changeset: dc1b0b5f
Author: Jonas Norlinder <jnorlinder at openjdk.org>
Committer: David Holmes <dholmes at openjdk.org>
URL: https://git.openjdk.org/jdk/commit/dc1b0b5f81b6c3de85a0234d0315370b6413c077
Stats: 11 lines in 2 files changed: 0 ins; 10 del; 1 mod
8373557: Remove stale comments after JDK-8372584
Reviewed-by: dholmes, jsjolen
-------------
PR: https://git.openjdk.org/jdk/pull/28777
More information about the hotspot-runtime-dev
mailing list