pgsql: Add isExtend to the parameters of the buffer_read_start and

From: tgl(at)postgresql(dot)org (Tom Lane)
To: pgsql-committers(at)postgresql(dot)org
Subject: pgsql: Add isExtend to the parameters of the buffer_read_start and
Date: 2009-03-22 22:39:05
Message-ID: 20090322223905.5CC30754ADE@cvs.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Log Message:
-----------
Add isExtend to the parameters of the buffer_read_start and buffer_read_done
DTrace probes, so that ordinary reads can be distinguished from relation
extension operations. Move buffer_read_start probe to before the
smgrnblocks() call that's needed in the isExtend case, since really that step
should be charged as part of the time needed for the extension operation.
(This makes it slightly harder to match the read_start with the associated
read_done, since now you can't match them on blockNumber, but it should still
be possible since isExtend operations on the same relation can never be
interleaved.) Per recent discussion.

In passing, add the page identity (forkNum/blockNum) to the parameters of the
buffer_flush_start/buffer_flush_done probes, which were unaccountably lacking
the info.

Modified Files:
--------------
pgsql/src/backend/storage/buffer:
bufmgr.c (r1.247 -> r1.248)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/backend/storage/buffer/bufmgr.c?r1=1.247&r2=1.248)
pgsql/src/backend/utils:
probes.d (r1.7 -> r1.8)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/backend/utils/probes.d?r1=1.7&r2=1.8)

Browse pgsql-committers by date

  From Date Subject
Next Message Bruce Momjian 2009-03-23 01:45:29 pgsql: Clarify libpq 'sslverify' documentation wording.
Previous Message Bruce Momjian 2009-03-22 21:55:07 pgsql: Clarify 'cert' authentication documention.