Adam Sitnik [Sat, 26 Jun 2021 21:20:32 +0000 (23:20 +0200)]
More file stream options tests (#53982)
* add more tests
* StreamWriter and StreamReader require FileStreamOptions with necessary access
* allow for bufferSize == 0
* extend the test
* refactor the tests
* Revert "allow for bufferSize == 0"
This reverts commit
f1259be18a3446c1b8939883f484cc28347c74cf.
Joseph Da Silva [Sat, 26 Jun 2021 19:33:01 +0000 (12:33 -0700)]
Fix BigInteger hex string parsing bug (#54251) (#54262)
Stephen Toub [Sat, 26 Jun 2021 19:30:56 +0000 (15:30 -0400)]
Fix WaitForNextTickAsync_CanceledWaitThenWaitAgain_Succeeds test (#54775)
There's a race condition in the test between the timer firing and cancellation being requested. It repros more on Linux because there's a smaller quantum on Linux than on Windows.
Danny Friar [Sat, 26 Jun 2021 19:29:30 +0000 (20:29 +0100)]
Expose underlying unix file descriptor in SafeMemoryMappedFileHandle (#53538)
* expose unix file descriptor safe memory map
* store handle after DangerousAddRef
* add test verifying handle matches filestream handle
* Update src/libraries/System.IO.MemoryMappedFiles/src/Microsoft/Win32/SafeMemoryMappedFileHandle.Unix.cs
Co-authored-by: Stephen Toub <stoub@microsoft.com>
* Update src/libraries/System.IO.MemoryMappedFiles/src/Microsoft/Win32/SafeMemoryMappedFileHandle.Unix.cs
Co-authored-by: Stephen Toub <stoub@microsoft.com>
* Update src/libraries/System.IO.MemoryMappedFiles/src/Microsoft/Win32/SafeMemoryMappedFileHandle.Unix.cs
Co-authored-by: Stephen Toub <stoub@microsoft.com>
* Update src/libraries/System.IO.MemoryMappedFiles/src/Microsoft/Win32/SafeMemoryMappedFileHandle.Unix.cs
Co-authored-by: Stephen Toub <stoub@microsoft.com>
* Update src/libraries/System.IO.MemoryMappedFiles/tests/MemoryMappedFile.CreateFromFile.Tests.cs
Co-authored-by: Stephen Toub <stoub@microsoft.com>
* Dispose MemoryMappedFile in MapHandleMatchesFileStreamHandle test
* Update src/libraries/System.IO.MemoryMappedFiles/src/Microsoft/Win32/SafeMemoryMappedFileHandle.Unix.cs
Co-authored-by: Stephen Toub <stoub@microsoft.com>
* Update MemoryMappedFile.CreateFromFile.Tests.cs
* Move mmf in test to using block
* Fix test
Co-authored-by: Danny Friar <danny@Dannys-MacBook-Pro.local>
Co-authored-by: Stephen Toub <stoub@microsoft.com>
Theodore Tsirpanis [Sat, 26 Jun 2021 19:11:08 +0000 (22:11 +0300)]
Use an untyped `Task` instead of a `Task<bool>` in `TimerQueueTimer`. (#54455)
Tomas Weinfurt [Sat, 26 Jun 2021 18:20:39 +0000 (20:20 +0200)]
add pre-cancellation check and enable ReadWriteAsync_PrecanceledOperations_ThrowsCancellationException for QUIC (#54540)
Vlad Brezae [Sat, 26 Jun 2021 18:15:41 +0000 (21:15 +0300)]
[interp] Don't cprop between vt vars of different sizes (#54734)
Via unsafe code, getting the lower Vector128 from a Vector256 ends up as a move of `sizeof (Vector128)` from a Vector256 var. However, the destination var is not a valid copy of the source var, having a different type
Juan Hoyos [Sat, 26 Jun 2021 17:52:23 +0000 (10:52 -0700)]
Revert "Fix 54025 (#54070)" (#54728)
This reverts commit
b2fe6678282503374aeea7ab5f468f7a3cc85a2f.
Kevin Ransom (msft) [Sat, 26 Jun 2021 13:51:19 +0000 (06:51 -0700)]
Add metadatatoken override to SymbolMethod (#54656)
* Add metadatatoken override to SymbolMethod
* Add test
Co-authored-by: Jan Kotas <jkotas@microsoft.com>
Oded Hanson [Sat, 26 Jun 2021 13:37:59 +0000 (16:37 +0300)]
minor optimization: move creation of X509 Subject Summary into catch block on MacOS. (#54774)
Sergey Andreenko [Sat, 26 Jun 2021 08:24:54 +0000 (01:24 -0700)]
Extract fgMorph(Init/Copy)Block into their own classes. (#53983)
* Rewrite fgMorph(Copy/Init)Block.
* fix a last squash error.
* fix nits
David Fowler [Sat, 26 Jun 2021 07:52:38 +0000 (00:52 -0700)]
Clean up some DI code (#54732)
- Remove empty CreateInstanceCallSite file
- Removed ServiceScopeFactoryCallsite and used ConstantCallsite instead.
- Added fast path if the value is already cached on the callsite.
Aleksey Kliger (λgeek) [Sat, 26 Jun 2021 07:30:00 +0000 (03:30 -0400)]
Miscellaneous MetadataUpdater cleanups (#54751)
Geoff Kizer [Sat, 26 Jun 2021 04:10:32 +0000 (21:10 -0700)]
Move remote server redirect and decompression tests to HttpClientHandlerTest.RemoteServer.cs (#54738)
* move remote redirect and decompression tests to HttpClientHandler_RemoteServerTest
Co-authored-by: Geoffrey Kizer <geoffrek@windows.microsoft.com>
David Wrighton [Sat, 26 Jun 2021 03:21:40 +0000 (20:21 -0700)]
Fix gc hole in IO thread pool (#54769)
Geoff Kizer [Sat, 26 Jun 2021 01:45:42 +0000 (18:45 -0700)]
HTTP3: Re-enable cookie and cancellation tests (#54727)
* fix an issue with GOAWAY handling and enable cookie tests
* disable ConnectTimeout test for HTTP3
* fix Expect 100 continue handling in HTTP3
* add and use GenericLoopbackServer.SendPartialResponseHeadersAsync
* enable cancellation tests for HTTP3
* disable failing interop tests
Co-authored-by: Geoffrey Kizer <geoffrek@windows.microsoft.com>
Layomi Akinrinade [Fri, 25 Jun 2021 23:09:23 +0000 (16:09 -0700)]
Clean up JSON source gen APIs (#54527)
* Clean up JSON source gen APIs
* Address review feedback
* Simplify GenerateX computation
David Wrighton [Fri, 25 Jun 2021 22:06:22 +0000 (15:06 -0700)]
Fix handling of static virtual method implementation checking (#54710)
- It turns out that GetMethodDescFromMemberDefOrRefOrSpec and FindOrCreateAssociatedMethodDesc are not safe to use on a MemberRef whent the associated MethodTable is not fully loaded.
- Instead only use that feature when working with a MethodDef or a fully loaded type, and when working with a not fully loaded type, use MemberLoader::FindMethod instead.
- When running the resolution algorithm for doing constraint validation, it also is not necessary to fully resolve to the exact correct MethodDesc, which as that process uses FindOrCreateAssociatedMethodDesc needs to be avoided.
- The above was not evident as in many cases, the validation algorithm did not run as it was misplaced and located directly before the call to SetIsFullyLoaded. That code path is only followed if the type is able to fully load without circular dependencies. (Test case CuriouslyRecurringGenericWithUnimplementedMethod added to cover that scenario)
- In addition, while investigating these issues, I realized we were lacking checks that the constraints on the impl and decl method were not checked at during type load, but that work was instead deferred to dispatch time. Along with the constraint check there was also a set of accessibility checks that had been missed that are common to all MethodImpl handling. Fix by adding tweaking the logic to share most of that code.
David Wrighton [Fri, 25 Jun 2021 22:05:56 +0000 (15:05 -0700)]
Fix thread safety of _typeRefsInCompilationModuleSet (#54760)
David Wrighton [Fri, 25 Jun 2021 18:10:50 +0000 (11:10 -0700)]
Tweak classcompat loader to skip MethodImpls associated with static methods (#54658)
Jan Vorlicek [Fri, 25 Jun 2021 17:06:37 +0000 (19:06 +0200)]
Fix BackoutJitData (#54711)
* Fix BackoutJitData
The RemoveJitData that the BackoutJitData calls requires the code
header to be copied to the final location. This change fixes it.
I've also found that in one of my previous changes, I've accidentally
enabled jitting into a scratch buffer by default by adding the
FEATURE_WXORX define unconditionally. So I am removing it in this change
for non Apple Silicon, it will be replaced by a dynamic check whether W^X
is enabled in the final W^X change.
Zoltan Varga [Fri, 25 Jun 2021 16:55:02 +0000 (12:55 -0400)]
[mono][jit] Refactor the generic sharing code a bit. NFC. (#54705)
* [jit] Refactor the emit_rgctx code a bit.
* Add an enum to specify the way the rgctx is accessed from a gshared method.
Use it to simplify some logic.
Pavel Savara [Fri, 25 Jun 2021 14:14:54 +0000 (16:14 +0200)]
Revert "[wasm] browser http response stream could be seekable (#54603)" (#54742)
This reverts commit
c139d00640485e8c231c54d1573dc02d7c9e8daf.
Pavel Savara [Fri, 25 Jun 2021 08:56:28 +0000 (10:56 +0200)]
[wasm] browser http response stream could be seekable (#54603)
- browser http response stream could be seekable
- test WebAssemblyEnableStreamingResponse
Stephen Toub [Fri, 25 Jun 2021 07:35:29 +0000 (03:35 -0400)]
Override more Stream members on System.IO.Compression streams (#54518)
Přemek Vysoký [Fri, 25 Jun 2021 07:07:51 +0000 (09:07 +0200)]
[main] Update dependencies from 9 repositories (#54543)
SingleAccretion [Fri, 25 Jun 2021 06:37:09 +0000 (09:37 +0300)]
Fix CQ regression & correctness bug in morphing of long muls (#53566)
* Add a test covering GTF_MUL_64RSLT transform
* Disable the test on Mono
* Add genActualTypeIsInt helper
* Add some gtFlags helpers
* Prepare decomposition for new long muls
* Update gtIsValid64RsltMul
To understand the new format for long muls.
* Rework morphing of long muls
Previously, morph was looking for the exact pattern of
MUL(CAST(long <- int), CAST(long <- int)) when assessing
candidacy of GT_MUL for being marked with "GTF_MUL_64RSLT" and
emitted as "long mul".
This worked fine, until the importer was changed to fold
all casts with constant operands. This broke the pattern
matching and thus all MULs in the form of (long)value * 10
started being emitted as helper calls.
This change updates morph to understand the new folded casts
and in general updates the "format" of long mul from
"CAST * CAST" to "CAST * (CAST | CONST)".
In the process, new helper functions have been introduced, to
avoid bloating fgMorphSmpOp with the new sizeable logic.
Recognition of overflowing cases has been upgraded, and a correctness
bug, where "checked((long)uint.MaxValue * (long)uint.MaxValue)"
was wrongly treated as non-overflowing, fixed.
Additionally, the logic to emit intermediate NOPs has been
changed to instead always skip morphing the casts themselves,
even when remorphing.
* Add the script to generate the longmul test
The test itself has been regenerated using it and
there were no diffs, as expected.
SingleAccretion [Fri, 25 Jun 2021 06:32:54 +0000 (09:32 +0300)]
Eliminate chained casts to small types (#52561)
We can take advantage of the implicit zero/sign-extension for
small integer types and eliminate some casts.
SingleAccretion [Fri, 25 Jun 2021 06:32:44 +0000 (09:32 +0300)]
Optimize `CAST(int <- long)` on 32 bit targets (#53040)
* Optimize CAST(int <- long) on 32 bit targets
* Revert "Optimize CAST(int <- long) on 32 bit targets"
Revert the implementation in lowering
* Optimize CAST(int <- long) on 32 bit targets
Move the code from lowering to long decomposition.
* Fixed the "Arguments" note for DecomposeNode
* Added the function header
For OptimizeCastFromDecomposedLong.
* Remove the TODO comment
While correct, it has questionable value.
* Add a more detailed dump output
* Do not try to optimize checked casts
It is easy to get it wrong. Let the frontend handle this.
* Do not depend on tree order
Previous version of the code assumed that there could be
no nodes between the cast and its operand. That is not
a correct assumption to make in LIR.
Mike McLaughlin [Fri, 25 Jun 2021 05:54:28 +0000 (22:54 -0700)]
Move the metadata update related APIs to the MetadataUpdater class (#54590)
Move the metadata update related APIs to the MetadataUpdater class
The old APIs AssemblyExtensions.ApplyUpdate() and AssemblyExtensions.GetApplyUpdateCapabilities() will be removed
after all the references to them have been changed to the new APIs.
Add the new IsSupported API described in issue https://github.com/dotnet/runtime/issues/51159.
Change the tests to use the MetadataUpdater APIs.
Fix the ApplyUpdate qcalls and icalls
Add the ILLink substitutions for MetadataUpdater.IsSupported property
Change the old APIs to call the new ones
Update mono's MetadataUpdater.IsSupported property
Update feature switch doc
Fixed the argument checking in coreclr's MetadataUpdater.ApplyUpdate().
Marie Píchová [Fri, 25 Jun 2021 04:37:07 +0000 (06:37 +0200)]
S.N.Quic made private (#54610)
Note this will need reaction on ASP.NET side.
Egor Chesakov [Fri, 25 Jun 2021 03:26:08 +0000 (20:26 -0700)]
Disallow promotion of HVA structs when their fields of TYP_SIMD8 were promoted as plain structs (#54694)
Thays Grazia [Fri, 25 Jun 2021 01:21:57 +0000 (22:21 -0300)]
[wasm][debugger] Detect initial status of pause on exceptions. (#54040)
* Detect initial status of pause on exceptions.
* Changing what @radical suggested.
* Changing more things.
* Test case created.
I could not test the pause on "all" exceptions because if I enable the pause on caught exceptions and reload the page it will stop in a lot of exceptions other then the one that I inserted in AttachToTarget.
* Adding a test for Reload page with ALL set.
* Fixing merge conflicts.
* setting icordebug = false.
* Removing unrelated change.
Jeremy Koritzinsky [Thu, 24 Jun 2021 23:40:40 +0000 (16:40 -0700)]
defMAC construction up a scope to make it live long enough. (#54702)
Fixes #54649
Steve Pfister [Thu, 24 Jun 2021 23:19:11 +0000 (19:19 -0400)]
Fix MonoCrossAOT.UnixFilePermissions for wasm (#54693)
* Fix MonoCrossAOT.UnixFilePermissions for wasm
We introduced Microsoft.NETCore.App.MonoCrossAOT.UnixFilePermissions.xml in https://github.com/dotnet/runtime/pull/54501 to make sure the right permissions are set when installing AOT compiler workload packs. We hardcoded the list to include mono-aot-cross, llc, and opt. However, in wasm's case they only have mono-aot-cross.
This change makes the xml file a template and only includes mono-aot-cross for browser and all three for the other configurations.
Fixes https://github.com/dotnet/runtime/issues/54612
* Use property instead of item for the file nodes
* Ankit suggestion
Co-authored-by: Steve Pfister <steve.pfister@microsoft.com>
Maxim Lipnin [Thu, 24 Jun 2021 21:01:06 +0000 (00:01 +0300)]
Mark AssemblyDependencyResolver as unsupported on Browser and mobile (#54601)
* Mark AssemblyDependencyResolver as unsupported on Browser and mobile
* Apply AlekseyK's workaround to fix android CI builds
* Add an annotation for maccatalyst
imhameed [Thu, 24 Jun 2021 20:56:27 +0000 (13:56 -0700)]
[mono] Implement Rdm and Dp (#49737)
See https://github.com/dotnet/runtime/issues/42322 and
https://github.com/dotnet/runtime/issues/42280.
Tested manually on an arm64 Linux VM running on an M1 Mac Mini. Does not
enable RDM or DP when AOT-compiling the associated runtime tests; our CI
hardware doesn't support these extensions yet.
Aleksey Kliger (λgeek) [Thu, 24 Jun 2021 20:56:12 +0000 (16:56 -0400)]
Pass the default ALC gchandle to the v3 preload hook (#54686)
* Pass the default ALC gchandle to the v3 preload hook
Fixes crashes early during startup when the gchandle is then used by the hooks
to call mono API functions. The default ALC exists, but its managed object
doesn't, so the gchandle target is null. The mono APIs detect the the special
gchandle and resolve it to the default ALC
Example crash
https://gist.github.com/grendello/
b4ab24587a055725cc5e1416b86ad7ca
* [alc] Assert that we never see a null target from a managed ALC gchandle
Miha Zupan [Thu, 24 Jun 2021 20:45:52 +0000 (13:45 -0700)]
Add ActivitySource support to DiagnosticsHandler (#54437)
* Add ActivitySource support to DiagnosticsHandler
* Use ActivitySource as another enabled condition
* Make IsGloballyEnabled a property
* Simplify enabled check
* Revert using the exception filter
* Update HTTP ILLink.Substitutions.xml
Tom McDonald [Thu, 24 Jun 2021 20:17:39 +0000 (13:17 -0700)]
Add CustomAttributes ApplyUpdate Capability (#54619)
* Add CustomAttributes ApplyUpdate Capability
* Rename CustomAttributes ChangeCustomAttributes per #54284
Steve Harter [Thu, 24 Jun 2021 19:35:35 +0000 (14:35 -0500)]
Support synchronous Stream-based serialization methods (#54632)
SingleAccretion [Thu, 24 Jun 2021 19:34:08 +0000 (22:34 +0300)]
Add args descriptions for VNF_MapStore and VNF_MapSelect (#54108)
Anton Lapounov [Thu, 24 Jun 2021 19:09:36 +0000 (12:09 -0700)]
Ensure relocation blocks are 4-byte aligned (#54668)
Zoltan Varga [Thu, 24 Jun 2021 19:05:15 +0000 (15:05 -0400)]
[mono][wasm] Reenable the remove finally pass. (#54450)
* [mono][wasm] Reenable the remove finally pass.
* [mono][wasm] Run methods which need stack walks using the interpreter instead of pushing an LMF.
* Disable vtype based sharing for vtypes with explicit sizes.
Ryan Lucia [Thu, 24 Jun 2021 18:56:04 +0000 (14:56 -0400)]
Remove CoffeeFlux from CODEOWNERS (#54696)
Thays Grazia [Thu, 24 Jun 2021 18:36:56 +0000 (15:36 -0300)]
[wasm][debugger] Fixing debugger tests errors (#54664)
* Fixing new debugger tests errors.
Errno is not being assigned anymore.
DebuggerTests.SetVariableValueTests
* Passing icordbg flag as false.
* Adding new tests.
Bill Wert [Thu, 24 Jun 2021 17:46:31 +0000 (10:46 -0700)]
Add non-LLVM iOS size scenario (#54585)
* Add non-LLVM iOS size scenario
* add category
* remove parens
* variables in the wrong place.
* move setting configs to before it is written
Jakob Botsch Nielsen [Thu, 24 Jun 2021 17:25:54 +0000 (19:25 +0200)]
Fix instruction hex display (#54675)
Natalia Kondratyeva [Thu, 24 Jun 2021 17:24:56 +0000 (19:24 +0200)]
[QUIC] Add Windows version check to QUIC initialization (#54488)
Add Windows version check to QUIC initialization, log min supported and current Windows version
Krzysztof Wicher [Thu, 24 Jun 2021 17:20:36 +0000 (19:20 +0200)]
Fix Activator.CreateInstance(GetType(), ...) related ILLink warnings in (#54680)
System.Data.Common
Alexander Köplinger [Thu, 24 Jun 2021 15:43:55 +0000 (17:43 +0200)]
Disable MacCatalyst arm64 PR test runs on staging pipeline (#54678)
We don't have enough capacity right now on Helix to handle the load.
Daniel Genkin [Thu, 24 Jun 2021 15:39:04 +0000 (11:39 -0400)]
[WASM] Fix async/await in config loading (#54652)
* Fixed config issue
* Updated Hot Reload test
monojenkins [Thu, 24 Jun 2021 15:33:14 +0000 (11:33 -0400)]
Fix for heap_use_after_free flagged by sanitizer (#54679)
Copy of https://github.com/mono/mono/pull/21120
Co-authored-by: dseshadri <dseshadri@users.noreply.github.com>
Radek Doulik [Thu, 24 Jun 2021 15:20:26 +0000 (17:20 +0200)]
[wasm] Bump emscripten to 2.0.23 (#53603)
Bumps emscripten to 2.0.23
The Browser AOT tests now use `-Wl,-lto-O0` option to reduce memory usage of `wasm-ld` tool, which was in some cases going over avaiable 8GB on helix machines.
* Revert "Add ActiveIssue to the MemoryMappedFiles tests"
This reverts commit
ec1ae530606ef1061680600fc046226cc1c4cbc3.
* Revert "Add ActiveIssue attr to the FileSystem tests"
This reverts commit
356b3ff2a703980ac01b9df697a594e8c341c436.
* Bump emscripten version to 2.0.23
* Use newer docker images with 2.0.23
* Update docs
* Use 2.0.23 emscripten nuget packages
* Revert "Revert "Add ActiveIssue attr to the FileSystem tests""
This reverts commit
eb2f9548b08c114b359fab8d867ba50de098fe48.
The fix is not present in 2.0.23
* Revert "Revert "Add ActiveIssue to the MemoryMappedFiles tests""
This reverts commit
8be39f583499a8d8451034c65260a785330b0795.
The fix is not present in 2.0.23
* Increase timeout for AOT tests
* Add description of emscripten bump to README
* Try to get information about resources
* Get all limits
* Escape & chars
* Reduce platform matrix
* Lets try one more build with doubled timeout
* Revert "Lets try one more build with doubled timeout"
This reverts commit
67dd7754bb79218b2c6b687034162d041715093e.
* Try -Wl,-O0 on CI
To be sure it behaves the same as in local build
* Use -Wl,-lto-O0 do lower link time optimization
It looks like it reduces the memory load a lot
* Set EmccLinkOptimizationFlag for AOT tests
And reset the default value
* Escape commas
* Revert "Reduce platform matrix"
This reverts commit
fec0e557208eb165824e75cd57b895a74d164de4.
* Remove resource info retrieval
* Bump emsdk versions
Co-authored-by: Larry Ewing <lewing@microsoft.com>
Viktor Hofer [Thu, 24 Jun 2021 14:46:39 +0000 (16:46 +0200)]
Fix compiler references when building inside VS (#54614)
If for a source project a contract project exists, then the contract project's TargetPath should be passed to the compiler. This is handled by the SDK by default when `ProduceReferenceAssembly` is true. As dotnet/runtime doesn't use the `ProduceReferenceAssembly` feature yet, a custom target adds the necessary `ReferenceAssembly` metadata to the `TargetPathWithTargetPlatformMoniker` item which then is transformed to references for the compiler. That works fine on the CLI as the `GetTargetPathWithTargetPlatformMoniker` target runs after the ProjectReference to the ContractProject is resolved and its target path is available.
Inside VS the target ordering is different and the `ResolvedMatchingContract` item was empty as the ProjectReference to the contract wasn't yet resolved. The fix for that is to add a dependency onto the `ResolveProjectReferences` target to guarantee that the `ResolvedMatchingContract` item is populated in time.
Noticed this when the build of System.ComponentModel.Composition.Registration failed because the implementation assembly of System.ComponentModel.Composition was passed to the compiler instead of the reference assembly.
Tomas Weinfurt [Thu, 24 Jun 2021 14:31:07 +0000 (16:31 +0200)]
process more TLS frames at one when available (#50815)
* process more TLS frames when available
* add SslStream.Implementation.cs
* remove extra comment
* add back DefaultRequestHeaders_SentUnparsed
* feedback from review
* fix winhttp
* fix linker test
* feedback from review
* feedback from review
* feedback from review
* make EnsureFullTlsFrame async
* feedback from review
Stephen Toub [Thu, 24 Jun 2021 14:24:47 +0000 (10:24 -0400)]
Add PeriodicTimer (#53899)
LateApexEarlySpeed [Thu, 24 Jun 2021 12:30:55 +0000 (20:30 +0800)]
UdpClient with span support (#53429)
Add API from #864
Pavel Savara [Thu, 24 Jun 2021 12:06:06 +0000 (14:06 +0200)]
exclude fragile tests (#54671)
Adam Sitnik [Thu, 24 Jun 2021 10:38:53 +0000 (12:38 +0200)]
get last error before calling a method that might fail as well (#54667)
Adam Sitnik [Thu, 24 Jun 2021 06:58:33 +0000 (08:58 +0200)]
[FileStream] add tests for device and UNC paths (#54545)
* add a test for unseekable device by using a path to named pipe
* add a test for seekable device by using DeviceID instead of drive letter
* add a test for a UNC file path (local file share)
Dan Moseley [Thu, 24 Jun 2021 04:35:39 +0000 (22:35 -0600)]
Fix sporadic double fd close (#54660)
Fix https://github.com/dotnet/runtime/issues/54589
Stephen Toub [Thu, 24 Jun 2021 04:00:01 +0000 (00:00 -0400)]
Remove Version.Clone from AssemblyName.Clone (#54621)
Version is immutable.
Radek Doulik [Thu, 24 Jun 2021 03:52:40 +0000 (05:52 +0200)]
[wasm] Enable fixed libraries tests (#54641)
Fixed by https://github.com/dotnet/runtime/pull/52705 and
https://github.com/dotnet/runtime/pull/52707
Ankit Jain [Thu, 24 Jun 2021 02:29:00 +0000 (22:29 -0400)]
[wasm] Fix blazor/aot builds (#54651)
`dotnet\packs\Microsoft.NET.Runtime.WebAssembly.Sdk\6.0.0-preview.7.21321.15\Sdk\WasmApp.Native.targets(342,5): error : Could not find AOT cross compiler at $(_MonoAotCrossCompilerPath)=`
Make sure this is set for the aot path.
Thays Grazia [Thu, 24 Jun 2021 01:49:43 +0000 (22:49 -0300)]
[mono][wasm] Fix compilation error on wasm (#54659)
Miha Zupan [Thu, 24 Jun 2021 00:21:42 +0000 (17:21 -0700)]
Fix telemetry for Socket connects to Dns endpoints (#54071)
Aleksey Kliger (λgeek) [Thu, 24 Jun 2021 00:13:37 +0000 (20:13 -0400)]
[wasm] Build static components; include hot_reload in runtime (#54568)
* [wasm] Build static components; include hot_reload in runtime
Workaround until https://github.com/dotnet/runtime/issues/54565 is fixed
Build the runtime always with support for hot_reload, and without diagnostics_tracing
* Update wasm.proj
* Add a browser functional test for hot reload
Just check that the capabilities are non-empty which is a good proxy for hot reload being enabled in the runtime.
* Turn off trimming for hot reload functional test
* Disable test on browser AOT
* fix whitespace
Co-authored-by: Thays Grazia <thaystg@gmail.com>
Thays Grazia [Wed, 23 Jun 2021 23:06:58 +0000 (20:06 -0300)]
[wasm][debugger] Reuse debugger-agent on wasm debugger (#52300)
* Trying to reuse debugger-agent on wasm debugger. This will remove a lot of code that does the same thing on mini-wasm-debugger.
* Replace remove_breakpoint and clear_all_breakpoints with the ones on debugger-agent and remove unused code.
* Stepping and callstack using debugger-agent.
* Remove more code.
* Get frame values using debugger-agent.
* Working valuetypes and call function on valuetypes.
make -C src/mono/wasm/ run-debugger-tests TEST_FILTER=DebuggerTests.SteppingTests.InspectValueTypeMethodArgsWhileStepping is working without use_cfo.
* Failed: 316, Passed: 175
* Failed: 307, Passed: 184, Skipped: 0, Total: 491
* Failed: 280, Passed: 211
* Failed: 277, Passed: 214
* Implemented boxed value.
Failed: 271, Passed: 220
* Implementing get properties on objects.
Implementing handling error on debugger-agent.
* Implementing callfunctionon object.
Failed: 248, Passed: 243
* Implementing get pointer values.
Failed: 243, Passed: 248
* Fixing pointer values and implement call on function with pointers.
Failed: 226, Passed: 265
* Reimplement call function on, and implement set values.
Failed: 192, Passed: 299
* Failed: 192, Passed: 299
* Fixing valuetype with null values.
Failed: 184, Passed: 307
* Implemented Evaluate expressions, conditional breakpoints, all breakpoints tests are passing.
Failed: 172, Passed: 319
* Fixing evaluate with value type.
Failed: 156, Passed: 335
* Trim part and add cache.
Failed: 148, Passed: 343
* Fixing evaluate expression.
Failed: 99, Passed: 392
* GetPropertiesTests working.
Failed: 53, Passed: 438
* Passing delegate tests.
Failed: 31, Passed: 460
* Removing unused code.
* Implementing exception handler.
Failed: 30, Passed: 461
* Fixing cfo returning array.
Removing more code.
Removing 2 tests that tests functions which does not exist anymore.
Failed: 18, Passed: 471
* Fix CallFunctionOn returning primitive types and null.
Failed: 9, Passed: 480
* Failed: 7, Passed: 482
* Fixing some tests.
Failed: 2, Passed: 488
* Removing a lot of code.
Failed: 4, Passed: 485
* 0 ERRORS!
* Removing more code.
No errors.
* Fixing added tests.
* Return javascript callstack after managed callstack.
Step out from managed code return to native wasm or javascript.
Adding debug info to Wasm.Browser.Sample to help testing debugger with sample.
* Change what Ankit suggested.
Clear cache with valuetypes and pointers after resume or step.
* Fixing suggestions.
* Fix error on wasm build.
* Apply suggestions from code review
Co-authored-by: Larry Ewing <lewing@microsoft.com>
* Changing what was suggested by @lewing.
* Fix pointer tests.
* Refactoring CreateJObjectForVariableValue
* Changing what @lewing suggested.
* Apply suggestions from code review
Co-authored-by: Larry Ewing <lewing@microsoft.com>
* Update src/mono/wasm/debugger/BrowserDebugProxy/MonoProxy.cs
Co-authored-by: Larry Ewing <lewing@microsoft.com>
* Update src/mono/wasm/debugger/BrowserDebugProxy/MonoSDBHelper.cs
Co-authored-by: Larry Ewing <lewing@microsoft.com>
* Fixing @lewing changes.
* Trying to fix CI.
Co-authored-by: Larry Ewing <lewing@microsoft.com>
Tomáš Rylek [Wed, 23 Jun 2021 21:39:28 +0000 (23:39 +0200)]
Put Crossgen2 in sync with #54235 (#54438)
Eirik Tsarpalis [Wed, 23 Jun 2021 21:37:17 +0000 (00:37 +0300)]
Move System.Object serialization to ObjectConverter (#54436)
* move System.Object serialization to ObjectConverter
* simply customized object converter test
Tomáš Rylek [Wed, 23 Jun 2021 21:24:06 +0000 (23:24 +0200)]
Move setting fHasVirtualStaticMethods out of sanity check section (#54574)
Ankit Jain [Wed, 23 Jun 2021 16:53:56 +0000 (12:53 -0400)]
[wasm] Compile .bc->.o in parallel, before passing to the linker (#54053)
Aleksey Kliger (λgeek) [Wed, 23 Jun 2021 13:44:41 +0000 (09:44 -0400)]
Change PathInternal.IsCaseSensitive to a constant (#54340)
* Return constants in PathInternal.GetIsCaseSensitive() on mobile platforms
In particular on Android probing using I/O is slow and contributes to slow app startup.
Fixes https://github.com/dotnet/runtime/issues/54339
* Implement Path.IsCaseSensitive as PathInternal.IsCaseSensitive
Also Path.StringComparison => PathInternal.StringComparison
* Add test for PathInternal.IsCaseSensitive
Move GetIsCaseSensitiveByProbing to FileSystemTest
* Drop PathInternal.s_isCaseSensitive cache field
* Delete Path.IsCaseSensitive and Path.StringComparison
update callers to use PathInternal.IsCaseSensitive and PathInternal.StringComparison
* Remove catch clause from GetIsCaseSensitiveByProbing
* Mark new test [OuterLoop]
* Apply suggestions from code review
Co-authored-by: Stephen Toub <stoub@microsoft.com>
Co-authored-by: Jan Kotas <jkotas@microsoft.com>
Co-authored-by: Adam Sitnik <adam.sitnik@gmail.com>
Co-authored-by: Stephen Toub <stoub@microsoft.com>
Steve Pfister [Wed, 23 Jun 2021 13:38:02 +0000 (09:38 -0400)]
Make mono_polling_required a public symbol (#54592)
Resolves build failure with the Android aot+llvm functional test
Marek Safar [Wed, 23 Jun 2021 12:56:44 +0000 (14:56 +0200)]
Respect EventSource::IsSupported setting in more codepaths (#51977)
Lakshan Fernando [Wed, 23 Jun 2021 12:11:42 +0000 (05:11 -0700)]
Root ComActivator for hosting (#54524)
* rooting ComActivator that is needed for hosting
* FB
Krzysztof Wicher [Wed, 23 Jun 2021 12:02:37 +0000 (14:02 +0200)]
Add ILLink annotations to S.D.Common related to DbConnectionStringBuilder (#54280)
* Add ILLink annotations to S.D.Common related to DbConnectionStringBuilder
* address some feedback
* Make GetEvents() safe
* make GetProperties safe
* Mark GetProperties with RUC
Peter Sollich [Wed, 23 Jun 2021 10:09:38 +0000 (12:09 +0200)]
Fix finalizer issue with regions (#54550)
This fixes an issue in Server GC where an item in the finalizer queue became stale due to not being relocated.
The problem was that a finalizable object was allocated on one heap, but registered in the finalizer queue of another heap (this is possible due to heap balancing). In CFinalize::UpdatePromotedGenerations, we ask for the generation of an object, and move the object to the correct section of the finalizer queue. In the error case, we obtained the wrong result for the generation of the object because it lived on another heap, and that heap hadn't set the final generation for the region containing the object yet. So we ended up moving the finalizer entry to the section corresponding to gen 2, and missed a relocation of the object occurring in a gen 1 collection afterwards.
Fix: It seems best to make sure an object is always registered for finalization on the heap it's allocated from, so the fix simply fetches the heap from the alloc context after the allocation in the case of SOH, or determines it by calling gc_heap::heap_of in the case of LOH and POH. In the case of SOH, I added an assert to ensure that the heap obtained agrees with the result of calling gc_heap::heap_of.
I also added some dprintf calls to the finalizer logic to aid in future investigations.
Mateo Torres-Ruiz [Wed, 23 Jun 2021 07:19:10 +0000 (00:19 -0700)]
Add support for multi-arch install locations (#53763)
* Add support for multiple architectures inside install_locations
* Add install_location tests
* Fallback to DOTNET_ROOT on win32
Noah Falk [Wed, 23 Jun 2021 07:00:47 +0000 (00:00 -0700)]
Update library testing docs page to reduce confusion (#54324)
* Add warning on unmaintained testing doc page
* Update testing.md
Some example text that seems more clear to me, but only offered as a suggestion.
Feel free to adjust it or if you want to use it as-is please double check what I wrote is accurate : )
I think the useful elements are:
1. Being explicit about what workflow steps need to happen in total
2. Being explicit about which commands are covering the entire workflow and which ones are only covering a part of it
3. Show the simple "do-it-all" options first before showing more complex partial options. Glancing at the first example and blindly copying it should land in the pit of success.
Co-authored-by: Viktor Hofer <viktor.hofer@microsoft.com>
Adam Sitnik [Wed, 23 Jun 2021 06:53:55 +0000 (08:53 +0200)]
[FileStream] handle UNC and device paths (#54483)
* stop using NtCreateFile as there is no public and reliable way of mapping DOS to NT paths
Stephen Toub [Wed, 23 Jun 2021 06:34:38 +0000 (02:34 -0400)]
Update NetAnalyzers version (#54511)
* Update NetAnalyzers version
* Add NetAnalyzers to dependency flow
Co-authored-by: Viktor Hofer <viktor.hofer@microsoft.com>
Daniel Genkin [Wed, 23 Jun 2021 04:46:38 +0000 (00:46 -0400)]
Added runtime dependency to fix the intermittent test failures (#54587)
* Added runtime dependency to hopefully fix the intermittent test failures
* addressed comments
* cleanup
* cleanup accidental spaces and tabs cleanup
* added Larry's comments
Bruce Forstall [Wed, 23 Jun 2021 04:42:14 +0000 (21:42 -0700)]
Disable failing System.Reflection.Tests.ModuleTests.GetMethods (#54564)
Tracking: https://github.com/dotnet/runtime/issues/50831
Ankit Jain [Wed, 23 Jun 2021 03:46:33 +0000 (23:46 -0400)]
[wasm] Move AOT builds from `runtime-staging` to `runtime` (#54577)
These builds have had ~2-3 failures in the last 14 days (~90 builds).
Sergey Andreenko [Wed, 23 Jun 2021 01:58:23 +0000 (18:58 -0700)]
Keep obj node for ArrayIndex. (#54584)
Dan Moseley [Wed, 23 Jun 2021 01:08:12 +0000 (19:08 -0600)]
Disable another failing MemoryCache test (#54578)
Anton Lapounov [Tue, 22 Jun 2021 22:45:36 +0000 (15:45 -0700)]
Set DLL flag on R2R binaries (#54533)
This is required for R2R relocations to be processed by the OS loader on Windows 7.
Maryam Ariyan [Tue, 22 Jun 2021 21:25:36 +0000 (17:25 -0400)]
Add `SkipEnabledCheck` on `LoggerMessageAttribute` (#54305)
* Add SkipEnabledCheck on LoggerMessageAttribute
* Make logging generator more robust with null input
* Adds LoggerMessageAttribute ctor overload
* properly identify misconfigured input
David Fowler [Tue, 22 Jun 2021 21:07:54 +0000 (14:07 -0700)]
Improve performance of IsRootScope check (#54555)
- Stash a field instead of doing an equality comparison.
Fixes #54351
Andrew Au [Tue, 22 Jun 2021 20:28:59 +0000 (13:28 -0700)]
Fix a memory corruption caused by an integer overflow (#54510)
Maoni Stephens [Tue, 22 Jun 2021 20:27:31 +0000 (13:27 -0700)]
get rid of an unnecessary join (#54312)
I think this join was left there by accident when we were doing the write watch feature. I'm puzzled why I didn't notice it. the comment was also inconsistent which strengthens my belief that this was a mistake. we don't need a join here (there used to not be a join) and disabling the dirty pages tracking can be done anywhere while the EE is stopped since write barriers cannot be invoked.
Bruce Forstall [Tue, 22 Jun 2021 20:12:05 +0000 (13:12 -0700)]
Disable failing test BasicTestWithMcj under GCStress (#54566)
Tracking: https://github.com/dotnet/runtime/issues/54203
Steve Pfister [Tue, 22 Jun 2021 19:00:17 +0000 (15:00 -0400)]
[Android] Fix AndroidAppBuilder to work w/ AOT+LLVM (#53643)
We were missing a few key additions to make sure we can test against AOT+LLVM. This change will make sure we link against all the .dll-llvm.o files produced from the AOT compiler and include the right mtriple for each architecture.
Fixes #53628
Fan Yang [Tue, 22 Jun 2021 17:36:57 +0000 (13:36 -0400)]
[mono]Re-enable runtime tests on Android arm64 (#49662)
* Re-enable runtime tests on Android arm64
* disable irrelevant CI lanes
* Disable more dotnet-linker-tests and runtime-dev-innerloop CI lanes
* Comment out definition for DISABLE_LOGGING
* Remove defining logging for android
* Extend app installation timeout limit
* Fix command prefix for windows
* Enable tests excluded only for Android arm64
* Disable failed test on arm64
* Revert hacks
* Revert unintended changes
* More...
* Only run tests on arm64 for rolling build
* Fix merge error
Bruce Forstall [Tue, 22 Jun 2021 16:57:09 +0000 (09:57 -0700)]
Disable failing tests under GCStress (#54532)
https://github.com/dotnet/runtime/issues/51477
https://github.com/dotnet/runtime/issues/53359
Steve Pfister [Tue, 22 Jun 2021 16:44:55 +0000 (12:44 -0400)]
Import the correct MacCatalyst workload pack (#54558)
In https://github.com/dotnet/runtime/pull/54361 there was an incorrect import of the maccatalyst aot workload pack. This fix corrects the problem.
Fixes https://github.com/dotnet/runtime/issues/54494
Ankit Jain [Tue, 22 Jun 2021 16:35:38 +0000 (12:35 -0400)]
[wasm] Collect, and process satellite assemblies automatically (#53656)
Jeremy Koritzinsky [Tue, 22 Jun 2021 16:32:53 +0000 (09:32 -0700)]
Fail COM CoClass creation during construction instead of during jitting (#54508)