Put Marshal.Read/Write with Object under FEATURE_LEGACYSURFACE

The Marshal.Read/Write methods with Object arguments are not part of the
refactored .NET Core System.Runtime.InteropServices contract. Mark them
with FEATURE_LEGACYSURFACE so these get physically excluded from
mscorlib.dll on Linux/Mac.

Merge pull request #437 from mikem8361/clrstack

The sos "ClrStack" command now works

Code review feedback.

Merge pull request #4 from kangaroo/clrstack

Fixes OSx problems.

Change THREADSilentGetCurrentThreadId to return the pthread_threadid, which matches the tid from lldb

Ensure we find_library on LLDB as well to find LLDB.framework on OSX

Merge pull request #439 from janvorli/add-unhandled-exception-trace-print

Add unhandled exception stack trace print

Add unhandled exception stack trace print

This change adds printing of unhandled managed exception stack trace to
console before exiting.

Fix CI build failures. Undefined functions.

Merge pull request #438 from janvorli/fix-frames-popping

Fix managed exception unwinding through CallDescrWorkerInternal

Exclude the Frame destructor from DAC build

The destructor is not needed on DAC builds and the definition was not being
built for DAC already. Conditionally disabling the declaration too.

Fix managed exception unwinding through CallDescrWorkerInternal

This change fixes issue with exception unwinding in the case when the unwinding
passed through a frame of the CallDescrWorkerInternal function. This function
had personality routine on it, but a windows style personality routine was specified.
The windows one has a completely different signature, so the code was crashing.
When looking into that, I've found that even if I have implemented a proper
Unix style personality routine, it cannot work the same way on Linux as it
used to work on Windows.
This personality routine's goal is to pop Frames from the Frame list in the current
thread so that all frames upto the frame handling the exception are popped.
There are two problems on Linux. First, unlike on Windows, the personality
routine is not passed the RSP of the frame handling the exception in an official
way. Although it can be extracted from the private_2 member of the exception
object during the 2nd pass, it is an implementation detail that we cannot rely on.
Moreover, even if we used that, it would still not be the right frame in all cases
due to the fact that we implement exception filters by catching and rethrowing and
so the frame we would get would be the frame of a filtering catch in case
there was one.
My solution to this problem is to add destructor to the Frame type and let it
pop the frame being destroyed if it is still in the list in the current thread.
That way the native code unwinding automatically takes care of popping the frames.
As an additional changes, I've added handling of the case when the
Thread::VirtualUnwindToFirstManagedCallFrame walks out of stack, fixed a stack
alignment issue in the recently added StartUnwindingNativeFrames function and
a cosmetic change in the UnwindManagedExceptionPass1.

Merge remote-tracking branch 'upstream/master' into clrstack

Fix PAL printf functions.  Didn't handle "%*.*p" correctly which is supported under windbg/windows runtime.  Didn't actually support "*" for the width parameter at all for any format char.

Merge pull request #435 from janvorli/nested-exception-handling

Add support for exceptions crossing native frames

Merge pull request #436 from janvorli/resource-loading-workaround

Workaround native resource string loading temporarily

Add support for exceptions crossing native frames

This change adds support for unwinding exceptions that cross native frames.
These are for example exceptions thrown / rethrown from catch blocks.
The exceptions are unwound in an interleaved manner in this case. First,
all managed frames upto the first native frame are unwound (both the first
and second pass), then the native frames are unwound by standard c++ exception
handling, then the next block of managed frames is unwound etc.
The change also implements RtlCaptureContext and changes the managed exception
handling to use it instead of the GetThreadContext. The difference is that the
RtlCaptureContext gets context of the caller while the GetThreadContext gets
a context somewhere deep in the PAL and so unwinding from such a context to the
first managed frame would be walking old stack frames that can already be corrupted.
As an additional change, I have fixed a problem that prevented unwinding of
assembler functions that use the PROLOG_WITH_TRANSITION_BLOCK macro.
The macro was using CFI annotation for the xmm registers and the libunwind
doesn't support that.
Moreover, the PAL_VirtualUnwind return status was not being checked in the
`Thread::VirtualUnwindToFirstManagedCallFrame`, so the failure resulted in an
infinite loop in there.
Finally, the `debug/di/amd64/floatconversion.S` was including the unixasmmacros.inc
using a relative path, now after I've moved it to a different place, I've removed
the relative path since the new path is in the include paths.

