31.10.2024 00:46, Hank Knox wrote:
Two followup observations:
I built commit 4bba839 as described below and ran it and it froze after a few
moments, as expected.
I did a git diff against the previous commit to see what changed. As only a few files had a few lines of code removed, I checked out v9.1.1, re-added
the removed lines, rebuilt and reran the VM, and it works fine.
The 'bad' commit removed dpy_cursor_define_supported(). I don't understand what that function is intended to do but on my machine, removing it
appears to cause the freeze.
Hank, you did an excellent job here, much more than I'd expect
given the previous conversation.
One of the very first thoughts about this is that the test
for the bisection is a flaky one, since you have to wait for
"some" time for the freeze to happen, and it isn't deterministic.
So the first suspect was the faulty bisection. But you has proven
your point now, much better than I'd thought.
I pinged the patch in question on the mailinglist. It received
at least single reply - also suggesting a possible flaky bisection
as a first possibility. Nope, this is definitely not it.
I'll do some more pinging tomorrow.
Thank you for a really excellent job!
/mjt
