From: | Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com> |
---|---|
To: | Donald Dong <xdong(at)csumb(dot)edu> |
Cc: | Andres Freund <andres(at)anarazel(dot)de>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Unified logging system for command-line programs |
Date: | 2019-01-11 17:14:29 |
Message-ID: | b8238aca-6987-5ee8-dab3-2b3a7bfb1e6b@2ndquadrant.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 10/01/2019 05:57, Donald Dong wrote:
> I think the logging system can also be applied on pg_regress. Perhaps even
> for the external frontend applications?
Could be done, yes. A bit at a time. ;-)
> The patch cannot be applied directly on HEAD. So I patched it on top of
> 60d99797bf.
Here is an updated patch with the merge conflicts of my own design
resolved. No functionality changes.
> When I call pg_log_error() in initdb, I see
>
> Program received signal SIGSEGV, Segmentation fault.
> __strlen_avx2 () at ../sysdeps/x86_64/multiarch/strlen-avx2.S:62
> 62 ../sysdeps/x86_64/multiarch/strlen-avx2.S: No such file or directory.
> (gdb) bt
> #0 __strlen_avx2 () at ../sysdeps/x86_64/multiarch/strlen-avx2.S:62
> #1 0x0000555555568f96 in dopr.constprop ()
> #2 0x0000555555569ddb in pg_vsnprintf ()
> #3 0x0000555555564236 in pg_log_generic ()
> #4 0x000055555555c240 in main ()
What do you mean exactly by "I call pg_log_error()"? The existing calls
in initdb clearly work, at least some of them, that is covered by the
test suite. Are you adding new calls?
> I'm not sure what would be causing this behavior. I would appreciate
> references or docs for testing and debugging patches more efficiently.
> Now I'm having difficulties loading symbols of initdb in gdb.
The above looks like you'd probably get a better insight by compiling
with -O0 or some other lower optimization setting.
There is also this:
https://wiki.postgresql.org/wiki/Getting_a_stack_trace_of_a_running_PostgreSQL_backend_on_Linux/BSD
--
Peter Eisentraut http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
Attachment | Content-Type | Size |
---|---|---|
v2-0001-Unified-logging-system-for-command-line-programs.patch | text/plain | 482.0 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Andrew Gierth | 2019-01-11 17:33:19 | Re: Ryu floating point output patch |
Previous Message | Robert Haas | 2019-01-11 17:13:50 | Re: Ryu floating point output patch |