Make GHC work on armv7l-linux #33405

dhess · 2018-01-04T00:19:43Z

Motivation for this change

Get GHC working on armv7l-linux

Things done

master on armv7l-linux is currently broken due to #33403, so I can't test this yet on master, but it is currently building on the nixos-unstable-small channel. I will update this PR when I can make progress on master, but these changes are straightforward and only affect ARM builds.

Note that in a few places there are fixes that are also obviously needed for aarch64-linux (e.g., LLVM support), but these patches do not address aarch64-linux in general as there is more work needed to bootstrap that platform due to GHC 7.10.3 and 8.0.2 dependencies in the toolchain for building 8.2.2.

There will also inevitably be more haskellPackages fixes needed for armv7l-linux (usually a dontCheck here and there, in my experience with my own GHC ARM overlay), but I will tackle those as they come up, as compiling even a small subset of haskellPackages on armv7l-linux is a multi-day effort.

Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)
- NixOS
- macOS
- other Linux distributions
Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
Tested execution of all binary files (usually in ./result/bin/)
Fits CONTRIBUTING.md.

dhess · 2018-01-06T20:32:11Z

Here is an update on the status of this PR:

I am able to build ghc-8.0.2 now on armv7l-linux on Nixpkgs master with these changes. However, the ghc-8.2.2 build reliably segfaults at the same stage of the build (3 times in a row). Here is the tail of the log from my personal Hydra: https://gist.github.com/dhess/97e177541f1c047905919042f8fad075 All 3 builds broke at the same point and approximately the same time in the build (5 hours).

As it's a segfault, I assume the compile is running out of memory. Before this last attempt, I bumped the swap on my Jetson TK1 up to 2GB (so 4GB total including RAM), but that did not help. I will now try to reduce the number of concurrent jobs to 2, which will slow things down quite a bit but will hopefully address this issue and get the build to finish.

In any case, I think these changes are ready to merge -- they do not affect other platforms and are sufficient to get ghc-8.0.2 building on armv7l-linux, in any case. More work may yet need to be done to make the ghc-8.2.2 build reliable, but I don't believe that any of the commits here will be affected by such changes; if it does turn out to be memory pressure issue, then likely the number of concurrent jobs will have to be limited during the ghc-8.2.2 build.

The main thing is that I would like to get these changes committed before more of the cross-compile stuff hits master, so that I don't have to maintain these fixes across multiple upstream commits.

@peti can you please have a look?

Ericson2314 · 2018-01-09T18:53:15Z

Oh no, I didn't review this in time. stdenv.is* is very confusing in terms of cross cause we don't know whether we are referring to the build or host platforms.

Ericson2314

@dhess might you have any time to change these accordingly?

Ericson2314 · 2018-01-09T18:56:04Z

pkgs/development/compilers/ghc/7.10.3-binary.nix

@@ -43,6 +43,7 @@ stdenv.mkDerivation rec {
    or (throw "cannot bootstrap GHC on this platform"));

  nativeBuildInputs = [ perl ];
+  buildInputs = stdenv.lib.optionals stdenv.isArm [ llvm_35 ];


stdenv.targetPlatform.isArm

Ericson2314 · 2018-01-09T18:56:20Z

pkgs/development/compilers/ghc/7.10.3.nix

@@ -44,7 +44,7 @@ stdenv.mkDerivation rec {
    ./relocation.patch
  ];

-  buildInputs = [ ghc perl libxml2 libxslt docbook_xsl docbook_xml_dtd_45 docbook_xml_dtd_42 hscolour ];
+  buildInputs = [ ghc perl libxml2 libxslt docbook_xsl docbook_xml_dtd_45 docbook_xml_dtd_42 hscolour ] ++ stdenv.lib.optionals stdenv.isArm [ llvm_35 ];


targetPlatform.isArm

Ericson2314 · 2018-01-09T18:57:11Z

pkgs/development/compilers/ghc/8.2.2.nix

