From: Reuben Thomas <rrt@sc3d.org>
To: Glenn Morris <rgm@gnu.org>
Cc: 28319@debbugs.gnu.org
Subject: bug#28319: emacsclient tests interfere with running Emacs, hang
Date: Tue, 5 Sep 2017 22:11:15 +0100 [thread overview]
Message-ID: <CAOnWdojxodtc2pQahsseO0FRysXQeKiQ-A22X=rgT5WKB8vWQQ@mail.gmail.com> (raw)
In-Reply-To: <m2a828vs1g.fsf@fencepost.gnu.org>
[-- Attachment #1: Type: text/plain, Size: 1711 bytes --]
On 5 September 2017 at 20:55, Glenn Morris <rgm@gnu.org> wrote:
>
> The tests still fail on hydra with "Profiling timer expired".
> I can reproduce this locally if I configure with --enable-profiling.
>
Thanks for the reproduction method, and for the tidy-up work you did on
the tests; I can reproduce the "timer expired" problem.
I see that --enable-profiling stops profiler.el from working, so it's
nothing to do with profiler.el (this suggests incidentally that there's
nothing I can do directly about the problem from Elisp, hence in the test
code itself). I also see that the error message "Profiling timer expired"
is caused by SIGPROF. Looking at the code in Emacs controlled by
-DPROFILING (set to 1 by --enable-profiling), I find there's very little,
but what there is seems to refer to the mon* routines, for which I find no
man pages on my system, but a little searching online suggests are
BSD-derived.
I see that Emacs, in src/sysdep.c, sets SIGPROF's handler to SIG_IGN. So
Emacs ignores the signal, but clearly from the call-process return value,
it's being delivered to the emacsclient child process.
The main thing I don't understand is why this behavior isn't a problem for
other tests that use call-process, except that they don't seem to check the
return value; but unlike the emacsclient test, at least some of them rely
on some action having been taken by the called process, so presumably it
must have completed successfully before receiving SIGPROF? If this is
indeed the case, then the emacsclient test could simply allow the error
message to signal success.
Can you enlighten me any further?
--
https://rrt.sc3d.org <http://rrt.sc3d.org>
[-- Attachment #2: Type: text/html, Size: 2542 bytes --]
next prev parent reply other threads:[~2017-09-05 21:11 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-09-01 16:40 bug#28319: emacsclient tests interfere with running Emacs, hang Glenn Morris
2017-09-01 17:23 ` Reuben Thomas
2017-09-01 20:37 ` Reuben Thomas
2017-09-01 22:19 ` Glenn Morris
2017-09-01 22:44 ` Reuben Thomas
2017-09-05 19:55 ` Glenn Morris
2017-09-05 21:11 ` Reuben Thomas [this message]
2017-09-05 22:24 ` Glenn Morris
2017-09-06 1:14 ` Glenn Morris
2017-09-06 16:02 ` Eli Zaretskii
2017-09-07 23:10 ` Reuben Thomas
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='CAOnWdojxodtc2pQahsseO0FRysXQeKiQ-A22X=rgT5WKB8vWQQ@mail.gmail.com' \
--to=rrt@sc3d.org \
--cc=28319@debbugs.gnu.org \
--cc=rgm@gnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/emacs.git
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.