GNU bug report logs - #74805
30.0.92; Trying to build scratch/igc on Cygwin

Previous Next

Package: emacs;

Reported by: Ken Brown <kbrown <at> cornell.edu>

Date: Wed, 11 Dec 2024 22:56:02 UTC

Severity: normal

Found in version 30.0.92

Done: Ken Brown <kbrown <at> cornell.edu>

Bug is archived. No further changes may be made.

Full log


Message #95 received at 74805 <at> debbugs.gnu.org (full text, mbox):

From: Pip Cet <pipcet <at> protonmail.com>
To: Ken Brown <kbrown <at> cornell.edu>
Cc: Gerd Möllmann <gerd.moellmann <at> gmail.com>,
 74805 <at> debbugs.gnu.org, Eli Zaretskii <eliz <at> gnu.org>,
 Richard Brooksby <rb <at> ravenbrook.com>
Subject: Re: bug#74805: 30.0.92; Trying to build scratch/igc on Cygwin
Date: Mon, 27 Jan 2025 22:31:25 +0000
"Ken Brown" <kbrown <at> cornell.edu> writes:

> On 1/27/2025 1:00 PM, Pip Cet wrote:
>> Pip Cet <pipcet <at> protonmail.com> writes:
>>
>>> The most unusual thing seems to be that pageSize / ArenaGrainSize is so
>>> large (64 KB).  Maybe that simply means that TEST_ARENA_SIZE needs to be
>>> increased?
>>
>> Changing ARENA_CLIENT_GRAIN_SIZE to 65536 makes the test fail here, too,
>> also when the counter reaches 2730 tracts per page in the page table.
>>
>> Changing
>>
>> #define TEST_ARENA_SIZE              ((Size)16<<22)
>>
>> to
>>
>> #define TEST_ARENA_SIZE              ((Size)16<<25)
>>
>> fixes it again.  My best guess is that
>>
>> #define TEST_ARENA_SIZE (ARENA_CLIENT_GRAIN_SIZE << 13)
>>
>> would make the arena large enough not to fail the test on all systems.
>>
>> If you can confirm, we should open an issue for this in the MPS
>> repository (still at GitHub, I assume).  If tehre isn't one already :-)
> Thank you very much!!

> I had to use 17 instead of 13 to get all the tests to pass
> consistently, but then they did in several consecutive runs.  (Note
> that there is a random element to the testsuite because of the use of
> randomize in testlib.c.)

IIUC, that makes TEST_ARENA_SIZE 8 GiB.  That's a lot, and such a large
constant would probably not be acceptable in the test suite.  Maybe
there is a bug somewhere where we don't reuse VA addresses,...

> I appreciate your help.  I guess the authors of the tests never imagined
> a platform with a 64K page size.

Is that actually the physical page size on your system?

> I hope it's simply the test suite
> that's affected and not the rest of the MPS code.

But discovering such bugs in MPS is important, too! We won't find
unusual bugs if we all run the some configuration.

> Ken
>
> Ken

Thanks to both of you :-)

Pip





This bug report was last modified 155 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.