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

Incorrect information in the ponysuspendthreshold help string #3987

Open
SeanTAllen opened this issue Feb 2, 2022 · 5 comments
Open

Incorrect information in the ponysuspendthreshold help string #3987

SeanTAllen opened this issue Feb 2, 2022 · 5 comments
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed

Comments

@SeanTAllen
Copy link
Member

The help string states:

  --ponysuspendthreshold
                   Amount of idle time before a scheduler thread suspends
                   itself to minimize resource consumption (max 1000 ms,
                   min 1 ms).
                   Defaults to 1 ms.

but the max isn't enforced.

Either the help string needs to be changed or we need to enforce a max.
This is also settable via RuntimeOptions so that needs to be taken into consideration.

The value is stored in an unsigned 64 which is multipled by 1000000 to get a final "ticks" value so, we need to take overflow into account.

@SeanTAllen SeanTAllen added the needs discussion Needs to be discussed further label Feb 2, 2022
@github-actions github-actions bot added the discuss during sync Should be discussed during an upcoming sync label Feb 2, 2022
@SeanTAllen SeanTAllen added enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed and removed needs discussion Needs to be discussed further labels Feb 15, 2022
@csos95
Copy link
Contributor

csos95 commented Feb 15, 2022

Instead of stating that the value given is the number of milliseconds to idle before a thread is suspended, maybe directly state that the value multiplied by one million is the number of ticks to idle before a thread is suspended and in parenthesis give an estimate of how many ticks are in a millisecond on some given hardware.

That way it's clear to the user that this isn't a constant amount of time and that it's something they need to be aware of if used on different hardware.

@SeanTAllen SeanTAllen removed the discuss during sync Should be discussed during an upcoming sync label Feb 15, 2022
@SeanTAllen
Copy link
Member Author

@csos95 would you like to handle the PR to fix this?

@ponylang-main ponylang-main added the discuss during sync Should be discussed during an upcoming sync label Feb 20, 2022
@csos95
Copy link
Contributor

csos95 commented Feb 20, 2022

Sure, I’ll get it done later today.

@jemc jemc removed the discuss during sync Should be discussed during an upcoming sync label Mar 15, 2022
@rhagenson
Copy link
Member

@csos95 Are you still handling this issue?

@ponylang-main ponylang-main added the discuss during sync Should be discussed during an upcoming sync label Aug 15, 2022
@SeanTAllen
Copy link
Member Author

I think we can assume that this is available for anyone to work on at this point and that @csos95 has moved on to other things.

@SeanTAllen SeanTAllen removed the discuss during sync Should be discussed during an upcoming sync label Aug 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

5 participants