Re: revise a comment in CreateCheckPoint()

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: "Qingqing Zhou" <zhouqq(at)cs(dot)toronto(dot)edu>
Cc: pgsql-patches(at)postgresql(dot)org
Subject: Re: revise a comment in CreateCheckPoint()
Date: 2005-04-23 18:52:16
Message-ID: 13228.1114282336@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-patches

"Qingqing Zhou" <zhouqq(at)cs(dot)toronto(dot)edu> writes:
> The original comment of why we are safe without protection of critical
> section is confusing.

I thought your version was even more so :-(. I've applied the attached
patch instead.

regards, tom lane

Index: xlog.c
===================================================================
RCS file: /cvsroot/pgsql/src/backend/access/transam/xlog.c,v
retrieving revision 1.187
diff -c -r1.187 xlog.c
*** xlog.c 17 Apr 2005 03:04:29 -0000 1.187
--- xlog.c 23 Apr 2005 18:48:45 -0000
***************
*** 4925,4931 ****
*
* This I/O could fail for various reasons. If so, we will fail to
* complete the checkpoint, but there is no reason to force a system
! * panic. Accordingly, exit critical section while doing it.
*/
END_CRIT_SECTION();

--- 4925,4934 ----
*
* This I/O could fail for various reasons. If so, we will fail to
* complete the checkpoint, but there is no reason to force a system
! * panic. Accordingly, exit critical section while doing it. (If
! * we are doing a shutdown checkpoint, we probably *should* panic ---
! * but that will happen anyway because we'll still be inside the
! * critical section established by ShutdownXLOG.)
*/
END_CRIT_SECTION();

In response to

Responses

Browse pgsql-patches by date

  From Date Subject
Next Message Bruce Momjian 2005-04-23 20:55:37 VACUUM VERBOSE FSM info
Previous Message Bruce Momjian 2005-04-23 18:13:06 Re: Updated FSM display for VACUUM VERBOSE