Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

gh-141794: Reduce size of compiler stress tests to fix Android warnings#142263

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
Eclips4 merged 3 commits intopython:mainfrommhsmith:android-oom
Dec 7, 2025

Conversation

@mhsmith
Copy link
Member

@mhsmithmhsmith commentedDec 4, 2025•
edited
Loading

This fixes the Android low memory warnings intest_ast andtest_compile, previously discussed in#137186 (comment).

These warnings haven't led to a crash so far. However, the warning will only appear during the first test that brings the memory below a threshold. If a later test exhausts the memory completely, then the warning won't appear again, and the process will simply crash. This separation between the warning and the crash makes the crash much more difficult to understand.

I think this may have been what happened in#142228 (comment), which introduced some leaks. Notice inthis run, the warning appeared duringtest_ast, and the crash happened intest_random.

@mhsmith
Copy link
MemberAuthor

I've updated this to change the sizes only on Android, and the tests are now passing on all platforms, so this PR is ready to review.

Due to#142289, the tests are now running on API level 35, so the low memory warnings are much less detailed. Search for thescudo : Can't populate more pages warning in themost recent main branch test, and notice that the warning is not present in the log of this PR.

@mhsmithmhsmith marked this pull request as ready for reviewDecember 7, 2025 19:48
@Eclips4Eclips4 merged commitf193c8f intopython:mainDec 7, 2025
50 checks passed
@miss-islington-app
Copy link

Thanks@mhsmith for the PR, and@Eclips4 for merging it 🌮🎉.. I'm working now to backport this PR to: 3.13, 3.14.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull requestDec 7, 2025
…warnings (pythonGH-142263)(cherry picked from commitf193c8f)Co-authored-by: Malcolm Smith <smith@chaquo.com>
@miss-islington-app
Copy link

Sorry,@mhsmith and@Eclips4, I could not cleanly backport this to3.13 due to a conflict.
Please backport usingcherry_picker on command line.

cherry_picker f193c8fe9e1d722c9a7f9a2b15f8f1b913755491 3.13

@bedevere-app
Copy link

GH-142386 is a backport of this pull request to the3.14 branch.

@bedevere-appbedevere-appbot removed the needs backport to 3.14bugs and security fixes labelDec 7, 2025
Eclips4 pushed a commit that referenced this pull requestDec 8, 2025
… warnings (GH-142263) (#142386)gh-141794: Reduce size of compiler stress tests to fix Android warnings (GH-142263)(cherry picked from commitf193c8f)Co-authored-by: Malcolm Smith <smith@chaquo.com>
mhsmith added a commit to mhsmith/cpython that referenced this pull requestDec 8, 2025
…ndroid warnings (pythonGH-142263)(cherry picked from commitf193c8f)Co-authored-by: Malcolm Smith <smith@chaquo.com>
@bedevere-app
Copy link

GH-142409 is a backport of this pull request to the3.13 branch.

@bedevere-appbedevere-appbot removed the needs backport to 3.13bugs and security fixes labelDec 8, 2025
Eclips4 pushed a commit that referenced this pull requestDec 8, 2025
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@Eclips4Eclips4Eclips4 approved these changes

@tomasr8tomasr8Awaiting requested review from tomasr8tomasr8 is a code owner

Assignees

@Eclips4Eclips4

Labels

skip newstestsTests in the Lib/test dir

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@mhsmith@Eclips4
[8]ページ先頭
©2009-2026 Movatter.jp