Mark the profiler attach thread during attach

Fix OSX floating point state extraction (#25295)

There was a bug reported on a very recent Mac with Intel i9 processor. A
crash in the RtlRestoreContext was happening at the fxrstor instruction
due to the fact that the floating point state data were garbage.
The investigation has shown that sometimes, the x86_FLOAT_STATE64
cannot be obtained using the thread_get_state API. And it was also found
that at the same time, the x86_AVX_STATE64 can be obtained. The state
extracted by the AVX variant contains all the registers that the FLOAT
variant would extract.
However, in some cases, even the x86_AVX_STATE64 cannot be obtained and
there is a third flavor that we can get - x86_AVX512_STATE64.
Unfortunately, there are cases where none of those can be obtained.
It is not clear what causes these cases, it seems only kernel debugging
can give us an answer to that.

This change modifies the way we extract the floating point state. We
first try to get the AVX state, if we fail, we try the AVX512 and
finally we fall back to the FLOAT state. If we fail to get the floating
point state with any of these, we return context without the floating
point state flag set. Also, if only getting the FLOAT state succeeds,
we return context without the XSTATE flag set.

Reduce string allocation in DiagnosticCounter.GetMetadataString (#25219)

When the _metadata dictionary contains a single key/value pair, the method ends up allocating an unnecessary StringBuilder, char[] beneath it, and input string.

When the _metadata dictionary contains more key/value pairs, the method ends up allocating an unnecessary string per key/value pair.

ensure process-wide fence when updating GC write barrier on ARM64 (#25130)

* ensure process-wide fences when updating GC write barrier on ARM64

[master] Update dependencies from dotnet/arcade (#25253)

* Update dependencies from https://github.com/dotnet/arcade build 20190619.25

- Microsoft.DotNet.Arcade.Sdk - 1.0.0-beta.19319.25
- Microsoft.DotNet.Build.Tasks.Feed - 2.2.0-beta.19319.25
- Microsoft.DotNet.Build.Tasks.Packaging - 1.0.0-beta.19319.25
- Microsoft.DotNet.Helix.Sdk - 2.0.0-beta.19319.25

[master] Update dependencies from dnceng/internal/dotnet-optimization (#25255)

* Update dependencies from https://dev.azure.com/dnceng/internal/_git/dotnet-optimization build 20190620.1

- optimization.IBC.CoreCLR - 99.99.99-master-20190620.1
- optimization.PGO.CoreCLR - 99.99.99-master-20190620.1

Revert "ILMarshaler Refactor (#21227)" (#25292)

This reverts commit 70febbad4bbe23326c26480b2848cdfe9fc8450c.

Add custom default base address option to crossgen (#25227)

* Add crossgen option to setup default base address for native image

This is enabled only with -DFEATURE_ENABLE_NO_ADDRESS_SPACE_RANDOMIZATION.

* Mmap native images at default base address if env variable COMPlus_UseDefaultBaseAddr=0x1 is setup.

This is enabled only with -DFEATURE_ENABLE_NO_ADDRESS_SPACE_RANDOMIZATION.

Fix eventpipe filtering (#25221)

* Add DOTNET_TRACE_CONTEXT and change macros to use DOTNET_TRACE_CONTEXT instead of MCGEN_TRACE_CONTEXT

* Fixing macro definitions

* eventing codegen scripts now generates EventPipe trace contexts

* Fix macros to use the EVENTPIPE_TRACE_Context

* Fix linux build

* Fix windows build

* Update Eventpipe provider context at EtwCallbackComon

* break in switch

* Update rundown provider context manually

* PR feedback

* Eventpipe->EventPipe

* cleanup in codegen script

Update linux-performace-tracing doc to add eventpipe/dotnet-trace (#25212)

* Update linux-performace-tracing doc to add eventpipe/dotnet-trace

* typo

* Add more detailed doc instead of just a link :)

* remove redundant part

* Update based on noah's feedback

Remove dead code related to copy-constructors and intercept stubs (#25264)

* Remove dead code related to copy-constructors and intercept stubs

* PR Feedback.

Disable NativeVarargsTest on Windows Nano Server. (#25284)

Move resources/containers out of azure-pipelines.yml to platform-matrix.yml (#25155)

Initial check-in of R2R format documentation (#25267)

In this initial check-in I have converted Jan Kotas' R2R format doc
from an internal Word document to the md format and I removed some
references to other internal docs. I plan to follow up by updating
the description to the current format version (e.g. apparently the
description doesn't yet include the instance entrypoint table) and
by adding specific extensions for the composite R2R format.

Thanks

Tomas

Merge pull request #25285 from MattGal/update-arm32-dockerimages

Update ARM32 Helix Images to include Python 3

Remove SOS and lldb plugin (#25220)

* Remove SOS and lldb plugin

Issue: https://github.com/dotnet/coreclr/issues/24092

Triage issues in #25098 (#25240)

JIT: use register indirect for indirect jumps (#25251)

If we are jumping to another method via an indirection cell, the cell address
may be too far from the jump to address via a RIP-relative indirection. So
instead, load the target address into RAX and jump register indirect.

Fixes #22342.

missed a spot

Update ARM32 Helix Images to include Python 3

Remove test 366085 (#25277)

This test was checking for "Parameter name:" substring in the
ArgumentNullException message. But a recent change #25185 modified
that string to just "Parameter".

We also have a coverage for this in corefx tests, so I am removing
this test.

Fix visibility on Exception.InnerExceptionPrefix (#25278)

Merge pull request #25265 from BruceForstall/FixCorefxTestingScript

Update run-corefx-tests.py for new 3.0.0 directory name

Restrict copyprop if address-taken (#25247)

* Restrict copyprop if address-taken

Fix #25231

Update run-corefx-tests.py for new 3.0.0 directory name

Address nullable feedback for S.T.Overlapped, S.Memory, S.R.InteropServices (#25214)

* Address nullable feedback for System.Threading.Overlapped

* Address nullable feedback for System.Memory

* Address nullable for System.Runtime.InteropServices

* Update xml comments and make CultureInfo nullable to match String.ToLower/Upper feedback

* PR feedback: revert accepting null comparer in BinarySearch

* Disable test that needs to be updated in corefx

Change Diagnostic Server transport name to align with spec (#25031)

resolves 24859

Exception format cleanup subset (#25185)

* Change ArgumentException message to one line

* Disable 2 tests

* Remove duplication in AggregateException.ToString()

* Inner exceptions on new lines

* Caps letter

* Typo

ILMarshaler Refactor (#21227)

* Add support for fields to ILStubMarshalHome objects.

* Make the specific code streams in the ILMarshaler private members.

* Make home members private.

* Move the IL code stream members back to being protected for the marshalers that override EmitMarshalArgumentCLRToNative instead of EmitMarshalArgumentContents/Space.

* Convert virtual specifiers in overrides to override specifiers.

* Remove unused and undefined member.

* Cleanup a few missed virtual/override specifiers.

* Refactor setup methods. Refactor some of the overrides to duplicate less code.

* Refactor the setup stream and add some comments around how the ArgumentOverride/ReturnOverride marshalling system works.

* Use the cleanup work list in SafeHandle marshalling since the work list is now implemented entirely in managed code.

* Generalize DelegateCleanupWorkListElement -> KeepAliveCleanupWorkListElement and rename the managed entry point to match.

* Refactor direct usage of the cleanup code stream in non-overridden marshalers to use the cleanup work list instead.

* Refactor AsAny marshalling to fit into the regular ILMarshaler code-paths.

* Move ILArgIteratorMarshaler over to the full ILMarshaler infrastructure.

* Port ILBlittablePtrMarshaler over to not directly reference m_pcsMarshal.

* Make the specific code streams private and have ILNativeArrayMarshaler fetch the one it needs from the NDirectStubLinker.

* Devirtualize a method on ILMarshaler.

* Fix broken metasig

* Revert "Use the cleanup work list in SafeHandle marshalling since the work list is now implemented entirely in managed code."

This reverts commit aedcdfb4830475ff8f4d582147cdc0eb497dd681.

* Fix ILArgIteratorMarshaler.

* Take 2 on using the cleanup work list for SafeHandles

* Remove unused field

* SafeHandleMarshaler doesn't need to have extra cleanup code when using the CleanupWorkList.

* Move the rest of the SafeHandle marshalling into ArgumentOverride.

* Moved Pinned Native Array special case into an ArgumentOverride.

* Devirtualize

* Remove invalid field hometype.

* Make ILMarshaler::m_pslNDirect private.

* Native Array marshalling fixes.

* Fix STLOC->LDLOC mistakes.

* Add override hook to allow a marshaler to request that byval contents are always converted to native even when not marked as In (i.e. explicitly marked only as Out). Used in AsAny for byval arrays.

* PR Feedback.

* Add explicit pinning path to ilmarshalers.

* Move implementation of ILNativeArrayMarshaler::CanMarshalViaPinning to ilmarshalers.cpp.

* Add missing override specifier.

* Don't create a managed marshaler if we are able to marshal via pinning. Remove extraneous checks against pinning.

* Convert ILWSTRMarshaler to use the MarshalViaPinning functions for its pinning fast-path.

* Enable LPWSTR marshalling to fall back to allocating on the heap for large strings when stack-allocating for native space (byref in-only semantics).

* PR Feedback.

Update dependencies from https://github.com/dotnet/core-setup build 20190618.02 (#25223)

- Microsoft.NETCore.App - 3.0.0-preview7-27818-02

Relax signed/unsigned checks in Array.Copy to match other places (#25209)

Disable failing tests. Also enable tests for changes that propagated through the system.

Fixes dotnet/corefx#38535

Don't free the loader handle if the LoaderAllocator is already dead. (#25208)

Reduce corefx testing load (#25235)

* Add new helixQueueGroup to allow restricting platforms tested

This is used to force corefx testing to only run on one Helix queue
for each platform, instead of multiple.

Fix 25233 (#25234)

* Fix ibcmerge.csproj to use IbcMergePackageVersion

This fixes a copy/paste bug.

* Fix more copy/paste in comment

* Use ! in place of %

Sign the DAC with MicrosoftSHA2 cert (#25205)

The current Microsoft certificate breaks minidumps/WER. The dbghelp
MiniDumpWriteDump code rejected the default cert. This started
failing when the obsolete SHA1 cert was removed from all of our
binaries.

Issue: https://devdiv.visualstudio.com/DevDiv/_workitems/edit/919061

Fix contained LEA handling (#25135)

* Fix contained LEA handling

This adds an LEA case to both `LinearScan::BuildOperandUses` and `CodeGen::genConsumeRegs`.

Fix #25039

Start cleaning emitxarch::GetSize estimates. (#25160)

* Add comments for _idCodeSize and a few asserts.

* Delete emitSizeOfJump.

* Delete emitInstCodeSz.

* Response review.

Update dependencies from https://dev.azure.com/dnceng/internal/_git/dotnet-optimization build 20190618.1 (#25224)

- optimization.IBC.CoreCLR - 99.99.99-master-20190618.1
- optimization.PGO.CoreCLR - 99.99.99-master-20190618.1

Prevent loading byref-like types with invalid layout (#25200)

First approximation of a fix for #25057.

This has two problems:
* We're checking for any byref-like typed fields. Types that don't actually contain interior pointers but were marked as `ref struct` will fail to load when not aligned properly.
* We're not doing the deep validation that we do for reference types to make sure the `ByReference<T>` field doesn't overlap with another non-byreference field.

Question is whether we're okay with those limitations, or whether we need a better fix. Better fix would likely entail inefficiently walking over the fields à la `FindByRefPointerOffsetsInByRefLikeObject` (doing the more efficient thing that we do for object references below would require a GCDesc representation of byrefness).

Contributes to #25057.

Allow PEImageLayout to be loadable for R2R images loaded from streams. (#25159)

* Allow PEImageLayout to be loadable for R2R images loaded from streams.

* Add regression test coverage

* Change CheckILOnly to return true for R2R images (similar to the ILOnly API)

* Early out for R2R images

* Remove R2R check

Delete dead uapaot code (dotnet/corefx#38330)

* Delete dead uapaot code

Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>

Remove unused TRACE_LEVEL_NONE (#23711)

Removed define TRACE_LEVEL_NONE from eventtracebase.h.
Removed static_assert(GCEventLevel_None == TRACE_LEVEL_NONE, "GCEventLevel_None value mismatch"); from eventtrace.cpp

Enhancement #23711

Add IJW native varargs tests. (#24983)

* Refactor IJW test infra. Add some tests for some  interesting simple native varargs.

* Add missing include of IjwHelper.cs

* Fix Cmake test project name.

* PR Feedback.

* Fix compilation error.

* Deploy msvcp*d.dll as part of the copy-local'd CRT.

Stop ignoring crashes around IUnknown::Release calls (#25210)

We do not want to be masking any memory corruption and bugs in the runtime.

[master] Update dependencies from dnceng/internal/dotnet-optimization (#25181)

* Update dependencies from https://dev.azure.com/dnceng/internal/_git/dotnet-optimization build 20190615.1

- optimization.IBC.CoreCLR - 99.99.99-master-20190615.1
- optimization.PGO.CoreCLR - 99.99.99-master-20190615.1

* Update dependencies from https://dev.azure.com/dnceng/internal/_git/dotnet-optimization build 20190616.1

- optimization.IBC.CoreCLR - 99.99.99-master-20190616.1
- optimization.PGO.CoreCLR - 99.99.99-master-20190616.1

* Update dependencies from https://dev.azure.com/dnceng/internal/_git/dotnet-optimization build 20190616.3

- optimization.IBC.CoreCLR - 99.99.99-master-20190616.3
- optimization.PGO.CoreCLR - 99.99.99-master-20190616.3

[master] Update dependencies from dotnet/arcade (#25179)

* Update dependencies from https://github.com/dotnet/arcade build 20190614.27

- Microsoft.DotNet.Arcade.Sdk - 1.0.0-beta.19314.27
- Microsoft.DotNet.Build.Tasks.Feed - 2.2.0-beta.19314.27
- Microsoft.DotNet.Build.Tasks.Packaging - 1.0.0-beta.19314.27
- Microsoft.DotNet.Helix.Sdk - 2.0.0-beta.19314.27

* Update dependencies from https://github.com/dotnet/arcade build 20190615.2

- Microsoft.DotNet.Arcade.Sdk - 1.0.0-beta.19315.2
- Microsoft.DotNet.Build.Tasks.Feed - 2.2.0-beta.19315.2
- Microsoft.DotNet.Build.Tasks.Packaging - 1.0.0-beta.19315.2
- Microsoft.DotNet.Helix.Sdk - 2.0.0-beta.19315.2

make DiagnosticCounter and its children lock 'this' instead of an arbitrary String (#25173)

Enable running SIGSEGV handler on non-alternate stack (#25196)

When a third party library that someone loads into a coreclr process registers
its own SIGSEGV handler and then chain-calls coreclr SIGSEGV handler on a
non-alternate stack, coreclr would currently crash.
This fix enables it to execute the SIGSEGV handler on the non-alternate stack
(original stack of the interrupted thread) in such case.
The disadvantage is that stack overflow would lead to silent crash in such a
case, but we cannot do anything about it.

Fix missing MethodEvents for R2R images (#25199)

Fixes #25014

Enable createdump on arm and arm64  (#24832)

* Enable build of ARM64 createdump build

- Adds definitions of necessary constructs and aligments for Elf formats in aarch64
- Work around changes in ptrace for aarch64

* Package createdump in runtime transport for arm and arm64

* Use special name register definitions from the PAL

* Add createdump_lib to arm64 sos and package plugin

Fix official build - specify platformGroup for internal/Release/build-job.yml (#25204)

Update dependencies from https://github.com/dotnet/core-setup build 20190617.01 (#25197)

- Microsoft.NETCore.App - 3.0.0-preview7-27817-01

IndexOfAny(values.Length == 0) == -1 (#25075)

* IndexOfAny(values.Length == 0) == -1

* Skip tests

Fix GetFunctionEnter3Info on linux (#25147)

Replace 'assert' with '_ASSERTE' (#25177)

Update linux-instructions.md (#25187)

Update ubuntu arm container version

[master] Update dependencies from dotnet/core-setup (#25180)

* Update dependencies from https://github.com/dotnet/core-setup build 20190614.05

- Microsoft.NETCore.App - 3.0.0-preview7-27814-05

* Update dependencies from https://github.com/dotnet/core-setup build 20190615.05

- Microsoft.NETCore.App - 3.0.0-preview7-27815-05

avoid expensive ref counting for CompareInfo on Linux (#25117)

* change SafeSortHandle to be CriticalSortHandle to avoid expensive ref counting

* seal the CriticalSortHandle class

* use IntPtr instead of CriticalHandle to avoid resurrection issues. It's ok to never free the handle

* introduce cache for sort handles to prevent from memory leak in certain scenarios

* move the  sort key logic to a dedicated helper type to avoid static field initialization order issues

* lock the dictionary

* simplify the code

* set the charset to Ansi

* Apply suggestions from code review

Co-Authored-By: Jan Kotas <jkotas@microsoft.com>
* don't use var

Do not export GC entrypoints outside standalone build (#25184)

It doesn't seem like something we would want to export outside standalone build.

Add active timer counter (#25186)

* Add active timer counter

* fix typo

don't use var

Apply suggestions from code review

Co-Authored-By: Jan Kotas <jkotas@microsoft.com>

set the charset to Ansi

simplify the code

Fix duplicate imports in managed product build (#25174)

Don't change LCL_FLD node type when replacing param with its shadow copy (#20900)

Fix an issue that is reproduced when:

1) a method requires GSSecurityCookie (e.g. when allocating unsafe buffer using localloc) and

2) one of the method parameters is "vulnerable" and requires a shadow variable to be created and used instead of the original parameter variable and

3) the method parameter has small signed integer type (i.e. int8 or int16) and

4) the value of the parameter is converted to unsigned type (e.g. uint8 or uint16).

abort build-tests if failed to restore stress dependencies (#25140)

* abort build-tests if failed to restore stress dependencies

* Fix arm condition.

* Add arm64 condition.

* Skip runtime dependencies for arm32/arm64.

Merge pull request #25162 from sandreenko/FixGCStressSettingForTheTest

Do not force GcStress=0xc for GitHub_23199 on OSX.

Move parts of RuntimeType to shared partition (#25176)

Moving non-EventPipe types out of eventpipe.* (#25161)

* Replace runtime check with compile time assert.
* Move EventPipeEventPayload to its own file.
* Move `StackContents` to its own file.
* Move other classes declaration/definition out of eventpipe.*

Delete OSX condition.

Do not force GCStress at all.

It doesn't work on OSX and Alpine.
It shows random failures on Windows as well, that we do not want to see in our PR/CI outerloop testing.

Fix crash/corruption in VSD hash tables when backpatching entry point slots is enabled (#25158)

Fixes https://github.com/dotnet/coreclr/issues/25080
- The prober used to look for an item (`DispatchStub` or `ResolveCacheElem`) stores information specific to the table
- Cooperative GC mode guarantees that the information stored in the prober remains valid when the prober is reused to add a new item when one is not found
- The lock taken to record/backpatch an item's slot exits and reenters cooperative GC mode, which can cause the table to be reclaimed and replaced with a new table, and the prober still refers to the old table
- Upon adding the new item it may crash or corrupt some other memory
- Fixed by resetting the prober if a path is taken that may reenter cooperative GC mode

Add a build option to skip building managed tools (#25171)

This specifically will set __BuildManagedTools=0 and skip building R2RDump and runincontext.

Update dependencies from https://dev.azure.com/dnceng/internal/_git/dotnet-optimization build 20190614.1 (#25170)

- optimization.IBC.CoreCLR - 99.99.99-master-20190614.1
- optimization.PGO.CoreCLR - 99.99.99-master-20190614.1

Update the SDK version (#25141)

* Update the SDK version

The updated SDK version has a fix for the issue causing msbuild to crash during the build on Linux environment.

* Use the version recommended by Eric

lock the dictionary

move the  sort key logic to a dedicated helper type to avoid static field initialization order issues

Make DisplayUnits string.Empty by default instead of null (#25104)

* Make DisplayUnits string.Empty by default instead of null

* PR feedback on DisplayName and setter validation

Fix French Decimal number parsing (#25154)

Update dependencies from https://github.com/dotnet/core-setup build 20190613.08 (#25169)

- Microsoft.NETCore.App - 3.0.0-preview7-27813-08

Update dependencies from https://github.com/dotnet/arcade build 20190613.18 (#25168)

- Microsoft.DotNet.Arcade.Sdk - 1.0.0-beta.19313.18
- Microsoft.DotNet.Build.Tasks.Feed - 2.2.0-beta.19313.18
- Microsoft.DotNet.Build.Tasks.Packaging - 1.0.0-beta.19313.18
- Microsoft.DotNet.Helix.Sdk - 2.0.0-beta.19313.18

Merge pull request #25138 from jashook/write_pgo_opt_version_to_intermediates_dir

Change how build.sh/build.cmd parse the PGO and IBC versions

Address all feedback

introduce cache for sort handles to prevent from memory leak in certain scenarios

Optimize Activator.CreateInstance (#25145)

* Optimize Activator.CreateInstance
- Short-circuit Activator.CreateInstance<T>() for value types without default constructor
- Cache default constructor delegate on RuntimeType instead of fixed-size singleton cache

use IntPtr instead of CriticalHandle to avoid resurrection issues. It's ok to never free the handle

Partially address comments

Update Corelib to adapt to compiler nullability updates

Update dependencies from https://github.com/dotnet/arcade build 20190612.21

- Microsoft.DotNet.Arcade.Sdk - 1.0.0-beta.19312.21
- Microsoft.DotNet.Build.Tasks.Feed - 2.2.0-beta.19312.21
- Microsoft.DotNet.Build.Tasks.Packaging - 1.0.0-beta.19312.21
- Microsoft.DotNet.Helix.Sdk - 2.0.0-beta.19312.21

Fix code version table lock / coop GC mode switch ordering (#25129)

Fixes https://github.com/dotnet/coreclr/issues/25086
- The lock is taken inside other unsafe locks in coop mode, in some paths it may not be legal to switch to preemptive mode before taking the lock
- Iterating over the cross loader allocator hash table to backpatch entry point slots needs coop GC mode, and was being done inside the lock
- Moved the switch to coop GC mode to before acquiring the lock to maintain consistent ordering between the two

Do not force GcStress=0xc for GitHub_23199.

There was a typo for Unix (it is COMPlus_GCStress, not COMPlus_GcStress, but COMPlus_GcStressOnDirectCalls was correct). That was why the test passed on OSX.

Add platformGroup parameter to platform-matrix.yml (#25111)

Allow pregenerating most HW intrinsics in CoreLib (#24917)

* Allow pregenerating all HW intrinsics in CoreLib

This is a follow up to #24689 that lets us pregenerate all hardware intrinsics in CoreLib.

We ensures the potentially unsupported code will never be reachable at runtime on CPUs that don't support it by not reporting the `IsSupported` property as intrinsic in crossgen. This ensures the support checks are always JITted. JITting the support checks is very cheap.

There is cost in the form of an extra call and failure to do constant propagation of the return value, but the cost is negligible in practice and gets eliminated once the tiered JIT tiers the method up.

We only do this in CoreLib because user code could technically not guard intrinsic use in `IsSupported` checks and pregenerating the code could lead to illegal instruction traps at runtime (instead of `PlatformNotSupportedException` throws) - it's a bad user experience.

[master] Update dependencies from dotnet/core-setup (#25121)

* Update dependencies from https://github.com/dotnet/core-setup build 20190612.01

- Microsoft.NETCore.App - 3.0.0-preview7-27812-01

* Update dependencies from https://github.com/dotnet/core-setup build 20190612.11

- Microsoft.NETCore.App - 3.0.0-preview7-27812-11

[master] Update dependencies from dnceng/internal/dotnet-optimization (#25122)

* Update dependencies from https://dev.azure.com/dnceng/internal/_git/dotnet-optimization build 20190612.1

- optimization.IBC.CoreCLR - 99.99.99-master-20190612.1
- optimization.PGO.CoreCLR - 99.99.99-master-20190612.1

* Update dependencies from https://dev.azure.com/dnceng/internal/_git/dotnet-optimization build 20190613.1

- optimization.IBC.CoreCLR - 99.99.99-master-20190613.1
- optimization.PGO.CoreCLR - 99.99.99-master-20190613.1

Update crossgen.md

Add instructions on how to use the /r switch. Update instructions to use /p instead of /Platform_Assemblies_Paths (shorter version)

Merge pull request #25137 from fadimounir/SuppressWarnings

Add a switch to suppress crossgen warnings

Intrinsicify SpanHelpers.IndexOf(char) (#22505)

* Helpers to support Intrinsics in SpanHelpers.Char

* Intrinsicify SpanHelpers.IndexOf(char)

* Feedback

* fix

* Fix assert

* Improve comment warning

* fix

* fix

* Fix

* Fix

Remove duplicate not