Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

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

Jump to bottom

GetAndSet_AreThreadSafe_AndUpdatesNeverLeavesNullValues failing #72879

Open
danmoseley opened this issue Jul 26, 2022 · 6 comments
Open

GetAndSet_AreThreadSafe_AndUpdatesNeverLeavesNullValues failing #72879

danmoseley opened this issue Jul 26, 2022 · 6 comments
Labels
arch-arm32 arch-arm64 area-Extensions-Caching disabled-test The test is disabled in source code against the issue
Milestone
Future

Comments

@danmoseley
Copy link
Member

danmoseley commented Jul 26, 2022 •
edited by runfoapp bot
Loading 

Assert.False() Failure
Expected: False
Actual:   True


Stack trace
   at Microsoft.Extensions.Caching.Memory.MemoryCacheSetAndRemoveTests.GetAndSet_AreThreadSafe_AndUpdatesNeverLeavesNullValues() in /_/src/libraries/Microsoft.Extensions.Caching.Memory/tests/MemoryCacheSetAndRemoveTests.cs:line 588
   at System.RuntimeMethodHandle.InvokeMethod(Object target, Void** arguments, Signature sig, Boolean isConstructor)
   at System.Reflection.MethodInvoker.Invoke(Object obj, IntPtr* args, BindingFlags invokeAttr) in /_/src/libraries/System.Private.CoreLib/src/System/Reflection/MethodInvoker.cs:line 6

net7.0-Linux-Release-arm-CoreCLR_checked-(Alpine.314.Arm32.Open)Ubuntu.1804.ArmArch.Open@mcr.microsoft.com/dotnet-buildtools/prereqs:alpine-3.14-helix-arm32v7-20210910135806-8a6f4f3

Perhaps Linux specific? Creating issue so I can reference when disabling.

Runfo Tracking Issue: GetAndSet_AreThreadSafe_AndUpdatesNeverLeavesNullValues failing

Build Definition Kind Run Name

Build Result Summary

Day Hit Count Week Hit Count Month Hit Count
0 0 0
@danmoseley danmoseley added blocking-clean-ci Blocking PR or rolling runs of 'runtime' or 'runtime-extra-platforms' area-Extensions-Caching labels Jul 26, 2022
@ghost ghost added the untriaged New issue has not been triaged by the area owner label Jul 26, 2022
@danmoseley danmoseley mentioned this issue Jul 26, 2022
Closed
@ghost
Copy link

ghost commented Jul 26, 2022

Tagging subscribers to this area: @dotnet/area-extensions-caching
See info in area-owners.md if you want to be subscribed.

Issue Details 
Assert.False() Failure
Expected: False
Actual:   True


Stack trace
   at Microsoft.Extensions.Caching.Memory.MemoryCacheSetAndRemoveTests.GetAndSet_AreThreadSafe_AndUpdatesNeverLeavesNullValues() in /_/src/libraries/Microsoft.Extensions.Caching.Memory/tests/MemoryCacheSetAndRemoveTests.cs:line 588
   at System.RuntimeMethodHandle.InvokeMethod(Object target, Void** arguments, Signature sig, Boolean isConstructor)
   at System.Reflection.MethodInvoker.Invoke(Object obj, IntPtr* args, BindingFlags invokeAttr) in /_/src/libraries/System.Private.CoreLib/src/System/Reflection/MethodInvoker.cs:line 6

net7.0-Linux-Release-arm-CoreCLR_checked-(Alpine.314.Arm32.Open)Ubuntu.1804.ArmArch.Open@mcr.microsoft.com/dotnet-buildtools/prereqs:alpine-3.14-helix-arm32v7-20210910135806-8a6f4f3

Perhaps Linux specific? Creating issue so I can reference when disabling.

Author: danmoseley
Assignees: -
Labels:

blocking-clean-ci, area-Extensions-Caching
Milestone: -

@runfoapp runfoapp bot mentioned this issue Jul 26, 2022
Closed
@danmoseley
Copy link
Member Author

It's Arm/Arm64 only?

