all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Eli Zaretskii <eliz@gnu.org>
To: Ken Raeburn <raeburn@raeburn.org>
Cc: lokedhs@gmail.com, emacs-devel@gnu.org
Subject: Re: "concurrency" branch updated
Date: Tue, 03 Nov 2015 18:26:47 +0200	[thread overview]
Message-ID: <8337wnvyk8.fsf@gnu.org> (raw)
In-Reply-To: <3C732C2C-27AA-44D3-8E00-7742C004960C@raeburn.org>

> From: Ken Raeburn <raeburn@raeburn.org>
> Date: Tue, 3 Nov 2015 04:40:11 -0500
> Cc: emacs-devel <emacs-devel@gnu.org>
> 
> I do suspect that some things dealing with user input (reading, testing
> availability, even the flag indicating we’re waiting) might get confused in a
> multithreaded setup. The test I was using recently that reliably crashed Emacs
> was:
> 
> (make-thread (lambda () (signal ‘foo nil)))
> 
> …though it has to be in an interactive Emacs, where the waiting_for_input flag
> can be set, which the “signal” routine really doesn’t like. Possibly
> waiting_for_input needs to be thread-local… or if it stays global, maybe
> Fsignal() just shouldn’t be looking at it any more.
> 
> A previous test which broke the pre-merged concurrency branch but which I
> haven’t tested recently was to create a thread and have it prompt for input… I
> think it was either yes-or-no-p or y-or-n-p; it seemed to put Emacs into a
> state where there were no key bindings, so it would complain about every
> character I typed.
> 
> So…yeah. Input seems to be a fragile area on the branch.

IMO, the concurrency branch needs someone motivated enough and with
enough resources to work on it, before it will be ready for prime
time.  Ideally, that should be Tom, but we can only hope he will find
time for that.  If not, then someone else.  I don't see how this could
be moved forward without some non-trivial effort.  The design is
simple and robust, and the implementation fairly simple and easy to
understand.  What's needed, IMO, is a lot of testing and debugging,
and quite a lot of the latter on the C level.




  reply	other threads:[~2015-11-03 16:26 UTC|newest]

Thread overview: 40+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-11-01  6:19 "concurrency" branch updated Ken Raeburn
2015-11-02 17:09 ` Eli Zaretskii
2015-11-02 20:23 ` John Wiegley
2015-11-02 20:35   ` Eli Zaretskii
2015-11-02 20:41     ` Eli Zaretskii
2015-11-02 21:57       ` John Wiegley
2015-11-03  3:58         ` Elias Mårtenson
2015-11-03  9:40           ` Ken Raeburn
2015-11-03 16:26             ` Eli Zaretskii [this message]
2015-11-03 16:23           ` Eli Zaretskii
2015-11-03  9:40   ` Ken Raeburn
2015-11-03 11:50     ` other "concurrency" approaches Nic Ferrier
2015-11-03 15:44       ` Eli Zaretskii
2015-11-03 17:16         ` Nic Ferrier
2015-11-03 17:23           ` Eli Zaretskii
2015-11-03 22:28             ` Nic Ferrier
2015-11-04  3:48               ` Eli Zaretskii
2015-11-03 15:14     ` "concurrency" branch updated Filipp Gunbin
2015-11-03 15:35       ` Michael Albinus
2015-11-03 16:38         ` Thierry Volpiatto
2015-11-03 16:29     ` Eli Zaretskii
2015-11-04  9:20       ` Ken Raeburn
2015-11-04 15:40         ` Eli Zaretskii
2015-11-04 19:48           ` Ken Raeburn
2015-11-04 20:51             ` Eli Zaretskii
2015-11-05  5:16               ` Ken Raeburn
2015-11-04 23:09         ` Richard Stallman
2015-11-05  3:41           ` Eli Zaretskii
2015-11-05  6:29           ` Ken Raeburn
2015-11-05 13:17             ` John Wiegley
2015-11-05 14:17               ` David Kastrup
2015-11-05 15:07                 ` John Wiegley
2015-11-05 21:55               ` Tom Tromey
2015-11-05 22:01                 ` John Wiegley
2015-11-05 22:46             ` Richard Stallman
2015-11-06  8:37               ` Eli Zaretskii
2015-11-05 21:49           ` Tom Tromey
2015-11-05 21:46         ` Tom Tromey
2015-11-06  7:58           ` Eli Zaretskii
2015-11-06 14:58             ` Tom Tromey

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=8337wnvyk8.fsf@gnu.org \
    --to=eliz@gnu.org \
    --cc=emacs-devel@gnu.org \
    --cc=lokedhs@gmail.com \
    --cc=raeburn@raeburn.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.