-
Notifications
You must be signed in to change notification settings - Fork 56
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
AGX Wireless bug fix #259
AGX Wireless bug fix #259
Conversation
https://ssrc.atlassian.net/browse/SP-2906 Bug was introduced by netvmExtraModules in hardware was overriding the netvmExtraModules in target. Fixed that issue. There were unnecessary if statements in som type checking for enabling wireless, fixed. Added a fix for docker if somehow being added but this will not introduce any delay as if docker support is not included the fix is also not included. |
I'd prefer the WiFi fix and Docker fix as separate commits/PRs as they are not related to each other. There's practical outcome from this - if the Docker overlay was a separate commit, it's easier to revert the overlay commit when we eventually get the fix from the upstream. The whole docker daemon enabling and the docker group are also separate from overlaying the Docker. |
I already rebooted the remote builder, but it repeats. I'll check if it's build host related. |
Nope, still the same. Ends up stuck in |
|
I checked the builds for many times for both flash-script and image flakes of agx and image flakes of nx of but I'll try a rebuild switch to check again. |
And a small note to add. The issue in docker was due to a bug in Go compiler. So that fix will also solve issues in many other Go compiler based applications. It is a docker fix but I'll rename the commit of docker fix as "Docker fix due to Go issue". |
Ok, thanks for confirming. I'm building it now directly on AGX without remote ssh builder connection. |
I've removed the docker fix from my local. After confirming build is correct I'll commit and rename the pull request for Wifi Fix. Then I'll later add a PR with Docker fix due to Go compiler issue. |
Gets stuck on:
This happens also on local build on |
Apparently I'm facing this NixOS/nix#2029 |
I'll commit a Wireless fix only PR just after check. So you could try on that. |
Signed-off-by: Emrah Billur <[email protected]> Remove docker Signed-off-by: Emrah Billur <[email protected]>
Replaced the PR into only wireless for agx fix. |
Still the same:
|
Still the same: [nix-shell:~/ghaf]$ git log --oneline -1 [nix-shell:~/ghaf]$ nix build .#packages.aarch64-linux.nvidia-jetson-orin-agx-debug
Still the same: [nix-shell:~/ghaf]$ git log --oneline -1 [nix-shell:~/ghaf]$ nix build .#packages.aarch64-linux.nvidia-jetson-orin-agx-debug
Still the same: [nix-shell:~/ghaf]$ git log --oneline -1 [nix-shell:~/ghaf]$ nix build .#packages.aarch64-linux.nvidia-jetson-orin-agx-debug root@emrah-ThinkPad-P14s-Gen-3:/home/emrah/ghaf-support-for-nx# nix build .#packages.aarch64-linux.nvidia-jetson-orin-nx-debug |
I get following lines to the kernel log - multiple times:
|
I'm trying to reproduce the same error but could not manage to get this. I remember in some cross compilation configuration cipher part caused errors but still not sure. I'll also try cross build now. |
Are we ready to merge? I'm just testing this and if everything goes ok I will push the merge button |
In my tests everything go ok, and the NetVM connects to the WLAN network |
I changed my remote builder from Orin AGX (ghaf/nixox) to M1 (nixos) and it passed building the problematic package - it's still building the |
Separation of netvmExtraModules caused a bug in agx wireless, where this was fixed by adding appropriate corrections. Wireless condition for agx is improved. A small fix was added for docker (where it is not included if it is not enabled) in case it is planned to add for support.