Setup OSX.1100.ARM64.* helix queues#47919

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.

Sign up for GitHub

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

Merged
sdmaclea merged 9 commits into from
Mar 2, 2021
Merged

Setup OSX.1100.ARM64.* helix queues #47919

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
ddcafc8
Setup OSX.1100.ARM64.* helix queues
sdmaclea Feb 5, 2021
a1de69c
Limit OSX.1100.ARM64.* queues to CI builds until we understand capacity
sdmaclea Feb 5, 2021
783393b
For CI testing, add OSX_arm64 to the platformGroup all and gcstress
sdmaclea Feb 5, 2021
136f9a0
Run jitstress on Apple Silicon
sdmaclea Feb 24, 2021
b57b635
Enable Apple Silicon on more pipelines
sdmaclea Feb 25, 2021
6146fd2
Enable more Apple Silicon pipelines
sdmaclea Feb 25, 2021
965efb3
Fixup libraries outerloop
sdmaclea Feb 26, 2021
763770d
Disable failing Apple Silicon PAL tests
sdmaclea Feb 26, 2021
144d483
Revert "Apple Silicon disable failing PAL tests (#48782)"
sdmaclea Feb 26, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 12 additions & 0 deletions eng/pipelines/coreclr/ci.yml
View file
Open in desktop
Original file line numberDiff line numberDiff line change
Expand Up@@ -54,6 +54,10 @@ jobs:
jobTemplate: /eng/pipelines/coreclr/templates/build-job.yml
buildConfig: checked
platformGroup: all
platforms:
# It is too early to include OSX_arm64 in platform group all
# Adding it here will enable it also
- OSX_arm64
jobParameters:
testGroup: outerloop

Expand DownExpand Up@@ -82,6 +86,10 @@ jobs:
jobTemplate: /eng/pipelines/libraries/build-job.yml
buildConfig: Release
platformGroup: all
platforms:
# It is too early to include OSX_arm64 in platform group all
# Adding it here will enable it also
- OSX_arm64
jobParameters:
isOfficialBuild: false
liveRuntimeBuildConfig: checked
Expand All@@ -107,6 +115,10 @@ jobs:
jobTemplate: /eng/pipelines/common/templates/runtimes/run-test-job.yml
buildConfig: checked
platformGroup: all
platforms:
# It is too early to include OSX_arm64 in platform group all
# Adding it here will enable it to also run this test
- OSX_arm64
helixQueueGroup: ci
helixQueuesTemplate: /eng/pipelines/coreclr/templates/helix-queues-setup.yml
jobParameters:
Expand Down
4 changes: 2 additions & 2 deletions eng/pipelines/coreclr/ilasm.yml
View file
Open in desktop
Original file line numberDiff line numberDiff line change
Expand Up@@ -24,7 +24,7 @@ jobs:
jobTemplate: /eng/pipelines/common/build-coreclr-and-libraries-job.yml
buildConfig: checked
platforms:
# Linux tests are built on the OSX machines.
- OSX_arm64
- OSX_x64
- Linux_arm
- Linux_arm64
Expand All@@ -51,7 +51,7 @@ jobs:
jobTemplate: /eng/pipelines/common/templates/runtimes/run-test-job.yml
buildConfig: checked
platforms:
# Linux tests are built on the OSX machines.
- OSX_arm64
- OSX_x64
- Linux_arm
- Linux_arm64
Expand Down
1 change: 1 addition & 0 deletions eng/pipelines/coreclr/jitrollingbuild.yml
View file
Open in desktop
Original file line numberDiff line numberDiff line change
Expand Up@@ -19,6 +19,7 @@ jobs:
jobTemplate: /eng/pipelines/coreclr/templates/build-jit-job.yml
buildConfig: checked
platforms:
- OSX_arm64
- OSX_x64
# Currently, Linux arm/arm64 machines don't have the Python 'pip3' tool, nor the azure-storage-blob package that
# is required to do the JIT upload to Azure Storage. Thus, these platforms are disabled. If we can figure out how
Expand Down
2 changes: 2 additions & 0 deletions eng/pipelines/coreclr/jitstress-isas-arm.yml
View file
Open in desktop
Original file line numberDiff line numberDiff line change
Expand Up@@ -16,6 +16,7 @@ jobs:
buildConfig: checked
platforms:
- Linux_arm64
- OSX_arm64
- windows_arm64
- CoreClrTestBuildHost # Either OSX_x64 or Linux_x64
jobParameters:
Expand All@@ -37,6 +38,7 @@ jobs:
buildConfig: checked
platforms:
- Linux_arm64
- OSX_arm64
- windows_arm64
helixQueueGroup: ci
helixQueuesTemplate: /eng/pipelines/coreclr/templates/helix-queues-setup.yml
Expand Down
4 changes: 2 additions & 2 deletions eng/pipelines/coreclr/jitstress.yml
View file
Open in desktop
Original file line numberDiff line numberDiff line change
Expand Up@@ -21,7 +21,7 @@ jobs:
jobTemplate: /eng/pipelines/common/build-coreclr-and-libraries-job.yml
buildConfig: checked
platforms:
# Linux tests are built on the OSX machines.
- OSX_arm64
- OSX_x64
- Linux_arm
- Linux_arm64
Expand All@@ -48,7 +48,7 @@ jobs:
jobTemplate: /eng/pipelines/common/templates/runtimes/run-test-job.yml
buildConfig: checked
platforms:
# Linux tests are built on the OSX machines.
- OSX_arm64
- OSX_x64
- Linux_arm
- Linux_arm64
Expand Down
4 changes: 2 additions & 2 deletions eng/pipelines/coreclr/jitstress2-jitstressregs.yml
View file
Open in desktop
Original file line numberDiff line numberDiff line change
Expand Up@@ -15,7 +15,7 @@ jobs:
jobTemplate: /eng/pipelines/common/build-coreclr-and-libraries-job.yml
buildConfig: checked
platforms:
# Linux tests are built on the OSX machines.
- OSX_arm64
- OSX_x64
- Linux_arm
- Linux_arm64
Expand All@@ -42,7 +42,7 @@ jobs:
jobTemplate: /eng/pipelines/common/templates/runtimes/run-test-job.yml
buildConfig: checked
platforms:
# Linux tests are built on the OSX machines.
- OSX_arm64
- OSX_x64
- Linux_arm
- Linux_arm64
Expand Down
4 changes: 2 additions & 2 deletions eng/pipelines/coreclr/jitstressregs.yml
View file
Open in desktop
Original file line numberDiff line numberDiff line change
Expand Up@@ -15,7 +15,7 @@ jobs:
jobTemplate: /eng/pipelines/common/build-coreclr-and-libraries-job.yml
buildConfig: checked
platforms:
# Linux tests are built on the OSX machines.
- OSX_arm64
- OSX_x64
- Linux_arm
- Linux_arm64
Expand All@@ -42,7 +42,7 @@ jobs:
jobTemplate: /eng/pipelines/common/templates/runtimes/run-test-job.yml
buildConfig: checked
platforms:
# Linux tests are built on the OSX machines.
- OSX_arm64
- OSX_x64
- Linux_arm
- Linux_arm64
Expand Down
8 changes: 8 additions & 0 deletions eng/pipelines/coreclr/release-tests.yml
View file
Open in desktop
Original file line numberDiff line numberDiff line change
Expand Up@@ -18,6 +18,10 @@ jobs:
jobTemplate: /eng/pipelines/common/build-coreclr-and-libraries-job.yml
buildConfig: release
platformGroup: all
platforms:
# It is too early to include OSX_arm64 in platform group all
# Adding it here will enable it also
- OSX_arm64
jobParameters:
isOfficialBuild: false

Expand All@@ -42,6 +46,10 @@ jobs:
jobTemplate: /eng/pipelines/common/templates/runtimes/run-test-job.yml
buildConfig: release
platformGroup: all
platforms:
# It is too early to include OSX_arm64 in platform group all
# Adding it here will enable it also
- OSX_arm64
helixQueueGroup: ci
helixQueuesTemplate: /eng/pipelines/coreclr/templates/helix-queues-setup.yml
jobParameters:
Expand Down
7 changes: 7 additions & 0 deletions eng/pipelines/coreclr/templates/helix-queues-setup.yml
View file
Open in desktop
Original file line numberDiff line numberDiff line change
Expand Up@@ -95,6 +95,13 @@ jobs:
- (Fedora.30.Amd64)Ubuntu.1604.amd64@mcr.microsoft.com/dotnet-buildtools/prereqs:fedora-30-helix-20200512010621-4f8cef7
- RedHat.7.Amd64

# OSX arm64
- ${{ if eq(parameters.platform, 'OSX_arm64') }}:
- ${{ if and(eq(variables['System.TeamProject'], 'public'), in(parameters.jobParameters.helixQueueGroup, 'ci', 'libraries')) }}:
- OSX.1100.ARM64.Open
- ${{ if eq(variables['System.TeamProject'], 'internal') }}:
- OSX.1100.ARM64

# OSX x64
- ${{ if eq(parameters.platform, 'OSX_x64') }}:
- ${{ if and(eq(variables['System.TeamProject'], 'public'), in(parameters.jobParameters.helixQueueGroup, 'pr', 'ci', 'libraries')) }}:
Expand Down
5 changes: 5 additions & 0 deletions eng/pipelines/libraries/helix-queues-setup.yml
View file
Open in desktop
Original file line numberDiff line numberDiff line change
Expand Up@@ -85,6 +85,11 @@ jobs:
# Limiting interp runs as we don't need as much coverage.
- Debian.9.Amd64.Open

# OSX arm64
- ${{ if eq(parameters.platform, 'OSX_arm64') }}:
Comment thread
sdmaclea marked this conversation as resolved.
Outdated
- ${{ if eq(parameters.jobParameters.isFullMatrix, true) }}:

@safernsafernMar 1, 2021

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I know we discussed this earlier, but I believe we should remove this condition for libraries and then instead condition the test runs in the yml entry point?

- OSX.1100.ARM64.Open

# OSX x64
- ${{ if eq(parameters.platform, 'OSX_x64') }}:
- ${{ if eq(parameters.jobParameters.isFullMatrix, true) }}:
Expand Down
5 changes: 5 additions & 0 deletions eng/pipelines/libraries/outerloop.yml
View file
Open in desktop
Original file line numberDiff line numberDiff line change
Expand Up@@ -32,6 +32,8 @@ jobs:
- Linux_arm64
- Linux_musl_arm64
- ${{ if eq(variables['includeOsxOuterloop'], true) }}:
- ${{ if eq(variables['isFullMatrix'], true) }}:
- OSX_arm64
- OSX_x64
jobParameters:
testGroup: innerloop
Expand All@@ -55,6 +57,7 @@ jobs:
- Linux_musl_x64
- Linux_musl_arm64
- ${{ if and(eq(variables['includeOsxOuterloop'], true), eq(variables['isFullMatrix'], true)) }}:
- OSX_arm64
- OSX_x64
helixQueuesTemplate: /eng/pipelines/libraries/helix-queues-setup.yml
jobParameters:
Expand All@@ -76,6 +79,8 @@ jobs:
- Linux_x64
- Linux_musl_x64
- ${{ if eq(variables['includeOsxOuterloop'], true) }}:
- ${{ if eq(variables['isFullMatrix'], true) }}:
- OSX_arm64
- OSX_x64
helixQueuesTemplate: /eng/pipelines/libraries/helix-queues-setup.yml
jobParameters:
Expand Down
8 changes: 0 additions & 8 deletions src/coreclr/pal/tests/palsuite/exception_handling/pal_sxs/test1/exceptionsxs.cpp
View file
Open in desktop
Original file line numberDiff line numberDiff line change
Expand Up@@ -68,13 +68,6 @@ void sigsegv_handler(int code, siginfo_t *siginfo, void *context)

int main(int argc, char *argv[])
{
#if defined(TARGET_OSX) && defined(TARGET_ARM64)
// Helix and/or build error...
// dyld: Library not loaded: libpaltest_pal_sxs_test1_dll1.dylib
printf("paltest_pal_sxs_test1 has been disabled on this platform\n");

return PASS;
#else // defined(TARGET_OSX) && defined(TARGET_ARM64)
struct sigaction newAction;
struct sigaction oldAction;
newAction.sa_flags = SA_SIGINFO | SA_RESTART;
Expand DownExpand Up@@ -120,5 +113,4 @@ int main(int argc, char *argv[])

printf("ERROR: code was executed after the access violation.\n");
return FAIL;
#endif // defined(TARGET_OSX) && defined(TARGET_ARM64)
}
5 changes: 0 additions & 5 deletions src/coreclr/pal/tests/palsuite/file_io/ReadFile/test2/ReadFile.cpp
View file
Open in desktop
Original file line numberDiff line numberDiff line change
Expand Up@@ -120,10 +120,6 @@ BOOL readTest_ReadFile_test2(DWORD dwByteCount, char cResult)