Merge pull request #432 from kangaroo/pal-glccfos

Implement GetLogicalCpuCountFromOS in the PAL for OSX

Reflect feedback

Changed #ifndef FEATURE_PAL to #if 0 since the file is in PAL.

Merge pull request #427 from LLITCHEV/work

Fix some struct passing and tail call Jit problems on System V type of O...

Fix some struct passing and tail call Jit problems on System V type of OSs.

This change set contains the following:
1. Removes the PUSH/POP for RDI/RSI functionality from the Jit for Linux. Replaced with
standard register tracking and spilling by the RA.
2. Initialize the non stack homed, register passed param stack offset to 0
and allow the fixup routines to operate on the offset. Tail calls
implementation depends on the offset of the first param be set properly.
3. Initial work to allow for frame chaining using a FP register.

Implement GetLogicalCpuCountFromOS in the PAL for OSX

Workaround native resource string loading temporarily

This change is a temporary workaround for native resource string loading until we have
the resource string support implemented.
Instead of asserting in the resource loading methods, return a string representing
the id for the requested resource.
The change implements a similar workaround for the FMTMSG_GetMessageString which now
returns a string containing "Error errorCode" where the errorCode is the error code
passed to the method.
This way, exceptions that use those two functions can be raised without asserting.

The ClrStack command mostly works except for some address formating (related to %p).  Needed to change how the PAL got the thread id (pthread_self to gettid) and implemented a simple GetThreadContext function.

Merge pull request #434 from AlexGhiondea/concurrentDict

Remove the CDS_COMPILE_JUST_THIS define from the ConcurrentDictionary.cs file

Merge pull request #418 from AlexGhiondea/AppContextFix

Make sure that AppContext respects CompatibilitySwitches.IsAppSilverlight81 flag

Remove the CDS_COMPILE_JUST_THIS define from the ConcurrentDictionary.cs file.

We are not going to build that file stand-alone anymore.

Merge pull request #429 from jkotas/testfixes

Fix error handling in REGUTIL::EnvGetString

Fix error handling in REGUTIL::EnvGetString

Merge pull request #428 from ellismg/custom-entry-point

Add custom entry point support.

Add custom entry point support.

Hosts like ASP.net don't want to call ExecuteAssembly, because their
entry point is library, not a managed exe.  However, forcing cross
platform hosts to actually call CreateAppDomainWithManager and
CreateDelegate themselves requires a bunch of tedious code, since these
hosts don't have access to mscoree.h and the PAL.

This change adds the ability to specificy an assembly, type and method
which can be used as a custom entrypoint.  The signiture of the managed
entry point needs to be `static int E(int, char**)`.

Fix some more build problems/missing globals. The ClrStack command builds and almost runs except for a thread id problem.

Fix some missing variables and other build problems.  The ClrStack command now builds and almost runs. There is a problem with thread ids between the debugger/DAC and the target/VM/PAL.

The "ClrStack" command and related functions now build.

Merge pull request #426 from ramarag/merge-from-tfs

Merge from tfs

Merge remote-tracking branch 'upstream/master' into merge-from-tfs

Resolved Merge Conflict:
tests/src/AllTestProjects.sln

Make sure that AppContext respects CompatibilitySwitches.IsAppSilverlight81 flag.

This flag is used in cases where we don't have a Main() which means we won't be
able to read the TFM. Setting this compatibility flag will force the TFM to be
'WindowsPhone, Version=8.1".

Merge pull request #425 from jkotas/jitpinvokefix

Fix trashing of P/Invoke return value by GC probe

Merge pull request #416 from janhenke/master

Do not hardcode path to bash into the shell scripts.

Merge pull request #423 from jkotas/timerfix

Avoid call to QueryUnbiasedInterruptTime for PAL

Do not hardcode path to bash into the shell scripts.

Use /usr/bin/env bash to discover bash, independet of actual path. Closes #204.

Avoid call to QueryUnbiasedInterruptTime for PAL

Merge pull request #400 from mikem8361/dacglobals1

Implement the DAC global pointer table for Linux

Merge pull request #419 from ellismg/unblock-datetime-now

Return null for GetLocalTzFile

Merge remote-tracking branch 'upstream/master' into dacglobals1

Fix sos on window's build.

Return null for GetLocalTzFile