@@ -46,7 +46,7 @@ stdenv.mkDerivation (rec {
    sed 's|#BuildFlavour  = quick-cross|BuildFlavour  = perf-cross|' mk/build.mk.sample > mk/build.mk
  '';

-  buildInputs = [ alex autoconf automake ghc happy hscolour perl python3 sphinx ];
+  buildInputs = [ alex autoconf automake ghc happy hscolour perl python3 sphinx ] ++ stdenv.lib.optionals (stdenv.isArm || stdenv.isAarch64) [ llvm_39 ];


(targetPlatform.isArm || targetPlatform.isAarch64)

Ericson2314 · 2018-01-09T18:57:46Z

pkgs/development/compilers/ghc/8.2.1-binary.nix

@@ -46,6 +46,7 @@ stdenv.mkDerivation rec {
    or (throw "cannot bootstrap GHC on this platform"));

  nativeBuildInputs = [ perl ];
+  buildInputs = stdenv.lib.optionals (stdenv.isArm || stdenv.isAarch64) [ llvm_39 ];


(with stdenv.targetPlatform; isArm || isAarch64)

Ericson2314 · 2018-01-09T18:58:30Z

pkgs/development/haskell-modules/generic-builder.nix

@@ -130,7 +130,7 @@ let
    (optionalString (enableSharedExecutables && stdenv.isDarwin) "--ghc-option=-optl=-Wl,-headerpad_max_install_names")
    (optionalString enableParallelBuilding "--ghc-option=-j$NIX_BUILD_CORES")
    (optionalString useCpphs "--with-cpphs=${cpphs}/bin/cpphs --ghc-options=-cpp --ghc-options=-pgmP${cpphs}/bin/cpphs --ghc-options=-optP--cpp")
-    (enableFeature (enableDeadCodeElimination && (versionAtLeast "8.0.1" ghc.version)) "split-objs")
+    (enableFeature (enableDeadCodeElimination && !stdenv.isArm && !stdenv.isAarch64 && (versionAtLeast "8.0.1" ghc.version)) "split-objs")


(hostPlatform.isArm || hostPlatform.isAarch64)

dhess · 2018-01-09T20:51:34Z

@Ericson2314 Yes, of course. I figured there would be cross fixes coming down the pipe.

Ericson2314 · 2018-01-09T21:08:25Z

@dhess Thanks! Yeah just would to confirm those additions will work for you. Then, when I land the cross stuff it will be one fewer likely regression :).

dhess · 2018-01-09T21:24:59Z

@Ericson2314 It might take me a day or two to get to this (because that is roughly the turnaround time on GHC builds for armv7l-linux, unfortunately) but ping me if you haven't seen it by then.

dhess · 2018-01-11T06:17:13Z

@Ericson2314 PR made, see #33733. Please review and make sure I didn't miss something.

dhess added 4 commits January 3, 2018 15:57

ghc: ARM builds have an LLVM runtime dependency.

461139e

ghc: 8.2.2 on armv7l doesn't build without explicit use of ld.gold.

8ad5937

haskell-modules: split-objs is not supported on ARM.

260749c

haskell-modules: disable happy tests on armv7l-linux.

05824e6

dhess requested a review from peti as a code owner January 4, 2018 00:19

GrahamcOfBorg added 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux labels Jan 4, 2018

dhess mentioned this pull request Jan 4, 2018

GHC on ARM platforms #33353

Closed

5 tasks

peti merged commit 48a6d1a into NixOS:master Jan 8, 2018

dhess deleted the ghc-armv7l branch January 8, 2018 18:33

Ericson2314 reviewed Jan 9, 2018

View reviewed changes

dhess mentioned this pull request Jan 11, 2018

ghc, haskell-modules: ARM cross build fixes. #33733

Merged

8 tasks

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Make GHC work on armv7l-linux #33405

Make GHC work on armv7l-linux #33405

dhess commented Jan 4, 2018

dhess commented Jan 6, 2018

Ericson2314 commented Jan 9, 2018

Ericson2314 left a comment

Ericson2314 Jan 9, 2018

Ericson2314 Jan 9, 2018

Ericson2314 Jan 9, 2018

Ericson2314 Jan 9, 2018 •

edited

Loading

Ericson2314 Jan 9, 2018

dhess commented Jan 9, 2018

Ericson2314 commented Jan 9, 2018

dhess commented Jan 9, 2018 •

edited

Loading

dhess commented Jan 11, 2018

Make GHC work on armv7l-linux #33405

Make GHC work on armv7l-linux #33405

Conversation

dhess commented Jan 4, 2018

Motivation for this change

Things done

dhess commented Jan 6, 2018

Ericson2314 commented Jan 9, 2018

Ericson2314 left a comment

Choose a reason for hiding this comment

Ericson2314 Jan 9, 2018

Choose a reason for hiding this comment

Ericson2314 Jan 9, 2018

Choose a reason for hiding this comment

Ericson2314 Jan 9, 2018

Choose a reason for hiding this comment

Ericson2314 Jan 9, 2018 • edited Loading

Choose a reason for hiding this comment

Ericson2314 Jan 9, 2018

Choose a reason for hiding this comment

dhess commented Jan 9, 2018

Ericson2314 commented Jan 9, 2018

dhess commented Jan 9, 2018 • edited Loading

dhess commented Jan 11, 2018

Ericson2314 Jan 9, 2018 •

edited

Loading

dhess commented Jan 9, 2018 •

edited

Loading