platform/upstream/coreclr.git
8 years agoControlling Xplat Event logging:
Rama [Thu, 15 Oct 2015 22:16:24 +0000 (15:16 -0700)]
Controlling Xplat Event logging:
Adding an environment variable COMPlus_EnableEventLog to turn on/off
Xplat Event Logging, by default Xplat Event logging is turned off

8 years agoMerge pull request #1783 from nguerrera/explicit-location-model
Nick Guerrera [Fri, 16 Oct 2015 20:41:10 +0000 (13:41 -0700)]
Merge pull request #1783 from nguerrera/explicit-location-model

Be explicit about exposing Assembly.Location in model.xml

8 years agoMerge pull request #1781 from nguerrera/expose-mvid
Nick Guerrera [Fri, 16 Oct 2015 18:41:39 +0000 (11:41 -0700)]
Merge pull request #1781 from nguerrera/expose-mvid

Expose Module.ModuleVersionId

8 years agoPass __IntermediatesDir to msbuild in build.sh
Nate Amundson [Fri, 16 Oct 2015 16:32:06 +0000 (11:32 -0500)]
Pass __IntermediatesDir to msbuild in build.sh

Currently, building coreclr x-plat fails if not running as root due to
the build trying to copy files to "/ToolRuntime".  The issue is that
$(IntermediatesOutputRootPath) is not set when running from build.sh,
and testruntime.targets in buildtools defines ToolRuntimePath as
"$(IntermediateOutputRootPath)ToolRuntime\".  The fix is to pass
__IntermediatesDir to msbuild in build_mscorlib, which results in
$(IntermediatesOutputRootPath) being properly set.

This change also adds a skipcoreclr flag to allow the user to skip the
CoreCLR part of the build, similar to the existing skipmscorlib flag.

8 years agoBe explicit about exposing Assembly.Location in model.xml
Nick Guerrera [Fri, 16 Oct 2015 01:07:29 +0000 (18:07 -0700)]
Be explicit about exposing Assembly.Location in model.xml

8 years agoExpose Module.ModuleVersionId
Nick Guerrera [Thu, 15 Oct 2015 23:15:14 +0000 (16:15 -0700)]
Expose Module.ModuleVersionId

8 years agoMerge pull request #1778 from swaroop-sridhar/comment
Jan Kotas [Thu, 15 Oct 2015 21:24:29 +0000 (14:24 -0700)]
Merge pull request #1778 from swaroop-sridhar/comment

Fix a comment in GcInfoEncoder.h

8 years agoMerge pull request #1777 from dotnet-bot/from-tfs
Matt Ellis [Thu, 15 Oct 2015 21:18:51 +0000 (14:18 -0700)]
Merge pull request #1777 from dotnet-bot/from-tfs

Merge changes from TFS

8 years agoFix a comment in GcInfoEncoder.h
Swaroop Sridhar [Thu, 15 Oct 2015 20:44:32 +0000 (13:44 -0700)]
Fix a comment in GcInfoEncoder.h

Fix a stale comment about stack-slot-base
in the GcInfo Encoder interface.

8 years agoMerge pull request #1690 from bryanar/changes-to-test-tree
Bryan P. Arant [Thu, 15 Oct 2015 20:24:28 +0000 (20:24 +0000)]
Merge pull request #1690 from bryanar/changes-to-test-tree

Changes to Test Tree Infrastructure

8 years agoAdd missing .gitmirror file.
Matt Ellis [Thu, 15 Oct 2015 19:11:32 +0000 (12:11 -0700)]
Add missing .gitmirror file.

This new folder was introduced in 465f56ef0c9c4b125d890e7ac0757e77d54e193c and needs a .gitmirror file.

[tfs-changeset: 1538359]

8 years agoMerge pull request #1770 from adiaaida/addSkipMscorlib
Michelle McDaniel [Thu, 15 Oct 2015 17:30:03 +0000 (10:30 -0700)]
Merge pull request #1770 from adiaaida/addSkipMscorlib

Add skipmscorlib option to build.cmd

8 years agoMerge pull request #1773 from jkotas/gcee-cleanup
Jan Kotas [Thu, 15 Oct 2015 16:56:13 +0000 (09:56 -0700)]
Merge pull request #1773 from jkotas/gcee-cleanup

Cleanup GC-EE Interface

8 years agoSkip Crossgen of mscorlib for x86
Michelle McDaniel [Wed, 14 Oct 2015 23:55:47 +0000 (16:55 -0700)]
Skip Crossgen of mscorlib for x86

