From: | Gurjeet Singh <singh(dot)gurjeet(at)gmail(dot)com> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | PGSQL Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Launching debugger on self on SIGSEGV |
Date: | 2011-07-11 17:26:34 |
Message-ID: | CABwTF4XeFc0wVtny0Zob0NcdS5RRESohdwk2u=JsJLM092i3Aw@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Mon, Jul 11, 2011 at 12:56 PM, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> Gurjeet Singh <singh(dot)gurjeet(at)gmail(dot)com> writes:
> > The attached patch registers a signal handler for SIGSEGV and
> launches
> > GDB in batch mode on its own pid so that the stack leading to the SEGV
> can
> > be dumped in the server logs.
>
> Did you not read the thread last week about how we did not want any such
> thing?
>
Unfortunately, I did not. I'll catch up on it.
> Quite aside from any postgres-specific reasons not to have any added
> delay in the signal-to-database-shutdown path, this patch makes a bunch
> of untenable assumptions about whether or where gdb is installed,
> whether there are usable debug symbols available, whether gdb's output
> will go somewhere useful, etc etc. And on top of all that, it adds *no
> functionality whatsoever* compared to a post-mortem gdb run on the core
> file.
>
I agree that it makes a bunch of assumptions, that's why I proposed that we
make it user configurable parameter, like archive_command, so that users (or
their packagers) can provide the command and all the relevant options.
Regards,
--
Gurjeet Singh
EnterpriseDB Corporation
The Enterprise PostgreSQL Company
From | Date | Subject | |
---|---|---|---|
Next Message | Patrick Earl | 2011-07-11 17:28:37 | Re: Full GUID support |
Previous Message | Joshua D. Drake | 2011-07-11 17:19:53 | Re: Full GUID support |