Date: Fri, 9 Jun 2006 09:47:43 -0400 (EDT) From: Daniel Eischen <deischen@freebsd.org> To: Maxim Konovalov <maxim@macomnet.ru> Cc: threads@freebsd.org, Mikhail Teterin <mi+mx@aldan.algebra.com> Subject: Re: SIGINFO and pthreads Message-ID: <Pine.GSO.4.64.0606090944530.10705@sea.ntplx.net> In-Reply-To: <20060609164120.X11643@mp2.macomnet.net> References: <200606071906.25776.mi%2Bmx@aldan.algebra.com> <200606071916.38538.mi%2Bmx@aldan.algebra.com> <20060608081626.X6097@mp2.macomnet.net> <200606081218.17131.mi%2Bmx@aldan.algebra.com> <20060608213618.O17062@mp2.macomnet.net> <Pine.GSO.4.64.0606081851330.6968@sea.ntplx.net> <20060609164120.X11643@mp2.macomnet.net>
next in thread | previous in thread | raw e-mail | index | archive | help
This message is in MIME format. The first part should be readable text, while the remaining parts are likely unreadable without MIME-aware tools. ---559023410-758783491-1149860863=:10705 Content-Type: TEXT/PLAIN; charset=KOI8-R; format=flowed Content-Transfer-Encoding: QUOTED-PRINTABLE On Fri, 9 Jun 2006, Maxim Konovalov wrote: > Daniel, > > On Thu, 8 Jun 2006, 18:52-0400, Daniel Eischen wrote: > >> On Thu, 8 Jun 2006, Maxim Konovalov wrote: >> >>> On Thu, 8 Jun 2006, 12:18-0400, Mikhail Teterin wrote: >>> >>>> =DE=C5=D4=D7=C5=D2 08 =DE=C5=D2=D7=C5=CE=D8 2006 00:17, Maxim Konovalo= v =CE=C1=D0=C9=D3=C1=D7: >>>>> Inline it if it's small. =9AMailman strips most attachments. >>>> >>>> Damn... Yes, it is small -- this is just a demo program. Compile it fi= rst >>>> as >>>> >>>> cc -o t t.c >>>> >>>> Then -- run and try various keyboard signals like Ctrl-C, Ctrl-Z, >>>> Ctrl-T (SIGINFO), or Ctrl-\. (It will not quit on its own, you'll >>>> have to kill it from another prompt.) >>>> >>>> They will all work. Then -- recompile as >>>> >>>> cc -o t -pthread t.c >>>> >>>> And run... This time around it will ignore the Ctrl-T, while >>>> continuing to respond to others. >>>> >>>>> libpthread uses SIGINFO for dumping thread information to a file but >>>>> looking over the code it seems it does allow to use SIGINFO to the >>>>> app. >>>> >>>> I'd like to be able to disable the debugging feature or, at least, >>>> force it to call my signal-handler AS EXPECTED AND DOCUMENTED, after >>>> it is done with its own dumping. >>> >>> A funny thing your test program does work if you run it as >>> >>> LIBPTHREAD_DEBUG=3Dyes ./t >>> >>> Try this patch: >> >> [ ... ] >> >>> For the first glance and quick test this check is not needed as we >>> dump threads in already installed for SIGINFO _thr_sig_handler(). Of >>> course we need our libpthreads experts review. >> >> Yes, that is a correct patch (remove the commented out section). >> Please go ahead and commit! > > There is still a problem if user decided to SIG_IGN or SIG_DFL > SIGINFO. In this case threads dump stops working. I don't know how > important this fuctionality is so here is a more aggressive patch: I could go either way on the issue. This second patch looks fine to me, so if you want to commit this one, that is fine with me. Thanks! --=20 DE ---559023410-758783491-1149860863=:10705--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?Pine.GSO.4.64.0606090944530.10705>