-
Notifications
You must be signed in to change notification settings - Fork 2.7k
Conversation
I'm thinking if a better error message should be presented if But maybe we could tackle it in the host or CLI? |
@Maoni0 I was wrong, Linux does have a concept of Reserve and Commit. |
@mjsabby one thing I missed was we should simply not do anything for
|
I have to introduce |
I don't see a reason for using LoadLibrary for these, you can use them directly. They are supported since Windows XP. |
OK, I can see you have already done it that way. |
bbfed48
to
86b01c7
Compare
@Maoni0 What are your thoughts on hard limit + numa awareness + now large pages? I'm thinking to start with we keep Large Pages NUMA-unaware, otherwise it gets a bit complicated where and how much should the memory be split and commuted at startup. Already the multiple heaps overcommits by 3X I think. For 3.0 maybe we start with GCHardLimit + Large Pages does a 3X overcommit at startup then in the future we add configs for splitting the memory across LOH, SOH + NUMA Nodes? |
this sounds fine to me if it's fine with you since you are our 1st customer for this :) also I'm working on reducing this 3x to 2x.
most likely we wouldn't need another config - it'll just be improved naturally in the future versions. |
603cce1
to
77efbd4
Compare
This is ready for review again. After this change, the situation will be:
Next up will be Large Pages support for the GC Heap on Linux. Documentation Items: (1) Noting that GCHeapHardLimit is required (GCHeapHardLimit takes input as hex bytes) |
LGTM - comments above. |
574b9af
to
28ae9ab
Compare
ad544ed
to
ae4f3f1
Compare
/azp run coreclr-ci |
How does one trigger these CI jobs again? |
@dotnet-bot test coreclr-ci |
@dotnet-bot test Ubuntu x64 Checked CoreFX Tests |
@dotnet-bot test coreclr-ci (Test Pri0 Windows_NT x64 checked) |
LGTM. |
The coreclr-ci says the following: Failed: 0 (0.00 %) But it still marked as failed. |
Looks like the error is ##[Error 1] |
@dotnet-bot test coreclr-ci (Test Pri0 OSX x64 checked) |
2612370
to
d36896f
Compare
Add Large pages support in GC Commit migrated from dotnet/coreclr@1874101
Fixes https://github.com/dotnet/coreclr/issues/18371.