Skip to content
This repository was archived by the owner on Jan 23, 2023. It is now read-only.

Fix build breaks in Linux ARM port#2935

Merged
jkotas merged 3 commits into
dotnet:masterfrom
jkotas:arm-fixes
Jan 31, 2016
Merged

Fix build breaks in Linux ARM port#2935
jkotas merged 3 commits into
dotnet:masterfrom
jkotas:arm-fixes

Conversation

@jkotas
Copy link
Copy Markdown
Member

@jkotas jkotas commented Jan 31, 2016

  • Add missing files for ARM legacy JIT
  • Cleanup top-level defines
  • Introduce central location for turning FEATURE_EVENT_TRACE on/off

There are likely more fixes required ... waiting for a new SSD card for my Raspberry Pi2 to arrive.

@jkotas
Copy link
Copy Markdown
Member Author

jkotas commented Jan 31, 2016

cc @benpye @kangaroo

@jkotas
Copy link
Copy Markdown
Member Author

jkotas commented Jan 31, 2016

@ramarag Could you please take a look?

@jkotas jkotas force-pushed the arm-fixes branch 2 times, most recently from c0de392 to 81fd0f0 Compare January 31, 2016 05:48
- Delete left-over cruft from internal build
- Move Windows-specific defines under WIN32
@kangaroo
Copy link
Copy Markdown

FEATURE_EVENT_TRACE stuff LGTM.

armv7 stuff looks GTM, but I haven't bootstrapped it locally.

@benpye
Copy link
Copy Markdown

benpye commented Jan 31, 2016

LGTM on a quick glance, I will try and build this locally tomorrow.

On Sun, 31 Jan 2016, 06:01 Geoff Norton notifications@github.com wrote:

FEATURE_EVENT_TRACE stuff LGTM.

armv7 stuff looks GTM, but I haven't bootstrapped it locally.


Reply to this email directly or view it on GitHub
#2935 (comment).

@ramarag
Copy link
Copy Markdown
Member

ramarag commented Jan 31, 2016

FEATURE_EVENT_TRACE changes LGTM

Turn FEATURE_EVENT_TRACE on for x64 Linux only. aarch64 and arm do not
have working lttng in their ubuntu/debian repos. We can expand to other
platforms in the future as they become supported.
jkotas added a commit that referenced this pull request Jan 31, 2016
Fix build breaks in Linux ARM port
@jkotas jkotas merged commit c4a0fb1 into dotnet:master Jan 31, 2016
@richlander
Copy link
Copy Markdown
Member

@jkotas how big of an SD card do you recommend?

@jkotas
Copy link
Copy Markdown
Member Author

jkotas commented Feb 1, 2016

8GB was not enough for Raspbian + CoreCLR build. 16GB should be enough. If you are buying a new SD card, I would recommend 64GB - it is where the best $/GB point seems to be these days.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants