Re: Deadlock condition in current sources - Mailing list pgsql-hackers

From Mikheev, Vadim
Subject Re: Deadlock condition in current sources
Date
Msg-id 3705826352029646A3E91C53F7189E3251844F@sectorbase2.sectorbase.com
Whole thread Raw
In response to Deadlock condition in current sources  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: Deadlock condition in current sources  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
> However, there's still a problem: GetUndoRecPtr also gets SInvalLock
> while its caller holds WALInsertLock, and therefore this routine
> could create the second leg of the deadlock too.  Removing the
> SInvalLock lock there creates the problem that backends might be
> added to or deleted from the PROC array while GetUndoRecPtr runs.
> I think it might be possible to survive that, by adding an assumption
> that logRec.xrecoff can be set to zero atomically, but it 
> seems tricky.

Checkpoint' undo is not used currently so just comment out GetUndoRecPtr
call in CreateCheckPoint - we'll find solution later.

Vadim


pgsql-hackers by date:

Previous
From: "Brian Hirt"
Date:
Subject: Re: problems with table corruption continued
Next
From: Tom Lane
Date:
Subject: Re: problems with table corruption continued