Started QueueName OS Architecture
2022-07-26 06:58:30.5840000 ubuntu.1804.armarch.open Debian.10.Arm32.Open arm
2022-07-26 06:58:31.6350000 ubuntu.1804.armarch.open Debian.11.Arm32.Open arm
2022-07-26 07:40:01.1380000 ubuntu.1804.armarch.open Ubuntu.1804.Arm32.Open arm
2022-07-26 07:40:02.3870000 ubuntu.1804.armarch.open Ubuntu.1804.Arm32.Open arm
2022-07-26 08:21:28.9620000 windows.10.arm64v8.open Windows.10.Arm64v8.Open arm64
2022-07-26 08:21:32.9770000 windows.10.arm64v8.open Windows.10.Arm64v8.Open arm64
2022-07-26 08:28:14.1640000 ubuntu.1804.armarch.open Ubuntu.1804.Arm32.Open arm
2022-07-26 09:01:46.0320000 windows.10.arm64v8.open Windows.10.Arm64v8.Open arm64
2022-07-26 09:17:31.4570000 windows.10.arm64v8.open Windows.10.Arm64v8.Open arm64
2022-07-26 09:34:12.9690000 windows.10.arm64v8.open Windows.10.Arm64v8.Open arm64
2022-07-26 09:48:48.9630000 windows.10.arm64v8.open Windows.10.Arm64v8.Open arm64
2022-07-26 09:51:14.7440000 ubuntu.1804.armarch.open Debian.11.Arm32.Open arm
2022-07-26 10:04:30.5840000 ubuntu.1804.armarch.open Alpine.314.Arm32.Open arm
2022-07-26 10:49:40.2350000 ubuntu.1804.armarch.open Alpine.314.Arm32.Open arm
2022-07-26 10:57:55.9570000 ubuntu.1804.armarch.open Debian.10.Arm32.Open arm
2022-07-26 13:13:47.7310000 ubuntu.1804.armarch.open Debian.10.Arm32.Open arm
2022-07-26 13:13:49.0750000 ubuntu.1804.armarch.open Debian.11.Arm32.Open arm
2022-07-26 15:00:56.3730000 ubuntu.1804.armarch.open Alpine.314.Arm32.Open arm
2022-07-26 15:59:14.2010000 ubuntu.1804.armarch.open Debian.11.Arm32.Open arm

@danmoseley danmoseley mentioned this issue Jul 26, 2022
Merged
@danmoseley
Copy link
Member Author

@vonzshik pointed out a memory model bug here #72821 (comment)

Such a bug could be Arm-specific because it has a weaker memory model, so missing 'volatile' is more likely to show up there.

@dotnet/area-extensions-caching owners, there is no evidence this is a regression in 7.0, so I'd be inclined to leave as-is until we fork. thoughts?

@maryamariyan maryamariyan removed the untriaged New issue has not been triaged by the area owner label Jul 26, 2022
@maryamariyan maryamariyan added this to the Future milestone Jul 26, 2022
@VincentBu
Copy link
Contributor

Failed again in: runtime-coreclr libraries-jitstress 20220726.1

Failed test:

net7.0-windows-Release-arm64-CoreCLR_checked-jitstress1_tiered-Windows.10.Arm64v8.Open
- Microsoft.Extensions.Caching.Memory.MemoryCacheSetAndRemoveTests.GetAndSet_AreThreadSafe_AndUpdatesNeverLeavesNullValues

net7.0-windows-Release-arm64-CoreCLR_checked-tailcallstress-Windows.10.Arm64v8.Open
- Microsoft.Extensions.Caching.Memory.MemoryCacheSetAndRemoveTests.GetAndSet_AreThreadSafe_AndUpdatesNeverLeavesNullValues

net7.0-windows-Release-arm64-CoreCLR_checked-jitstress2_tiered-Windows.10.Arm64v8.Open
- Microsoft.Extensions.Caching.Memory.MemoryCacheSetAndRemoveTests.GetAndSet_AreThreadSafe_AndUpdatesNeverLeavesNullValues

net7.0-windows-Release-arm64-CoreCLR_checked-zapdisable-Windows.10.Arm64v8.Open
- Microsoft.Extensions.Caching.Memory.MemoryCacheSetAndRemoveTests.GetAndSet_AreThreadSafe_AndUpdatesNeverLeavesNullValues

net7.0-windows-Release-arm64-CoreCLR_checked-jitstress2-Windows.10.Arm64v8.Open
- Microsoft.Extensions.Caching.Memory.MemoryCacheSetAndRemoveTests.GetAndSet_AreThreadSafe_AndUpdatesNeverLeavesNullValues

