fix(arm): improve ARM build process and compatibility

Signed-off-by: Max Newcomer <[email protected]>

fix: echo string formating

Signed-off-by: Max Newcomer <[email protected]>

fix: remove neon flag

Signed-off-by: Max Newcomer <[email protected]>

fix(wip): compile with gcc

Signed-off-by: Max Newcomer <[email protected]>

Author: maxwnewcomer

Diff for: nix/pkgs/libspdk/default.nix

@@ -31,7 +31,6 @@
 , libtool
 , liburing
 , libuuid
-, llvmPackages
 , meson
 , nasm
 , ncurses
@@ -69,6 +68,8 @@ let
         else
           "--without-fio";
 
+      # Only set crossPrefix if we're actually cross-compiling 
+      # (which we aren't, but let's keep the logic).
       crossPrefix =
         if targetPlatform.config != buildPlatform.config then
           "--crossPrefix=${targetPlatform.config}"
@@ -90,13 +91,13 @@ let
     version = "24.05-${lib.substring 0 7 rev}";
     name = "${pname}-${version}";
   };
+
   drvAttrs = rec {
     pname = spdk.pname;
     version = spdk.version;
 
     src = [
       (fetchFromGitHub {
-        # Note that this would only rebuild if the first 7 chars differ, but in practice should be fine
         name = spdk.name;
         owner = "openebs";
         repo = "spdk";
@@ -113,16 +114,13 @@ let
       cmake
       gcc
       help2man
-      llvmPackages.bintools
-      llvmPackages.clang
-      llvmPackages.libclang
       meson
       ninja
       pkg-config
       procps
       udev
       utillinux
-      (python3.withPackages (ps: with ps; [ pyelftools ]))
+      pkgs.python3Packages.pyelftools
     ] ++ extraBuildInputs;
 
     buildInputs = [
@@ -158,9 +156,7 @@ let
     enableParallelBuilding = true;
     hardeningDisable = [ "all" ];
 
-    #
-    # Phases.
-    #
+    # Our phases
     prePatch = ''
       pushd ..
       chmod -R u+w build_scripts
@@ -181,4 +177,4 @@ let
     '';
   };
 in
-llvmPackages.stdenv.mkDerivation drvAttrs
+pkgs.stdenv.mkDerivation drvAttrs

Diff for: nix/shell/spdk.nix

@@ -18,6 +18,14 @@ let
     echo "FIO path        : $FIO"
   '';
 
+  # Determine the system architecture
+  system = builtins.currentSystem;
+
+  # Define CFLAGS based on the architecture
+  cflagsValue = if system == "aarch64-linux"
+                then "-march=armv8-a"
+                else "-msse4";
+
   # spdk-path argument overrides spdk argument.
   spdkCfg = if spdk-path != null then "none" else spdk;
 
@@ -63,14 +71,14 @@ let
     };
 
     # Do not use Nix libspdk. User must provide SPDK.
-    # Build environment for development libspdk packahe is provided.
+    # Build environment for development libspdk package is provided.
     none = {
       drv = null;
 
       buildInputs = with pkgs; libspdk-dev.nativeBuildInputs ++ libspdk-dev.buildInputs;
 
       shellEnv = {
-        CFLAGS = "-msse4";
+        CFLAGS = cflagsValue;
         SPDK_RS_BUILD_USE_LOGS = "yes"; # Tells spdk-rs build.rs script to rerun when build_logs dir is updated.
       };