RFR: 8373557: Remove stale comments after JDK-8372584
duke
duke at openjdk.org
Fri Dec 12 10:39:54 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.
@JonasNorlinder
Your change (at version 7b056ce8fe7c22e5d8a848c9060086aa681c1ccf) is now ready to be sponsored by a Committer.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/28777#issuecomment-3645922563
More information about the hotspot-runtime-dev
mailing list