From: | Alvaro Herrera <alvherre(at)commandprompt(dot)com> |
---|---|
To: | Jeff Davis <pgsql(at)j-davis(dot)com> |
Cc: | pgsql-bugs(at)postgresql(dot)org |
Subject: | Re: Out of memory error causes Abort, Abort tries to allocate memory |
Date: | 2006-10-25 19:20:49 |
Message-ID: | 20061025192049.GA9340@alvh.no-ip.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-bugs pgsql-hackers |
Jeff Davis wrote:
> I found the root cause of the bug I reported at:
>
> http://archives.postgresql.org/pgsql-bugs/2006-10/msg00211.php
>
> What happens is this:
> * Out of memory condition causes an ERROR
> * ERROR triggers an AbortTransaction()
> * AbortTransaction() calls RecordTransactionAbort()
> * RecordTransactionAbort calls smgrGetPendingDeletes()
> * smgrGetPendingDeletes() calls palloc()
> * palloc() fails, resulting in ERROR, causing infinite recursion
> * elog.c detects infinite recursion, and elevates it to PANIC
>
> I'm not sure how easy this is to fix, but I asked on IRC and got some
> agreement that this is a bug.
Hmm, maybe we could have AbortTransaction switch to ErrorContext, which
has some preallocated space, before calling RecordTransactionAbort (or
maybe have RecordTransactionAbort itself do it).
Problem is, what happens if ErrorContext is filled up by doing this? At
that point we will be severely fscked up, and you probably won't get the
PANIC either. (Maybe it doesn't happen in this particular case, but
seems a real risk.)
--
Alvaro Herrera http://www.CommandPrompt.com/
The PostgreSQL Company - Command Prompt, Inc.
From | Date | Subject | |
---|---|---|---|
Next Message | Jeff Davis | 2006-10-25 21:03:42 | Re: Out of memory error causes Abort, Abort tries to |
Previous Message | Jeff Davis | 2006-10-25 18:54:38 | Out of memory error causes Abort, Abort tries to allocate memory |
From | Date | Subject | |
---|---|---|---|
Next Message | Bruce Momjian | 2006-10-25 19:31:09 | Re: [HACKERS] Replication documentation addition |
Previous Message | Josh Berkus | 2006-10-25 18:59:28 | Re: [HACKERS] Replication documentation addition |