x86 is currently cannot crossgen mscorlib, but we want to get a build in
the lab so that we make sure we do not break the x86 build of coreclr when
we make changes. This change defaults to skipping crossgen for x86 unless
docrossgen is passed to build.cmd.

8 years agoCleanup GC-EE Interface
Jan Kotas [Thu, 15 Oct 2015 14:14:05 +0000 (07:14 -0700)]
Cleanup GC-EE Interface

- Add proper abstractions for threading methods to GCToEEInterface
- Split sample gcenv into multiple files to avoid redundant content
- Add handle table methods needed by .NET Native WinRT implementation

Part of #1550

8 years agoMerge pull request #1769 from mmitche/fix-type-assert
Matt Mitchell [Thu, 15 Oct 2015 14:44:53 +0000 (07:44 -0700)]
Merge pull request #1769 from mmitche/fix-type-assert

Remove assert found in internal testing

8 years agoMerge pull request #1772 from anurse/anurse/1771-coreconsole-wpgmptr
Jan Kotas [Thu, 15 Oct 2015 14:21:58 +0000 (07:21 -0700)]
Merge pull request #1772 from anurse/anurse/1771-coreconsole-wpgmptr

change coreconsole to use _get_wpgmptr

8 years agoChange CoreConsole to use _get_wpgmptr
Andrew Stanton-Nurse [Thu, 15 Oct 2015 05:08:09 +0000 (22:08 -0700)]
Change CoreConsole to use _get_wpgmptr

The first value in argv only contains the string that was used to
launch the process. This means that when the CoreConsole-based
application is on the PATH, some invocations of it (most notably
those done from 'cmd') cause that value to be incomplete.

Changing to use _get_wpgmptr ensures that we always get the full
path to the executable, no matter how it is executed.

Fix #1771

8 years agoChanges to Test Infrastructure
Koundinya Veluri [Wed, 30 Sep 2015 19:20:52 +0000 (12:20 -0700)]
Changes to Test Infrastructure

The following changes are being submitted in order to prepare for a large collection of tests to be ported from the internal legacy test tree.