net7.0-Linux-Release-arm-CoreCLR_checked-no_tiered_compilation-(Ubuntu.1804.Arm32.Open)Ubuntu.1804.Armarch.Open@mcr.microsoft.com/dotnet-buildtools/prereqs:ubuntu-18.04-helix-arm32v7-bfcd90a-20200121150440
- Microsoft.Extensions.Caching.Memory.MemoryCacheSetAndRemoveTests.GetAndSet_AreThreadSafe_AndUpdatesNeverLeavesNullValues

net7.0-Linux-Release-arm-CoreCLR_checked-zapdisable-(Ubuntu.1804.Arm32.Open)Ubuntu.1804.Armarch.Open@mcr.microsoft.com/dotnet-buildtools/prereqs:ubuntu-18.04-helix-arm32v7-bfcd90a-20200121150440
- Microsoft.Extensions.Caching.Memory.MemoryCacheSetAndRemoveTests.GetAndSet_AreThreadSafe_AndUpdatesNeverLeavesNullValues

net7.0-Linux-Release-arm-CoreCLR_checked-jitminopts-(Ubuntu.1804.Arm32.Open)Ubuntu.1804.Armarch.Open@mcr.microsoft.com/dotnet-buildtools/prereqs:ubuntu-18.04-helix-arm32v7-bfcd90a-20200121150440
- Microsoft.Extensions.Caching.Memory.MemoryCacheSetAndRemoveTests.GetAndSet_AreThreadSafe_AndUpdatesNeverLeavesNullValues

net7.0-windows-Release-arm64-CoreCLR_checked-no_tiered_compilation-Windows.10.Arm64v8.Open
- Microsoft.Extensions.Caching.Memory.MemoryCacheSetAndRemoveTests.GetAndSet_AreThreadSafe_AndUpdatesNeverLeavesNullValues

net7.0-windows-Release-x86-CoreCLR_checked-jitstress2-Windows.10.Amd64.Open
- System.Runtime.Intrinsics.Tests.Vectors.Vector64Tests.Vector64Int64SumTest

Error message:

Assert.False() Failure
Expected: False
Actual:   True


Stack trace
   at Microsoft.Extensions.Caching.Memory.MemoryCacheSetAndRemoveTests.GetAndSet_AreThreadSafe_AndUpdatesNeverLeavesNullValues() in /_/src/libraries/Microsoft.Extensions.Caching.Memory/tests/MemoryCacheSetAndRemoveTests.cs:line 588
   at System.RuntimeMethodHandle.InvokeMethod(Object target, Void** arguments, Signature sig, Boolean isConstructor)
   at System.Reflection.MethodInvoker.Invoke(Object obj, IntPtr* args, BindingFlags invokeAttr) in /_/src/libraries/System.Private.CoreLib/src/System/Reflection/MethodInvoker.cs:line 64

@ericstj ericstj mentioned this issue Jul 27, 2022
Closed
@runfoapp runfoapp bot removed this from the Future milestone Jul 27, 2022
@runfoapp runfoapp bot closed this as completed Jul 27, 2022
@ericstj ericstj reopened this Jul 27, 2022
@ghost ghost added the untriaged New issue has not been triaged by the area owner label Jul 27, 2022
@danmoseley danmoseley removed the blocking-clean-ci Blocking PR or rolling runs of 'runtime' or 'runtime-extra-platforms' label Jul 27, 2022
This was referenced Jul 27, 2022
Merged
Merged
@eerhardt eerhardt added the disabled-test The test is disabled in source code against the issue label Aug 1, 2022
@eerhardt eerhardt added this to the Future milestone Aug 1, 2022
@ghost ghost removed the untriaged New issue has not been triaged by the area owner label Aug 1, 2022
@eerhardt
Copy link
Member

eerhardt commented Aug 9, 2022

@v-wenyuxu - that stack trace looks like a different test. Also the test being tracked here has been disabled for over a week.

@VincentBu
Copy link
Contributor

@v-wenyuxu , you need to open a new issue for that test and delete that comment.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
arch-arm32 arch-arm64 area-Extensions-Caching disabled-test The test is disabled in source code against the issue
Projects
None yet
Milestone
Future
Development

No branches or pull requests
6 participants
@maryamariyan @adamsitnik @danmoseley @eerhardt @ericstj @VincentBu