Skip to content

erts: Report the effective atom_limit - #11364

Merged
sverker merged 1 commit into
erlang:masterfrom
naserca:effective-atom-limit
Aug 5, 2026
Merged

erts: Report the effective atom_limit#11364
sverker merged 1 commit into
erlang:masterfrom
naserca:effective-atom-limit

Conversation

@naserca

@naserca naserca commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

erts_index_init() grows its table one INDEX_PAGE_SIZE page at a time and only checks the limit when allocating a new page, so the effective limit is the requested value rounded up to a whole number of pages. Store that rounded value in t->limit so that erlang:system_info(atom_limit) reports the limit actually enforced, like process_limit and port_limit do, rather than echoing back the raw +t value.

erts_index_init() grows its table one INDEX_PAGE_SIZE page at a time and
only checks the limit when allocating a new page, so the effective limit
is the requested value rounded up to a whole number of pages. Store that
rounded value in t->limit so that erlang:system_info(atom_limit) reports
the limit actually enforced, like process_limit and port_limit do, rather
than echoing back the raw +t value.
@github-actions

github-actions Bot commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

CT Test Results

    3 files    136 suites   49m 57s ⏱️
1 682 tests 1 626 ✅ 56 💤 0 ❌
2 325 runs  2 251 ✅ 74 💤 0 ❌

Results for commit 672b8e8.

♻️ This comment has been updated with latest results.

To speed up review, make sure that you have read Contributing to Erlang/OTP and that all checks pass.

See the TESTING and DEVELOPMENT HowTo guides for details about how to run test locally.

Artifacts

// Erlang/OTP Github Action Bot

@naserca

naserca commented Jul 14, 2026

Copy link
Copy Markdown
Contributor Author

Suggested here: #11264 (comment)

@lucioleKi lucioleKi added the team:VM Assigned to OTP team VM label Jul 16, 2026
@sverker sverker added enhancement testing currently being tested, tag is used by OTP internal CI labels Aug 3, 2026
@sverker
sverker merged commit 0652aaa into erlang:master Aug 5, 2026
35 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement team:VM Assigned to OTP team VM testing currently being tested, tag is used by OTP internal CI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants