pgsql: pgstat: Track more detailed relation IO statistics

From: Andres Freund <andres(at)anarazel(dot)de>
To: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: pgsql: pgstat: Track more detailed relation IO statistics
Date: 2023-02-10 06:24:44
Message-ID: E1pQMpw-000MPf-4r@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers pgsql-hackers

pgstat: Track more detailed relation IO statistics

Commit 28e626bde00 introduced the infrastructure for tracking more detailed IO
statistics. This commit adds the actual collection of the new IO statistics
for relations and temporary relations. See aforementioned commit for goals and
high-level design.

The changes in this commit are fairly straight-forward. The bulk of the change
is to passing sufficient information to the callsites of pgstat_count_io_op().

A somewhat unsightly detail is that it currently is hard to find a better
place to count fsyncs than in md.c, whereas the other pgstat_count_io_op()
calls are in bufmgr.c/localbuf.c. As the number of fsyncs is tied to md.c
implementation details, it's not obvious there is a better answer.

Author: Melanie Plageman <melanieplageman(at)gmail(dot)com>
Reviewed-by: Andres Freund <andres(at)anarazel(dot)de>
Discussion: https://postgr.es/m/20200124195226.lth52iydq2n2uilq@alap3.anarazel.de

Branch
------
master

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

Modified Files
--------------
src/backend/storage/buffer/bufmgr.c | 110 +++++++++++++++++++++++++++++-----
src/backend/storage/buffer/freelist.c | 58 +++++++++++++-----
src/backend/storage/buffer/localbuf.c | 13 +++-
src/backend/storage/smgr/md.c | 24 ++++++++
src/include/storage/buf_internals.h | 8 ++-
src/include/storage/bufmgr.h | 7 ++-
6 files changed, 184 insertions(+), 36 deletions(-)

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Michael Paquier 2023-02-10 06:39:18 pgsql: Fix typo in parsenodes.h
Previous Message Amit Kapila 2023-02-10 02:06:25 Re: pgsql: Use appropriate wait event when sending data in the apply worker

Browse pgsql-hackers by date

  From Date Subject
Next Message Peter Eisentraut 2023-02-10 06:42:31 Re: ICU locale validation / canonicalization
Previous Message Michael Paquier 2023-02-10 05:59:13 Re: Introduce a new view for checkpointer related stats