PALTEST(file_io_ReadFile_test2_paltest_readfile_test2, "file_io/ReadFile/test2/paltest_readfile_test2")
{
#if defined(TARGET_OSX) && defined(TARGET_ARM64)
// Test hard codes PAGE_SIZE = 4096. Apple Silicon does not have 4K pages.
printf("file_io_ReadFile_test2_paltest_readfile_test2 has been disabled on this platform\n");
#else // defined(TARGET_OSX) && defined(TARGET_ARM64)
HANDLE hFile = NULL;
const int BUFFER_SIZE = 2 * PAGESIZE;

Expand DownExpand Up@@ -191,7 +187,6 @@ PALTEST(file_io_ReadFile_test2_paltest_readfile_test2, "file_io/ReadFile/test2/p

VirtualFree(readBuffer_ReadFile_test2, BUFFER_SIZE, MEM_RELEASE);
PAL_Terminate();
#endif // defined(TARGET_OSX) && defined(TARGET_ARM64)
return PASS;
}

5 changes: 0 additions & 5 deletions src/coreclr/pal/tests/palsuite/filemapping_memmgt/VirtualProtect/test4/VirtualProtect.cpp
View file
Open in desktop
Original file line numberDiff line numberDiff line change
Expand Up@@ -28,10 +28,6 @@ PALTEST(filemapping_memmgt_VirtualProtect_test4_paltest_virtualprotect_test4, "f
ExitProcess(FAIL);
}

#if defined(TARGET_OSX) && defined(TARGET_ARM64)
// AppleSilcon does not allow mapping general pages as RWX
printf("filemapping_memmgt_VirtualProtect_test4_paltest_virtualprotect_test4 has been disabled on this platform\n");
#else // defined(TARGET_OSX) && defined(TARGET_ARM64)
//Allocate the physical storage in memory or in the paging file on disk
lpVirtualAddress = VirtualAlloc(NULL,//determine where to allocate the region
REGIONSIZE, //specify the size
Expand DownExpand Up@@ -65,7 +61,6 @@ PALTEST(filemapping_memmgt_VirtualProtect_test4_paltest_virtualprotect_test4, "f
{
Fail("\nFailed to call VirtualFree API!\n");
}
#endif // defined(TARGET_OSX) && defined(TARGET_ARM64)

PAL_Terminate();
return PASS;
Expand Down
17 changes: 17 additions & 0 deletions src/coreclr/pal/tests/palsuite/issues.targets
View file
Open in desktop
Original file line numberDiff line numberDiff line change
Expand Up@@ -32,4 +32,21 @@
<Issue>https://github.com/dotnet/runtime/issues/42292</Issue>
</ExcludeList>
</ItemGroup>

<ItemGroup Condition="'$(TargetArchitecture)' == 'arm64' and '$(TargetOS)' == 'OSX'">
<ExcludeList Include="miscellaneous/queryperformancecounter/test1/paltest_queryperformancecounter_test1">
<Issue>https://github.com/dotnet/runtime/issues/7639</Issue>
</ExcludeList>
<ExcludeList Include="exception_handling/pal_sxs/test1/paltest_pal_sxs_test1">
<Issue>https://github.com/dotnet/runtime/issues/42292</Issue>
</ExcludeList>
<ExcludeList Include="file_io/ReadFile/test2/paltest_readfile_test2">
<!-- Test hard codes PAGE_SIZE = 4096. Apple Silicon does not have 4K pages. -->
<Issue>https://github.com/dotnet/runtime/issues/42292</Issue>
</ExcludeList>
<ExcludeList Include="filemapping_memmgt/VirtualProtect/test4/paltest_virtualprotect_test4">
<!-- Test tries to set RWX on general memory page -->
<Issue>https://github.com/dotnet/runtime/issues/7639</Issue>
</ExcludeList>
</ItemGroup>
</Project>
5 changes: 0 additions & 5 deletions src/coreclr/pal/tests/palsuite/miscellaneous/queryperformancecounter/test1/test1.cpp
View file
Open in desktop
Original file line numberDiff line numberDiff line change
Expand Up@@ -17,10 +17,6 @@

PALTEST(miscellaneous_queryperformancecounter_test1_paltest_queryperformancecounter_test1, "miscellaneous/queryperformancecounter/test1/paltest_queryperformancecounter_test1")
{
#if defined(TARGET_OSX) && defined(TARGET_ARM64)
// We are see > 100 ms of average error on this platform
printf("miscellaneous_queryperformancecounter_test1_paltest_queryperformancecounter_test1 has been disabled on this platform\n");
#else // defined(TARGET_OSX) && defined(TARGET_ARM64)
/* Milliseconds of error which are acceptable Function execution time, etc.
FreeBSD has a "standard" resolution of 50ms for waiting operations, so we
must take that into account as well */
Expand DownExpand Up@@ -104,7 +100,6 @@ PALTEST(miscellaneous_queryperformancecounter_test1_paltest_queryperformancecoun
/* Terminate the PAL.
*/
PAL_Terminate();
#endif // defined(TARGET_OSX) && defined(TARGET_ARM64)
return PASS;
}

Expand Down
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Add copy buttons to all 
 blocks\n(function() {\n function addCopyButtons() {\n document.querySelectorAll('pre code').forEach(function(codeBlock) {\n if (codeBlock.parentElement.hasAttribute('data-copy-added')) return;\n codeBlock.parentElement.setAttribute('data-copy-added', 'true');\n \n var btn = document.createElement('button');\n btn.textContent = 'Copy';\n btn.style.cssText = 'position:absolute;top:4px;right:4px;padding:2px 8px;font-size:11px;background:#4ecdc4;border:none;border-radius:4px;color:#1a1a2e;cursor:pointer;opacity:0.7;transition:opacity 0.2s;';\n btn.onmouseover = function() { this.style.opacity = '1'; };\n btn.onmouseout = function() { this.style.opacity = '0.7'; };\n btn.onclick = function() {\n navigator.clipboard.writeText(codeBlock.textContent).then(function() {\n btn.textContent = 'Copied!';\n setTimeout(function() { btn.textContent = 'Copy'; }, 1500);\n });\n };\n codeBlock.parentElement.style.position = 'relative';\n codeBlock.parentElement.appendChild(btn);\n });\n }\n \n addCopyButtons();\n \n // Re-run on dynamic content\n var observer = new MutationObserver(addCopyButtons);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Add Copy Buttons to Code Blocks");
}
} catch(__e) { console.warn('[Userscript:Add Copy Buttons to Code Blocks]', __e); }
})();
(function(){
try {
var __m = "github.com";
var __re = new RegExp('^' + "github\\.com" + '

Skip to content
Setup OSX.1100.ARM64.* helix queues#47919

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.

 Sign up for GitHub

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

 Merged

sdmaclea
merged 9 commits into

from

Mar 2, 2021
 Merged


Setup OSX.1100.ARM64.* helix queues
#47919
Show file tree
Hide file tree
 Changes from all commits
Commits

Show all changes


9 commits


Select commit
Hold shift + click to select a range
ddcafc8

Setup OSX.1100.ARM64.* helix queues


sdmaclea Feb 5, 2021
a1de69c

Limit OSX.1100.ARM64.* queues to CI builds until we understand capacity


sdmaclea Feb 5, 2021
783393b

For CI testing, add OSX_arm64 to the platformGroup all and gcstress


sdmaclea Feb 5, 2021
136f9a0

Run jitstress on Apple Silicon


sdmaclea Feb 24, 2021
b57b635

Enable Apple Silicon on more pipelines


sdmaclea Feb 25, 2021
6146fd2

Enable more Apple Silicon pipelines


sdmaclea Feb 25, 2021
965efb3

Fixup libraries outerloop


sdmaclea Feb 26, 2021
763770d

Disable failing Apple Silicon PAL tests


sdmaclea Feb 26, 2021
144d483

Revert "Apple Silicon disable failing PAL tests (#48782)"


sdmaclea Feb 26, 2021

File filter

Filter by extension
Filter by extension
Conversations

Failed to load comments. 
Loading
Jump to

Jump to file


Failed to load files. 
Loading
 Diff view

 Diff view


12 changes: 12 additions & 0 deletions
eng/pipelines/coreclr/ci.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -54,6 +54,10 @@ jobs:
 jobTemplate: /eng/pipelines/coreclr/templates/build-job.yml
 buildConfig: checked
 platformGroup: all
 platforms:
 # It is too early to include OSX_arm64 in platform group all
 # Adding it here will enable it also
 - OSX_arm64
 jobParameters:
 testGroup: outerloop

Expand DownExpand Up@@ -82,6 +86,10 @@ jobs:
 jobTemplate: /eng/pipelines/libraries/build-job.yml
 buildConfig: Release
 platformGroup: all
 platforms:
 # It is too early to include OSX_arm64 in platform group all
 # Adding it here will enable it also
 - OSX_arm64
 jobParameters:
 isOfficialBuild: false
 liveRuntimeBuildConfig: checked
Expand All@@ -107,6 +115,10 @@ jobs:
 jobTemplate: /eng/pipelines/common/templates/runtimes/run-test-job.yml
 buildConfig: checked
 platformGroup: all
 platforms:
 # It is too early to include OSX_arm64 in platform group all
 # Adding it here will enable it to also run this test
 - OSX_arm64
 helixQueueGroup: ci
 helixQueuesTemplate: /eng/pipelines/coreclr/templates/helix-queues-setup.yml
 jobParameters:
Expand Down

4 changes: 2 additions & 2 deletions
eng/pipelines/coreclr/ilasm.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -24,7 +24,7 @@ jobs:
 jobTemplate: /eng/pipelines/common/build-coreclr-and-libraries-job.yml
 buildConfig: checked
 platforms:
# Linux tests are built on the OSX machines.
- OSX_arm64
 - OSX_x64
 - Linux_arm
 - Linux_arm64
Expand All@@ -51,7 +51,7 @@ jobs:
 jobTemplate: /eng/pipelines/common/templates/runtimes/run-test-job.yml
 buildConfig: checked
 platforms:
# Linux tests are built on the OSX machines.
- OSX_arm64
 - OSX_x64
 - Linux_arm
 - Linux_arm64
Expand Down

1 change: 1 addition & 0 deletions
eng/pipelines/coreclr/jitrollingbuild.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -19,6 +19,7 @@ jobs:
 jobTemplate: /eng/pipelines/coreclr/templates/build-jit-job.yml
 buildConfig: checked
 platforms:
 - OSX_arm64
 - OSX_x64
 # Currently, Linux arm/arm64 machines don't have the Python 'pip3' tool, nor the azure-storage-blob package that
 # is required to do the JIT upload to Azure Storage. Thus, these platforms are disabled. If we can figure out how
Expand Down

2 changes: 2 additions & 0 deletions
eng/pipelines/coreclr/jitstress-isas-arm.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -16,6 +16,7 @@ jobs:
 buildConfig: checked
 platforms:
 - Linux_arm64
 - OSX_arm64
 - windows_arm64
 - CoreClrTestBuildHost # Either OSX_x64 or Linux_x64
 jobParameters:
Expand All@@ -37,6 +38,7 @@ jobs:
 buildConfig: checked
 platforms:
 - Linux_arm64
 - OSX_arm64
 - windows_arm64
 helixQueueGroup: ci
 helixQueuesTemplate: /eng/pipelines/coreclr/templates/helix-queues-setup.yml
Expand Down

4 changes: 2 additions & 2 deletions
eng/pipelines/coreclr/jitstress.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -21,7 +21,7 @@ jobs:
 jobTemplate: /eng/pipelines/common/build-coreclr-and-libraries-job.yml
 buildConfig: checked
 platforms:
# Linux tests are built on the OSX machines.
- OSX_arm64
 - OSX_x64
 - Linux_arm
 - Linux_arm64
Expand All@@ -48,7 +48,7 @@ jobs:
 jobTemplate: /eng/pipelines/common/templates/runtimes/run-test-job.yml
 buildConfig: checked
 platforms:
# Linux tests are built on the OSX machines.
- OSX_arm64
 - OSX_x64
 - Linux_arm
 - Linux_arm64
Expand Down

4 changes: 2 additions & 2 deletions
eng/pipelines/coreclr/jitstress2-jitstressregs.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -15,7 +15,7 @@ jobs:
 jobTemplate: /eng/pipelines/common/build-coreclr-and-libraries-job.yml
 buildConfig: checked
 platforms:
# Linux tests are built on the OSX machines.
- OSX_arm64
 - OSX_x64
 - Linux_arm
 - Linux_arm64
Expand All@@ -42,7 +42,7 @@ jobs:
 jobTemplate: /eng/pipelines/common/templates/runtimes/run-test-job.yml
 buildConfig: checked
 platforms:
# Linux tests are built on the OSX machines.
- OSX_arm64
 - OSX_x64
 - Linux_arm
 - Linux_arm64
Expand Down

4 changes: 2 additions & 2 deletions
eng/pipelines/coreclr/jitstressregs.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -15,7 +15,7 @@ jobs:
 jobTemplate: /eng/pipelines/common/build-coreclr-and-libraries-job.yml
 buildConfig: checked
 platforms:
# Linux tests are built on the OSX machines.
- OSX_arm64
 - OSX_x64
 - Linux_arm
 - Linux_arm64
Expand All@@ -42,7 +42,7 @@ jobs:
 jobTemplate: /eng/pipelines/common/templates/runtimes/run-test-job.yml
 buildConfig: checked
 platforms:
# Linux tests are built on the OSX machines.
- OSX_arm64
 - OSX_x64
 - Linux_arm
 - Linux_arm64
Expand Down

8 changes: 8 additions & 0 deletions
eng/pipelines/coreclr/release-tests.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -18,6 +18,10 @@ jobs:
 jobTemplate: /eng/pipelines/common/build-coreclr-and-libraries-job.yml
 buildConfig: release
 platformGroup: all
 platforms:
 # It is too early to include OSX_arm64 in platform group all
 # Adding it here will enable it also
 - OSX_arm64
 jobParameters:
 isOfficialBuild: false

Expand All@@ -42,6 +46,10 @@ jobs:
 jobTemplate: /eng/pipelines/common/templates/runtimes/run-test-job.yml
 buildConfig: release
 platformGroup: all
 platforms:
 # It is too early to include OSX_arm64 in platform group all
 # Adding it here will enable it also
 - OSX_arm64
 helixQueueGroup: ci
 helixQueuesTemplate: /eng/pipelines/coreclr/templates/helix-queues-setup.yml
 jobParameters:
Expand Down

7 changes: 7 additions & 0 deletions
eng/pipelines/coreclr/templates/helix-queues-setup.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -95,6 +95,13 @@ jobs:
 - (Fedora.30.Amd64)Ubuntu.1604.amd64@mcr.microsoft.com/dotnet-buildtools/prereqs:fedora-30-helix-20200512010621-4f8cef7
 - RedHat.7.Amd64

 # OSX arm64
 - ${{ if eq(parameters.platform, 'OSX_arm64') }}:
 - ${{ if and(eq(variables['System.TeamProject'], 'public'), in(parameters.jobParameters.helixQueueGroup, 'ci', 'libraries')) }}:
 - OSX.1100.ARM64.Open
 - ${{ if eq(variables['System.TeamProject'], 'internal') }}:
 - OSX.1100.ARM64

 # OSX x64
 - ${{ if eq(parameters.platform, 'OSX_x64') }}:
 - ${{ if and(eq(variables['System.TeamProject'], 'public'), in(parameters.jobParameters.helixQueueGroup, 'pr', 'ci', 'libraries')) }}:
Expand Down

5 changes: 5 additions & 0 deletions
eng/pipelines/libraries/helix-queues-setup.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -85,6 +85,11 @@ jobs:
 # Limiting interp runs as we don't need as much coverage.
 - Debian.9.Amd64.Open

 # OSX arm64
 - ${{ if eq(parameters.platform, 'OSX_arm64') }}:
Comment thread
sdmaclea marked this conversation as resolved.


Outdated

 - ${{ if eq(parameters.jobParameters.isFullMatrix, true) }}:
@safernsafernMar 1, 2021

Copy link


Copy Markdown

Member
Choose a reason for hiding this comment

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

I know we discussed this earlier, but I believe we should remove this condition for libraries and then instead condition the test runs in the yml entry point?
 - OSX.1100.ARM64.Open

 # OSX x64
 - ${{ if eq(parameters.platform, 'OSX_x64') }}:
 - ${{ if eq(parameters.jobParameters.isFullMatrix, true) }}:
Expand Down

5 changes: 5 additions & 0 deletions
eng/pipelines/libraries/outerloop.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -32,6 +32,8 @@ jobs:
 - Linux_arm64
 - Linux_musl_arm64
 - ${{ if eq(variables['includeOsxOuterloop'], true) }}: 
 - ${{ if eq(variables['isFullMatrix'], true) }}:
 - OSX_arm64
 - OSX_x64
 jobParameters:
 testGroup: innerloop
Expand All@@ -55,6 +57,7 @@ jobs:
 - Linux_musl_x64
 - Linux_musl_arm64
 - ${{ if and(eq(variables['includeOsxOuterloop'], true), eq(variables['isFullMatrix'], true)) }}:
 - OSX_arm64
 - OSX_x64
 helixQueuesTemplate: /eng/pipelines/libraries/helix-queues-setup.yml
 jobParameters:
Expand All@@ -76,6 +79,8 @@ jobs:
 - Linux_x64
 - Linux_musl_x64
 - ${{ if eq(variables['includeOsxOuterloop'], true) }}:
 - ${{ if eq(variables['isFullMatrix'], true) }}:
 - OSX_arm64
 - OSX_x64
 helixQueuesTemplate: /eng/pipelines/libraries/helix-queues-setup.yml
 jobParameters:
Expand Down

8 changes: 0 additions & 8 deletions
src/coreclr/pal/tests/palsuite/exception_handling/pal_sxs/test1/exceptionsxs.cpp

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -68,13 +68,6 @@ void sigsegv_handler(int code, siginfo_t *siginfo, void *context)

int main(int argc, char *argv[])
{
#if defined(TARGET_OSX) && defined(TARGET_ARM64)
 // Helix and/or build error...
 // dyld: Library not loaded: libpaltest_pal_sxs_test1_dll1.dylib
 printf("paltest_pal_sxs_test1 has been disabled on this platform\n");

 return PASS;
#else // defined(TARGET_OSX) && defined(TARGET_ARM64)
 struct sigaction newAction;
 struct sigaction oldAction;
 newAction.sa_flags = SA_SIGINFO | SA_RESTART;
Expand DownExpand Up@@ -120,5 +113,4 @@ int main(int argc, char *argv[])

 printf("ERROR: code was executed after the access violation.\n");
 return FAIL;
#endif // defined(TARGET_OSX) && defined(TARGET_ARM64)
}

5 changes: 0 additions & 5 deletions
src/coreclr/pal/tests/palsuite/file_io/ReadFile/test2/ReadFile.cpp

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -120,10 +120,6 @@ BOOL readTest_ReadFile_test2(DWORD dwByteCount, char cResult)

PALTEST(file_io_ReadFile_test2_paltest_readfile_test2, "file_io/ReadFile/test2/paltest_readfile_test2")
{
#if defined(TARGET_OSX) && defined(TARGET_ARM64)
 // Test hard codes PAGE_SIZE = 4096. Apple Silicon does not have 4K pages.
 printf("file_io_ReadFile_test2_paltest_readfile_test2 has been disabled on this platform\n");
#else // defined(TARGET_OSX) && defined(TARGET_ARM64)
 HANDLE hFile = NULL;
 const int BUFFER_SIZE = 2 * PAGESIZE;

Expand DownExpand Up@@ -191,7 +187,6 @@ PALTEST(file_io_ReadFile_test2_paltest_readfile_test2, "file_io/ReadFile/test2/p

	VirtualFree(readBuffer_ReadFile_test2, BUFFER_SIZE, MEM_RELEASE);
 PAL_Terminate();
#endif // defined(TARGET_OSX) && defined(TARGET_ARM64)
 return PASS;
}


5 changes: 0 additions & 5 deletions
src/coreclr/pal/tests/palsuite/filemapping_memmgt/VirtualProtect/test4/VirtualProtect.cpp

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -28,10 +28,6 @@ PALTEST(filemapping_memmgt_VirtualProtect_test4_paltest_virtualprotect_test4, "f
 ExitProcess(FAIL);
 }

#if defined(TARGET_OSX) && defined(TARGET_ARM64)
 // AppleSilcon does not allow mapping general pages as RWX
 printf("filemapping_memmgt_VirtualProtect_test4_paltest_virtualprotect_test4 has been disabled on this platform\n");
#else // defined(TARGET_OSX) && defined(TARGET_ARM64)
 //Allocate the physical storage in memory or in the paging file on disk 
 lpVirtualAddress = VirtualAlloc(NULL,//determine where to allocate the region
 REGIONSIZE, //specify the size
Expand DownExpand Up@@ -65,7 +61,6 @@ PALTEST(filemapping_memmgt_VirtualProtect_test4_paltest_virtualprotect_test4, "f
 {
 Fail("\nFailed to call VirtualFree API!\n");
 }
#endif // defined(TARGET_OSX) && defined(TARGET_ARM64)

 PAL_Terminate();
 return PASS;
Expand Down

17 changes: 17 additions & 0 deletions
src/coreclr/pal/tests/palsuite/issues.targets

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -32,4 +32,21 @@
 <Issue>https://github.com/dotnet/runtime/issues/42292</Issue>
 </ExcludeList>
 </ItemGroup>

 <ItemGroup Condition="'$(TargetArchitecture)' == 'arm64' and '$(TargetOS)' == 'OSX'">
 <ExcludeList Include="miscellaneous/queryperformancecounter/test1/paltest_queryperformancecounter_test1">
 <Issue>https://github.com/dotnet/runtime/issues/7639</Issue>
 </ExcludeList>
 <ExcludeList Include="exception_handling/pal_sxs/test1/paltest_pal_sxs_test1">
 <Issue>https://github.com/dotnet/runtime/issues/42292</Issue>
 </ExcludeList>
 <ExcludeList Include="file_io/ReadFile/test2/paltest_readfile_test2">
 <!-- Test hard codes PAGE_SIZE = 4096. Apple Silicon does not have 4K pages. -->
 <Issue>https://github.com/dotnet/runtime/issues/42292</Issue>
 </ExcludeList>
 <ExcludeList Include="filemapping_memmgt/VirtualProtect/test4/paltest_virtualprotect_test4">
 <!-- Test tries to set RWX on general memory page -->
 <Issue>https://github.com/dotnet/runtime/issues/7639</Issue>
 </ExcludeList>
 </ItemGroup>
</Project>

5 changes: 0 additions & 5 deletions
src/coreclr/pal/tests/palsuite/miscellaneous/queryperformancecounter/test1/test1.cpp

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -17,10 +17,6 @@

PALTEST(miscellaneous_queryperformancecounter_test1_paltest_queryperformancecounter_test1, "miscellaneous/queryperformancecounter/test1/paltest_queryperformancecounter_test1")
{
#if defined(TARGET_OSX) && defined(TARGET_ARM64)
 // We are see > 100 ms of average error on this platform
 printf("miscellaneous_queryperformancecounter_test1_paltest_queryperformancecounter_test1 has been disabled on this platform\n");
#else // defined(TARGET_OSX) && defined(TARGET_ARM64)
 /* Milliseconds of error which are acceptable Function execution time, etc.
 FreeBSD has a "standard" resolution of 50ms for waiting operations, so we
 must take that into account as well */
Expand DownExpand Up@@ -104,7 +100,6 @@ PALTEST(miscellaneous_queryperformancecounter_test1_paltest_queryperformancecoun
 /* Terminate the PAL.
 */ 
 PAL_Terminate();
#endif // defined(TARGET_OSX) && defined(TARGET_ARM64)
 return PASS;
}

Expand Down

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Force GitHub README to respect dark mode\n(function() {\n var style = document.createElement('style');\n style.textContent = '\n .markdown-body {\n color-scheme: dark light;\n }\n .markdown-body pre { background: #161b22 !important; }\n .markdown-body code { background: rgba(110, 118, 129, 0.4) !important; }\n .markdown-body table th, .markdown-body table td { border-color: #30363d !important; }\n .markdown-body img { background: #0d1117; }\n .markdown-body blockquote { border-left-color: #8b949e; }\n .markdown-body hr { border-color: #30363d; }\n ';\n document.head.appendChild(style);\n})();", "GitHub Dark Mode README Fix");
}
} catch(__e) { console.warn('[Userscript:GitHub Dark Mode README Fix]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content
Setup OSX.1100.ARM64.* helix queues#47919

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.

 Sign up for GitHub

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

 Merged

sdmaclea
merged 9 commits into

from

Mar 2, 2021
 Merged


Setup OSX.1100.ARM64.* helix queues
#47919
Show file tree
Hide file tree
 Changes from all commits
Commits

Show all changes


9 commits


Select commit
Hold shift + click to select a range
ddcafc8

Setup OSX.1100.ARM64.* helix queues


sdmaclea Feb 5, 2021
a1de69c

Limit OSX.1100.ARM64.* queues to CI builds until we understand capacity


sdmaclea Feb 5, 2021
783393b

For CI testing, add OSX_arm64 to the platformGroup all and gcstress


sdmaclea Feb 5, 2021
136f9a0

Run jitstress on Apple Silicon


sdmaclea Feb 24, 2021
b57b635

Enable Apple Silicon on more pipelines


sdmaclea Feb 25, 2021
6146fd2

Enable more Apple Silicon pipelines


sdmaclea Feb 25, 2021
965efb3

Fixup libraries outerloop


sdmaclea Feb 26, 2021
763770d

Disable failing Apple Silicon PAL tests


sdmaclea Feb 26, 2021
144d483

Revert "Apple Silicon disable failing PAL tests (#48782)"


sdmaclea Feb 26, 2021

File filter

Filter by extension
Filter by extension
Conversations

Failed to load comments. 
Loading
Jump to

Jump to file


Failed to load files. 
Loading
 Diff view

 Diff view


12 changes: 12 additions & 0 deletions
eng/pipelines/coreclr/ci.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -54,6 +54,10 @@ jobs:
 jobTemplate: /eng/pipelines/coreclr/templates/build-job.yml
 buildConfig: checked
 platformGroup: all
 platforms:
 # It is too early to include OSX_arm64 in platform group all
 # Adding it here will enable it also
 - OSX_arm64
 jobParameters:
 testGroup: outerloop

Expand DownExpand Up@@ -82,6 +86,10 @@ jobs:
 jobTemplate: /eng/pipelines/libraries/build-job.yml
 buildConfig: Release
 platformGroup: all
 platforms:
 # It is too early to include OSX_arm64 in platform group all
 # Adding it here will enable it also
 - OSX_arm64
 jobParameters:
 isOfficialBuild: false
 liveRuntimeBuildConfig: checked
Expand All@@ -107,6 +115,10 @@ jobs:
 jobTemplate: /eng/pipelines/common/templates/runtimes/run-test-job.yml
 buildConfig: checked
 platformGroup: all
 platforms:
 # It is too early to include OSX_arm64 in platform group all
 # Adding it here will enable it to also run this test
 - OSX_arm64
 helixQueueGroup: ci
 helixQueuesTemplate: /eng/pipelines/coreclr/templates/helix-queues-setup.yml
 jobParameters:
Expand Down

4 changes: 2 additions & 2 deletions
eng/pipelines/coreclr/ilasm.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -24,7 +24,7 @@ jobs:
 jobTemplate: /eng/pipelines/common/build-coreclr-and-libraries-job.yml
 buildConfig: checked
 platforms:
# Linux tests are built on the OSX machines.
- OSX_arm64
 - OSX_x64
 - Linux_arm
 - Linux_arm64
Expand All@@ -51,7 +51,7 @@ jobs:
 jobTemplate: /eng/pipelines/common/templates/runtimes/run-test-job.yml
 buildConfig: checked
 platforms:
# Linux tests are built on the OSX machines.
- OSX_arm64
 - OSX_x64
 - Linux_arm
 - Linux_arm64
Expand Down

1 change: 1 addition & 0 deletions
eng/pipelines/coreclr/jitrollingbuild.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -19,6 +19,7 @@ jobs:
 jobTemplate: /eng/pipelines/coreclr/templates/build-jit-job.yml
 buildConfig: checked
 platforms:
 - OSX_arm64
 - OSX_x64
 # Currently, Linux arm/arm64 machines don't have the Python 'pip3' tool, nor the azure-storage-blob package that
 # is required to do the JIT upload to Azure Storage. Thus, these platforms are disabled. If we can figure out how
Expand Down

2 changes: 2 additions & 0 deletions
eng/pipelines/coreclr/jitstress-isas-arm.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -16,6 +16,7 @@ jobs:
 buildConfig: checked
 platforms:
 - Linux_arm64
 - OSX_arm64
 - windows_arm64
 - CoreClrTestBuildHost # Either OSX_x64 or Linux_x64
 jobParameters:
Expand All@@ -37,6 +38,7 @@ jobs:
 buildConfig: checked
 platforms:
 - Linux_arm64
 - OSX_arm64
 - windows_arm64
 helixQueueGroup: ci
 helixQueuesTemplate: /eng/pipelines/coreclr/templates/helix-queues-setup.yml
Expand Down

4 changes: 2 additions & 2 deletions
eng/pipelines/coreclr/jitstress.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -21,7 +21,7 @@ jobs:
 jobTemplate: /eng/pipelines/common/build-coreclr-and-libraries-job.yml
 buildConfig: checked
 platforms:
# Linux tests are built on the OSX machines.
- OSX_arm64
 - OSX_x64
 - Linux_arm
 - Linux_arm64
Expand All@@ -48,7 +48,7 @@ jobs:
 jobTemplate: /eng/pipelines/common/templates/runtimes/run-test-job.yml
 buildConfig: checked
 platforms:
# Linux tests are built on the OSX machines.
- OSX_arm64
 - OSX_x64
 - Linux_arm
 - Linux_arm64
Expand Down

4 changes: 2 additions & 2 deletions
eng/pipelines/coreclr/jitstress2-jitstressregs.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -15,7 +15,7 @@ jobs:
 jobTemplate: /eng/pipelines/common/build-coreclr-and-libraries-job.yml
 buildConfig: checked
 platforms:
# Linux tests are built on the OSX machines.
- OSX_arm64
 - OSX_x64
 - Linux_arm
 - Linux_arm64
Expand All@@ -42,7 +42,7 @@ jobs:
 jobTemplate: /eng/pipelines/common/templates/runtimes/run-test-job.yml
 buildConfig: checked
 platforms:
# Linux tests are built on the OSX machines.
- OSX_arm64
 - OSX_x64
 - Linux_arm
 - Linux_arm64
Expand Down

4 changes: 2 additions & 2 deletions
eng/pipelines/coreclr/jitstressregs.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -15,7 +15,7 @@ jobs:
 jobTemplate: /eng/pipelines/common/build-coreclr-and-libraries-job.yml
 buildConfig: checked
 platforms:
# Linux tests are built on the OSX machines.
- OSX_arm64
 - OSX_x64
 - Linux_arm
 - Linux_arm64
Expand All@@ -42,7 +42,7 @@ jobs:
 jobTemplate: /eng/pipelines/common/templates/runtimes/run-test-job.yml
 buildConfig: checked
 platforms:
# Linux tests are built on the OSX machines.
- OSX_arm64
 - OSX_x64
 - Linux_arm
 - Linux_arm64
Expand Down

8 changes: 8 additions & 0 deletions
eng/pipelines/coreclr/release-tests.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -18,6 +18,10 @@ jobs:
 jobTemplate: /eng/pipelines/common/build-coreclr-and-libraries-job.yml
 buildConfig: release
 platformGroup: all
 platforms:
 # It is too early to include OSX_arm64 in platform group all
 # Adding it here will enable it also
 - OSX_arm64
 jobParameters:
 isOfficialBuild: false

Expand All@@ -42,6 +46,10 @@ jobs:
 jobTemplate: /eng/pipelines/common/templates/runtimes/run-test-job.yml
 buildConfig: release
 platformGroup: all
 platforms:
 # It is too early to include OSX_arm64 in platform group all
 # Adding it here will enable it also
 - OSX_arm64
 helixQueueGroup: ci
 helixQueuesTemplate: /eng/pipelines/coreclr/templates/helix-queues-setup.yml
 jobParameters:
Expand Down

7 changes: 7 additions & 0 deletions
eng/pipelines/coreclr/templates/helix-queues-setup.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -95,6 +95,13 @@ jobs:
 - (Fedora.30.Amd64)Ubuntu.1604.amd64@mcr.microsoft.com/dotnet-buildtools/prereqs:fedora-30-helix-20200512010621-4f8cef7
 - RedHat.7.Amd64

 # OSX arm64
 - ${{ if eq(parameters.platform, 'OSX_arm64') }}:
 - ${{ if and(eq(variables['System.TeamProject'], 'public'), in(parameters.jobParameters.helixQueueGroup, 'ci', 'libraries')) }}:
 - OSX.1100.ARM64.Open
 - ${{ if eq(variables['System.TeamProject'], 'internal') }}:
 - OSX.1100.ARM64

 # OSX x64
 - ${{ if eq(parameters.platform, 'OSX_x64') }}:
 - ${{ if and(eq(variables['System.TeamProject'], 'public'), in(parameters.jobParameters.helixQueueGroup, 'pr', 'ci', 'libraries')) }}:
Expand Down

5 changes: 5 additions & 0 deletions
eng/pipelines/libraries/helix-queues-setup.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -85,6 +85,11 @@ jobs:
 # Limiting interp runs as we don't need as much coverage.
 - Debian.9.Amd64.Open

 # OSX arm64
 - ${{ if eq(parameters.platform, 'OSX_arm64') }}:
Comment thread
sdmaclea marked this conversation as resolved.


Outdated

 - ${{ if eq(parameters.jobParameters.isFullMatrix, true) }}:
@safernsafernMar 1, 2021

Copy link


Copy Markdown

Member
Choose a reason for hiding this comment

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

I know we discussed this earlier, but I believe we should remove this condition for libraries and then instead condition the test runs in the yml entry point?
 - OSX.1100.ARM64.Open

 # OSX x64
 - ${{ if eq(parameters.platform, 'OSX_x64') }}:
 - ${{ if eq(parameters.jobParameters.isFullMatrix, true) }}:
Expand Down

5 changes: 5 additions & 0 deletions
eng/pipelines/libraries/outerloop.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -32,6 +32,8 @@ jobs:
 - Linux_arm64
 - Linux_musl_arm64
 - ${{ if eq(variables['includeOsxOuterloop'], true) }}: 
 - ${{ if eq(variables['isFullMatrix'], true) }}:
 - OSX_arm64
 - OSX_x64
 jobParameters:
 testGroup: innerloop
Expand All@@ -55,6 +57,7 @@ jobs:
 - Linux_musl_x64
 - Linux_musl_arm64
 - ${{ if and(eq(variables['includeOsxOuterloop'], true), eq(variables['isFullMatrix'], true)) }}:
 - OSX_arm64
 - OSX_x64
 helixQueuesTemplate: /eng/pipelines/libraries/helix-queues-setup.yml
 jobParameters:
Expand All@@ -76,6 +79,8 @@ jobs:
 - Linux_x64
 - Linux_musl_x64
 - ${{ if eq(variables['includeOsxOuterloop'], true) }}:
 - ${{ if eq(variables['isFullMatrix'], true) }}:
 - OSX_arm64
 - OSX_x64
 helixQueuesTemplate: /eng/pipelines/libraries/helix-queues-setup.yml
 jobParameters:
Expand Down

8 changes: 0 additions & 8 deletions
src/coreclr/pal/tests/palsuite/exception_handling/pal_sxs/test1/exceptionsxs.cpp

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -68,13 +68,6 @@ void sigsegv_handler(int code, siginfo_t *siginfo, void *context)

int main(int argc, char *argv[])
{
#if defined(TARGET_OSX) && defined(TARGET_ARM64)
 // Helix and/or build error...
 // dyld: Library not loaded: libpaltest_pal_sxs_test1_dll1.dylib
 printf("paltest_pal_sxs_test1 has been disabled on this platform\n");

 return PASS;
#else // defined(TARGET_OSX) && defined(TARGET_ARM64)
 struct sigaction newAction;
 struct sigaction oldAction;
 newAction.sa_flags = SA_SIGINFO | SA_RESTART;
Expand DownExpand Up@@ -120,5 +113,4 @@ int main(int argc, char *argv[])

 printf("ERROR: code was executed after the access violation.\n");
 return FAIL;
#endif // defined(TARGET_OSX) && defined(TARGET_ARM64)
}

5 changes: 0 additions & 5 deletions
src/coreclr/pal/tests/palsuite/file_io/ReadFile/test2/ReadFile.cpp

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -120,10 +120,6 @@ BOOL readTest_ReadFile_test2(DWORD dwByteCount, char cResult)

PALTEST(file_io_ReadFile_test2_paltest_readfile_test2, "file_io/ReadFile/test2/paltest_readfile_test2")
{
#if defined(TARGET_OSX) && defined(TARGET_ARM64)
 // Test hard codes PAGE_SIZE = 4096. Apple Silicon does not have 4K pages.
 printf("file_io_ReadFile_test2_paltest_readfile_test2 has been disabled on this platform\n");
#else // defined(TARGET_OSX) && defined(TARGET_ARM64)
 HANDLE hFile = NULL;
 const int BUFFER_SIZE = 2 * PAGESIZE;

Expand DownExpand Up@@ -191,7 +187,6 @@ PALTEST(file_io_ReadFile_test2_paltest_readfile_test2, "file_io/ReadFile/test2/p

	VirtualFree(readBuffer_ReadFile_test2, BUFFER_SIZE, MEM_RELEASE);
 PAL_Terminate();
#endif // defined(TARGET_OSX) && defined(TARGET_ARM64)
 return PASS;
}


5 changes: 0 additions & 5 deletions
src/coreclr/pal/tests/palsuite/filemapping_memmgt/VirtualProtect/test4/VirtualProtect.cpp

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -28,10 +28,6 @@ PALTEST(filemapping_memmgt_VirtualProtect_test4_paltest_virtualprotect_test4, "f
 ExitProcess(FAIL);
 }

#if defined(TARGET_OSX) && defined(TARGET_ARM64)
 // AppleSilcon does not allow mapping general pages as RWX
 printf("filemapping_memmgt_VirtualProtect_test4_paltest_virtualprotect_test4 has been disabled on this platform\n");
#else // defined(TARGET_OSX) && defined(TARGET_ARM64)
 //Allocate the physical storage in memory or in the paging file on disk 
 lpVirtualAddress = VirtualAlloc(NULL,//determine where to allocate the region
 REGIONSIZE, //specify the size
Expand DownExpand Up@@ -65,7 +61,6 @@ PALTEST(filemapping_memmgt_VirtualProtect_test4_paltest_virtualprotect_test4, "f
 {
 Fail("\nFailed to call VirtualFree API!\n");
 }
#endif // defined(TARGET_OSX) && defined(TARGET_ARM64)

 PAL_Terminate();
 return PASS;
Expand Down

17 changes: 17 additions & 0 deletions
src/coreclr/pal/tests/palsuite/issues.targets

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -32,4 +32,21 @@
 <Issue>https://github.com/dotnet/runtime/issues/42292</Issue>
 </ExcludeList>
 </ItemGroup>

 <ItemGroup Condition="'$(TargetArchitecture)' == 'arm64' and '$(TargetOS)' == 'OSX'">
 <ExcludeList Include="miscellaneous/queryperformancecounter/test1/paltest_queryperformancecounter_test1">
 <Issue>https://github.com/dotnet/runtime/issues/7639</Issue>
 </ExcludeList>
 <ExcludeList Include="exception_handling/pal_sxs/test1/paltest_pal_sxs_test1">
 <Issue>https://github.com/dotnet/runtime/issues/42292</Issue>
 </ExcludeList>
 <ExcludeList Include="file_io/ReadFile/test2/paltest_readfile_test2">
 <!-- Test hard codes PAGE_SIZE = 4096. Apple Silicon does not have 4K pages. -->
 <Issue>https://github.com/dotnet/runtime/issues/42292</Issue>
 </ExcludeList>
 <ExcludeList Include="filemapping_memmgt/VirtualProtect/test4/paltest_virtualprotect_test4">
 <!-- Test tries to set RWX on general memory page -->
 <Issue>https://github.com/dotnet/runtime/issues/7639</Issue>
 </ExcludeList>
 </ItemGroup>
</Project>

5 changes: 0 additions & 5 deletions
src/coreclr/pal/tests/palsuite/miscellaneous/queryperformancecounter/test1/test1.cpp

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -17,10 +17,6 @@

PALTEST(miscellaneous_queryperformancecounter_test1_paltest_queryperformancecounter_test1, "miscellaneous/queryperformancecounter/test1/paltest_queryperformancecounter_test1")
{
#if defined(TARGET_OSX) && defined(TARGET_ARM64)
 // We are see > 100 ms of average error on this platform
 printf("miscellaneous_queryperformancecounter_test1_paltest_queryperformancecounter_test1 has been disabled on this platform\n");
#else // defined(TARGET_OSX) && defined(TARGET_ARM64)
 /* Milliseconds of error which are acceptable Function execution time, etc.
 FreeBSD has a "standard" resolution of 50ms for waiting operations, so we
 must take that into account as well */
Expand DownExpand Up@@ -104,7 +100,6 @@ PALTEST(miscellaneous_queryperformancecounter_test1_paltest_queryperformancecoun
 /* Terminate the PAL.
 */ 
 PAL_Terminate();
#endif // defined(TARGET_OSX) && defined(TARGET_ARM64)
 return PASS;
}

Expand Down

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Highlight search terms from Google/DuckDuckGo/Bing referrer\n(function() {\n var ref = document.referrer;\n var terms = [];\n \n if (ref.includes('google.com') || ref.includes('duckduckgo.com') || ref.includes('bing.com')) {\n var url = new URL(ref);\n var q = url.searchParams.get('q') || url.searchParams.get('p');\n if (q) {\n terms = q.split(/\\s+/).filter(function(t) { return t.length > 2; });\n }\n }\n \n if (terms.length === 0) return;\n \n var style = document.createElement('style');\n style.textContent = '.userscript-highlight { background: #fbbf24; color: #1a1a2e; padding: 1px 3px; border-radius: 2px; }';\n document.head.appendChild(style);\n \n function highlight(node) {\n if (node.nodeType === 3) { // text node\n var text = node.textContent;\n var found = false;\n terms.forEach(function(term) {\n var regex = new RegExp('(' + term.replace(/[.*+?^${}()|[\\]\\\\]/g, '\\\\') + ')', 'gi');\n if (regex.test(text)) {\n found = true;\n var frag = document.createDocumentFragment();\n var parts = text.split(regex);\n parts.forEach(function(part, i) {\n if (i % 2 === 0) {\n frag.appendChild(document.createTextNode(part));\n } else {\n var span = document.createElement('span');\n span.className = 'userscript-highlight';\n span.textContent = part;\n frag.appendChild(span);\n }\n });\n node.parentNode.replaceChild(frag, node);\n }\n });\n } else if (node.nodeType === 1 && node.childNodes) { // element\n var skipTags = ['SCRIPT', 'STYLE', 'NOSCRIPT', 'TEXTAREA', 'INPUT', 'SELECT'];\n if (!skipTags.includes(node.tagName)) {\n Array.from(node.childNodes).forEach(highlight);\n }\n }\n }\n \n highlight(document.body);\n \n // Re-highlight on dynamic content\n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1 || node.nodeType === 3) highlight(node);\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Highlight Search Terms");
}
} catch(__e) { console.warn('[Userscript:Highlight Search Terms]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content
Setup OSX.1100.ARM64.* helix queues#47919

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.

 Sign up for GitHub

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

 Merged

sdmaclea
merged 9 commits into

from

Mar 2, 2021
 Merged


Setup OSX.1100.ARM64.* helix queues
#47919
Show file tree
Hide file tree
 Changes from all commits
Commits

Show all changes


9 commits


Select commit
Hold shift + click to select a range
ddcafc8

Setup OSX.1100.ARM64.* helix queues


sdmaclea Feb 5, 2021
a1de69c

Limit OSX.1100.ARM64.* queues to CI builds until we understand capacity


sdmaclea Feb 5, 2021
783393b

For CI testing, add OSX_arm64 to the platformGroup all and gcstress


sdmaclea Feb 5, 2021
136f9a0

Run jitstress on Apple Silicon


sdmaclea Feb 24, 2021
b57b635

Enable Apple Silicon on more pipelines


sdmaclea Feb 25, 2021
6146fd2

Enable more Apple Silicon pipelines


sdmaclea Feb 25, 2021
965efb3

Fixup libraries outerloop


sdmaclea Feb 26, 2021
763770d

Disable failing Apple Silicon PAL tests


sdmaclea Feb 26, 2021
144d483

Revert "Apple Silicon disable failing PAL tests (#48782)"


sdmaclea Feb 26, 2021

File filter

Filter by extension
Filter by extension
Conversations

Failed to load comments. 
Loading
Jump to

Jump to file


Failed to load files. 
Loading
 Diff view

 Diff view


12 changes: 12 additions & 0 deletions
eng/pipelines/coreclr/ci.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -54,6 +54,10 @@ jobs:
 jobTemplate: /eng/pipelines/coreclr/templates/build-job.yml
 buildConfig: checked
 platformGroup: all
 platforms:
 # It is too early to include OSX_arm64 in platform group all
 # Adding it here will enable it also
 - OSX_arm64
 jobParameters:
 testGroup: outerloop

Expand DownExpand Up@@ -82,6 +86,10 @@ jobs:
 jobTemplate: /eng/pipelines/libraries/build-job.yml
 buildConfig: Release
 platformGroup: all
 platforms:
 # It is too early to include OSX_arm64 in platform group all
 # Adding it here will enable it also
 - OSX_arm64
 jobParameters:
 isOfficialBuild: false
 liveRuntimeBuildConfig: checked
Expand All@@ -107,6 +115,10 @@ jobs:
 jobTemplate: /eng/pipelines/common/templates/runtimes/run-test-job.yml
 buildConfig: checked
 platformGroup: all
 platforms:
 # It is too early to include OSX_arm64 in platform group all
 # Adding it here will enable it to also run this test
 - OSX_arm64
 helixQueueGroup: ci
 helixQueuesTemplate: /eng/pipelines/coreclr/templates/helix-queues-setup.yml
 jobParameters:
Expand Down

4 changes: 2 additions & 2 deletions
eng/pipelines/coreclr/ilasm.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -24,7 +24,7 @@ jobs:
 jobTemplate: /eng/pipelines/common/build-coreclr-and-libraries-job.yml
 buildConfig: checked
 platforms:
# Linux tests are built on the OSX machines.
- OSX_arm64
 - OSX_x64
 - Linux_arm
 - Linux_arm64
Expand All@@ -51,7 +51,7 @@ jobs:
 jobTemplate: /eng/pipelines/common/templates/runtimes/run-test-job.yml
 buildConfig: checked
 platforms:
# Linux tests are built on the OSX machines.
- OSX_arm64
 - OSX_x64
 - Linux_arm
 - Linux_arm64
Expand Down

1 change: 1 addition & 0 deletions
eng/pipelines/coreclr/jitrollingbuild.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -19,6 +19,7 @@ jobs:
 jobTemplate: /eng/pipelines/coreclr/templates/build-jit-job.yml
 buildConfig: checked
 platforms:
 - OSX_arm64
 - OSX_x64
 # Currently, Linux arm/arm64 machines don't have the Python 'pip3' tool, nor the azure-storage-blob package that
 # is required to do the JIT upload to Azure Storage. Thus, these platforms are disabled. If we can figure out how
Expand Down

2 changes: 2 additions & 0 deletions
eng/pipelines/coreclr/jitstress-isas-arm.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -16,6 +16,7 @@ jobs:
 buildConfig: checked
 platforms:
 - Linux_arm64
 - OSX_arm64
 - windows_arm64
 - CoreClrTestBuildHost # Either OSX_x64 or Linux_x64
 jobParameters:
Expand All@@ -37,6 +38,7 @@ jobs:
 buildConfig: checked
 platforms:
 - Linux_arm64
 - OSX_arm64
 - windows_arm64
 helixQueueGroup: ci
 helixQueuesTemplate: /eng/pipelines/coreclr/templates/helix-queues-setup.yml
Expand Down

4 changes: 2 additions & 2 deletions
eng/pipelines/coreclr/jitstress.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -21,7 +21,7 @@ jobs:
 jobTemplate: /eng/pipelines/common/build-coreclr-and-libraries-job.yml
 buildConfig: checked
 platforms:
# Linux tests are built on the OSX machines.
- OSX_arm64
 - OSX_x64
 - Linux_arm
 - Linux_arm64
Expand All@@ -48,7 +48,7 @@ jobs:
 jobTemplate: /eng/pipelines/common/templates/runtimes/run-test-job.yml
 buildConfig: checked
 platforms:
# Linux tests are built on the OSX machines.
- OSX_arm64
 - OSX_x64
 - Linux_arm
 - Linux_arm64
Expand Down

4 changes: 2 additions & 2 deletions
eng/pipelines/coreclr/jitstress2-jitstressregs.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -15,7 +15,7 @@ jobs:
 jobTemplate: /eng/pipelines/common/build-coreclr-and-libraries-job.yml
 buildConfig: checked
 platforms:
# Linux tests are built on the OSX machines.
- OSX_arm64
 - OSX_x64
 - Linux_arm
 - Linux_arm64
Expand All@@ -42,7 +42,7 @@ jobs:
 jobTemplate: /eng/pipelines/common/templates/runtimes/run-test-job.yml
 buildConfig: checked
 platforms:
# Linux tests are built on the OSX machines.
- OSX_arm64
 - OSX_x64
 - Linux_arm
 - Linux_arm64
Expand Down

4 changes: 2 additions & 2 deletions
eng/pipelines/coreclr/jitstressregs.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -15,7 +15,7 @@ jobs:
 jobTemplate: /eng/pipelines/common/build-coreclr-and-libraries-job.yml
 buildConfig: checked
 platforms:
# Linux tests are built on the OSX machines.
- OSX_arm64
 - OSX_x64
 - Linux_arm
 - Linux_arm64
Expand All@@ -42,7 +42,7 @@ jobs:
 jobTemplate: /eng/pipelines/common/templates/runtimes/run-test-job.yml
 buildConfig: checked
 platforms:
# Linux tests are built on the OSX machines.
- OSX_arm64
 - OSX_x64
 - Linux_arm
 - Linux_arm64
Expand Down

8 changes: 8 additions & 0 deletions
eng/pipelines/coreclr/release-tests.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -18,6 +18,10 @@ jobs:
 jobTemplate: /eng/pipelines/common/build-coreclr-and-libraries-job.yml
 buildConfig: release
 platformGroup: all
 platforms:
 # It is too early to include OSX_arm64 in platform group all
 # Adding it here will enable it also
 - OSX_arm64
 jobParameters:
 isOfficialBuild: false

Expand All@@ -42,6 +46,10 @@ jobs:
 jobTemplate: /eng/pipelines/common/templates/runtimes/run-test-job.yml
 buildConfig: release
 platformGroup: all
 platforms:
 # It is too early to include OSX_arm64 in platform group all
 # Adding it here will enable it also
 - OSX_arm64
 helixQueueGroup: ci
 helixQueuesTemplate: /eng/pipelines/coreclr/templates/helix-queues-setup.yml
 jobParameters:
Expand Down

7 changes: 7 additions & 0 deletions
eng/pipelines/coreclr/templates/helix-queues-setup.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -95,6 +95,13 @@ jobs:
 - (Fedora.30.Amd64)Ubuntu.1604.amd64@mcr.microsoft.com/dotnet-buildtools/prereqs:fedora-30-helix-20200512010621-4f8cef7
 - RedHat.7.Amd64

 # OSX arm64
 - ${{ if eq(parameters.platform, 'OSX_arm64') }}:
 - ${{ if and(eq(variables['System.TeamProject'], 'public'), in(parameters.jobParameters.helixQueueGroup, 'ci', 'libraries')) }}:
 - OSX.1100.ARM64.Open
 - ${{ if eq(variables['System.TeamProject'], 'internal') }}:
 - OSX.1100.ARM64

 # OSX x64
 - ${{ if eq(parameters.platform, 'OSX_x64') }}:
 - ${{ if and(eq(variables['System.TeamProject'], 'public'), in(parameters.jobParameters.helixQueueGroup, 'pr', 'ci', 'libraries')) }}:
Expand Down

5 changes: 5 additions & 0 deletions
eng/pipelines/libraries/helix-queues-setup.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -85,6 +85,11 @@ jobs:
 # Limiting interp runs as we don't need as much coverage.
 - Debian.9.Amd64.Open

 # OSX arm64
 - ${{ if eq(parameters.platform, 'OSX_arm64') }}:
Comment thread
sdmaclea marked this conversation as resolved.


Outdated

 - ${{ if eq(parameters.jobParameters.isFullMatrix, true) }}:
@safernsafernMar 1, 2021

Copy link


Copy Markdown

Member
Choose a reason for hiding this comment

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

I know we discussed this earlier, but I believe we should remove this condition for libraries and then instead condition the test runs in the yml entry point?
 - OSX.1100.ARM64.Open

 # OSX x64
 - ${{ if eq(parameters.platform, 'OSX_x64') }}:
 - ${{ if eq(parameters.jobParameters.isFullMatrix, true) }}:
Expand Down

5 changes: 5 additions & 0 deletions
eng/pipelines/libraries/outerloop.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -32,6 +32,8 @@ jobs:
 - Linux_arm64
 - Linux_musl_arm64
 - ${{ if eq(variables['includeOsxOuterloop'], true) }}: 
 - ${{ if eq(variables['isFullMatrix'], true) }}:
 - OSX_arm64
 - OSX_x64
 jobParameters:
 testGroup: innerloop
Expand All@@ -55,6 +57,7 @@ jobs:
 - Linux_musl_x64
 - Linux_musl_arm64
 - ${{ if and(eq(variables['includeOsxOuterloop'], true), eq(variables['isFullMatrix'], true)) }}:
 - OSX_arm64
 - OSX_x64
 helixQueuesTemplate: /eng/pipelines/libraries/helix-queues-setup.yml
 jobParameters:
Expand All@@ -76,6 +79,8 @@ jobs:
 - Linux_x64
 - Linux_musl_x64
 - ${{ if eq(variables['includeOsxOuterloop'], true) }}:
 - ${{ if eq(variables['isFullMatrix'], true) }}:
 - OSX_arm64
 - OSX_x64
 helixQueuesTemplate: /eng/pipelines/libraries/helix-queues-setup.yml
 jobParameters:
Expand Down

8 changes: 0 additions & 8 deletions
src/coreclr/pal/tests/palsuite/exception_handling/pal_sxs/test1/exceptionsxs.cpp

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -68,13 +68,6 @@ void sigsegv_handler(int code, siginfo_t *siginfo, void *context)

int main(int argc, char *argv[])
{
#if defined(TARGET_OSX) && defined(TARGET_ARM64)
 // Helix and/or build error...
 // dyld: Library not loaded: libpaltest_pal_sxs_test1_dll1.dylib
 printf("paltest_pal_sxs_test1 has been disabled on this platform\n");

 return PASS;
#else // defined(TARGET_OSX) && defined(TARGET_ARM64)
 struct sigaction newAction;
 struct sigaction oldAction;
 newAction.sa_flags = SA_SIGINFO | SA_RESTART;
Expand DownExpand Up@@ -120,5 +113,4 @@ int main(int argc, char *argv[])

 printf("ERROR: code was executed after the access violation.\n");
 return FAIL;
#endif // defined(TARGET_OSX) && defined(TARGET_ARM64)
}

5 changes: 0 additions & 5 deletions
src/coreclr/pal/tests/palsuite/file_io/ReadFile/test2/ReadFile.cpp

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -120,10 +120,6 @@ BOOL readTest_ReadFile_test2(DWORD dwByteCount, char cResult)

PALTEST(file_io_ReadFile_test2_paltest_readfile_test2, "file_io/ReadFile/test2/paltest_readfile_test2")
{
#if defined(TARGET_OSX) && defined(TARGET_ARM64)
 // Test hard codes PAGE_SIZE = 4096. Apple Silicon does not have 4K pages.
 printf("file_io_ReadFile_test2_paltest_readfile_test2 has been disabled on this platform\n");
#else // defined(TARGET_OSX) && defined(TARGET_ARM64)
 HANDLE hFile = NULL;
 const int BUFFER_SIZE = 2 * PAGESIZE;

Expand DownExpand Up@@ -191,7 +187,6 @@ PALTEST(file_io_ReadFile_test2_paltest_readfile_test2, "file_io/ReadFile/test2/p

	VirtualFree(readBuffer_ReadFile_test2, BUFFER_SIZE, MEM_RELEASE);
 PAL_Terminate();
#endif // defined(TARGET_OSX) && defined(TARGET_ARM64)
 return PASS;
}


5 changes: 0 additions & 5 deletions
src/coreclr/pal/tests/palsuite/filemapping_memmgt/VirtualProtect/test4/VirtualProtect.cpp

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -28,10 +28,6 @@ PALTEST(filemapping_memmgt_VirtualProtect_test4_paltest_virtualprotect_test4, "f
 ExitProcess(FAIL);
 }

#if defined(TARGET_OSX) && defined(TARGET_ARM64)
 // AppleSilcon does not allow mapping general pages as RWX
 printf("filemapping_memmgt_VirtualProtect_test4_paltest_virtualprotect_test4 has been disabled on this platform\n");
#else // defined(TARGET_OSX) && defined(TARGET_ARM64)
 //Allocate the physical storage in memory or in the paging file on disk 
 lpVirtualAddress = VirtualAlloc(NULL,//determine where to allocate the region
 REGIONSIZE, //specify the size
Expand DownExpand Up@@ -65,7 +61,6 @@ PALTEST(filemapping_memmgt_VirtualProtect_test4_paltest_virtualprotect_test4, "f
 {
 Fail("\nFailed to call VirtualFree API!\n");
 }
#endif // defined(TARGET_OSX) && defined(TARGET_ARM64)

 PAL_Terminate();
 return PASS;
Expand Down

17 changes: 17 additions & 0 deletions
src/coreclr/pal/tests/palsuite/issues.targets

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -32,4 +32,21 @@
 <Issue>https://github.com/dotnet/runtime/issues/42292</Issue>
 </ExcludeList>
 </ItemGroup>

 <ItemGroup Condition="'$(TargetArchitecture)' == 'arm64' and '$(TargetOS)' == 'OSX'">
 <ExcludeList Include="miscellaneous/queryperformancecounter/test1/paltest_queryperformancecounter_test1">
 <Issue>https://github.com/dotnet/runtime/issues/7639</Issue>
 </ExcludeList>
 <ExcludeList Include="exception_handling/pal_sxs/test1/paltest_pal_sxs_test1">
 <Issue>https://github.com/dotnet/runtime/issues/42292</Issue>
 </ExcludeList>
 <ExcludeList Include="file_io/ReadFile/test2/paltest_readfile_test2">
 <!-- Test hard codes PAGE_SIZE = 4096. Apple Silicon does not have 4K pages. -->
 <Issue>https://github.com/dotnet/runtime/issues/42292</Issue>
 </ExcludeList>
 <ExcludeList Include="filemapping_memmgt/VirtualProtect/test4/paltest_virtualprotect_test4">
 <!-- Test tries to set RWX on general memory page -->
 <Issue>https://github.com/dotnet/runtime/issues/7639</Issue>
 </ExcludeList>
 </ItemGroup>
</Project>

5 changes: 0 additions & 5 deletions
src/coreclr/pal/tests/palsuite/miscellaneous/queryperformancecounter/test1/test1.cpp

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -17,10 +17,6 @@

PALTEST(miscellaneous_queryperformancecounter_test1_paltest_queryperformancecounter_test1, "miscellaneous/queryperformancecounter/test1/paltest_queryperformancecounter_test1")
{
#if defined(TARGET_OSX) && defined(TARGET_ARM64)
 // We are see > 100 ms of average error on this platform
 printf("miscellaneous_queryperformancecounter_test1_paltest_queryperformancecounter_test1 has been disabled on this platform\n");
#else // defined(TARGET_OSX) && defined(TARGET_ARM64)
 /* Milliseconds of error which are acceptable Function execution time, etc.
 FreeBSD has a "standard" resolution of 50ms for waiting operations, so we
 must take that into account as well */
Expand DownExpand Up@@ -104,7 +100,6 @@ PALTEST(miscellaneous_queryperformancecounter_test1_paltest_queryperformancecoun
 /* Terminate the PAL.
 */ 
 PAL_Terminate();
#endif // defined(TARGET_OSX) && defined(TARGET_ARM64)
 return PASS;
}

Expand Down

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Strip utm_, fbclid, gclid, etc. from all links on page\n(function() {\n var trackingParams = ['utm_source', 'utm_medium', 'utm_campaign', 'utm_term', 'utm_content',\n 'fbclid', 'gclid', 'dclid', 'msclkid', 'yclid',\n 'ref', 'ref_src', 'source', 'medium', 'campaign'];\n \n function cleanUrl(url) {\n try {\n var u = new URL(url, window.location.origin);\n var changed = false;\n trackingParams.forEach(function(p) {\n if (u.searchParams.has(p)) {\n u.searchParams.delete(p);\n changed = true;\n }\n });\n return changed ? u.toString() : url;\n } catch (e) {\n return url;\n }\n }\n \n function cleanLinks() {\n document.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n \n cleanLinks();\n \n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1) {\n if (node.tagName === 'A') cleanLinks();\n node.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Remove Tracking Parameters from Links");
}
} catch(__e) { console.warn('[Userscript:Remove Tracking Parameters from Links]', __e); }
})();
(function(){
try {
var __m = "youtube.com";
var __re = new RegExp('^' + "youtube\\.com" + '

Skip to content
Setup OSX.1100.ARM64.* helix queues#47919

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.

 Sign up for GitHub

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

 Merged

sdmaclea
merged 9 commits into

from

Mar 2, 2021
 Merged


Setup OSX.1100.ARM64.* helix queues
#47919
Show file tree
Hide file tree
 Changes from all commits
Commits

Show all changes


9 commits


Select commit
Hold shift + click to select a range
ddcafc8

Setup OSX.1100.ARM64.* helix queues


sdmaclea Feb 5, 2021
a1de69c

Limit OSX.1100.ARM64.* queues to CI builds until we understand capacity


sdmaclea Feb 5, 2021
783393b

For CI testing, add OSX_arm64 to the platformGroup all and gcstress


sdmaclea Feb 5, 2021
136f9a0

Run jitstress on Apple Silicon


sdmaclea Feb 24, 2021
b57b635

Enable Apple Silicon on more pipelines


sdmaclea Feb 25, 2021
6146fd2

Enable more Apple Silicon pipelines


sdmaclea Feb 25, 2021
965efb3

Fixup libraries outerloop


sdmaclea Feb 26, 2021
763770d

Disable failing Apple Silicon PAL tests


sdmaclea Feb 26, 2021
144d483

Revert "Apple Silicon disable failing PAL tests (#48782)"


sdmaclea Feb 26, 2021

File filter

Filter by extension
Filter by extension
Conversations

Failed to load comments. 
Loading
Jump to

Jump to file


Failed to load files. 
Loading
 Diff view

 Diff view


12 changes: 12 additions & 0 deletions
eng/pipelines/coreclr/ci.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -54,6 +54,10 @@ jobs:
 jobTemplate: /eng/pipelines/coreclr/templates/build-job.yml
 buildConfig: checked
 platformGroup: all
 platforms:
 # It is too early to include OSX_arm64 in platform group all
 # Adding it here will enable it also
 - OSX_arm64
 jobParameters:
 testGroup: outerloop

Expand DownExpand Up@@ -82,6 +86,10 @@ jobs:
 jobTemplate: /eng/pipelines/libraries/build-job.yml
 buildConfig: Release
 platformGroup: all
 platforms:
 # It is too early to include OSX_arm64 in platform group all
 # Adding it here will enable it also
 - OSX_arm64
 jobParameters:
 isOfficialBuild: false
 liveRuntimeBuildConfig: checked
Expand All@@ -107,6 +115,10 @@ jobs:
 jobTemplate: /eng/pipelines/common/templates/runtimes/run-test-job.yml
 buildConfig: checked
 platformGroup: all
 platforms:
 # It is too early to include OSX_arm64 in platform group all
 # Adding it here will enable it to also run this test
 - OSX_arm64
 helixQueueGroup: ci
 helixQueuesTemplate: /eng/pipelines/coreclr/templates/helix-queues-setup.yml
 jobParameters:
Expand Down

4 changes: 2 additions & 2 deletions
eng/pipelines/coreclr/ilasm.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -24,7 +24,7 @@ jobs:
 jobTemplate: /eng/pipelines/common/build-coreclr-and-libraries-job.yml
 buildConfig: checked
 platforms:
# Linux tests are built on the OSX machines.
- OSX_arm64
 - OSX_x64
 - Linux_arm
 - Linux_arm64
Expand All@@ -51,7 +51,7 @@ jobs:
 jobTemplate: /eng/pipelines/common/templates/runtimes/run-test-job.yml
 buildConfig: checked
 platforms:
# Linux tests are built on the OSX machines.
- OSX_arm64
 - OSX_x64
 - Linux_arm
 - Linux_arm64
Expand Down

1 change: 1 addition & 0 deletions
eng/pipelines/coreclr/jitrollingbuild.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -19,6 +19,7 @@ jobs:
 jobTemplate: /eng/pipelines/coreclr/templates/build-jit-job.yml
 buildConfig: checked
 platforms:
 - OSX_arm64
 - OSX_x64
 # Currently, Linux arm/arm64 machines don't have the Python 'pip3' tool, nor the azure-storage-blob package that
 # is required to do the JIT upload to Azure Storage. Thus, these platforms are disabled. If we can figure out how
Expand Down

2 changes: 2 additions & 0 deletions
eng/pipelines/coreclr/jitstress-isas-arm.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -16,6 +16,7 @@ jobs:
 buildConfig: checked
 platforms:
 - Linux_arm64
 - OSX_arm64
 - windows_arm64
 - CoreClrTestBuildHost # Either OSX_x64 or Linux_x64
 jobParameters:
Expand All@@ -37,6 +38,7 @@ jobs:
 buildConfig: checked
 platforms:
 - Linux_arm64
 - OSX_arm64
 - windows_arm64
 helixQueueGroup: ci
 helixQueuesTemplate: /eng/pipelines/coreclr/templates/helix-queues-setup.yml
Expand Down

4 changes: 2 additions & 2 deletions
eng/pipelines/coreclr/jitstress.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -21,7 +21,7 @@ jobs:
 jobTemplate: /eng/pipelines/common/build-coreclr-and-libraries-job.yml
 buildConfig: checked
 platforms:
# Linux tests are built on the OSX machines.
- OSX_arm64
 - OSX_x64
 - Linux_arm
 - Linux_arm64
Expand All@@ -48,7 +48,7 @@ jobs:
 jobTemplate: /eng/pipelines/common/templates/runtimes/run-test-job.yml
 buildConfig: checked
 platforms:
# Linux tests are built on the OSX machines.
- OSX_arm64
 - OSX_x64
 - Linux_arm
 - Linux_arm64
Expand Down

4 changes: 2 additions & 2 deletions
eng/pipelines/coreclr/jitstress2-jitstressregs.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -15,7 +15,7 @@ jobs:
 jobTemplate: /eng/pipelines/common/build-coreclr-and-libraries-job.yml
 buildConfig: checked
 platforms:
# Linux tests are built on the OSX machines.
- OSX_arm64
 - OSX_x64
 - Linux_arm
 - Linux_arm64
Expand All@@ -42,7 +42,7 @@ jobs:
 jobTemplate: /eng/pipelines/common/templates/runtimes/run-test-job.yml
 buildConfig: checked
 platforms:
# Linux tests are built on the OSX machines.
- OSX_arm64
 - OSX_x64
 - Linux_arm
 - Linux_arm64
Expand Down

4 changes: 2 additions & 2 deletions
eng/pipelines/coreclr/jitstressregs.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -15,7 +15,7 @@ jobs:
 jobTemplate: /eng/pipelines/common/build-coreclr-and-libraries-job.yml
 buildConfig: checked
 platforms:
# Linux tests are built on the OSX machines.
- OSX_arm64
 - OSX_x64
 - Linux_arm
 - Linux_arm64
Expand All@@ -42,7 +42,7 @@ jobs:
 jobTemplate: /eng/pipelines/common/templates/runtimes/run-test-job.yml
 buildConfig: checked
 platforms:
# Linux tests are built on the OSX machines.
- OSX_arm64
 - OSX_x64
 - Linux_arm
 - Linux_arm64
Expand Down

8 changes: 8 additions & 0 deletions
eng/pipelines/coreclr/release-tests.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -18,6 +18,10 @@ jobs:
 jobTemplate: /eng/pipelines/common/build-coreclr-and-libraries-job.yml
 buildConfig: release
 platformGroup: all
 platforms:
 # It is too early to include OSX_arm64 in platform group all
 # Adding it here will enable it also
 - OSX_arm64
 jobParameters:
 isOfficialBuild: false

Expand All@@ -42,6 +46,10 @@ jobs:
 jobTemplate: /eng/pipelines/common/templates/runtimes/run-test-job.yml
 buildConfig: release
 platformGroup: all
 platforms:
 # It is too early to include OSX_arm64 in platform group all
 # Adding it here will enable it also
 - OSX_arm64
 helixQueueGroup: ci
 helixQueuesTemplate: /eng/pipelines/coreclr/templates/helix-queues-setup.yml
 jobParameters:
Expand Down

7 changes: 7 additions & 0 deletions
eng/pipelines/coreclr/templates/helix-queues-setup.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -95,6 +95,13 @@ jobs:
 - (Fedora.30.Amd64)Ubuntu.1604.amd64@mcr.microsoft.com/dotnet-buildtools/prereqs:fedora-30-helix-20200512010621-4f8cef7
 - RedHat.7.Amd64

 # OSX arm64
 - ${{ if eq(parameters.platform, 'OSX_arm64') }}:
 - ${{ if and(eq(variables['System.TeamProject'], 'public'), in(parameters.jobParameters.helixQueueGroup, 'ci', 'libraries')) }}:
 - OSX.1100.ARM64.Open
 - ${{ if eq(variables['System.TeamProject'], 'internal') }}:
 - OSX.1100.ARM64

 # OSX x64
 - ${{ if eq(parameters.platform, 'OSX_x64') }}:
 - ${{ if and(eq(variables['System.TeamProject'], 'public'), in(parameters.jobParameters.helixQueueGroup, 'pr', 'ci', 'libraries')) }}:
Expand Down

5 changes: 5 additions & 0 deletions
eng/pipelines/libraries/helix-queues-setup.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -85,6 +85,11 @@ jobs:
 # Limiting interp runs as we don't need as much coverage.
 - Debian.9.Amd64.Open

 # OSX arm64
 - ${{ if eq(parameters.platform, 'OSX_arm64') }}:
Comment thread
sdmaclea marked this conversation as resolved.


Outdated

 - ${{ if eq(parameters.jobParameters.isFullMatrix, true) }}:
@safernsafernMar 1, 2021

Copy link


Copy Markdown

Member
Choose a reason for hiding this comment

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

I know we discussed this earlier, but I believe we should remove this condition for libraries and then instead condition the test runs in the yml entry point?
 - OSX.1100.ARM64.Open

 # OSX x64
 - ${{ if eq(parameters.platform, 'OSX_x64') }}:
 - ${{ if eq(parameters.jobParameters.isFullMatrix, true) }}:
Expand Down

5 changes: 5 additions & 0 deletions
eng/pipelines/libraries/outerloop.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -32,6 +32,8 @@ jobs:
 - Linux_arm64
 - Linux_musl_arm64
 - ${{ if eq(variables['includeOsxOuterloop'], true) }}: 
 - ${{ if eq(variables['isFullMatrix'], true) }}:
 - OSX_arm64
 - OSX_x64
 jobParameters:
 testGroup: innerloop
Expand All@@ -55,6 +57,7 @@ jobs:
 - Linux_musl_x64
 - Linux_musl_arm64
 - ${{ if and(eq(variables['includeOsxOuterloop'], true), eq(variables['isFullMatrix'], true)) }}:
 - OSX_arm64
 - OSX_x64
 helixQueuesTemplate: /eng/pipelines/libraries/helix-queues-setup.yml
 jobParameters:
Expand All@@ -76,6 +79,8 @@ jobs:
 - Linux_x64
 - Linux_musl_x64
 - ${{ if eq(variables['includeOsxOuterloop'], true) }}:
 - ${{ if eq(variables['isFullMatrix'], true) }}:
 - OSX_arm64
 - OSX_x64
 helixQueuesTemplate: /eng/pipelines/libraries/helix-queues-setup.yml
 jobParameters:
Expand Down

8 changes: 0 additions & 8 deletions
src/coreclr/pal/tests/palsuite/exception_handling/pal_sxs/test1/exceptionsxs.cpp

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -68,13 +68,6 @@ void sigsegv_handler(int code, siginfo_t *siginfo, void *context)

int main(int argc, char *argv[])
{
#if defined(TARGET_OSX) && defined(TARGET_ARM64)
 // Helix and/or build error...
 // dyld: Library not loaded: libpaltest_pal_sxs_test1_dll1.dylib
 printf("paltest_pal_sxs_test1 has been disabled on this platform\n");

 return PASS;
#else // defined(TARGET_OSX) && defined(TARGET_ARM64)
 struct sigaction newAction;
 struct sigaction oldAction;
 newAction.sa_flags = SA_SIGINFO | SA_RESTART;
Expand DownExpand Up@@ -120,5 +113,4 @@ int main(int argc, char *argv[])

 printf("ERROR: code was executed after the access violation.\n");
 return FAIL;
#endif // defined(TARGET_OSX) && defined(TARGET_ARM64)
}

5 changes: 0 additions & 5 deletions
src/coreclr/pal/tests/palsuite/file_io/ReadFile/test2/ReadFile.cpp

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -120,10 +120,6 @@ BOOL readTest_ReadFile_test2(DWORD dwByteCount, char cResult)

PALTEST(file_io_ReadFile_test2_paltest_readfile_test2, "file_io/ReadFile/test2/paltest_readfile_test2")
{
#if defined(TARGET_OSX) && defined(TARGET_ARM64)
 // Test hard codes PAGE_SIZE = 4096. Apple Silicon does not have 4K pages.
 printf("file_io_ReadFile_test2_paltest_readfile_test2 has been disabled on this platform\n");
#else // defined(TARGET_OSX) && defined(TARGET_ARM64)
 HANDLE hFile = NULL;
 const int BUFFER_SIZE = 2 * PAGESIZE;

Expand DownExpand Up@@ -191,7 +187,6 @@ PALTEST(file_io_ReadFile_test2_paltest_readfile_test2, "file_io/ReadFile/test2/p

	VirtualFree(readBuffer_ReadFile_test2, BUFFER_SIZE, MEM_RELEASE);
 PAL_Terminate();
#endif // defined(TARGET_OSX) && defined(TARGET_ARM64)
 return PASS;
}


5 changes: 0 additions & 5 deletions
src/coreclr/pal/tests/palsuite/filemapping_memmgt/VirtualProtect/test4/VirtualProtect.cpp

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -28,10 +28,6 @@ PALTEST(filemapping_memmgt_VirtualProtect_test4_paltest_virtualprotect_test4, "f
 ExitProcess(FAIL);
 }

#if defined(TARGET_OSX) && defined(TARGET_ARM64)
 // AppleSilcon does not allow mapping general pages as RWX
 printf("filemapping_memmgt_VirtualProtect_test4_paltest_virtualprotect_test4 has been disabled on this platform\n");
#else // defined(TARGET_OSX) && defined(TARGET_ARM64)
 //Allocate the physical storage in memory or in the paging file on disk 
 lpVirtualAddress = VirtualAlloc(NULL,//determine where to allocate the region
 REGIONSIZE, //specify the size
Expand DownExpand Up@@ -65,7 +61,6 @@ PALTEST(filemapping_memmgt_VirtualProtect_test4_paltest_virtualprotect_test4, "f
 {
 Fail("\nFailed to call VirtualFree API!\n");
 }
#endif // defined(TARGET_OSX) && defined(TARGET_ARM64)

 PAL_Terminate();
 return PASS;
Expand Down

17 changes: 17 additions & 0 deletions
src/coreclr/pal/tests/palsuite/issues.targets

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -32,4 +32,21 @@
 <Issue>https://github.com/dotnet/runtime/issues/42292</Issue>
 </ExcludeList>
 </ItemGroup>

 <ItemGroup Condition="'$(TargetArchitecture)' == 'arm64' and '$(TargetOS)' == 'OSX'">
 <ExcludeList Include="miscellaneous/queryperformancecounter/test1/paltest_queryperformancecounter_test1">
 <Issue>https://github.com/dotnet/runtime/issues/7639</Issue>
 </ExcludeList>
 <ExcludeList Include="exception_handling/pal_sxs/test1/paltest_pal_sxs_test1">
 <Issue>https://github.com/dotnet/runtime/issues/42292</Issue>
 </ExcludeList>
 <ExcludeList Include="file_io/ReadFile/test2/paltest_readfile_test2">
 <!-- Test hard codes PAGE_SIZE = 4096. Apple Silicon does not have 4K pages. -->
 <Issue>https://github.com/dotnet/runtime/issues/42292</Issue>
 </ExcludeList>
 <ExcludeList Include="filemapping_memmgt/VirtualProtect/test4/paltest_virtualprotect_test4">
 <!-- Test tries to set RWX on general memory page -->
 <Issue>https://github.com/dotnet/runtime/issues/7639</Issue>
 </ExcludeList>
 </ItemGroup>
</Project>

5 changes: 0 additions & 5 deletions
src/coreclr/pal/tests/palsuite/miscellaneous/queryperformancecounter/test1/test1.cpp

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -17,10 +17,6 @@

PALTEST(miscellaneous_queryperformancecounter_test1_paltest_queryperformancecounter_test1, "miscellaneous/queryperformancecounter/test1/paltest_queryperformancecounter_test1")
{
#if defined(TARGET_OSX) && defined(TARGET_ARM64)
 // We are see > 100 ms of average error on this platform
 printf("miscellaneous_queryperformancecounter_test1_paltest_queryperformancecounter_test1 has been disabled on this platform\n");
#else // defined(TARGET_OSX) && defined(TARGET_ARM64)
 /* Milliseconds of error which are acceptable Function execution time, etc.
 FreeBSD has a "standard" resolution of 50ms for waiting operations, so we
 must take that into account as well */
Expand DownExpand Up@@ -104,7 +100,6 @@ PALTEST(miscellaneous_queryperformancecounter_test1_paltest_queryperformancecoun
 /* Terminate the PAL.
 */ 
 PAL_Terminate();
#endif // defined(TARGET_OSX) && defined(TARGET_ARM64)
 return PASS;
}

Expand Down

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Auto-enable theater mode on YouTube\n(function() {\n function tryTheater() {\n var btn = document.querySelector('button[aria-label=\"Theater mode\"], ytd-player #player button[title=\"Theater mode\"]');\n if (btn && !btn.classList.contains('activated')) {\n btn.click();\n }\n }\n \n // Try immediately\n tryTheater();\n \n // Try after navigation (SPA)\n var lastUrl = location.href;\n setInterval(function() {\n if (location.href !== lastUrl) {\n lastUrl = location.href;\n setTimeout(tryTheater, 500);\n }\n }, 1000);\n \n // Also try on player load\n var observer = new MutationObserver(tryTheater);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "YouTube Theater Mode Default");
}
} catch(__e) { console.warn('[Userscript:YouTube Theater Mode Default]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content
Setup OSX.1100.ARM64.* helix queues#47919

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.

 Sign up for GitHub

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

 Merged

sdmaclea
merged 9 commits into

from

Mar 2, 2021
 Merged


Setup OSX.1100.ARM64.* helix queues
#47919
Show file tree
Hide file tree
 Changes from all commits
Commits

Show all changes


9 commits


Select commit
Hold shift + click to select a range
ddcafc8

Setup OSX.1100.ARM64.* helix queues


sdmaclea Feb 5, 2021
a1de69c

Limit OSX.1100.ARM64.* queues to CI builds until we understand capacity


sdmaclea Feb 5, 2021
783393b

For CI testing, add OSX_arm64 to the platformGroup all and gcstress


sdmaclea Feb 5, 2021
136f9a0

Run jitstress on Apple Silicon


sdmaclea Feb 24, 2021
b57b635

Enable Apple Silicon on more pipelines


sdmaclea Feb 25, 2021
6146fd2

Enable more Apple Silicon pipelines


sdmaclea Feb 25, 2021
965efb3

Fixup libraries outerloop


sdmaclea Feb 26, 2021
763770d

Disable failing Apple Silicon PAL tests


sdmaclea Feb 26, 2021
144d483

Revert "Apple Silicon disable failing PAL tests (#48782)"


sdmaclea Feb 26, 2021

File filter

Filter by extension
Filter by extension
Conversations

Failed to load comments. 
Loading
Jump to

Jump to file


Failed to load files. 
Loading
 Diff view

 Diff view


12 changes: 12 additions & 0 deletions
eng/pipelines/coreclr/ci.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -54,6 +54,10 @@ jobs:
 jobTemplate: /eng/pipelines/coreclr/templates/build-job.yml
 buildConfig: checked
 platformGroup: all
 platforms:
 # It is too early to include OSX_arm64 in platform group all
 # Adding it here will enable it also
 - OSX_arm64
 jobParameters:
 testGroup: outerloop

Expand DownExpand Up@@ -82,6 +86,10 @@ jobs:
 jobTemplate: /eng/pipelines/libraries/build-job.yml
 buildConfig: Release
 platformGroup: all
 platforms:
 # It is too early to include OSX_arm64 in platform group all
 # Adding it here will enable it also
 - OSX_arm64
 jobParameters:
 isOfficialBuild: false
 liveRuntimeBuildConfig: checked
Expand All@@ -107,6 +115,10 @@ jobs:
 jobTemplate: /eng/pipelines/common/templates/runtimes/run-test-job.yml
 buildConfig: checked
 platformGroup: all
 platforms:
 # It is too early to include OSX_arm64 in platform group all
 # Adding it here will enable it to also run this test
 - OSX_arm64
 helixQueueGroup: ci
 helixQueuesTemplate: /eng/pipelines/coreclr/templates/helix-queues-setup.yml
 jobParameters:
Expand Down

4 changes: 2 additions & 2 deletions
eng/pipelines/coreclr/ilasm.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -24,7 +24,7 @@ jobs:
 jobTemplate: /eng/pipelines/common/build-coreclr-and-libraries-job.yml
 buildConfig: checked
 platforms:
# Linux tests are built on the OSX machines.
- OSX_arm64
 - OSX_x64
 - Linux_arm
 - Linux_arm64
Expand All@@ -51,7 +51,7 @@ jobs:
 jobTemplate: /eng/pipelines/common/templates/runtimes/run-test-job.yml
 buildConfig: checked
 platforms:
# Linux tests are built on the OSX machines.
- OSX_arm64
 - OSX_x64
 - Linux_arm
 - Linux_arm64
Expand Down

1 change: 1 addition & 0 deletions
eng/pipelines/coreclr/jitrollingbuild.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -19,6 +19,7 @@ jobs:
 jobTemplate: /eng/pipelines/coreclr/templates/build-jit-job.yml
 buildConfig: checked
 platforms:
 - OSX_arm64
 - OSX_x64
 # Currently, Linux arm/arm64 machines don't have the Python 'pip3' tool, nor the azure-storage-blob package that
 # is required to do the JIT upload to Azure Storage. Thus, these platforms are disabled. If we can figure out how
Expand Down

2 changes: 2 additions & 0 deletions
eng/pipelines/coreclr/jitstress-isas-arm.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -16,6 +16,7 @@ jobs:
 buildConfig: checked
 platforms:
 - Linux_arm64
 - OSX_arm64
 - windows_arm64
 - CoreClrTestBuildHost # Either OSX_x64 or Linux_x64
 jobParameters:
Expand All@@ -37,6 +38,7 @@ jobs:
 buildConfig: checked
 platforms:
 - Linux_arm64
 - OSX_arm64
 - windows_arm64
 helixQueueGroup: ci
 helixQueuesTemplate: /eng/pipelines/coreclr/templates/helix-queues-setup.yml
Expand Down

4 changes: 2 additions & 2 deletions
eng/pipelines/coreclr/jitstress.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -21,7 +21,7 @@ jobs:
 jobTemplate: /eng/pipelines/common/build-coreclr-and-libraries-job.yml
 buildConfig: checked
 platforms:
# Linux tests are built on the OSX machines.
- OSX_arm64
 - OSX_x64
 - Linux_arm
 - Linux_arm64
Expand All@@ -48,7 +48,7 @@ jobs:
 jobTemplate: /eng/pipelines/common/templates/runtimes/run-test-job.yml
 buildConfig: checked
 platforms:
# Linux tests are built on the OSX machines.
- OSX_arm64
 - OSX_x64
 - Linux_arm
 - Linux_arm64
Expand Down

4 changes: 2 additions & 2 deletions
eng/pipelines/coreclr/jitstress2-jitstressregs.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -15,7 +15,7 @@ jobs:
 jobTemplate: /eng/pipelines/common/build-coreclr-and-libraries-job.yml
 buildConfig: checked
 platforms:
# Linux tests are built on the OSX machines.
- OSX_arm64
 - OSX_x64
 - Linux_arm
 - Linux_arm64
Expand All@@ -42,7 +42,7 @@ jobs:
 jobTemplate: /eng/pipelines/common/templates/runtimes/run-test-job.yml
 buildConfig: checked
 platforms:
# Linux tests are built on the OSX machines.
- OSX_arm64
 - OSX_x64
 - Linux_arm
 - Linux_arm64
Expand Down

4 changes: 2 additions & 2 deletions
eng/pipelines/coreclr/jitstressregs.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -15,7 +15,7 @@ jobs:
 jobTemplate: /eng/pipelines/common/build-coreclr-and-libraries-job.yml
 buildConfig: checked
 platforms:
# Linux tests are built on the OSX machines.
- OSX_arm64
 - OSX_x64
 - Linux_arm
 - Linux_arm64
Expand All@@ -42,7 +42,7 @@ jobs:
 jobTemplate: /eng/pipelines/common/templates/runtimes/run-test-job.yml
 buildConfig: checked
 platforms:
# Linux tests are built on the OSX machines.
- OSX_arm64
 - OSX_x64
 - Linux_arm
 - Linux_arm64
Expand Down

8 changes: 8 additions & 0 deletions
eng/pipelines/coreclr/release-tests.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -18,6 +18,10 @@ jobs:
 jobTemplate: /eng/pipelines/common/build-coreclr-and-libraries-job.yml
 buildConfig: release
 platformGroup: all
 platforms:
 # It is too early to include OSX_arm64 in platform group all
 # Adding it here will enable it also
 - OSX_arm64
 jobParameters:
 isOfficialBuild: false

Expand All@@ -42,6 +46,10 @@ jobs:
 jobTemplate: /eng/pipelines/common/templates/runtimes/run-test-job.yml
 buildConfig: release
 platformGroup: all
 platforms:
 # It is too early to include OSX_arm64 in platform group all
 # Adding it here will enable it also
 - OSX_arm64
 helixQueueGroup: ci
 helixQueuesTemplate: /eng/pipelines/coreclr/templates/helix-queues-setup.yml
 jobParameters:
Expand Down

7 changes: 7 additions & 0 deletions
eng/pipelines/coreclr/templates/helix-queues-setup.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -95,6 +95,13 @@ jobs:
 - (Fedora.30.Amd64)Ubuntu.1604.amd64@mcr.microsoft.com/dotnet-buildtools/prereqs:fedora-30-helix-20200512010621-4f8cef7
 - RedHat.7.Amd64

 # OSX arm64
 - ${{ if eq(parameters.platform, 'OSX_arm64') }}:
 - ${{ if and(eq(variables['System.TeamProject'], 'public'), in(parameters.jobParameters.helixQueueGroup, 'ci', 'libraries')) }}:
 - OSX.1100.ARM64.Open
 - ${{ if eq(variables['System.TeamProject'], 'internal') }}:
 - OSX.1100.ARM64

 # OSX x64
 - ${{ if eq(parameters.platform, 'OSX_x64') }}:
 - ${{ if and(eq(variables['System.TeamProject'], 'public'), in(parameters.jobParameters.helixQueueGroup, 'pr', 'ci', 'libraries')) }}:
Expand Down

5 changes: 5 additions & 0 deletions
eng/pipelines/libraries/helix-queues-setup.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -85,6 +85,11 @@ jobs:
 # Limiting interp runs as we don't need as much coverage.
 - Debian.9.Amd64.Open

 # OSX arm64
 - ${{ if eq(parameters.platform, 'OSX_arm64') }}:
Comment thread
sdmaclea marked this conversation as resolved.


Outdated

 - ${{ if eq(parameters.jobParameters.isFullMatrix, true) }}:
@safernsafernMar 1, 2021

Copy link


Copy Markdown

Member
Choose a reason for hiding this comment

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

I know we discussed this earlier, but I believe we should remove this condition for libraries and then instead condition the test runs in the yml entry point?
 - OSX.1100.ARM64.Open

 # OSX x64
 - ${{ if eq(parameters.platform, 'OSX_x64') }}:
 - ${{ if eq(parameters.jobParameters.isFullMatrix, true) }}:
Expand Down

5 changes: 5 additions & 0 deletions
eng/pipelines/libraries/outerloop.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -32,6 +32,8 @@ jobs:
 - Linux_arm64
 - Linux_musl_arm64
 - ${{ if eq(variables['includeOsxOuterloop'], true) }}: 
 - ${{ if eq(variables['isFullMatrix'], true) }}:
 - OSX_arm64
 - OSX_x64
 jobParameters:
 testGroup: innerloop
Expand All@@ -55,6 +57,7 @@ jobs:
 - Linux_musl_x64
 - Linux_musl_arm64
 - ${{ if and(eq(variables['includeOsxOuterloop'], true), eq(variables['isFullMatrix'], true)) }}:
 - OSX_arm64
 - OSX_x64
 helixQueuesTemplate: /eng/pipelines/libraries/helix-queues-setup.yml
 jobParameters:
Expand All@@ -76,6 +79,8 @@ jobs:
 - Linux_x64
 - Linux_musl_x64
 - ${{ if eq(variables['includeOsxOuterloop'], true) }}:
 - ${{ if eq(variables['isFullMatrix'], true) }}:
 - OSX_arm64
 - OSX_x64
 helixQueuesTemplate: /eng/pipelines/libraries/helix-queues-setup.yml
 jobParameters:
Expand Down

8 changes: 0 additions & 8 deletions
src/coreclr/pal/tests/palsuite/exception_handling/pal_sxs/test1/exceptionsxs.cpp

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -68,13 +68,6 @@ void sigsegv_handler(int code, siginfo_t *siginfo, void *context)

int main(int argc, char *argv[])
{
#if defined(TARGET_OSX) && defined(TARGET_ARM64)
 // Helix and/or build error...
 // dyld: Library not loaded: libpaltest_pal_sxs_test1_dll1.dylib
 printf("paltest_pal_sxs_test1 has been disabled on this platform\n");

 return PASS;
#else // defined(TARGET_OSX) && defined(TARGET_ARM64)
 struct sigaction newAction;
 struct sigaction oldAction;
 newAction.sa_flags = SA_SIGINFO | SA_RESTART;
Expand DownExpand Up@@ -120,5 +113,4 @@ int main(int argc, char *argv[])

 printf("ERROR: code was executed after the access violation.\n");
 return FAIL;
#endif // defined(TARGET_OSX) && defined(TARGET_ARM64)
}

5 changes: 0 additions & 5 deletions
src/coreclr/pal/tests/palsuite/file_io/ReadFile/test2/ReadFile.cpp

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -120,10 +120,6 @@ BOOL readTest_ReadFile_test2(DWORD dwByteCount, char cResult)

PALTEST(file_io_ReadFile_test2_paltest_readfile_test2, "file_io/ReadFile/test2/paltest_readfile_test2")
{
#if defined(TARGET_OSX) && defined(TARGET_ARM64)
 // Test hard codes PAGE_SIZE = 4096. Apple Silicon does not have 4K pages.
 printf("file_io_ReadFile_test2_paltest_readfile_test2 has been disabled on this platform\n");
#else // defined(TARGET_OSX) && defined(TARGET_ARM64)
 HANDLE hFile = NULL;
 const int BUFFER_SIZE = 2 * PAGESIZE;

Expand DownExpand Up@@ -191,7 +187,6 @@ PALTEST(file_io_ReadFile_test2_paltest_readfile_test2, "file_io/ReadFile/test2/p

	VirtualFree(readBuffer_ReadFile_test2, BUFFER_SIZE, MEM_RELEASE);
 PAL_Terminate();
#endif // defined(TARGET_OSX) && defined(TARGET_ARM64)
 return PASS;
}


5 changes: 0 additions & 5 deletions
src/coreclr/pal/tests/palsuite/filemapping_memmgt/VirtualProtect/test4/VirtualProtect.cpp

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -28,10 +28,6 @@ PALTEST(filemapping_memmgt_VirtualProtect_test4_paltest_virtualprotect_test4, "f
 ExitProcess(FAIL);
 }

#if defined(TARGET_OSX) && defined(TARGET_ARM64)
 // AppleSilcon does not allow mapping general pages as RWX
 printf("filemapping_memmgt_VirtualProtect_test4_paltest_virtualprotect_test4 has been disabled on this platform\n");
#else // defined(TARGET_OSX) && defined(TARGET_ARM64)
 //Allocate the physical storage in memory or in the paging file on disk 
 lpVirtualAddress = VirtualAlloc(NULL,//determine where to allocate the region
 REGIONSIZE, //specify the size
Expand DownExpand Up@@ -65,7 +61,6 @@ PALTEST(filemapping_memmgt_VirtualProtect_test4_paltest_virtualprotect_test4, "f
 {
 Fail("\nFailed to call VirtualFree API!\n");
 }
#endif // defined(TARGET_OSX) && defined(TARGET_ARM64)

 PAL_Terminate();
 return PASS;
Expand Down

17 changes: 17 additions & 0 deletions
src/coreclr/pal/tests/palsuite/issues.targets

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -32,4 +32,21 @@
 <Issue>https://github.com/dotnet/runtime/issues/42292</Issue>
 </ExcludeList>
 </ItemGroup>

 <ItemGroup Condition="'$(TargetArchitecture)' == 'arm64' and '$(TargetOS)' == 'OSX'">
 <ExcludeList Include="miscellaneous/queryperformancecounter/test1/paltest_queryperformancecounter_test1">
 <Issue>https://github.com/dotnet/runtime/issues/7639</Issue>
 </ExcludeList>
 <ExcludeList Include="exception_handling/pal_sxs/test1/paltest_pal_sxs_test1">
 <Issue>https://github.com/dotnet/runtime/issues/42292</Issue>
 </ExcludeList>
 <ExcludeList Include="file_io/ReadFile/test2/paltest_readfile_test2">
 <!-- Test hard codes PAGE_SIZE = 4096. Apple Silicon does not have 4K pages. -->
 <Issue>https://github.com/dotnet/runtime/issues/42292</Issue>
 </ExcludeList>
 <ExcludeList Include="filemapping_memmgt/VirtualProtect/test4/paltest_virtualprotect_test4">
 <!-- Test tries to set RWX on general memory page -->
 <Issue>https://github.com/dotnet/runtime/issues/7639</Issue>
 </ExcludeList>
 </ItemGroup>
</Project>

5 changes: 0 additions & 5 deletions
src/coreclr/pal/tests/palsuite/miscellaneous/queryperformancecounter/test1/test1.cpp

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -17,10 +17,6 @@

PALTEST(miscellaneous_queryperformancecounter_test1_paltest_queryperformancecounter_test1, "miscellaneous/queryperformancecounter/test1/paltest_queryperformancecounter_test1")
{
#if defined(TARGET_OSX) && defined(TARGET_ARM64)
 // We are see > 100 ms of average error on this platform
 printf("miscellaneous_queryperformancecounter_test1_paltest_queryperformancecounter_test1 has been disabled on this platform\n");
#else // defined(TARGET_OSX) && defined(TARGET_ARM64)
 /* Milliseconds of error which are acceptable Function execution time, etc.
 FreeBSD has a "standard" resolution of 50ms for waiting operations, so we
 must take that into account as well */
Expand DownExpand Up@@ -104,7 +100,6 @@ PALTEST(miscellaneous_queryperformancecounter_test1_paltest_queryperformancecoun
 /* Terminate the PAL.
 */ 
 PAL_Terminate();
#endif // defined(TARGET_OSX) && defined(TARGET_ARM64)
 return PASS;
}

Expand Down

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Remove or un-stick sticky/fixed headers that block content\n(function() {\n function unstick() {\n document.querySelectorAll('header, nav, [role=\"banner\"], .header, .navbar, .sticky, .fixed-top, [style*=\"position: fixed\"], [style*=\"position:sticky\"]').forEach(function(el) {\n if (el.style.position === 'fixed' || el.style.position === 'sticky' || \n getComputedStyle(el).position === 'fixed' || getComputedStyle(el).position === 'sticky') {\n el.style.position = 'static';\n el.style.top = 'auto';\n el.style.zIndex = 'auto';\n }\n });\n }\n \n unstick();\n \n var observer = new MutationObserver(unstick);\n observer.observe(document.body, { childList: true, subtree: true, attributes: true, attributeFilter: ['style', 'class'] });\n})();", "Kill Sticky Headers");
}
} catch(__e) { console.warn('[Userscript:Kill Sticky Headers]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content
Setup OSX.1100.ARM64.* helix queues#47919

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.

 Sign up for GitHub

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

 Merged

sdmaclea
merged 9 commits into

from

Mar 2, 2021
 Merged


Setup OSX.1100.ARM64.* helix queues
#47919
Show file tree
Hide file tree
 Changes from all commits
Commits

Show all changes


9 commits


Select commit
Hold shift + click to select a range
ddcafc8

Setup OSX.1100.ARM64.* helix queues


sdmaclea Feb 5, 2021
a1de69c

Limit OSX.1100.ARM64.* queues to CI builds until we understand capacity


sdmaclea Feb 5, 2021
783393b

For CI testing, add OSX_arm64 to the platformGroup all and gcstress


sdmaclea Feb 5, 2021
136f9a0

Run jitstress on Apple Silicon


sdmaclea Feb 24, 2021
b57b635

Enable Apple Silicon on more pipelines


sdmaclea Feb 25, 2021
6146fd2

Enable more Apple Silicon pipelines


sdmaclea Feb 25, 2021
965efb3

Fixup libraries outerloop


sdmaclea Feb 26, 2021
763770d

Disable failing Apple Silicon PAL tests


sdmaclea Feb 26, 2021
144d483

Revert "Apple Silicon disable failing PAL tests (#48782)"


sdmaclea Feb 26, 2021

File filter

Filter by extension
Filter by extension
Conversations

Failed to load comments. 
Loading
Jump to

Jump to file


Failed to load files. 
Loading
 Diff view

 Diff view


12 changes: 12 additions & 0 deletions
eng/pipelines/coreclr/ci.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -54,6 +54,10 @@ jobs:
 jobTemplate: /eng/pipelines/coreclr/templates/build-job.yml
 buildConfig: checked
 platformGroup: all
 platforms:
 # It is too early to include OSX_arm64 in platform group all
 # Adding it here will enable it also
 - OSX_arm64
 jobParameters:
 testGroup: outerloop

Expand DownExpand Up@@ -82,6 +86,10 @@ jobs:
 jobTemplate: /eng/pipelines/libraries/build-job.yml
 buildConfig: Release
 platformGroup: all
 platforms:
 # It is too early to include OSX_arm64 in platform group all
 # Adding it here will enable it also
 - OSX_arm64
 jobParameters:
 isOfficialBuild: false
 liveRuntimeBuildConfig: checked
Expand All@@ -107,6 +115,10 @@ jobs:
 jobTemplate: /eng/pipelines/common/templates/runtimes/run-test-job.yml
 buildConfig: checked
 platformGroup: all
 platforms:
 # It is too early to include OSX_arm64 in platform group all
 # Adding it here will enable it to also run this test
 - OSX_arm64
 helixQueueGroup: ci
 helixQueuesTemplate: /eng/pipelines/coreclr/templates/helix-queues-setup.yml
 jobParameters:
Expand Down

4 changes: 2 additions & 2 deletions
eng/pipelines/coreclr/ilasm.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -24,7 +24,7 @@ jobs:
 jobTemplate: /eng/pipelines/common/build-coreclr-and-libraries-job.yml
 buildConfig: checked
 platforms:
# Linux tests are built on the OSX machines.
- OSX_arm64
 - OSX_x64
 - Linux_arm
 - Linux_arm64
Expand All@@ -51,7 +51,7 @@ jobs:
 jobTemplate: /eng/pipelines/common/templates/runtimes/run-test-job.yml
 buildConfig: checked
 platforms:
# Linux tests are built on the OSX machines.
- OSX_arm64
 - OSX_x64
 - Linux_arm
 - Linux_arm64
Expand Down

1 change: 1 addition & 0 deletions
eng/pipelines/coreclr/jitrollingbuild.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -19,6 +19,7 @@ jobs:
 jobTemplate: /eng/pipelines/coreclr/templates/build-jit-job.yml
 buildConfig: checked
 platforms:
 - OSX_arm64
 - OSX_x64
 # Currently, Linux arm/arm64 machines don't have the Python 'pip3' tool, nor the azure-storage-blob package that
 # is required to do the JIT upload to Azure Storage. Thus, these platforms are disabled. If we can figure out how
Expand Down

2 changes: 2 additions & 0 deletions
eng/pipelines/coreclr/jitstress-isas-arm.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -16,6 +16,7 @@ jobs:
 buildConfig: checked
 platforms:
 - Linux_arm64
 - OSX_arm64
 - windows_arm64
 - CoreClrTestBuildHost # Either OSX_x64 or Linux_x64
 jobParameters:
Expand All@@ -37,6 +38,7 @@ jobs:
 buildConfig: checked
 platforms:
 - Linux_arm64
 - OSX_arm64
 - windows_arm64
 helixQueueGroup: ci
 helixQueuesTemplate: /eng/pipelines/coreclr/templates/helix-queues-setup.yml
Expand Down

4 changes: 2 additions & 2 deletions
eng/pipelines/coreclr/jitstress.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -21,7 +21,7 @@ jobs:
 jobTemplate: /eng/pipelines/common/build-coreclr-and-libraries-job.yml
 buildConfig: checked
 platforms:
# Linux tests are built on the OSX machines.
- OSX_arm64
 - OSX_x64
 - Linux_arm
 - Linux_arm64
Expand All@@ -48,7 +48,7 @@ jobs:
 jobTemplate: /eng/pipelines/common/templates/runtimes/run-test-job.yml
 buildConfig: checked
 platforms:
# Linux tests are built on the OSX machines.
- OSX_arm64
 - OSX_x64
 - Linux_arm
 - Linux_arm64
Expand Down

4 changes: 2 additions & 2 deletions
eng/pipelines/coreclr/jitstress2-jitstressregs.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -15,7 +15,7 @@ jobs:
 jobTemplate: /eng/pipelines/common/build-coreclr-and-libraries-job.yml
 buildConfig: checked
 platforms:
# Linux tests are built on the OSX machines.
- OSX_arm64
 - OSX_x64
 - Linux_arm
 - Linux_arm64
Expand All@@ -42,7 +42,7 @@ jobs:
 jobTemplate: /eng/pipelines/common/templates/runtimes/run-test-job.yml
 buildConfig: checked
 platforms:
# Linux tests are built on the OSX machines.
- OSX_arm64
 - OSX_x64
 - Linux_arm
 - Linux_arm64
Expand Down

4 changes: 2 additions & 2 deletions
eng/pipelines/coreclr/jitstressregs.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -15,7 +15,7 @@ jobs:
 jobTemplate: /eng/pipelines/common/build-coreclr-and-libraries-job.yml
 buildConfig: checked
 platforms:
# Linux tests are built on the OSX machines.
- OSX_arm64
 - OSX_x64
 - Linux_arm
 - Linux_arm64
Expand All@@ -42,7 +42,7 @@ jobs:
 jobTemplate: /eng/pipelines/common/templates/runtimes/run-test-job.yml
 buildConfig: checked
 platforms:
# Linux tests are built on the OSX machines.
- OSX_arm64
 - OSX_x64
 - Linux_arm
 - Linux_arm64
Expand Down

8 changes: 8 additions & 0 deletions
eng/pipelines/coreclr/release-tests.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -18,6 +18,10 @@ jobs:
 jobTemplate: /eng/pipelines/common/build-coreclr-and-libraries-job.yml
 buildConfig: release
 platformGroup: all
 platforms:
 # It is too early to include OSX_arm64 in platform group all
 # Adding it here will enable it also
 - OSX_arm64
 jobParameters:
 isOfficialBuild: false

Expand All@@ -42,6 +46,10 @@ jobs:
 jobTemplate: /eng/pipelines/common/templates/runtimes/run-test-job.yml
 buildConfig: release
 platformGroup: all
 platforms:
 # It is too early to include OSX_arm64 in platform group all
 # Adding it here will enable it also
 - OSX_arm64
 helixQueueGroup: ci
 helixQueuesTemplate: /eng/pipelines/coreclr/templates/helix-queues-setup.yml
 jobParameters:
Expand Down

7 changes: 7 additions & 0 deletions
eng/pipelines/coreclr/templates/helix-queues-setup.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -95,6 +95,13 @@ jobs:
 - (Fedora.30.Amd64)Ubuntu.1604.amd64@mcr.microsoft.com/dotnet-buildtools/prereqs:fedora-30-helix-20200512010621-4f8cef7
 - RedHat.7.Amd64

 # OSX arm64
 - ${{ if eq(parameters.platform, 'OSX_arm64') }}:
 - ${{ if and(eq(variables['System.TeamProject'], 'public'), in(parameters.jobParameters.helixQueueGroup, 'ci', 'libraries')) }}:
 - OSX.1100.ARM64.Open
 - ${{ if eq(variables['System.TeamProject'], 'internal') }}:
 - OSX.1100.ARM64

 # OSX x64
 - ${{ if eq(parameters.platform, 'OSX_x64') }}:
 - ${{ if and(eq(variables['System.TeamProject'], 'public'), in(parameters.jobParameters.helixQueueGroup, 'pr', 'ci', 'libraries')) }}:
Expand Down

5 changes: 5 additions & 0 deletions
eng/pipelines/libraries/helix-queues-setup.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -85,6 +85,11 @@ jobs:
 # Limiting interp runs as we don't need as much coverage.
 - Debian.9.Amd64.Open

 # OSX arm64
 - ${{ if eq(parameters.platform, 'OSX_arm64') }}:
Comment thread
sdmaclea marked this conversation as resolved.


Outdated

 - ${{ if eq(parameters.jobParameters.isFullMatrix, true) }}:
@safernsafernMar 1, 2021

Copy link


Copy Markdown

Member
Choose a reason for hiding this comment

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

I know we discussed this earlier, but I believe we should remove this condition for libraries and then instead condition the test runs in the yml entry point?
 - OSX.1100.ARM64.Open

 # OSX x64
 - ${{ if eq(parameters.platform, 'OSX_x64') }}:
 - ${{ if eq(parameters.jobParameters.isFullMatrix, true) }}:
Expand Down

5 changes: 5 additions & 0 deletions
eng/pipelines/libraries/outerloop.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -32,6 +32,8 @@ jobs:
 - Linux_arm64
 - Linux_musl_arm64
 - ${{ if eq(variables['includeOsxOuterloop'], true) }}: 
 - ${{ if eq(variables['isFullMatrix'], true) }}:
 - OSX_arm64
 - OSX_x64
 jobParameters:
 testGroup: innerloop
Expand All@@ -55,6 +57,7 @@ jobs:
 - Linux_musl_x64
 - Linux_musl_arm64
 - ${{ if and(eq(variables['includeOsxOuterloop'], true), eq(variables['isFullMatrix'], true)) }}:
 - OSX_arm64
 - OSX_x64
 helixQueuesTemplate: /eng/pipelines/libraries/helix-queues-setup.yml
 jobParameters:
Expand All@@ -76,6 +79,8 @@ jobs:
 - Linux_x64
 - Linux_musl_x64
 - ${{ if eq(variables['includeOsxOuterloop'], true) }}:
 - ${{ if eq(variables['isFullMatrix'], true) }}:
 - OSX_arm64
 - OSX_x64
 helixQueuesTemplate: /eng/pipelines/libraries/helix-queues-setup.yml
 jobParameters:
Expand Down

8 changes: 0 additions & 8 deletions
src/coreclr/pal/tests/palsuite/exception_handling/pal_sxs/test1/exceptionsxs.cpp

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -68,13 +68,6 @@ void sigsegv_handler(int code, siginfo_t *siginfo, void *context)

int main(int argc, char *argv[])
{
#if defined(TARGET_OSX) && defined(TARGET_ARM64)
 // Helix and/or build error...
 // dyld: Library not loaded: libpaltest_pal_sxs_test1_dll1.dylib
 printf("paltest_pal_sxs_test1 has been disabled on this platform\n");

 return PASS;
#else // defined(TARGET_OSX) && defined(TARGET_ARM64)
 struct sigaction newAction;
 struct sigaction oldAction;
 newAction.sa_flags = SA_SIGINFO | SA_RESTART;
Expand DownExpand Up@@ -120,5 +113,4 @@ int main(int argc, char *argv[])

 printf("ERROR: code was executed after the access violation.\n");
 return FAIL;
#endif // defined(TARGET_OSX) && defined(TARGET_ARM64)
}

5 changes: 0 additions & 5 deletions
src/coreclr/pal/tests/palsuite/file_io/ReadFile/test2/ReadFile.cpp

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -120,10 +120,6 @@ BOOL readTest_ReadFile_test2(DWORD dwByteCount, char cResult)

PALTEST(file_io_ReadFile_test2_paltest_readfile_test2, "file_io/ReadFile/test2/paltest_readfile_test2")
{
#if defined(TARGET_OSX) && defined(TARGET_ARM64)
 // Test hard codes PAGE_SIZE = 4096. Apple Silicon does not have 4K pages.
 printf("file_io_ReadFile_test2_paltest_readfile_test2 has been disabled on this platform\n");
#else // defined(TARGET_OSX) && defined(TARGET_ARM64)
 HANDLE hFile = NULL;
 const int BUFFER_SIZE = 2 * PAGESIZE;

Expand DownExpand Up@@ -191,7 +187,6 @@ PALTEST(file_io_ReadFile_test2_paltest_readfile_test2, "file_io/ReadFile/test2/p

	VirtualFree(readBuffer_ReadFile_test2, BUFFER_SIZE, MEM_RELEASE);
 PAL_Terminate();
#endif // defined(TARGET_OSX) && defined(TARGET_ARM64)
 return PASS;
}


5 changes: 0 additions & 5 deletions
src/coreclr/pal/tests/palsuite/filemapping_memmgt/VirtualProtect/test4/VirtualProtect.cpp

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -28,10 +28,6 @@ PALTEST(filemapping_memmgt_VirtualProtect_test4_paltest_virtualprotect_test4, "f
 ExitProcess(FAIL);
 }

#if defined(TARGET_OSX) && defined(TARGET_ARM64)
 // AppleSilcon does not allow mapping general pages as RWX
 printf("filemapping_memmgt_VirtualProtect_test4_paltest_virtualprotect_test4 has been disabled on this platform\n");
#else // defined(TARGET_OSX) && defined(TARGET_ARM64)
 //Allocate the physical storage in memory or in the paging file on disk 
 lpVirtualAddress = VirtualAlloc(NULL,//determine where to allocate the region
 REGIONSIZE, //specify the size
Expand DownExpand Up@@ -65,7 +61,6 @@ PALTEST(filemapping_memmgt_VirtualProtect_test4_paltest_virtualprotect_test4, "f
 {
 Fail("\nFailed to call VirtualFree API!\n");
 }
#endif // defined(TARGET_OSX) && defined(TARGET_ARM64)

 PAL_Terminate();
 return PASS;
Expand Down

17 changes: 17 additions & 0 deletions
src/coreclr/pal/tests/palsuite/issues.targets

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -32,4 +32,21 @@
 <Issue>https://github.com/dotnet/runtime/issues/42292</Issue>
 </ExcludeList>
 </ItemGroup>

 <ItemGroup Condition="'$(TargetArchitecture)' == 'arm64' and '$(TargetOS)' == 'OSX'">
 <ExcludeList Include="miscellaneous/queryperformancecounter/test1/paltest_queryperformancecounter_test1">
 <Issue>https://github.com/dotnet/runtime/issues/7639</Issue>
 </ExcludeList>
 <ExcludeList Include="exception_handling/pal_sxs/test1/paltest_pal_sxs_test1">
 <Issue>https://github.com/dotnet/runtime/issues/42292</Issue>
 </ExcludeList>
 <ExcludeList Include="file_io/ReadFile/test2/paltest_readfile_test2">
 <!-- Test hard codes PAGE_SIZE = 4096. Apple Silicon does not have 4K pages. -->
 <Issue>https://github.com/dotnet/runtime/issues/42292</Issue>
 </ExcludeList>
 <ExcludeList Include="filemapping_memmgt/VirtualProtect/test4/paltest_virtualprotect_test4">
 <!-- Test tries to set RWX on general memory page -->
 <Issue>https://github.com/dotnet/runtime/issues/7639</Issue>
 </ExcludeList>
 </ItemGroup>
</Project>

5 changes: 0 additions & 5 deletions
src/coreclr/pal/tests/palsuite/miscellaneous/queryperformancecounter/test1/test1.cpp

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -17,10 +17,6 @@

PALTEST(miscellaneous_queryperformancecounter_test1_paltest_queryperformancecounter_test1, "miscellaneous/queryperformancecounter/test1/paltest_queryperformancecounter_test1")
{
#if defined(TARGET_OSX) && defined(TARGET_ARM64)
 // We are see > 100 ms of average error on this platform
 printf("miscellaneous_queryperformancecounter_test1_paltest_queryperformancecounter_test1 has been disabled on this platform\n");
#else // defined(TARGET_OSX) && defined(TARGET_ARM64)
 /* Milliseconds of error which are acceptable Function execution time, etc.
 FreeBSD has a "standard" resolution of 50ms for waiting operations, so we
 must take that into account as well */
Expand DownExpand Up@@ -104,7 +100,6 @@ PALTEST(miscellaneous_queryperformancecounter_test1_paltest_queryperformancecoun
 /* Terminate the PAL.
 */ 
 PAL_Terminate();
#endif // defined(TARGET_OSX) && defined(TARGET_ARM64)
 return PASS;
}

Expand Down

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Universal Dark Mode - works on any site\n(function() {\n var enabled = true;\n \n function applyDarkMode() {\n if (!enabled) return;\n \n // Create style element if it doesn't exist\n var style = document.getElementById('universal-dark-mode-style');\n if (!style) {\n style = document.createElement('style');\n style.id = 'universal-dark-mode-style';\n document.head.appendChild(style);\n }\n \n // Dark mode CSS - inverts colors but preserves images/video\n style.textContent = '\n /* Invert everything except media */\n html {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #1a1a2e !important;\n }\n \n /* Restore images, videos, iframes, canvas */\n img, video, iframe, canvas, svg, picture, [style*=\"background-image\"] {\n filter: invert(1) hue-rotate(180deg) !important;\n }\n \n /* Preserve specific elements that should not be inverted */\n .no-dark-mode, .no-dark-mode *,\n [data-theme=\"light\"], [data-theme=\"light\"],\n .ace_editor, .ace_editor *,\n .CodeMirror, .CodeMirror *,\n .monaco-editor, .monaco-editor *,\n .markdown-body pre, .markdown-body pre *,\n .highlight, .highlight *,\n pre code, pre code * {\n filter: none !important;\n }\n \n /* Fix common UI elements */\n .modal, .popup, .dropdown-menu, .tooltip, .popover {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #2d2d44 !important;\n border-color: #444 !important;\n }\n \n /* Scrollbars */\n ::-webkit-scrollbar { background: #1a1a2e !important; }\n ::-webkit-scrollbar-thumb { background: #444 !important; }\n ::-webkit-scrollbar-thumb:hover { background: #555 !important; }\n \n /* Selection */\n ::selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ::-moz-selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ';\n }\n \n function removeDarkMode() {\n var style = document.getElementById('universal-dark-mode-style');\n if (style) style.remove();\n }\n \n // Toggle with Alt+Shift+D\n document.addEventListener('keydown', function(e) {\n if (e.altKey && e.shiftKey && e.key === 'D') {\n e.preventDefault();\n enabled = !enabled;\n if (enabled) {\n applyDarkMode();\n console.log('[Universal Dark Mode] Enabled');\n } else {\n removeDarkMode();\n console.log('[Universal Dark Mode] Disabled');\n }\n }\n });\n \n // Apply on load\n applyDarkMode();\n \n // Re-apply on dynamic content\n var observer = new MutationObserver(function(mutations) {\n if (enabled && !document.getElementById('universal-dark-mode-style')) {\n applyDarkMode();\n }\n });\n observer.observe(document.head, { childList: true });\n \n console.log('[Universal Dark Mode] Loaded - Press Alt+Shift+D to toggle');\n})();", "Universal Dark Mode");
}
} catch(__e) { console.warn('[Userscript:Universal Dark Mode]', __e); }
})();
})();

Skip to content
Setup OSX.1100.ARM64.* helix queues#47919

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.

 Sign up for GitHub

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

 Merged

sdmaclea
merged 9 commits into

from

Mar 2, 2021
 Merged


Setup OSX.1100.ARM64.* helix queues
#47919
Show file tree
Hide file tree
 Changes from all commits
Commits

Show all changes


9 commits


Select commit
Hold shift + click to select a range
ddcafc8

Setup OSX.1100.ARM64.* helix queues


sdmaclea Feb 5, 2021
a1de69c

Limit OSX.1100.ARM64.* queues to CI builds until we understand capacity


sdmaclea Feb 5, 2021
783393b

For CI testing, add OSX_arm64 to the platformGroup all and gcstress


sdmaclea Feb 5, 2021
136f9a0

Run jitstress on Apple Silicon


sdmaclea Feb 24, 2021
b57b635

Enable Apple Silicon on more pipelines


sdmaclea Feb 25, 2021
6146fd2

Enable more Apple Silicon pipelines


sdmaclea Feb 25, 2021
965efb3

Fixup libraries outerloop


sdmaclea Feb 26, 2021
763770d

Disable failing Apple Silicon PAL tests


sdmaclea Feb 26, 2021
144d483

Revert "Apple Silicon disable failing PAL tests (#48782)"


sdmaclea Feb 26, 2021

File filter

Filter by extension
Filter by extension
Conversations

Failed to load comments. 
Loading
Jump to

Jump to file


Failed to load files. 
Loading
 Diff view

 Diff view


12 changes: 12 additions & 0 deletions
eng/pipelines/coreclr/ci.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -54,6 +54,10 @@ jobs:
 jobTemplate: /eng/pipelines/coreclr/templates/build-job.yml
 buildConfig: checked
 platformGroup: all
 platforms:
 # It is too early to include OSX_arm64 in platform group all
 # Adding it here will enable it also
 - OSX_arm64
 jobParameters:
 testGroup: outerloop

Expand DownExpand Up@@ -82,6 +86,10 @@ jobs:
 jobTemplate: /eng/pipelines/libraries/build-job.yml
 buildConfig: Release
 platformGroup: all
 platforms:
 # It is too early to include OSX_arm64 in platform group all
 # Adding it here will enable it also
 - OSX_arm64
 jobParameters:
 isOfficialBuild: false
 liveRuntimeBuildConfig: checked
Expand All@@ -107,6 +115,10 @@ jobs:
 jobTemplate: /eng/pipelines/common/templates/runtimes/run-test-job.yml
 buildConfig: checked
 platformGroup: all
 platforms:
 # It is too early to include OSX_arm64 in platform group all
 # Adding it here will enable it to also run this test
 - OSX_arm64
 helixQueueGroup: ci
 helixQueuesTemplate: /eng/pipelines/coreclr/templates/helix-queues-setup.yml
 jobParameters:
Expand Down

4 changes: 2 additions & 2 deletions
eng/pipelines/coreclr/ilasm.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -24,7 +24,7 @@ jobs:
 jobTemplate: /eng/pipelines/common/build-coreclr-and-libraries-job.yml
 buildConfig: checked
 platforms:
# Linux tests are built on the OSX machines.
- OSX_arm64
 - OSX_x64
 - Linux_arm
 - Linux_arm64
Expand All@@ -51,7 +51,7 @@ jobs:
 jobTemplate: /eng/pipelines/common/templates/runtimes/run-test-job.yml
 buildConfig: checked
 platforms:
# Linux tests are built on the OSX machines.
- OSX_arm64
 - OSX_x64
 - Linux_arm
 - Linux_arm64
Expand Down

1 change: 1 addition & 0 deletions
eng/pipelines/coreclr/jitrollingbuild.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -19,6 +19,7 @@ jobs:
 jobTemplate: /eng/pipelines/coreclr/templates/build-jit-job.yml
 buildConfig: checked
 platforms:
 - OSX_arm64
 - OSX_x64
 # Currently, Linux arm/arm64 machines don't have the Python 'pip3' tool, nor the azure-storage-blob package that
 # is required to do the JIT upload to Azure Storage. Thus, these platforms are disabled. If we can figure out how
Expand Down

2 changes: 2 additions & 0 deletions
eng/pipelines/coreclr/jitstress-isas-arm.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -16,6 +16,7 @@ jobs:
 buildConfig: checked
 platforms:
 - Linux_arm64
 - OSX_arm64
 - windows_arm64
 - CoreClrTestBuildHost # Either OSX_x64 or Linux_x64
 jobParameters:
Expand All@@ -37,6 +38,7 @@ jobs:
 buildConfig: checked
 platforms:
 - Linux_arm64
 - OSX_arm64
 - windows_arm64
 helixQueueGroup: ci
 helixQueuesTemplate: /eng/pipelines/coreclr/templates/helix-queues-setup.yml
Expand Down

4 changes: 2 additions & 2 deletions
eng/pipelines/coreclr/jitstress.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -21,7 +21,7 @@ jobs:
 jobTemplate: /eng/pipelines/common/build-coreclr-and-libraries-job.yml
 buildConfig: checked
 platforms:
# Linux tests are built on the OSX machines.
- OSX_arm64
 - OSX_x64
 - Linux_arm
 - Linux_arm64
Expand All@@ -48,7 +48,7 @@ jobs:
 jobTemplate: /eng/pipelines/common/templates/runtimes/run-test-job.yml
 buildConfig: checked
 platforms:
# Linux tests are built on the OSX machines.
- OSX_arm64
 - OSX_x64
 - Linux_arm
 - Linux_arm64
Expand Down

4 changes: 2 additions & 2 deletions
eng/pipelines/coreclr/jitstress2-jitstressregs.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -15,7 +15,7 @@ jobs:
 jobTemplate: /eng/pipelines/common/build-coreclr-and-libraries-job.yml
 buildConfig: checked
 platforms:
# Linux tests are built on the OSX machines.
- OSX_arm64
 - OSX_x64
 - Linux_arm
 - Linux_arm64
Expand All@@ -42,7 +42,7 @@ jobs:
 jobTemplate: /eng/pipelines/common/templates/runtimes/run-test-job.yml
 buildConfig: checked
 platforms:
# Linux tests are built on the OSX machines.
- OSX_arm64
 - OSX_x64
 - Linux_arm
 - Linux_arm64
Expand Down

4 changes: 2 additions & 2 deletions
eng/pipelines/coreclr/jitstressregs.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -15,7 +15,7 @@ jobs:
 jobTemplate: /eng/pipelines/common/build-coreclr-and-libraries-job.yml
 buildConfig: checked
 platforms:
# Linux tests are built on the OSX machines.
- OSX_arm64
 - OSX_x64
 - Linux_arm
 - Linux_arm64
Expand All@@ -42,7 +42,7 @@ jobs:
 jobTemplate: /eng/pipelines/common/templates/runtimes/run-test-job.yml
 buildConfig: checked
 platforms:
# Linux tests are built on the OSX machines.
- OSX_arm64
 - OSX_x64
 - Linux_arm
 - Linux_arm64
Expand Down

8 changes: 8 additions & 0 deletions
eng/pipelines/coreclr/release-tests.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -18,6 +18,10 @@ jobs:
 jobTemplate: /eng/pipelines/common/build-coreclr-and-libraries-job.yml
 buildConfig: release
 platformGroup: all
 platforms:
 # It is too early to include OSX_arm64 in platform group all
 # Adding it here will enable it also
 - OSX_arm64
 jobParameters:
 isOfficialBuild: false

Expand All@@ -42,6 +46,10 @@ jobs:
 jobTemplate: /eng/pipelines/common/templates/runtimes/run-test-job.yml
 buildConfig: release
 platformGroup: all
 platforms:
 # It is too early to include OSX_arm64 in platform group all
 # Adding it here will enable it also
 - OSX_arm64
 helixQueueGroup: ci
 helixQueuesTemplate: /eng/pipelines/coreclr/templates/helix-queues-setup.yml
 jobParameters:
Expand Down

7 changes: 7 additions & 0 deletions
eng/pipelines/coreclr/templates/helix-queues-setup.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -95,6 +95,13 @@ jobs:
 - (Fedora.30.Amd64)Ubuntu.1604.amd64@mcr.microsoft.com/dotnet-buildtools/prereqs:fedora-30-helix-20200512010621-4f8cef7
 - RedHat.7.Amd64

 # OSX arm64
 - ${{ if eq(parameters.platform, 'OSX_arm64') }}:
 - ${{ if and(eq(variables['System.TeamProject'], 'public'), in(parameters.jobParameters.helixQueueGroup, 'ci', 'libraries')) }}:
 - OSX.1100.ARM64.Open
 - ${{ if eq(variables['System.TeamProject'], 'internal') }}:
 - OSX.1100.ARM64

 # OSX x64
 - ${{ if eq(parameters.platform, 'OSX_x64') }}:
 - ${{ if and(eq(variables['System.TeamProject'], 'public'), in(parameters.jobParameters.helixQueueGroup, 'pr', 'ci', 'libraries')) }}:
Expand Down

5 changes: 5 additions & 0 deletions
eng/pipelines/libraries/helix-queues-setup.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -85,6 +85,11 @@ jobs:
 # Limiting interp runs as we don't need as much coverage.
 - Debian.9.Amd64.Open

 # OSX arm64
 - ${{ if eq(parameters.platform, 'OSX_arm64') }}:
Comment thread
sdmaclea marked this conversation as resolved.


Outdated

 - ${{ if eq(parameters.jobParameters.isFullMatrix, true) }}:
@safernsafernMar 1, 2021

Copy link


Copy Markdown

Member
Choose a reason for hiding this comment

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

I know we discussed this earlier, but I believe we should remove this condition for libraries and then instead condition the test runs in the yml entry point?
 - OSX.1100.ARM64.Open

 # OSX x64
 - ${{ if eq(parameters.platform, 'OSX_x64') }}:
 - ${{ if eq(parameters.jobParameters.isFullMatrix, true) }}:
Expand Down

5 changes: 5 additions & 0 deletions
eng/pipelines/libraries/outerloop.yml

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -32,6 +32,8 @@ jobs:
 - Linux_arm64
 - Linux_musl_arm64
 - ${{ if eq(variables['includeOsxOuterloop'], true) }}: 
 - ${{ if eq(variables['isFullMatrix'], true) }}:
 - OSX_arm64
 - OSX_x64
 jobParameters:
 testGroup: innerloop
Expand All@@ -55,6 +57,7 @@ jobs:
 - Linux_musl_x64
 - Linux_musl_arm64
 - ${{ if and(eq(variables['includeOsxOuterloop'], true), eq(variables['isFullMatrix'], true)) }}:
 - OSX_arm64
 - OSX_x64
 helixQueuesTemplate: /eng/pipelines/libraries/helix-queues-setup.yml
 jobParameters:
Expand All@@ -76,6 +79,8 @@ jobs:
 - Linux_x64
 - Linux_musl_x64
 - ${{ if eq(variables['includeOsxOuterloop'], true) }}:
 - ${{ if eq(variables['isFullMatrix'], true) }}:
 - OSX_arm64
 - OSX_x64
 helixQueuesTemplate: /eng/pipelines/libraries/helix-queues-setup.yml
 jobParameters:
Expand Down

8 changes: 0 additions & 8 deletions
src/coreclr/pal/tests/palsuite/exception_handling/pal_sxs/test1/exceptionsxs.cpp

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -68,13 +68,6 @@ void sigsegv_handler(int code, siginfo_t *siginfo, void *context)

int main(int argc, char *argv[])
{
#if defined(TARGET_OSX) && defined(TARGET_ARM64)
 // Helix and/or build error...
 // dyld: Library not loaded: libpaltest_pal_sxs_test1_dll1.dylib
 printf("paltest_pal_sxs_test1 has been disabled on this platform\n");

 return PASS;
#else // defined(TARGET_OSX) && defined(TARGET_ARM64)
 struct sigaction newAction;
 struct sigaction oldAction;
 newAction.sa_flags = SA_SIGINFO | SA_RESTART;
Expand DownExpand Up@@ -120,5 +113,4 @@ int main(int argc, char *argv[])

 printf("ERROR: code was executed after the access violation.\n");
 return FAIL;
#endif // defined(TARGET_OSX) && defined(TARGET_ARM64)
}

5 changes: 0 additions & 5 deletions
src/coreclr/pal/tests/palsuite/file_io/ReadFile/test2/ReadFile.cpp

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -120,10 +120,6 @@ BOOL readTest_ReadFile_test2(DWORD dwByteCount, char cResult)

PALTEST(file_io_ReadFile_test2_paltest_readfile_test2, "file_io/ReadFile/test2/paltest_readfile_test2")
{
#if defined(TARGET_OSX) && defined(TARGET_ARM64)
 // Test hard codes PAGE_SIZE = 4096. Apple Silicon does not have 4K pages.
 printf("file_io_ReadFile_test2_paltest_readfile_test2 has been disabled on this platform\n");
#else // defined(TARGET_OSX) && defined(TARGET_ARM64)
 HANDLE hFile = NULL;
 const int BUFFER_SIZE = 2 * PAGESIZE;

Expand DownExpand Up@@ -191,7 +187,6 @@ PALTEST(file_io_ReadFile_test2_paltest_readfile_test2, "file_io/ReadFile/test2/p

	VirtualFree(readBuffer_ReadFile_test2, BUFFER_SIZE, MEM_RELEASE);
 PAL_Terminate();
#endif // defined(TARGET_OSX) && defined(TARGET_ARM64)
 return PASS;
}


5 changes: 0 additions & 5 deletions
src/coreclr/pal/tests/palsuite/filemapping_memmgt/VirtualProtect/test4/VirtualProtect.cpp

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -28,10 +28,6 @@ PALTEST(filemapping_memmgt_VirtualProtect_test4_paltest_virtualprotect_test4, "f
 ExitProcess(FAIL);
 }

#if defined(TARGET_OSX) && defined(TARGET_ARM64)
 // AppleSilcon does not allow mapping general pages as RWX
 printf("filemapping_memmgt_VirtualProtect_test4_paltest_virtualprotect_test4 has been disabled on this platform\n");
#else // defined(TARGET_OSX) && defined(TARGET_ARM64)
 //Allocate the physical storage in memory or in the paging file on disk 
 lpVirtualAddress = VirtualAlloc(NULL,//determine where to allocate the region
 REGIONSIZE, //specify the size
Expand DownExpand Up@@ -65,7 +61,6 @@ PALTEST(filemapping_memmgt_VirtualProtect_test4_paltest_virtualprotect_test4, "f
 {
 Fail("\nFailed to call VirtualFree API!\n");
 }
#endif // defined(TARGET_OSX) && defined(TARGET_ARM64)

 PAL_Terminate();
 return PASS;
Expand Down

17 changes: 17 additions & 0 deletions
src/coreclr/pal/tests/palsuite/issues.targets

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -32,4 +32,21 @@
 <Issue>https://github.com/dotnet/runtime/issues/42292</Issue>
 </ExcludeList>
 </ItemGroup>

 <ItemGroup Condition="'$(TargetArchitecture)' == 'arm64' and '$(TargetOS)' == 'OSX'">
 <ExcludeList Include="miscellaneous/queryperformancecounter/test1/paltest_queryperformancecounter_test1">
 <Issue>https://github.com/dotnet/runtime/issues/7639</Issue>
 </ExcludeList>
 <ExcludeList Include="exception_handling/pal_sxs/test1/paltest_pal_sxs_test1">
 <Issue>https://github.com/dotnet/runtime/issues/42292</Issue>
 </ExcludeList>
 <ExcludeList Include="file_io/ReadFile/test2/paltest_readfile_test2">
 <!-- Test hard codes PAGE_SIZE = 4096. Apple Silicon does not have 4K pages. -->
 <Issue>https://github.com/dotnet/runtime/issues/42292</Issue>
 </ExcludeList>
 <ExcludeList Include="filemapping_memmgt/VirtualProtect/test4/paltest_virtualprotect_test4">
 <!-- Test tries to set RWX on general memory page -->
 <Issue>https://github.com/dotnet/runtime/issues/7639</Issue>
 </ExcludeList>
 </ItemGroup>
</Project>

5 changes: 0 additions & 5 deletions
src/coreclr/pal/tests/palsuite/miscellaneous/queryperformancecounter/test1/test1.cpp

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -17,10 +17,6 @@

PALTEST(miscellaneous_queryperformancecounter_test1_paltest_queryperformancecounter_test1, "miscellaneous/queryperformancecounter/test1/paltest_queryperformancecounter_test1")
{
#if defined(TARGET_OSX) && defined(TARGET_ARM64)
 // We are see > 100 ms of average error on this platform
 printf("miscellaneous_queryperformancecounter_test1_paltest_queryperformancecounter_test1 has been disabled on this platform\n");
#else // defined(TARGET_OSX) && defined(TARGET_ARM64)
 /* Milliseconds of error which are acceptable Function execution time, etc.
 FreeBSD has a "standard" resolution of 50ms for waiting operations, so we
 must take that into account as well */
Expand DownExpand Up@@ -104,7 +100,6 @@ PALTEST(miscellaneous_queryperformancecounter_test1_paltest_queryperformancecoun
 /* Terminate the PAL.
 */ 
 PAL_Terminate();
#endif // defined(TARGET_OSX) && defined(TARGET_ARM64)
 return PASS;
}

Expand Down