Skip to content

[3.15] gh-75876: Correct the memory use declared by some bigmem tests (GH-155307) - #155312

Open
miss-islington wants to merge 1 commit into
python:3.15from
miss-islington:backport-e469fa9-3.15
Open

[3.15] gh-75876: Correct the memory use declared by some bigmem tests (GH-155307)#155312
miss-islington wants to merge 1 commit into
python:3.15from
miss-islington:backport-e469fa9-3.15

Conversation

@miss-islington

@miss-islington miss-islington commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Measured with a limited address space and the peak RSS of the process:

  • TupleTest.test_repeat_large_2 keeps 3 tuples of pointers alive, as its
    siblings do, but declared the count for a 32-bit build.
  • The md5 tests hash the data and a io.BytesIO() copy of it.
  • testDecompress4G keeps the data, the compressed and the decompressed
    bytes alive, and the decompressor grows its output buffer.
  • test_huge_string_builtins uses 3 times the string size, not 2.
  • test_large_subn uses a bit more than the 18 times it declared.
    (cherry picked from commit e469fa9)

Co-authored-by: Serhiy Storchaka storchaka@gmail.com
Co-authored-by: Claude Opus 5 (1M context) noreply@anthropic.com

…ythonGH-155307)

Measured with a limited address space and the peak RSS of the process:

* TupleTest.test_repeat_large_2 keeps 3 tuples of pointers alive, as its
  siblings do, but declared the count for a 32-bit build.
* The md5 tests hash the data and a io.BytesIO() copy of it.
* testDecompress4G keeps the data, the compressed and the decompressed
  bytes alive, and the decompressor grows its output buffer.
* test_huge_string_builtins uses 3 times the string size, not 2.
* test_large_subn uses a bit more than the 18 times it declared.
(cherry picked from commit e469fa9c8077345c75da6e39d40534e7a6fc5057)

Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting review skip news tests Tests in the Lib/test dir

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants