-
Notifications
You must be signed in to change notification settings - Fork 4.9k
Update CoreClr, CoreSetup to preview2-26216-09, preview2-26216-04, respectively (master) #27212
Update CoreClr, CoreSetup to preview2-26216-09, preview2-26216-04, respectively (master) #27212
Conversation
|
These tests are failing - System.Runtime.Serialization.Formatters.Tests.BinaryFormatterTests - ValidateAgainstBlobs: Any ideas why? cc @Anipik, @ViktorHofer, @danmosemsft, @joperezr |
|
Serialization for stringComparer has changed for netcoreapp2.1 so this test needs to be changed. The PR is up for it but hasnot yet merged in corefx |
|
I have fixed these 2 tests in #27051 |
Edit: |
…aryFormatterTestData" This reverts commit acba72f.
|
@dotnet-bot test Alpine.3.6 x64 Debug Build |
|
Couldn't update this pull request: Head commit author 'ahsonkhan' is not 'dotnet-maestro-bot' |
|
Anyone know what is causing this failure for Alpine3.6? cc @dotnet/dnceng. @stephentoub @dotnet-bot test Alpine.3.6 x64 Debug Build |
|
That's a segfault. We'll look into it to see if it's an issue with the build system. |
|
Looks like the segfault occurred in ILASM, which suggests to me that it's a bug with the Alpine assembler. I don't know who owns that, but those folks are who you're going to want to ping about this. |
|
ILASM is produced in CoreCLR and serviced by the jit team. cc @dotnet/jit-contrib Do you have a crash dump or any other indication of where things go wrong? |
|
Unforutnately dumpling is not working at present. We only know: I recommend using the "repro" system to get a connection to an Alpine box on which you can run this command and get it under the debugger or get a dump. |
|
I don't see a repro link, unfortunately. So here's a pointer to the old leg; I am going to retry and see if this fails consistently. @dotnet-bot retest Alpine.3.6 x64 Debug Build |
|
Segmentation fault (core dumped) |
There is no repro link on ci3. @dotnet/dnceng, @maririos |
|
Repro is not available for ci3 server |
|
Couldn't update this pull request: Head commit author 'ahsonkhan' is not 'dotnet-maestro-bot' |
|
Couldn't update this pull request: Head commit author 'ahsonkhan' is not 'dotnet-maestro-bot' |
|
Couldn't update this pull request: Head commit author 'ahsonkhan' is not 'dotnet-maestro-bot' |
|
@janvorli any idea what might be going wrong? ILASM hasn't changed in a while so this looks like a build or runtime configuration issue. |
|
Couldn't update this pull request: Head commit author 'ahsonkhan' is not 'dotnet-maestro-bot' |
|
I built my own CHK ILASM (not sure if it's exactly what we get with official builds since I could not easily find what packages we use to build on alpine) and it hits an assert. So this might give some clues about what is going wrong above. So maybe something wrong with the method that gets the current SP? Rebuilding with debug to see if it tells me more. |
|
Couldn't update this pull request: Head commit author 'ahsonkhan' is not 'dotnet-maestro-bot' |
|
Couldn't update this pull request: Head commit author 'ahsonkhan' is not 'dotnet-maestro-bot' |
|
The error you showed was a PAX error. An attempt was made to execute code which was writable, when it was not enabled. @janvorli used |
|
Couldn't update this pull request: Head commit author 'ahsonkhan' is not 'dotnet-maestro-bot' |
|
I am going to close this PR so we can get new updates. We will need to add this commit to the PR (ahsonkhan@c1ffef1) once the Alpine issue has been resolved. |
…26522) (dotnet#27212) * Use win32 api directly for workingset counter * Fix build warnings * Removing useless code * more cleanup * remove size annotation * remove useless comment * Move all the changes to Environment.WorkingSet and remove it from RuntimeEventSourceHelper * removing useless usings * Use kernel32.dll instead of psapi.dll * Code review feedback * Remove newline change * More code review nits Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>



No description provided.