- Adds "build kinds" to the test tree.
- Adds some documentation for the test tree
- Adds a Test Priority feature (see documentation Documentation/project-docs/tests.md for more information.
- Dropped the cs_template.csproj types and converted to individual CSProject files.

8 years agoRemove assert found in internal testing
Matt Mitchell [Wed, 14 Oct 2015 23:09:59 +0000 (16:09 -0700)]
Remove assert found in internal testing

This assert was introduced with the fix for #1323 as a sanity check that truly incompatible types couldn't enter this function.  However, there are at least some cases where the types could be different.  In the case found, REF and LONG were the types and the operand was a GT_LEA, which is handled later in the function.
This assert was introduced with the fix for #1323 as a sanity check that truly incompatible types couldn't enter this function.  However, there are at least some cases where the types could be different.  In the case found, REF and LONG were the types and the operand was a GT_LEA, which is handled later in the function.

8 years agoMerge pull request #1747 from stephentoub/string_tolowerupper_ascii
Stephen Toub [Wed, 14 Oct 2015 22:24:34 +0000 (18:24 -0400)]
Merge pull request #1747 from stephentoub/string_tolowerupper_ascii

Add ASCII optimization to string.ToLower/ToUpper on Unix

8 years agoAvoid brancing in ToLower/UpperAsciiInvariant(char)
stephentoub [Wed, 14 Oct 2015 15:29:11 +0000 (11:29 -0400)]
Avoid brancing in ToLower/UpperAsciiInvariant(char)

Rather than comparing the input char against 'A'/'a' and 'Z'/'z', we simply trim off all but the lower 7 bits and do a lookup into a precomputed table.  This is ~3-4x faster than the branching version.

8 years agoMerge pull request #1761 from kouvel/Instructions
Sergiy Kuryata [Wed, 14 Oct 2015 17:47:11 +0000 (10:47 -0700)]
Merge pull request #1761 from kouvel/Instructions

Add some instructions (tests, new APIs)

8 years agoMerge pull request #1763 from kouvel/PerfFix
Sergiy Kuryata [Wed, 14 Oct 2015 16:36:12 +0000 (09:36 -0700)]
Merge pull request #1763 from kouvel/PerfFix

Add string allocation fast path outside Windows

8 years agoAdd ASCII optimization to string.ToLower/ToUpper on Unix
stephentoub [Mon, 12 Oct 2015 19:06:25 +0000 (15:06 -0400)]
Add ASCII optimization to string.ToLower/ToUpper on Unix

8 years agoMerge pull request #1758 from wtgodbe/failingTests
Aditya Mandaleeka [Wed, 14 Oct 2015 00:48:03 +0000 (17:48 -0700)]
Merge pull request #1758 from wtgodbe/failingTests

Removed working tests from failing tests file

8 years agoMerge pull request #1759 from ellismg/format-corefx-code
Matt Ellis [Wed, 14 Oct 2015 00:15:36 +0000 (17:15 -0700)]
Merge pull request #1759 from ellismg/format-corefx-code

Format native CoreFX code to follow conventions

8 years agoAdd string allocation fast path outside Windows
Koundinya Veluri [Tue, 13 Oct 2015 22:49:03 +0000 (15:49 -0700)]
Add string allocation fast path outside Windows

- On Linux, a microbenchmark is 140% faster with the portable fast path
- On Windows with the asm fast path, the microbenchmark was 300% faster than on Linux before, and is now 65% faster with the portable fast path
- On Windows, the portable fast path is 5% slower than the asm fast path

8 years agoMerge pull request #1669 from joperezr/UseNewBclRewriter
Jose Perez Rodriguez [Tue, 13 Oct 2015 23:12:59 +0000 (16:12 -0700)]
Merge pull request #1669 from joperezr/UseNewBclRewriter

Using DotNetCore version of BclRewriter in mscorlib build

8 years agoMerge pull request #1755 from kouvel/DisableTests
Koundinya Veluri [Tue, 13 Oct 2015 21:04:49 +0000 (14:04 -0700)]
Merge pull request #1755 from kouvel/DisableTests

Disable a couple of tests that are not supported outside Windows

8 years agoAdd some instructions (tests, new APIs)
Koundinya Veluri [Tue, 13 Oct 2015 19:58:48 +0000 (12:58 -0700)]
Add some instructions (tests, new APIs)

- Instructions for running CoreCLR and PAL tests outside Windows
- Brief instructions for running CoreFX tests with a private build of CoreCLR outside Windows
- Adding new public APIs to mscorlib

8 years agoRun format-code.sh
Matt Ellis [Tue, 13 Oct 2015 19:15:54 +0000 (12:15 -0700)]
Run format-code.sh

8 years agoAdd format-code.sh from CoreFX
Matt Ellis [Tue, 13 Oct 2015 19:11:50 +0000 (12:11 -0700)]
Add format-code.sh from CoreFX

8 years agoUsing DotNetCore version of BclRewriter in mscorlib build
Jose Perez Rodriguez [Fri, 2 Oct 2015 16:52:22 +0000 (09:52 -0700)]
Using DotNetCore version of BclRewriter in mscorlib build

8 years agoRemoved working tests from failing tests file
William Godbe [Tue, 13 Oct 2015 18:28:07 +0000 (11:28 -0700)]
Removed working tests from failing tests file

8 years agoMerge pull request #1756 from mmitche/add-arm-arm64-cross
Matt Mitchell [Tue, 13 Oct 2015 18:34:03 +0000 (11:34 -0700)]
Merge pull request #1756 from mmitche/add-arm-arm64-cross

Add arm64/arm cross builds to CI definition

8 years agoAdd missing quote
Matt Mitchell [Tue, 13 Oct 2015 18:23:48 +0000 (11:23 -0700)]
Add missing quote

8 years agoUse the correct comment style
Matt Mitchell [Tue, 13 Oct 2015 17:44:58 +0000 (10:44 -0700)]
Use the correct comment style

8 years agoAdd arm64/arm cross builds to CI definition
Matt Mitchell [Tue, 13 Oct 2015 17:38:23 +0000 (10:38 -0700)]
Add arm64/arm cross builds to CI definition

8 years agoDisable a couple of tests that are not supported outside Windows
Koundinya Veluri [Tue, 13 Oct 2015 17:29:10 +0000 (10:29 -0700)]
Disable a couple of tests that are not supported outside Windows

8 years agoMerge pull request #1738 from janvorli/fix-cocreate-guid
Jan Vorlicek [Tue, 13 Oct 2015 14:56:52 +0000 (16:56 +0200)]
Merge pull request #1738 from janvorli/fix-cocreate-guid

Implement CoCreateGuid using uuid_generate on Unix

8 years agoImplement CoCreateGuid using uuid_generate on Unix
Jan Vorlicek [Fri, 9 Oct 2015 18:56:43 +0000 (20:56 +0200)]
Implement CoCreateGuid using uuid_generate on Unix

We were generating GUIDs on Unix as random numbers. But that is not
correct since GUIDs have defined structure with bits having specific
meanings. For example, there are four bits that represent the type of
the GUID, which means whether the guid is randomly generated,
name based on SHA1, time based etc.
This change changes the CoCreateGuid to use the uuid_generate function
from the uuid library that should generate well-formed GUIDs on
Linux and OSX and the uuid_create on FreeBSD.

8 years agoMerge pull request #1753 from ramarag/lttng_check
Rama krishnan Raghupathy [Tue, 13 Oct 2015 05:27:21 +0000 (22:27 -0700)]
Merge pull request #1753 from ramarag/lttng_check

Add checks for liblttng-ust-dev at build time

8 years agoAdd checks for liblttng-ust-dev at build time
Rama [Mon, 12 Oct 2015 23:56:33 +0000 (16:56 -0700)]
Add checks for liblttng-ust-dev at build time

8 years agoMerge pull request #1749 from janvorli/improve-frame-destructor
Jan Vorlicek [Mon, 12 Oct 2015 22:55:35 +0000 (00:55 +0200)]
Merge pull request #1749 from janvorli/improve-frame-destructor

Improve frame destructor performance

8 years agoMerge pull request #1746 from kouvel/PerfFix2
Koundinya Veluri [Mon, 12 Oct 2015 22:12:11 +0000 (15:12 -0700)]
Merge pull request #1746 from kouvel/PerfFix2

Add some fast paths that are missing outside Windows

8 years agoImprove frame destructor performance
Jan Vorlicek [Tue, 6 Oct 2015 00:25:36 +0000 (02:25 +0200)]
Improve frame destructor performance

This change makes the Frame destructor access thread local storage much less often,
only in rare case when the Frame was not popped before the destructor is called.

8 years agoAdd some fast paths that are missing outside Windows
Koundinya Veluri [Fri, 9 Oct 2015 18:36:25 +0000 (11:36 -0700)]
Add some fast paths that are missing outside Windows

- Allocation fast path for arrays of object elements
  - On Linux, a microbenchmark is 25% faster with the portable fast path
  - On Windows with the asm fast path, the microbenchmark was 52% faster than on Linux before, and is now 22% faster with the portable fast path
  - On Windows, the portable fast path is within 4% slower than the asm fast path
- Allocation fast path for objects
  - On Linux, a microbenchmark is 200% faster with the portable fast path
  - On Windows with the asm fast path, the microbenchmark was 325% faster than on Linux before, and is now 43% faster with the portable fast path
  - On Windows, the portable fast path is within 1% slower than the asm fast path
- Skipped the Box fast path since that seems to be inlined into jitted code using the new object fast path. As a result of adding the new object fast path, boxing perf has also improved outside Windows similarly to above.

8 years agoMerge pull request #1739 from jasonwilliams200OK/master
Jan Kotas [Mon, 12 Oct 2015 14:24:11 +0000 (07:24 -0700)]
Merge pull request #1739 from jasonwilliams200OK/master

code,pal: Removes unused headers

8 years agoMerge pull request #1598 from ramarag/enablelttng
Rama krishnan Raghupathy [Mon, 12 Oct 2015 05:28:38 +0000 (22:28 -0700)]
Merge pull request #1598 from ramarag/enablelttng

Enablelttng

8 years agoThis enables Lttng Logging for CoreClr,
Rama [Fri, 9 Oct 2015 01:53:59 +0000 (18:53 -0700)]
This enables Lttng Logging for CoreClr,

The Tracepoint Providers are built as a separate shared object called libcoreclrtraceptprovider and it is
available in the same directory as libcoreclr.so

By Default the ability of Tracing will not be present
To enable the ability of  Tracing, the apps need to be run like:

LD_PRELOAD=libcoreclrtraceptprovider.so ./corerun HelloWorld.exe

For now to change Xplat Event Logging mechanism add any events to:
        <root>/src/vm/ClrEtwAll.man
Then regenerate files by running :
        <root>/src/inc/genXplatLtnng.pl

Conflicts:
Documentation/building/linux-instructions.md

8 years agocode,pal: Removes unused headers.
Peter Jas [Sat, 10 Oct 2015 19:05:53 +0000 (19:05 +0000)]
code,pal: Removes unused headers.
`<typeinfo>` and `<errno.h>` were unused in `seh.cpp`.
On FreeBSD, the `errno` macro was conflicting due to these
stale inclusions. The exact cause is still unknown that how
it is building on CI and only failing locally on clean FreeBSD 10.2
installation with Clang 3.5.

Fixes #1091.

8 years agoMerge pull request #1733 from kouvel/PerfFix
Jan Kotas [Sat, 10 Oct 2015 01:46:30 +0000 (18:46 -0700)]
Merge pull request #1733 from kouvel/PerfFix

Add allocation fast path for arrays of value type elements outside Wi…

8 years agoMerge pull request #1723 from steveharter/FixDefaultLocale
Steve Harter [Fri, 9 Oct 2015 21:11:25 +0000 (16:11 -0500)]
Merge pull request #1723 from steveharter/FixDefaultLocale

Add support for obtaining default locale in Linux and fix issue with collation not being passed to ICU

8 years agoMerge pull request #1737 from dotnet-bot/from-tfs
Jan Kotas [Fri, 9 Oct 2015 20:15:39 +0000 (13:15 -0700)]
Merge pull request #1737 from dotnet-bot/from-tfs

Merge changes from TFS

8 years agoAdd support for obtaining default locale in Linux and fix issue with @collation=...
Steve Harter [Thu, 8 Oct 2015 22:03:53 +0000 (17:03 -0500)]
Add support for obtaining default locale in Linux and fix issue with @collation= not being passed to ICU

8 years agoAdd allocation fast path for arrays of value type elements outside Windows
Koundinya Veluri [Thu, 8 Oct 2015 21:40:14 +0000 (14:40 -0700)]
Add allocation fast path for arrays of value type elements outside Windows

- A microbenchmark involving byte array allocation was about 200% faster on Windows compared to Linux
- On Windows, using the portable version of the fast path is about 5% slower than using the asm version on the microbenchmark
- On Linux, using the portable fast path improves the microbenchmark perf by 160%
- With the fast path enabled on Linux, the microbenchmark on Windows (with asm fast path) is now about 17% faster than on Linux.

8 years agoFix contracts so scanRuntime@x86chk task succeeds
Bruce Forstall [Fri, 9 Oct 2015 15:38:34 +0000 (08:38 -0700)]
Fix contracts so scanRuntime@x86chk task succeeds

[tfs-changeset: 1535699]

8 years agoMerge pull request #1719 from stephentoub/sb_direction
Stephen Toub [Fri, 9 Oct 2015 12:21:00 +0000 (08:21 -0400)]
Merge pull request #1719 from stephentoub/sb_direction

Add [Out] to a couple of StringBuilders in DllImports

8 years agoMerge pull request #1722 from stephentoub/casing_branches
Stephen Toub [Fri, 9 Oct 2015 12:18:53 +0000 (08:18 -0400)]
Merge pull request #1722 from stephentoub/casing_branches

Improve String.ToLower/ToUpper throughput on Unix

8 years agoMerge pull request #1721 from richardlford/fixruntimecontract
Richard L Ford [Fri, 9 Oct 2015 05:16:58 +0000 (22:16 -0700)]
Merge pull request #1721 from richardlford/fixruntimecontract

Fix incorrect runtime contract for FailedAssembly::Initialize

8 years agoCode changes for firing etw events when a pdb is dynamically loaded. This corresponds...
dotnet-bot [Fri, 9 Oct 2015 01:28:37 +0000 (18:28 -0700)]
Code changes for firing etw events when a pdb is dynamically loaded. This corresponds to the following user story: 1200238. As a profiler developer, I want to access PDB content for dynamic assemblies in the ETW event stream, so that I can show my users source for their callstack frames.

The main function in eventtrace.cpp, EmitCodeSymbols, divides the pdb stream into appropriate chunks and transmits them as events in order. There are helper functions which have been taken from the corresponding code in Desktop (Netfxdev1) to provide a dynamically loaded pdb through the profiler authored by Noah.

The test uses EventToSymbols, which is based on TraceEventParser to turn on the CodeSymbols and ModuleLoad keywords and listens for events that send out pdb chunks. It then concatenates the chunks to create the pdb on disk. The BigPdb.cs program loads a dll and pdb dynamically, which triggers the events by the runtime. Once the pdb is recreated, it is compared with the original to make sure there are no differences and if not, the test is declared as passed. The test also uses an EventSource event to transmit a unique cookie to make sure the pdb created corresponds to the process under test, in case there are mulitple instances of the test running.

[tfs-changeset: 1535373]

8 years agoMerge pull request #1726 from janvorli/fix-win-coreconsole
Jan Kotas [Fri, 9 Oct 2015 00:07:02 +0000 (17:07 -0700)]
Merge pull request #1726 from janvorli/fix-win-coreconsole

Fix Windows CoreConsole property list

8 years agoFix Windows CoreConsole property list
Jan Vorlicek [Thu, 8 Oct 2015 23:13:14 +0000 (01:13 +0200)]
Fix Windows CoreConsole property list

The property list was broken - the keys list was missing a comma between two entries,
so they got merged into one string. And the values list was missing one value too.

8 years agoImprove string.ToLower/ToUpper perf
stephentoub [Thu, 8 Oct 2015 21:38:06 +0000 (17:38 -0400)]
Improve string.ToLower/ToUpper perf

Our Unix implementation of changing case is currently slower than our implementation on Windows.  In our ChangeCase implementation in System.Globalization.Native that uses ICU, we have a loop that reads each code point, processes it, and writes out the result.  That processing involves branching into four cases based on whether we're going to upper or to lower, and whether we're using Turkish or not.  By manually hoisting the invariants and loop cloning in order to remove the branches from the inner loop, we can improve the performance of this routine by ~15-20%.

8 years agoMerge pull request #1706 from mmitche/update-badges
Matt Mitchell [Thu, 8 Oct 2015 22:25:09 +0000 (15:25 -0700)]
Merge pull request #1706 from mmitche/update-badges

Update badges to new job locations

8 years agoMerge pull request #1724 from mmitche/reenable-suse
Matt Mitchell [Thu, 8 Oct 2015 22:20:29 +0000 (15:20 -0700)]
Merge pull request #1724 from mmitche/reenable-suse

Reenable OpenSuSE PRs

8 years agoReenable OpenSuSE PRs
Matt Mitchell [Thu, 8 Oct 2015 22:19:39 +0000 (15:19 -0700)]
Reenable OpenSuSE PRs

8 years agoMerge pull request #1720 from janvorli/fix-osx-activation-and-exception-collision
Jan Vorlicek [Thu, 8 Oct 2015 20:46:22 +0000 (22:46 +0200)]
Merge pull request #1720 from janvorli/fix-osx-activation-and-exception-collision

Fix OSX hardware exception and activation collision

8 years agoMerge pull request #1689 from Priya91/palstring
Lakshmi Priya [Thu, 8 Oct 2015 20:32:02 +0000 (13:32 -0700)]
Merge pull request #1689 from Priya91/palstring

Apply StackString for pal MAX_LONGPATH references.

8 years agoFix incorrect runtime contract for FailedAssembly::Initialize
Richard L Ford [Thu, 8 Oct 2015 20:21:14 +0000 (13:21 -0700)]
Fix incorrect runtime contract for FailedAssembly::Initialize

FailedAssembly::Initialize may call CLRException::GetHR.
CLRException::GetHR is marked GC_TRIGGERS.
FailedAssembly::Initialize has to be marked GC_TRIGGERS
as well.

8 years agoMerge pull request #1718 from stephentoub/stringcasing_allocations
Jan Kotas [Thu, 8 Oct 2015 19:48:41 +0000 (12:48 -0700)]
Merge pull request #1718 from stephentoub/stringcasing_allocations

Reduce allocations in string.ToLower/ToUpper on Unix

8 years agoFix OSX hardware exception and activation collision
Jan Vorlicek [Thu, 8 Oct 2015 19:34:33 +0000 (21:34 +0200)]
Fix OSX hardware exception and activation collision

This change fixes a collision between hardware exception handling and activation
injection. Both require creating helper frame on stack and patching the target thread
context to inject invocation of a handler. When these two happened together,
they both tried to update the context and create helper frame on stack at the same
time, which lead to various issues including PAL_SEHException escaping from
the host application unhandled.

8 years agoMerge pull request #1683 from krixalis/master
Aditya Mandaleeka [Thu, 8 Oct 2015 19:20:23 +0000 (12:20 -0700)]
Merge pull request #1683 from krixalis/master

Typos in qcall.h

8 years agoFix additional StringBuilder directions in corefx Unix code
stephentoub [Thu, 8 Oct 2015 18:50:28 +0000 (14:50 -0400)]
Fix additional StringBuilder directions in corefx Unix code

8 years agoMerge pull request #1711 from mikem8361/unhand
Mike McLaughlin [Thu, 8 Oct 2015 18:35:30 +0000 (11:35 -0700)]
Merge pull request #1711 from mikem8361/unhand

Fix unhandled exception debugger notification

8 years agoAdd Null check to lib_name before getting string length.
Lakshmi Priya Sekar [Wed, 7 Oct 2015 18:04:19 +0000 (11:04 -0700)]
Add Null check to lib_name before getting string length.

8 years agoAdd [Out] to a couple of StringBuilders in DllImports
stephentoub [Thu, 8 Oct 2015 17:14:21 +0000 (13:14 -0400)]
Add [Out] to a couple of StringBuilders in DllImports

I audited all of the StringBuilders being used in P/Invokes in mscorlib.  Almost all of them were correctly annotated as to their direction of needed marshaling.  Two weren't.

8 years agoReduce allocations in string.ToLower/ToUpper on Unix
stephentoub [Thu, 8 Oct 2015 15:51:14 +0000 (11:51 -0400)]
Reduce allocations in string.ToLower/ToUpper on Unix

On Windows, ToLower/ToUpper calls into InternalChangeCaseString in the runtime which does two optiizations:
- a 0-length check to just return an empty string if the source is empty
- allocates the string and writes the results into it directly

In our current Unix implementation, the ChangeCase implementation doesn't do either of these, no special-casing empty strings and first allocating a char[] into which the results are written and then constructing a string from that.

This commit brings the Unix implementation more in line with the Windows one, adding the 0-length check, and writing the results directly into the result string so as to avoid the unnecessary char[] allocation and copy.

8 years agoMerge pull request #1715 from janvorli/add-unhandled-exception-trap
Jan Vorlicek [Thu, 8 Oct 2015 13:35:56 +0000 (15:35 +0200)]
Merge pull request #1715 from janvorli/add-unhandled-exception-trap

Add high level unhandled exception trap

8 years agoAdd high level unhandled exception trap
Jan Vorlicek [Thu, 8 Oct 2015 12:41:56 +0000 (14:41 +0200)]
Add high level unhandled exception trap

After the recent exception handling changes, we were missing a high level
exception trap in the ExecuteAssembly. While it was fine for exceptions
comming from or through managed code, it was a problem for exceptions
happening in the ExecuteAssembly call chain before the managed code was
called.
So for example when ExecuteAssembly was called for assembly that didn't
have proper file access rights for the current user, the PAL_SEHException
leaked out of the host app (corerun) and the C++ runtime displayed
message like this:
libc++abi.dylib: terminating with uncaught exception of type PAL_SEHException
Abort trap: 6
The fix is to put back the unhandled managed exception trap that I've removed
with the recent exception handling changes, but to implement it in a slightly
different way - to contain exception holder.
This way, the call to InternalUnhandledExceptionFilter_Worker can be removed
from the UnwindManagedExceptionPass1, since this new trap would ensure it
is called for both the case when the one in UnwindManagedExceptionPass1 would
kick in and the case that this issue is fixing.

8 years agoMerge pull request #1688 from kouvel/LoadFromNativeSearchPathsTest
Stephen Toub [Thu, 8 Oct 2015 12:31:36 +0000 (08:31 -0400)]
Merge pull request #1688 from kouvel/LoadFromNativeSearchPathsTest

Add unit test for loading native library from host-provided native se…

8 years agoMerge pull request #1702 from stephentoub/marshaling_allocs
Stephen Toub [Thu, 8 Oct 2015 02:37:47 +0000 (22:37 -0400)]
Merge pull request #1702 from stephentoub/marshaling_allocs

Reduce garbage generated in StringBuilder marshaling

8 years agoMerge pull request #1696 from brianrob/framepointers
Brian Robbins [Wed, 7 Oct 2015 22:47:02 +0000 (15:47 -0700)]
Merge pull request #1696 from brianrob/framepointers

Enable Frame Pointers for UNIX Builds

8 years agoMerge pull request #1693 from Kagamine/correct-word-spelling
Jan Kotas [Wed, 7 Oct 2015 22:42:37 +0000 (15:42 -0700)]
Merge pull request #1693 from Kagamine/correct-word-spelling

Correct word spelling

8 years agoMerge pull request #1705 from kyulee1/protojit
Jan Kotas [Wed, 7 Oct 2015 22:39:51 +0000 (15:39 -0700)]
Merge pull request #1705 from kyulee1/protojit

Producing standalone Jit for testing

8 years agoFix UnhandledException notifications. The IsDebuggerPresent check was left over...
Mike McLaughlin [Wed, 7 Oct 2015 04:10:39 +0000 (21:10 -0700)]
Fix UnhandledException notifications.  The IsDebuggerPresent check was left over from windows coreclr and prevented notifications from being sent.

8 years agoMerge pull request #1697 from mikem8361/osxbp
Mike McLaughlin [Wed, 7 Oct 2015 21:43:48 +0000 (14:43 -0700)]
Merge pull request #1697 from mikem8361/osxbp

Fix managed breakpoints on OSX

8 years agoAddress PR feedback
stephentoub [Wed, 7 Oct 2015 21:43:00 +0000 (17:43 -0400)]
Address PR feedback

Remove addition of null terminator, which should already be added by ConvertToAnsi.

8 years agoMerge pull request #1710 from mmitche/disable-centos-suse
Matt Mitchell [Wed, 7 Oct 2015 20:13:26 +0000 (13:13 -0700)]
Merge pull request #1710 from mmitche/disable-centos-suse

Disable PRs on SuSE and CentOS until we get more nodes spun up

8 years agoDisable PRs on SuSE and CentOS until we get more nodes spun up
Matt Mitchell [Wed, 7 Oct 2015 20:11:20 +0000 (13:11 -0700)]
Disable PRs on SuSE and CentOS until we get more nodes spun up

8 years agoProducing standalone Jit for testing
Kyungwoo Lee [Mon, 5 Oct 2015 21:16:32 +0000 (14:16 -0700)]
Producing standalone Jit for testing

protojit.dll (Windows) / libprotojit.so (*nix) is produced in addition, which is a standalone Jit that are not attached to CoreCLR.
Note CoreCLR still embeds such Jit by default same as before to not disrupt the existing clients.
This (same) standalone Jit can be used for other testing purpose and also this can be specified as an altjit as well in CoreCLR.
Added to nuget dev package.

8 years agoMerge pull request #1645 from eerhardt/Encoding
Eric Erhardt [Wed, 7 Oct 2015 19:15:43 +0000 (14:15 -0500)]
Merge pull request #1645 from eerhardt/Encoding

Implement Encodings on Linux

8 years agoMerge remote-tracking branch 'upstream/master' into osxbp
Mike McLaughlin [Wed, 7 Oct 2015 18:44:49 +0000 (11:44 -0700)]
Merge remote-tracking branch 'upstream/master' into osxbp

8 years agoUpdate badges to new job locations
Matt Mitchell [Wed, 7 Oct 2015 18:20:00 +0000 (11:20 -0700)]
Update badges to new job locations

8 years agoMerge pull request #1701 from janvorli/fix-osx-hardware-exceptions-2
Josh Free [Wed, 7 Oct 2015 17:39:54 +0000 (10:39 -0700)]
Merge pull request #1701 from janvorli/fix-osx-hardware-exceptions-2

Add missing file to the previous hardware exceptions fix

8 years agoReduce garbage generated in StringBuilder marshaling
stephentoub [Wed, 7 Oct 2015 16:48:53 +0000 (12:48 -0400)]
Reduce garbage generated in StringBuilder marshaling

When marshaling a StringBuilder In as Ansi (both done by default on Unix), we currently allocate enough native memory to hold the result.  But then we use AnsiCharMarshaler.DoAnsiConversion to allocate a managed byte array and fill it with the converted results, and then we copy those results into the native memory.  We can instead just call String's ConvertToAnsi directly, avoiding the extra managed array allocation and the extra memory copy.

8 years agoAdd missing file to the previous hardware exceptions fix
Jan Vorlicek [Wed, 7 Oct 2015 16:45:43 +0000 (18:45 +0200)]
Add missing file to the previous hardware exceptions fix

8 years agoMerge pull request #1692 from mmitche/add-netci
Matt Mitchell [Wed, 7 Oct 2015 15:41:51 +0000 (08:41 -0700)]
Merge pull request #1692 from mmitche/add-netci

Initial commit of CI definition for inner loop

8 years agoInitial commit of CI definition for inner loop
Matt Mitchell [Tue, 6 Oct 2015 22:56:08 +0000 (15:56 -0700)]
Initial commit of CI definition for inner loop