Skip to content
This repository has been archived by the owner on Dec 18, 2017. It is now read-only.

architecture is "x86' on arm #2063

Closed
moozzyk opened this issue Jun 13, 2015 · 8 comments
Closed

architecture is "x86' on arm #2063

moozzyk opened this issue Jun 13, 2015 · 8 comments
Milestone

Comments

@moozzyk
Copy link
Contributor

moozzyk commented Jun 13, 2015

To determine OS architecture we just check the size of the pointer (

RuntimeArchitecture = IntPtr.Size == 8 ? "x64" : "x86";
). As a result Win10 IoT one core on ARM is retported to be x86.

@BrennanConroy
Copy link
Member

Here too if we run tests on arm:

var runtimeArchitecture = IntPtr.Size == 8 ? "x64" : "x86";

@moozzyk
Copy link
Contributor Author

moozzyk commented Jun 15, 2015

And in kestrel where we use it to determine where to load libuv.dll.

@analogrelay
Copy link
Contributor

Shouldn't the native bootstrapper just pass this data in and the managed code just read it rather than trying to redetect it? The bootstrapper is unique per architecture and OS (and there's a bootstrapper for Mono too).

@moozzyk
Copy link
Contributor Author

moozzyk commented Jun 17, 2015

Possibly. Might part of #1970

@muratg
Copy link
Contributor

muratg commented Sep 29, 2015

@DamianEdwards is ARM in scope for V1?

cc @davidfowl @Eilon @danroth27

Also see aspnet/KestrelHttpServer#106

@moozzyk
Copy link
Contributor Author

moozzyk commented Sep 29, 2015

We seem to be doing this now only in one place: https://github.com/aspnet/dnx/blob/dev/src/Microsoft.Dnx.Loader/LoadContext.cs#L71

@davidfowl
Copy link
Member

Do we still need this?

@moozzyk
Copy link
Contributor Author

moozzyk commented Dec 7, 2015

Possibly - depending on what happens with LoadContext and if ARM is going to be officially supported.

@muratg muratg modified the milestone: Backlog Dec 11, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants