From: "Gerd Möllmann" <gerd.moellmann@gmail.com>
To: Alan Third <alan@idiocy.org>
Cc: 65843@debbugs.gnu.org, Christian Tanzer <tanzer@gg32.com>
Subject: bug#65843: 28.2; Too many iconified frames in .emacs.desktop -> crash (macOS)
Date: Mon, 11 Sep 2023 06:07:56 +0200 [thread overview]
Message-ID: <m2cyypl5cz.fsf@Pro.fritz.box> (raw)
In-Reply-To: <ZP4fCoUpDFgv+aYO@idiocy.org> (Alan Third's message of "Sun, 10 Sep 2023 20:54:50 +0100")
Alan Third <alan@idiocy.org> writes:
> Is this the same error as above, with the corrupt autorelease pool?
Probably not, one being an ABRT, and one a SEGV. Or maybe they are,
if the underlying cause is the same.
It's all pretty strange. Maybe it's a timing thing? I mean, ignoring
the possibility of an optimization bug for a moment, the fact that
optimization flags that make Emacs slower make the bug disappear. Or
the fact that it takes a "random" number of runs to make it happen. And
that it doesn't happen at all on my old Macbook, which is 2x or so
slower than my M1 Mac mini. I wouldn't be surprised if Christian had an
even faster CPU, so that he sees the bug more reliably.
Out of curiosity, Christian, what CPU do you have?
>> frame #27: 0x00000001a9b12b18 AppKit`-[NSApplication(NSEvent)
>> _nextEventMatchingEventMask:untilDate:inMode:dequeue:] + 716
>> frame #28: 0x000000010052e0d4 emacs`ns_send_appdefined(value=-1) at nsterm.m:4569:24 [opt]
>
> This is an interesting little bit of code with a comment saying it's
> been added to cover for what sounds like a bug in macOS 10.10...
> Perhaps it's worth removing it and seeing if that helps.
>
> The code's around nsterm.m line 4569, and the whole #ifdef'd section
> should be removed.
>
Thanks, I'll try that.
diff --git a/src/nsterm.m b/src/nsterm.m
index c809c0b824a..2cb508b5328 100644
--- a/src/nsterm.m
+++ b/src/nsterm.m
@@ -4560,6 +4560,7 @@ Function modeled after x_draw_glyph_string_box ().
the [NXApp run] main loop after having processed all events queued at
this moment. */
+#if 0
#ifdef NS_IMPL_COCOA
if (! send_appdefined)
{
@@ -4572,6 +4573,7 @@ in certain situations (rapid incoming events).
dequeue:NO];
if (! appev) send_appdefined = YES;
}
+#endif
#endif
if (send_appdefined)
> I still suspect that if we have a corrupt autorelease pool, the
> problem will be somewhere entirely different...
Yeah. Strange thing is that ASAN doesn't kick in. Hm. Maybe Apple has
something for debugging such situations? I guess it's worth googling.
next prev parent reply other threads:[~2023-09-11 4:07 UTC|newest]
Thread overview: 27+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-09-09 16:35 bug#65843: 28.2; Too many iconified frames in .emacs.desktop -> crash (macOS) Christian Tanzer via Bug reports for GNU Emacs, the Swiss army knife of text editors
2023-09-09 16:49 ` Eli Zaretskii
2023-09-09 17:36 ` Christian Tanzer via Bug reports for GNU Emacs, the Swiss army knife of text editors
2023-09-09 17:39 ` Eli Zaretskii
2023-09-10 14:11 ` Christian Tanzer via Bug reports for GNU Emacs, the Swiss army knife of text editors
2023-09-10 14:27 ` Eli Zaretskii
2023-09-10 15:07 ` Gerd Möllmann
2023-09-10 15:37 ` Christian Tanzer via Bug reports for GNU Emacs, the Swiss army knife of text editors
2023-09-10 16:04 ` Gerd Möllmann
2023-09-09 19:53 ` Alan Third
2023-09-10 4:20 ` Gerd Möllmann
2023-09-10 14:59 ` Christian Tanzer via Bug reports for GNU Emacs, the Swiss army knife of text editors
2023-09-10 15:37 ` Gerd Möllmann
2023-09-10 15:40 ` Christian Tanzer via Bug reports for GNU Emacs, the Swiss army knife of text editors
2023-09-10 17:09 ` Gerd Möllmann
2023-09-10 17:38 ` Gerd Möllmann
2023-09-10 17:09 ` Alan Third
2023-09-10 18:40 ` Gerd Möllmann
2023-09-10 19:54 ` Alan Third
2023-09-11 4:07 ` Gerd Möllmann [this message]
2023-09-11 4:50 ` Gerd Möllmann
2023-09-11 7:55 ` Christian Tanzer via Bug reports for GNU Emacs, the Swiss army knife of text editors
2023-09-11 8:06 ` Gerd Möllmann
2023-09-11 16:25 ` Alan Third
2023-09-11 17:16 ` Christian Tanzer via Bug reports for GNU Emacs, the Swiss army knife of text editors
2023-09-11 18:26 ` Alan Third
2023-09-12 17:43 ` Christian Tanzer via Bug reports for GNU Emacs, the Swiss army knife of text editors
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=m2cyypl5cz.fsf@Pro.fritz.box \
--to=gerd.moellmann@gmail.com \
--cc=65843@debbugs.gnu.org \
--cc=alan@idiocy.org \
--cc=tanzer@gg32.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).