Skip to content
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

Investigate whether we should increase the default nursery size #21435

Open
rolfbjarne opened this issue Oct 14, 2024 · 0 comments
Open

Investigate whether we should increase the default nursery size #21435

rolfbjarne opened this issue Oct 14, 2024 · 0 comments
Labels
performance If an issue or pull request is related to performance
Milestone

Comments

@rolfbjarne
Copy link
Member

Our default nursery size is 512k:

string ret = "nursery-size=512k";

We've had this default since 2012 (https://github.com/xamarin/maccore/commit/40d3775d9028edf0292ca625e30c1c3647b391d6), and in the meantime devices have a lot more memory and they're much faster too. The consequence of a small nursery is that there will be more frequent garbage collections, and this can slow down memory allocations (the consequence of a bigger nursery is longer GC pauses).

So it might be time to investigate whether we should increase the default.

@ivanpovazan suggested 4MB, but we should measure this (the most sensitive/visible part is typically dropped frames while scrolling a list/grid view).

We should also add an MSBuild property to control the nursery size, instead of having to put it in MtouchExtraArgs.

@rolfbjarne rolfbjarne added the performance If an issue or pull request is related to performance label Oct 14, 2024
@rolfbjarne rolfbjarne added this to the .NET 10 milestone Oct 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
performance If an issue or pull request is related to performance
Projects
None yet
Development

No branches or pull requests

1 participant