Throwing an exception from GetLocalTzFile causes DateTime.Now to throw
an exception and the runtime to tear down.  While we are bringing up
the globalization stack on Unix, we can just return null instead of
throwing.  This will cause us to use a zero UTC offset, so the local
times will not actually match the local clock, but we will be able to
get further along.

Merge pull request #415 from jkotas/zapwriter

Fix size computation in ZapWriter::WritePad

Merge pull request #411 from ramarag/fix_test_msbuild_warnings

Adding app.config to get rid of the warning MSB3276 in the Test Build

Fix size computation in ZapWriter::WritePad

Merge pull request #414 from jkotas/assertfix

Make assertion failures to break into debugger

Make assertion failures to break into debugger

Adding app.config to get rid of the warning MSB3276

With This change  the tests build all the native components first and then builds the managed components. The managed components can refer to the native projects by the following construct in the .csproj
             <ProjectReference Include="CMakeLists.txt">

[tfs-changeset: 1427574]

Merge pull request #409 from ellismg/fix-file-mode

Change file modes of .PNGs

Fix method desc name display by using the PAL vsnprintf instead of the regular linux runtime version that doesn't understand %S as an UTF16 string.

Change file modes of .PNGs

For some reason, some of our png files were marked as executable,
which is a very odd thing to do.

Move them back to the standard file mode.

Merge remote-tracking branch 'upstream/master' into dacglobals1

Conflicts:
src/debug/ee/CMakeLists.txt

Fix the LLDB.h search order and the format character used.

Fixed that IP2MD stopped working after using it, continuing the target and then trying to use it again.  Needed to always recreate the IXCLRDataProcess instance each time a command is run.

Allow either lldb-3.6 or lldb-3.5 to be used to build the sos plugin.

Add a fourth parameter to the DEFINE_DACVAR macro that is the actual fully qualified name of the static/global.

First pass at the global DAC table support for Linux. Explicitly build the table in the coreclr module instead of extract it from the pdb and put it in a resource.

Fixed the SVR gc globals in the DAC table. They had to be seperated and initialized in gceesvr.cpp.

Start on global pointer table.

PAL functions to write and read the DAC table address between processes.

The dac table is now copied from the coreclr process to the DAC/debugger process.  The tables were not being built with exactly the same defines so they weren't the same size. Fixed a bug in the read memory implementation. Still assumes pid = 0.

Changed the dacTable entries to be RVAs and renabled getting the corclr module base addres (m_globalBase). Added dac table address file cleanup on coreclr shutdown.

Filled in the vtable entries in the global dac table.  Changed some of the VPTR_* macros to be defined on the coreclr side (RS) to defined a constructor that is used to get the vtable in dactable.cpp. These changes required default constructors to be added to some of the classes.

Changed getting the vtable address to not invoke the destructors since the constructor used didn't do anything (like initialize variables, etc.).

Added a TODO comment about the debuggee pid not being available in the dac table address file name.

Fixed Windows build. Created a couple of new VPTR_* macros that add a default constructor only if building coreclr (not DAC) on Linux.

Comment on how these DAC table functions are temporary.

Fix trashing of P/Invoke return value by GC probe

Merge pull request #406 from akatakritos/patch-1

Fix typo in garbage-collection.md

Fix typo in garbage-collection.md

"improtant" -> "important"

Merge pull request #404 from stephentoub/fix_indexof_unix

Fix exception in CompareInfo.IndexOfOrdinal on Unix

Fix exception in CompareInfo.IndexOfOrdinal on Unix

A one-character typo is leading to an out of range exception.

Merge pull request #402 from jkotas/vsdfix

Fix register usage in ResolveWorkerChainLookupAsmStub

Fix register usage in ResolveWorkerChainLookupAsmStub

This stub has 100% custom calling convention. It gets the argument in rdx even on Unix.

Merge pull request #397 from Djuffin/net-debug

Pipe based communication between debugee and managed debugger on Linux

Merge pull request #398 from stephentoub/fix_getlasterror

Reimplement CPalThread::Get/SetLastError in terms of errno

Merge pull request #399 from richlander/master

Correct name of Ecma standards organization

Correct name of Ecma standards organization

Reimplement CPalThread::Get/SetLastError in terms of errno

