Fix variable names mentioned in comment to match the code.

Also, in another comment, explain why holding an insertion slot is a
critical section.

Per review by Amit Kapila.

diff --git a/src/backend/access/transam/xlog.c b/src/backend/access/transam/xlog.c
index 8d43660353f5f98d6b171b5f3b3895d37969f4a6..96aceb9e087e58cbc7ec2fb6b944e4a8dc6fbaae 100644 (file)
--- a/src/backend/access/transam/xlog.c
+++ b/src/backend/access/transam/xlog.c
@@ -1322,8 +1322,8 @@ ReserveXLogInsertLocation(int size, XLogRecPtr *StartPos, XLogRecPtr *EndPos,
  *
  * A log-switch record is handled slightly differently. The rest of the
  * segment will be reserved for this insertion, as indicated by the returned
- * *EndPos_p value. However, if we are already at the beginning of the current
- * segment, *StartPos_p and *EndPos_p are set to the current location without
+ * *EndPos value. However, if we are already at the beginning of the current
+ * segment, *StartPos and *EndPos are set to the current location without
  * reserving any space, and the function returns false.
 */
 static bool
@@ -1575,7 +1575,9 @@ WALInsertSlotAcquireOne(int slotno)
    /*
     * Lock out cancel/die interrupts until we exit the code section protected
     * by the slot.  This ensures that interrupts will not interfere with
-    * manipulations of data structures in shared memory.
+    * manipulations of data structures in shared memory. There is no cleanup
+    * mechanism to release the slot if the backend dies while holding one,
+    * so make this a critical section.
     */
    START_CRIT_SECTION();