Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@steveisok
Copy link
Contributor

@steveisok steveisok commented Aug 22, 2019

Currently, if the process is 32-bit, roslyn tries to determine how much
memory is available. To do that, it tries to pinvoke into GlobalMemoryStatusEx,
which is not supported on mono.

Without it, the compiler server bombs on https://github.com/dotnet/roslyn/blob/0e63260c5afb3fb5b74c357dd250e500172bcd63/src/Compilers/Server/VBCSCompiler/NamedPipeClientConnection.cs#L55-L59

…nux.

Currently, if the process is 32-bit, roslyn tries to determine how much
memory is available.  To do that, it tries to pinvoke into GlobalMemoryStatusEx,
which is not supported on mono.
@akoeplinger akoeplinger merged commit fbdad78 into mono:master Aug 22, 2019
@akoeplinger
Copy link
Member

@monojenkins backport 2019-08

@monojenkins
Copy link
Contributor

@akoeplinger backporting to 2019-08 failed, the patch results in conflicts:

Applying: Added a workaround for an issue with the compiler server on 32-bit linux.
Using index info to reconstruct a base tree...
M	mono/mini/intrinsics.c
Falling back to patching base and 3-way merge...
Auto-merging mono/mini/intrinsics.c
CONFLICT (content): Merge conflict in mono/mini/intrinsics.c
error: Failed to merge in the changes.
Patch failed at 0001 Added a workaround for an issue with the compiler server on 32-bit linux.

Please backport manually!

@akoeplinger
Copy link
Member

Roslyn issue: dotnet/roslyn#38211

steveisok pushed a commit to steveisok/mono that referenced this pull request Aug 22, 2019
Currently, if the process is 32-bit, roslyn tries to determine how much
memory is available. To do that, it tries to pinvoke into GlobalMemoryStatusEx,
which is not supported on mono.

Without it, the compiler server bombs on https://github.com/dotnet/roslyn/blob/0e63260c5afb3fb5b74c357dd250e500172bcd63/src/Compilers/Server/VBCSCompiler/NamedPipeClientConnection.cs#L55-L59

Backport of mono#16405
akoeplinger pushed a commit that referenced this pull request Aug 23, 2019
Currently, if the process is 32-bit, roslyn tries to determine how much
memory is available. To do that, it tries to pinvoke into GlobalMemoryStatusEx,
which is not supported on mono.

Without it, the compiler server bombs on https://github.com/dotnet/roslyn/blob/0e63260c5afb3fb5b74c357dd250e500172bcd63/src/Compilers/Server/VBCSCompiler/NamedPipeClientConnection.cs#L55-L5

Backport of #16405
akoeplinger pushed a commit that referenced this pull request Aug 23, 2019
Currently, if the process is 32-bit, roslyn tries to determine how much
memory is available. To do that, it tries to pinvoke into GlobalMemoryStatusEx,
which is not supported on mono.

Without it, the compiler server bombs on https://github.com/dotnet/roslyn/blob/0e63260c5afb3fb5b74c357dd250e500172bcd63/src/Compilers/Server/VBCSCompiler/NamedPipeClientConnection.cs#L55-L59

Backport of #16405
ManickaP pushed a commit to ManickaP/runtime that referenced this pull request Jan 20, 2020
…nux. (mono/mono#16405)

Currently, if the process is 32-bit, roslyn tries to determine how much
memory is available.  To do that, it tries to pinvoke into GlobalMemoryStatusEx,
which is not supported on mono.

Without it, the compiler server bombs on https://github.com/dotnet/roslyn/blob/mono/mono@0e63260c5afb3fb5b74c357dd250e500172bcd63/src/Compilers/Server/VBCSCompiler/NamedPipeClientConnection.cs#L55-L59

Commit migrated from mono/mono@fbdad78
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants