Merge pull request #890 from xoofx/master

[Mono.Linker] Add support for <Module> with static method and pre-mark t...

[Mono.Linker] Add support for using the linker with assemblies referencing WPF

[Mono.Linker] Add support for <Module> with static method and pre-mark them if necessary

[sgen] Fix an infinite loop in find_previous_pointer_fragment due to remove CAS'ing the wrong value.

The fragment list has one invariant, the next pointer is tagged if the current fragment is been deleted,
which means the list head can never point to a tagged pointer.

If this invariant is broken, the code in find_previous_pointer_fragment will end up in an infinite loop.

The infinite loop happens in this simplified fragment:

try_again:
prev = &allocator->alloc_head;
1) cur = unmask (*prev);
while (1) {
2) if (*prev != cur)
goto try_again;

So assume allocator->alloc_head points to 0x121, which is tagged.

So execution goes like this:

1) cur is set to 0x120
2) *prev (value 0x121) is different than cur (value 0x120) so retry.

The above will loop forever as the fixup code never touches allocator->alloc_head.

Now that we understand the infinite loop, let's see how the remove code can trigger it.

Use use the following notation in the event sequence:

-> a non tagged pointer
-*> a tagged pointer
first CAS: InterlockedCompareExchangePointer ((volatile gpointer*)&frag->next, mask (next, 1), next)
second CAS: InterlockedCompareExchangePointer ((volatile gpointer*)prev_ptr, next, frag)

Assuming the following initial fragment list and two threads removing two separate fragments:

fragment list; -> A -> B -> C

thread 1 removes A
thread 2 removes B

thread 2 succeed the first CAS
-> A -> B -*> C

thread 1 succeed the first CAS
-> A -*> B -*> C

thread 2 fail the second CAS because of thread 1
-> A -*> B -*> C

thread 1 succeed the second CAS
-> B -*> C

thread 2 succeed the second CAS and set prev_ptr to next,
which at this point is a tagged pointer. Leaving the fragment list like this:
-*> C

After the last CAS, we end up with alloc_head with a tagged pointer, which will cause
the infinite loop we previously explained.

The fix is to unmask the value we store in the second CAS, as the objective is to link
two live fragments and not flag the previous fragment for deletion.

The exact same bug exists in mono-linked-list-set and was fixed there.

[Http]: Cosmetic, remove some dead code.

The is no reason to ever invoke the callback on the ThreadPool,
it would in fact only cause trouble.

Marek's commit db92449f was correct, I removed the commented out
code and the no longer used method.

[mcs] Implement rethrow support in awaited catch clause

[Http]: Fix digest authentication (bug #18799).

We did not correctly parse key-value pairs without quotes in the
challenge.

[System.Net.Http]: Always add "Content-Length" in HttpClientHandler.SendAsync().

This has been reported on the forums:
http://forums.xamarin.com/discussion/17770/length-required-error-in-http-post-since-latest-release

[Mono.Debugger.Soft] Improved ILInterpreter to eval properties that return a primitive constant

[windows] Return early in the vectored exception handler if the thread is not managed by the runtime.

[Http]: Only recycle ServicePoints from the idle timer (fixes #19823).

Thanks a lot for Martin Potter to find and report the race condition
which was causing the problem.

ServicePointManager.RecycleServicePoints() was a left-over from the
old times when we didn't have an idle-timer based approach.

[Mono.Debugger.Soft] minor code cleanup

Fix mono exception handler crashing on windows.

Here's the scenario that causes the crash:

Exception is fired, which invokes mono's vectored exception handler. It
is not an exception mono recognizes, so falls through to the default:
case in the exception handler. The problem is the return value has been
initialized to EXCEPTION_CONTINUE_EXECUTION. This causes SEH not to
process any other handlers, and just attempt to continue execution at
the point the exception was thrown.

The problem is much of windows internals will catch various exceptions
internally and handle them. So instead of the exception being handled,
we would just crash in the bowels of windows.

The solution is to set mono_win_chained_exception_needs_run to TRUE in
the default case - this is what the various exception handlers do if the
exception is not in JIT'd code. This causes the handler to return
EXCEPTION_CONTINUE_SEARCH, which causes SEH to search for the next
handler, and no crash is produced.

Both x86 and x64 had this bug, so I fixed it in both places.

Fix contributed by Steve Anichini.

Changed mono_win_chained_exception_needs_run to be a JIT TLS variable.

This fixes a potential race condition that might happen if two threads try to handle an exception at the same time.

[sys.xaml] fix TypeId and get broken WindowsBase tests pass.

Attempt to sanitize the `vasprintf` situation in eglib.

The problem is that on some platforms we call the function
without having provided a prototype for it. This breaks when
building with `-Werror`.

Now we declare it in the private `vasprintf.h` header when
the platform doesn't have a declaration in `stdio.h`. This
should also let us remove the hack in `glib.h` for MSVC.

This may or may not break the build on some platforms; I've
tried on those that I have access to with success. Let me
know if this breaks anything.

Add build system plumbing for building with -Werror.

Merge pull request #1094 from LRFalk01/master

Identities not being set in ClaimsPrincipal Ctor

Identities not being set in ClaimsPrincipal Ctor

Class' identities member is not set in constructor ClaimsPrincipal
(IEnumerable<ClaimsIdentity> identities)

Remove unnecessary MPH_INTERNAL applied to a function definition.

This should be applied to prototypes, and it already is for
this function (in mph.h).

Remove the architecture conditions for using `perf` in the log profiler.

We now use generic atomics which work on all platforms, so there's
no reason to keep this constraint here. This allows the `perf`-backed
profiler to be used on mips, ppc, s390x, etc.

Revert "Disable `perf` support in the profiler when running on Android (for now)."

This reverts commit 8e35b7d01e14371851a806dad6196b36d9163c91.

The fallback implementation (as used on other platforms) appears to hang
the runtime. Having a maybe-working `perf`-backed sample profiler is better
than a not-working sample profiler.

Revert "[x86] Reenable the no pushes code."

This reverts commit 5093efd55883e8a8a093fdb43d938d7b9dde7a4e.

Revert this as it still breaks the build.

[x86] Fix the build.

[x86] Reenable the no pushes code.

[sys.xaml] remove extraneous NET_2_1 and use __MOBILE__ to make it build for XA/XiOS.

[jit] Align the stack when using the pushless code on x86.

Merge pull request #1090 from BrianDurham/patch-2

Update X11Display.cs - change "Check you DISPLAY"

Merge pull request #1089 from BrianDurham/patch-1

Update Graphics.cs - change "Check you DISPLAY"

Merge pull request #1091 from BrianDurham/patch-3

Update XplatUIX11.cs - change "Check you DISPLAY"

Merge pull request #1092 from BrianDurham/patch-4

Update XplatUIX11GTK.cs - change "Check you DISPLAY"

Update XplatUIX11GTK.cs - change "Check you DISPLAY"

Update XplatUIX11GTK.cs - change "Check you DISPLAY" text to "Check your DISPLAY"

Update XplatUIX11.cs - change "Check you DISPLAY"

Update XplatUIX11.cs - change "Check you DISPLAY" to "Check your DISPLAY"

Update X11Display.cs - change "Check you DISPLAY"

Change "Check you DISPLAY environment variable" to "Check your DISPLAY environment variable"

Update Graphics.cs - change "Check you DISPLAY"

Update Graphics.cs - change "Check you DISPLAY environment variable" to "Check your DISPLAY environment variable"

[x86] Disable the no-pushes code for now, it breaks many tests.

Apply the change in a99ebf9843f0cf92b3a9a856738378dcace20566 to mprof-report too.

Fix a ton of warnings when building the log profiler with newer zlib versions.

Merge pull request #1088 from BrzVlad/pushless

[x86] Arguments to calls are passed via moves, instead of pushes.

Silence another array subscript warning in eglib/src/sort.frag.h.

Fix typo in bf703c4a8f95db4859452474499038c7539ea3c7.

Fixed the build on MSVC.

Local variables need to be declared at the start of the block.

Add missing prototypes to supportw.c.

Comment an unused function in minizip.

Fix z_alloc () signature.

Yes, really. zlib really uses a 32-bit int here...

Add missing prototypes to Mono.Posix C sources.

Move shared_area_disabled () behind ifdef.

Wrap deadce_has_run variable in DISABLE_SSA ifdef.

Add g_assert_not_reached () to mono_gc_pthread_exit ().

On Android, pthread_exit () is not decorated as noreturn.

Silence unused mkstemp result warnings in jay.

Add missing prototypes in ikvm-native.

Mark an unused variable as such in eglib/test/file.c.

Disable -Warray-bounds in eglib/src/sort.frag.h!insert_list in GCC 4.6.0+.

Disable -Wreturn-local-addr in libgc/mark_rts.c!GC_approx_sp with GCC 4.8.0+.

Compile libgc with -Wno-deprecated-declarations as it uses some obsolete POSIX APIs.

Revert unintentional change.

Fix the build. We've been missing this include for ages...

Bring the implementation of the 64-bit CAS fallback in line with the 32-bit one.

Mark the `spin` variable in atomic.c as G_GNUC_UNUSED.

It would require an `#ifdef` forest to get the conditions
under which it is used right.

Merge branch 'master' of github.com:mono/mono

Revert "Fix a warning in atomic.c."

This reverts commit 1fdc8789a7f4bfc6fa7f67c126569765194a3b6c.

This breaks the ios build:

../../../../../mono/mono/utils/atomic.c: In function ‘InterlockedCompareExchange64’:
../../../../../mono/mono/utils/atomic.c:572: error: ‘spin’ undeclared (first use in this function)
../../../../../mono/mono/utils/atomic.c:572: error: (Each undeclared identifier is reported only once
../../../../../mono/mono/utils/atomic.c:572: error: for each function it appears in.)

[arm] Fix OP_ATOMIC_CAS_I4.

[x86] Arguments to calls are passed via moves, instead of pushes.

I release these changes under the MIT license.

Disable `perf` support in the profiler when running on Android (for now).

We need to assess the situation before we enable this everywhere. Apparently
some older devices have trouble with `perf`.

Fix a warning in atomic.c.

Remove a #warning in security.c which gets triggered on OS X.

[counters] Add counters documentation to man page

Fix printf formatting warnings in mono-counters.c.

Use G_GNUC_UNUSED instead of commenting out variable declarations.

The previous approach resulted in mixed decls/stmts warnings.

Fix a ton of printf formatting warnings in decode.c.

Silence some warnings in decode.c.

Fix volatile pointer casts in proflog.c.

Silence a bunch of unused result warnings in proflog.c.

Fix a bunch of warnings caused by unused functions.

Add suggested parentheses to RHS of ^ operator.

Fix formatting specifiers for some size_t arguments.

[runtime] Avoid joining attached threads. Fixes #19343.

[Http]: Read version from "HTTP CONNECT" response.

When using "HTTP CONNECT" to create a tunnel to a proxy server,
check the proxy server's response for the HTTP Version and close
the connection if it's HTTP/1.0.

[Http]: Fix the loop in WebConnection.ReadHeaders().

We need to reset `gotStatus' each time we enter the outer loop or we
would attempt to interpret the status line as a header.

Merge pull request #1084 from tritao/msvc-warnings

MSVC warning fixes

[runtime] Fix support for custom marshallers defined in other assemblies. Fixes #20020.

Fixed monodis MSVC build.

We need to include "mono/utils/mono-compiler.h" to get the proper isinf/isnan declaration.

[mcs] Return type inference of async anonymous methods

[mcs] Check for await expression inside new delegate statement

[runtime] Fix a jit assertion on a class which contains an empty struct as a static field. Fixes #20349.

Regenerate keyboards.resx resource file.

The existing one is reported as malformed XML by csc.

Fixed warning in MSVC: 'unreferenced local variable'.

Fixed warning in MSVC: 'different types for formal and actual parameter'.

Fixed warning in MSVC: 'void' function returning a value.

Fixed warnings in MSVC: 'unreferenced local variable'.

Removed an unused variable from the code, fixes a warning in MSVC.

Fixed warning in MSVC: 'unreferenced local variable'.

Fixed warning in MSVC: 'unreferenced local variable'.

Fixed warning in MSVC: 'different types for formal and actual parameter'.

Fixed warning in MSVC: 'getpid' undefined; assuming extern returning int.

Fixed unreferenced local variable warning in MSVC.

Fixed MSVC warning about unreferenced local variable.

Fixed MSVC warning about different types for formal and actual parameter.

Fixed unreferenced variables/label warnings in MSVC.

Fixed unsigned/signed mismatch warning.