GetLastError in the PAL isn't checking errno, and is instead using its own m_dwLastError.  As such, it doesn't properly handle arbitrary P/Inovkes, and all of our P/Invokes to libc are resulting in garbage GetLastWin32Error values, which throws off all of the error handling paths.

This commit just reimplements CPalThread::GetLastError and CPalThread::SetLastError to use errno (defined by POSIX.1c to be per-thread) as the backing store rather than using the m_dwLastError field, which I've removed.

Rename old Silverlight Mac debugging GUID to avoid confusion

Rearrange libs in cmakefile for libcoreclr in order to help ld find twowaypipe symbols

Fix build from msbuild environment

Merge pull request #396 from swaroop-sridhar/TestEnv

Add Support for Custom Test Environment Settings

Add Support for Custom Test Environment Settings

RunTest.Cmd had limited suport for setting test environment
(ex: Specifying use of alternate Jit)

This change generalizes this support so that the clients can specify a
test environment script that will be run to set custom settings in
the test-environment.

Usage is:

Runtest TestEnv <test-env-script> ...

Merge pull request #394 from ramarag/parallelize_tests_within_category

Enabling test within the same category to run in parallel

Merge pull request #395 from cherylws/patch-1

Update index.md

Update index.md

Update the MSDN link to most recent, locale-agnostic, version of the topic.

Enabling test within the same category to run in parallel

Merge pull request #386 from benpye/travis-ci-fix-2

FIx Travis CI release failing

Merge pull request #391 from jkotas/umthunk

Fix RBP-relative offsets in UMThunkStub

Merge pull request #392 from jkotas/TypedReference

Introduce FC_TypedByRef for passing TypedReferences as FCall arguments

Disable PAL tests

Introduce FC_TypedByRef for passing TypedReferences as FCall arguments

FC_TypedByRef is defined as TypedByRef& for now to workaround the mismatch between managed and native struct calling convention.

Merge pull request #389 from jkotas/memcpy

Fix copy&paste typo in JIT_MemCpy helper

Fix RBP-relative offsets in UMThunkStub

Fix copy&paste typo in JIT_MemCpy helper

Merge pull request #388 from richlander/master

Add Garbage Collection Book of the Runtime Doc

Add Garbage Collection Book of the Runtime Doc

Pipe based communication between debugee and managed debugger on Linux

Goal of this change is to make managed debugging on Linux possible. (It is not fully achieved, but we're getting there)

So far our provision for debugging on Linux is somewhat different from debugging on Windows.
Instead of using WaitForDebugEvent and RaiseException as means of communication between debugger and debuggee, we're gonna use pipes.
Thankfully from old times of Silverlight Mac debugging we had debugging via network sockets under ifdefs FEATURE_DBGIPC_TRANSPORT_DI and FEATURE_DBGIPC_TRANSPORT_VM.
So this change is taking that old way of debugging, changing sockets for network pipes, removes lots of unused stuff and implements whatever is missing on Linux.

Testing:
Due to infrastructural issues I wasn't able to test debugging on Linux yet. So my testing consistent of
1. End to end net pipe debugging on Windows
2. Testing of twowaypipe implementation on Linux
3. Testing of search for loaded CoreCLR module on Linux.

Merge pull request #380 from janvorli/exception-handling

Fix CONTEXTFromNativeContext / CONTEXTToNativeContext to handle FP regs

Merge pull request #335 from LLITCHEV/master

Fix a problem with accessing caller passed arguments on SystemV systems.

Merge pull request #381 from jkotas/unwinding

More asm code unwinding fixes

Fix GetThreadContext and reflect PR feedback

The GetThreadContext on Linux was not getting the floating point registers even if it was asked
to do so. Moreover, it indicated in the context's ContextFlags that the context contains the
floating point registers. So the RtlRestoreContext has attempted to restore the full context
and filled the floating point registers with junk.
I've modified the GetThreadContext to use the CONTEXTFromNativeContext which fixes the issue.
Also, the CONTEXTFromNativeContext can now get any requested subsets of registers (integer,
control, floating point) instead of enforcing that the integer and control are always present.
Last change is a removal of the contextFlags parameter from the CONTEXTToNativeContext,
since the flags are present in the context itself and it doesn't make sense to specify other
flags than those ones.

FIx Travis CI release failing

Travis CI release mode failed as the test path was always debug rather
than referencing the build matrix.

Merge pull request #373 from benpye/travis-ci-fix

Incorrect branch for travis-ci badge