-
Notifications
You must be signed in to change notification settings - Fork 85
Run MMQA tests as part of CI #116
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
…compiler warnings about overruns.
|
This is currently failing to pass test function/136.c https://github.com/Ravenbrook/mps/actions/runs/3939501021/jobs/6739430364#step:6:865 . Running this locally, I find an assertion failure here Lines 81 to 86 in ed73ba8
The test reduces the arena commit limit, but seems to assume that it will be able to allocate an 8-byte object in pool2. This call fails with result code 7, which means it's hitting the commit limit. The test appears to expect to fail to allocate, but only for the "large objects" that it tries next. This seems wrong to me, especially if the commit limit is reached exactly by allocating the descriptors for the pool creations. |
|
Possibly related to #88 but trying function/136.c from that branch does not seem to fix the problem here. EDIT: Disregard this. I needed to clean. |
This makes the test portable to XCA6LL, where the page size (and so the minimum arena grain size) can be 16 kB. (cherry picked from commit ff8c9c9)
There seems to be a sensitive boundary condition. Running but running with this in log/full.log: So it seems that |
Attempting to resolve #106