pgsql: pgstat: Acquire lock when reading variable-numbered stats - Mailing list pgsql-committers

From Andres Freund
Subject pgsql: pgstat: Acquire lock when reading variable-numbered stats
Date
Msg-id E1oQKSd-000mm0-1f@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
pgstat: Acquire lock when reading variable-numbered stats

Somewhere during the development of the patch acquiring a lock during read
access to variable-numbered stats got lost. The missing lock acquisition won't
cause corruption, but can lead to reading torn values when accessing
stats. Add the missing lock acquisitions.

Reported-by: Greg Stark <stark@mit.edu>
Reviewed-by: "Drouvot, Bertrand" <bdrouvot@amazon.com>
Reviewed-by: Andres Freund <andres@anarazel.de>
Author: Kyotaro Horiguchi <horikyota.ntt@gmail.com>
Discussion: https://postgr.es/m/CAM-w4HMYkM_DkYhWtUGV+qE_rrBxKOzOF0+5faozxO3vXrc9wA@mail.gmail.com
Backpatch: 15-

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/cd063344fb801a90a40923a5b8aefe4eb8d80762

Modified Files
--------------
src/backend/utils/activity/pgstat.c       |  9 +++++++++
src/backend/utils/activity/pgstat_shmem.c | 16 ++++++++++++++++
src/include/utils/pgstat_internal.h       |  1 +
3 files changed, 26 insertions(+)


pgsql-committers by date:

Previous
From: John Naylor
Date:
Subject: Re: pgsql: Refer to replication origin roident as "ID" in user facing messa
Next
From: Andres Freund
Date:
Subject: pgsql: Remove redundant call to pgstat_report_wal()