Re: DTrace probes patch - Mailing list pgsql-hackers

From Robert Lor
Subject Re: DTrace probes patch
Date
Msg-id 494933C7.6020203@sun.com
Whole thread Raw
In response to Re: DTrace probes patch  ("Fujii Masao" <masao.fujii@gmail.com>)
Responses Re: DTrace probes patch  (Alvaro Herrera <alvherre@commandprompt.com>)
List pgsql-hackers
Fujii Masao wrote:
> Hi,
>
> On Wed, Dec 17, 2008 at 4:53 AM, Robert Lor <Robert.Lor@sun.com> wrote:
>   
>> @@ -1313,12 +1318,14 @@ AdvanceXLInsertBuffer(bool new_segment)
>>                                 * Have to write buffers while holding insert
>> lock. This is
>>                                 * not good, so only write as much as we
>> absolutely must.
>>                                 */
>> +                               TRACE_POSTGRESQL_WAL_BUFFER_WRITE_START();
>>                                WriteRqst.Write = OldPageRqstPtr;
>>                                WriteRqst.Flush.xlogid = 0;
>>                                WriteRqst.Flush.xrecoff = 0;
>>                                XLogWrite(WriteRqst, false, false);
>>                                LWLockRelease(WALWriteLock);
>>                                Insert->LogwrtResult = LogwrtResult;
>> +                               TRACE_POSTGRESQL_WAL_BUFFER_WRITE_DONE();
>>     
>
> Why is TRACE_POSTGRESQL_WAL_BUFFER_WRITE_START/DONE called
> only in AdvanceXLInsertBuffer? We can trace only a part of WAL buffer write?
>
>   
The intention of these probes is to determine if wal_buffers is too 
small by monitoring how frequent the server has to write out the buffers 
along with the I/O time.

-Robert




pgsql-hackers by date:

Previous
From: Michael Meskes
Date:
Subject: Re: Looking for someone with MinGW
Next
From: Alvaro Herrera
Date:
Subject: Re: DTrace probes patch