]> granicus.if.org Git - procps-ng/commit
Set TZ to avoid repeated stat("/etc/localtime")
authorStephen Brennan <stephen.s.brennan@oracle.com>
Fri, 20 Nov 2020 00:03:58 +0000 (16:03 -0800)
committerCraig Small <csmall@dropbear.xyz>
Tue, 22 Dec 2020 00:08:36 +0000 (11:08 +1100)
commit31343570e17cb13605c2c5a5c8b9a21d443bbb2a
tree73f02452ba304c8e7dea58ebfaf4c4dc3e1a766c
parent930dad118e7c0e9b19fe6eef674ed6b8dba5c7c0
Set TZ to avoid repeated stat("/etc/localtime")

With glibc, each time the strftime() function is used (twice per process
in a typical ps -fe run), a stat("/etc/localtime") system call is used
to determine the timezone. Not only does this add extra system call
overhead, but when multiple ps processes are trying to access this
file (or multiple glibc programs using strftime) in parallel, this can
trigger significant lock contention within the OS kernel.

Since ps is not intended to run for long periods of time as a
daemon (during which the system timezone could be altered and PS might
reasonably be expected to adapt its output), there is no benefit to
repeatedly doing this stat(). To stop this behavior, explicitly set the
TZ variable to its default value (:/etc/localtime) whenever it is unset.
glibc will then cache the stat() result.
ps/display.c