From: Tom Tromey <tromey@redhat.com>
To: Stefan Monnier <monnier@iro.umontreal.ca>
Cc: emacs-devel@gnu.org, David De La Harpe Golden <david@harpegolden.net>
Subject: Re: Patch for fields of `struct buffer'
Date: Sun, 06 Feb 2011 19:44:15 -0700 [thread overview]
Message-ID: <m3pqr436o0.fsf@fleche.redhat.com> (raw)
In-Reply-To: <jwv39o76a81.fsf-monnier+emacs@gnu.org> (Stefan Monnier's message of "Tue, 01 Feb 2011 10:43:09 -0500")
>>>>> "Stefan" == Stefan Monnier <monnier@iro.umontreal.ca> writes:
Stefan> Erlang-style concurrency is nice and clean, but I'm not sure it'll
Stefan> integrate well with existing code which stomps over a global state all
Stefan> the time. This doesn't mean it won't work. If we can make it work
Stefan> something like "one agent per buffer" and turn `set-buffer' into a kind
Stefan> of message maybe we could get some good results, but it seems tricky.
I couldn't think of a way to make this work, at least not with
`set-buffer' as the primitive.
I think it could be done by having all other buffer-manipulating
primitives (e.g., `insert') work by message-passing to some other
thread. This is basically like a buffer lock.
Stefan> Threads and locks don't look too good: too low-level. If we go
Stefan> in that direction, then STM seems a lot more appealing.
I read up on STM and then spent some time thinking about it this
weekend. I don't see how it could be implemented in Emacs without
extreme hairiness in the C code. My thinking is along the lines of: a
transaction could conceivably touch any lisp object, including buffer
contents (I was thinking along the lines of: each call-interactively is
a transaction automatically). So, any change anywhere would have to be
logged and then potentially rolled back if a transaction is aborted.
In particular, all heap modifications in the C code would need to be
intercepted... orders of magnitude uglier than my proposed `struct
buffer' patch. (Though FWIW I have thought about this particular change
for other reasons -- you could introduce a software write barrier for
the GC this way :-)
Also I think heap reads would need to be intercepted and indirected to
make transactions not prematurely affect other threads. So, this is
quite hard and inefficient.
I am interested in reasoned arguments, grounded in real Emacs code, to
the contrary for either STM or CSP. Otherwise, sucky as it is, I think
the approach will be explicit locking. I know I can make that one work
ok, or "ok enough".
After thinking about the Bordeaux threads model a bit more, I have come
to the conclusion that I would prefer something simpler. E.g., if we
went "Java style" and only provided `with-lock-held' (and not separate
lock- and unlock- primitives), then we could eliminate a class of bugs.
Likewise, I think (and I know we already disagree here) that only having
recursive locks similarly eliminates a source of bugs. I think these
issues are important because such bugs will show up to Emacs users in a
particular un-fun way.
Tom
next prev parent reply other threads:[~2011-02-07 2:44 UTC|newest]
Thread overview: 123+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-01-27 20:18 Patch for fields of `struct buffer' Tom Tromey
2011-01-28 7:37 ` Eli Zaretskii
2011-01-28 8:58 ` Tom Tromey
2011-01-28 9:29 ` Eli Zaretskii
2011-01-29 12:50 ` Eli Zaretskii
2011-01-29 13:33 ` Juanma Barranquero
2011-01-29 13:42 ` Eli Zaretskii
2011-02-02 18:17 ` Juanma Barranquero
2011-02-02 19:51 ` Eli Zaretskii
2011-01-28 14:55 ` Stefan Monnier
2011-01-28 17:38 ` Tom Tromey
2011-01-28 18:27 ` Stefan Monnier
2011-01-28 18:42 ` Tom Tromey
2011-01-28 20:04 ` Stefan Monnier
2011-01-28 17:23 ` Kim F. Storm
2011-01-28 17:36 ` Tom Tromey
2011-01-28 18:29 ` Stefan Monnier
2011-01-29 2:34 ` Miles Bader
2011-01-29 3:58 ` Stefan Monnier
2011-01-29 16:18 ` Richard Stallman
2011-01-30 20:23 ` Tom Tromey
2011-01-31 4:04 ` Stefan Monnier
2011-01-31 14:29 ` Tom Tromey
2011-01-31 15:22 ` Stefan Monnier
2011-01-31 15:30 ` Tom Tromey
2011-01-31 17:02 ` Stefan Monnier
2011-01-31 18:38 ` Tom Tromey
2011-01-31 20:03 ` Stefan Monnier
2011-01-31 20:00 ` Stefan Monnier
2011-01-31 20:05 ` Tom Tromey
2011-01-31 20:40 ` Stefan Monnier
2011-01-31 16:46 ` David De La Harpe Golden
2011-01-31 19:04 ` Tom Tromey
2011-01-31 22:08 ` David De La Harpe Golden
2011-02-01 10:54 ` Daniel Colascione
2011-02-07 2:34 ` Tom Tromey
2011-02-07 19:48 ` concurrency suggestions for Gnus (was: Patch for fields of `struct buffer') Ted Zlatanov
2011-02-08 4:31 ` concurrency suggestions for Gnus Miles Bader
2011-02-08 12:55 ` Andy Moreton
2011-02-08 12:55 ` Justin Lilly
2011-02-08 14:41 ` bloom filters (was: concurrency suggestions for Gnus) Ted Zlatanov
2011-02-08 15:15 ` Stephen J. Turnbull
2011-02-10 8:17 ` concurrency suggestions for Gnus Lars Ingebrigtsen
2011-02-01 15:43 ` Patch for fields of `struct buffer' Stefan Monnier
2011-02-01 16:28 ` Helmut Eller
2011-02-07 2:47 ` Tom Tromey
2011-02-07 2:44 ` Tom Tromey [this message]
2011-02-07 8:05 ` Helmut Eller
2011-02-07 19:23 ` Richard Stallman
2011-02-08 16:30 ` Tom Tromey
2011-02-08 16:26 ` Tom Tromey
2011-02-08 17:57 ` Helmut Eller
2011-02-11 21:59 ` Tom Tromey
2011-02-12 9:16 ` Helmut Eller
2011-02-08 21:10 ` Stefan Monnier
2011-01-31 19:38 ` Richard Stallman
2011-02-01 18:26 ` Tom Tromey
2011-02-01 19:28 ` Paul Eggert
2011-02-01 19:42 ` Tom Tromey
2011-02-09 10:16 ` Jim Meyering
2011-02-10 4:42 ` Miles Bader
2011-02-01 20:34 ` Andreas Schwab
2011-02-01 21:56 ` Tom Tromey
2011-02-01 21:59 ` Tom Tromey
2011-02-02 0:44 ` Paul Eggert
2011-02-02 3:49 ` Stefan Monnier
2011-02-02 16:26 ` Tom Tromey
2011-02-02 16:37 ` Stefan Monnier
2011-02-08 15:07 ` Tom Tromey
2011-02-08 21:02 ` Stefan Monnier
2011-02-08 21:08 ` Tom Tromey
2011-02-08 21:21 ` Tom Tromey
2011-02-09 21:32 ` Stefan Monnier
2011-02-08 21:58 ` Andreas Schwab
2011-02-01 22:21 ` Stefan Monnier
2011-02-08 16:38 ` Tom Tromey
2011-02-11 21:48 ` Tom Tromey
2011-02-12 2:25 ` Stefan Monnier
2011-01-31 19:37 ` Richard Stallman
2011-01-31 19:57 ` Tom Tromey
2011-02-01 2:42 ` Tom Tromey
2011-02-01 4:09 ` Eli Zaretskii
2011-02-01 16:40 ` Richard Stallman
2011-02-01 16:44 ` Tom Tromey
2011-02-02 2:42 ` Richard Stallman
2011-01-31 20:30 ` Stefan Monnier
2011-01-31 20:49 ` Tom Tromey
2011-01-31 21:54 ` Stefan Monnier
2011-02-13 19:01 ` Richard Stallman
2011-02-14 17:47 ` Stefan Monnier
2011-02-14 19:34 ` Lennart Borgman
2011-02-15 15:58 ` Richard Stallman
2011-02-15 20:41 ` Stefan Monnier
2011-02-01 16:39 ` Richard Stallman
2011-02-01 12:09 ` Stephen J. Turnbull
2011-02-02 2:41 ` Richard Stallman
2011-02-01 10:26 ` Daniel Colascione
2011-02-01 12:10 ` Stephen J. Turnbull
2011-02-01 14:23 ` Lennart Borgman
2011-02-01 16:19 ` Stephen J. Turnbull
2011-02-01 17:08 ` Lennart Borgman
2011-02-01 19:57 ` Daniel Colascione
2011-02-01 20:06 ` Daniel Colascione
2011-02-01 16:41 ` Richard Stallman
2011-02-01 16:51 ` Tom Tromey
2011-02-02 2:42 ` Richard Stallman
2011-02-02 4:16 ` Tom Tromey
2011-02-02 5:04 ` Stefan Monnier
2011-02-11 21:51 ` Tom Tromey
2011-02-12 23:28 ` Richard Stallman
2011-02-13 20:26 ` Installing `struct buffer' patch (Was: Patch for fields of `struct buffer') Tom Tromey
2011-02-14 16:42 ` Installing `struct buffer' patch Chong Yidong
2011-02-14 16:48 ` Tom Tromey
2011-02-14 16:52 ` Chong Yidong
2011-02-14 16:58 ` Tom Tromey
2011-02-14 17:36 ` Stefan Monnier
2011-02-15 15:50 ` Chong Yidong
2011-02-16 14:55 ` Tom Tromey
2011-02-15 15:57 ` Richard Stallman
2011-02-15 4:07 ` Glenn Morris
2011-02-15 14:28 ` Tom Tromey
2011-02-01 20:04 ` Patch for fields of `struct buffer' Daniel Colascione
2011-02-02 2:43 ` Richard Stallman
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
List information: https://www.gnu.org/software/emacs/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=m3pqr436o0.fsf@fleche.redhat.com \
--to=tromey@redhat.com \
--cc=david@harpegolden.net \
--cc=emacs-devel@gnu.org \
--cc=monnier@iro.umontreal.ca \
/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 public inbox
https://git.savannah.gnu.org/cgit/emacs.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).