From: Eli Zaretskii <eliz@gnu.org>
To: "Mattias Engdegård" <mattias.engdegard@gmail.com>
Cc: emacs-devel@gnu.org, acorallo@gnu.org
Subject: Re: last_marked array is now ifdef'ed away
Date: Tue, 17 Sep 2024 16:22:48 +0300 [thread overview]
Message-ID: <86cyl2bejb.fsf@gnu.org> (raw)
In-Reply-To: <9354266E-9F9E-430E-A862-2169A62C9F67@gmail.com> (message from Mattias Engdegård on Tue, 17 Sep 2024 12:15:16 +0200)
> From: Mattias Engdegård <mattias.engdegard@gmail.com>
> Date: Tue, 17 Sep 2024 12:15:16 +0200
> Cc: emacs-devel@gnu.org,
> Andrea Corallo <acorallo@gnu.org>
>
> 15 sep. 2024 kl. 16.21 skrev Eli Zaretskii <eliz@gnu.org>:
>
> > There's no need for any evidence to keep the code which we always had.
>
> Well, we didn't have it in Emacs 29, so one might be excused for thinking it wasn't a strict necessity.
Because it was silently disabled. This change should have been
discussed in advance.
> > That won't help because GC crashes are seldom if ever reproducible.
> > So if the trace is off, the information is gone and cannot be
> > recovered in practice.
>
> My experience is rather that in such cases the crash was already gone because there was no core dump or debugger attached anyway, but we then take the necessary steps to catch the bug next time – turn on core dumps (if possible), run with a debugger, enable checking, etc – and we always end up trapping the gremlin eventually.
At least some (many?) users have core dumps available, even though
current systems foolishly disable them by default. I see this all the
time on debbugs: users report bugs and tell they have core files or
the crashed Emacs in a debugger.
> Anyway, I'm going to re-enable the mark trace buffer for the sake of development peace; since it's important to you, that's also worth something. I shall add a configuration option for disabling it, with its trade-off clearly documented, so that users can make an informed decision, but the buffer will be enabled by default.
Thank you. Having a knob to disable it is fine, as long as the
default is to have it enabled.
next prev parent reply other threads:[~2024-09-17 13:22 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-09-07 6:05 last_marked array is now ifdef'ed away Eli Zaretskii
2024-09-12 9:19 ` Eli Zaretskii
2024-09-13 14:43 ` Mattias Engdegård
2024-09-13 15:21 ` Eli Zaretskii
2024-09-14 17:05 ` Mattias Engdegård
2024-09-14 17:30 ` Eli Zaretskii
2024-09-14 20:30 ` Mattias Engdegård
2024-09-15 5:51 ` Eli Zaretskii
2024-09-15 13:58 ` Mattias Engdegård
2024-09-15 14:21 ` Eli Zaretskii
2024-09-17 10:15 ` Mattias Engdegård
2024-09-17 13:22 ` Eli Zaretskii [this message]
2024-09-14 22:50 ` Stefan Kangas
2024-09-15 6:01 ` Eli Zaretskii
2024-09-16 18:07 ` Andrea Corallo
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=86cyl2bejb.fsf@gnu.org \
--to=eliz@gnu.org \
--cc=acorallo@gnu.org \
--cc=emacs-devel@gnu.org \
--cc=mattias.engdegard@gmail.com \
/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).