Skip to content
This repository was archived by the owner on Nov 1, 2020. It is now read-only.

Conversation

@AntonLapounov
Copy link
Member

Thread class improvements:

  • Support custom stack size (maxStackSize parameter).
  • OOM hardening for new thread starting. Throw ThreadStartException in case of failure.
  • Create a new thread suspended and set its priority before resuming it.
  • In HasFinishedExecution method handle the case when an external thread died and its Thread object was resurrected.
  • Add test cases for resurrected Thread objects and maxStackSize parameter.

Also fix a couple of issues copied from CLR/CoreCLR code:

  • Better handle OS threads with intermediate priorities.
  • Obtain the actual thread priority from the OS instead of using the cached value.

* Support custom stack size (maxStackSize parameter).
* OOM hardening for new thread starting. Throw ThreadStartException in case of failure.
* Create a new thread suspended and set its priority before resuming it.
* In HasFinishedExecution method handle the case when an external thread died and its Thread object was resurrected.
* Add test cases for resurrected Thread objects and maxStackSize parameter.

Also fix a couple of issues copied from CLR/CoreCLR code:
* Better handle OS threads with intermediate priorities.
* Obtain the actual thread priority from the OS instead of using the cached value.
@dnfclas
Copy link

dnfclas commented Mar 2, 2017

@AntonLapounov,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

Copy link
Contributor

@sergiy-k sergiy-k left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Leave a comment

@AntonLapounov AntonLapounov merged commit feefb48 into dotnet:master Mar 2, 2017
@AntonLapounov
Copy link
Member Author

Thank you, Sergiy and Kount!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants