Skip to content
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

linuxPackages.rtl88xxau-aircrack: fix build for linux 5.15 #145697

Merged
merged 1 commit into from
Nov 13, 2021

Conversation

MatthewCroughan
Copy link
Contributor

Co-authored-by: nixinator 33lockdown33@protonmail.com

ZHF: #144627

@jonringer, @tomberek , @nrdxp

Things done
  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
  • Tested, as applicable:
  • Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
  • Tested basic functionality of all binary files (usually in ./result/bin/)
  • 21.11 Release Notes (or backporting 21.05 Release notes)
    • (Package updates) Added a release notes entry if the change is major or breaking
    • (Module updates) Added a release notes entry if the change is significant
    • (Module addition) Added a release notes entry if adding a new NixOS module
  • Fits CONTRIBUTING.md.

@nixinator
Copy link
Member

it compiles now after applying a upstream patch, don't have the hardware so can't test it running. Maybe some can do that?

Copy link
Member

@SuperSandro2000 SuperSandro2000 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why are we not bumping the package?

@MatthewCroughan
Copy link
Contributor Author

Why are we not bumping the package?

I saw the pattern of using the patch and decided to continue with it, just to get it compiling. Should we bump the version instead?

This removes the unnecessary patches and bumps the version to the rev of
tag v5.6.4.2

ZHF: NixOS#144627

Co-authored-by: nixinator <33lockdown33@protonmail.com>
@MatthewCroughan
Copy link
Contributor Author

@SuperSandro2000

The latest tagged release does compiles successfully with a 5.15 kernel, though only if we remove the 5.12 kernel patch, so I have done this.

Copy link
Contributor

@jonringer jonringer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

failures are broken on target branch

https://github.com/NixOS/nixpkgs/pull/145697

2 packages marked as broken and skipped:
linuxKernel.packages.hardkernel_4_14.rtl88xxau-aircrack linuxPackages_hardkernel_latest.rtl88xxau-aircrack

1 package failed to build:
linuxPackages_hardened.rtl88xxau-aircrack

18 packages built:
linuxKernel.packages.linux_4_14.rtl88xxau-aircrack linuxPackages_4_14_hardened.rtl88xxau-aircrack linuxKernel.packages.linux_4_19.rtl88xxau-aircrack linuxPackages_4_19_hardened.rtl88xxau-aircrack linuxKernel.packages.linux_4_4.rtl88xxau-aircrack linuxKernel.packages.linux_4_9.rtl88xxau-aircrack linuxPackages.rtl88xxau-aircrack linuxKernel.packages.linux_5_14.rtl88xxau-aircrack linuxPackages_5_14_hardened.rtl88xxau-aircrack linuxPackages_latest.rtl88xxau-aircrack linuxKernel.packages.linux_5_4.rtl88xxau-aircrack linuxPackages_5_4_hardened.rtl88xxau-aircrack linuxPackages_latest-libre.rtl88xxau-aircrack linuxPackages-libre.rtl88xxau-aircrack linuxPackages_lqx.rtl88xxau-aircrack linuxPackages_testing_bcachefs.rtl88xxau-aircrack linuxPackages_xanmod.rtl88xxau-aircrack linuxPackages_zen.rtl88xxau-aircrack

cc @SuperSandro2000

@Mic92 Mic92 merged commit 3f68612 into NixOS:master Nov 13, 2021
@@ -20,14 +20,6 @@ stdenv.mkDerivation rec {

NIX_CFLAGS_COMPILE="-Wno-error=incompatible-pointer-types";

# Fix build for 5.12 kernels
patches = [
(fetchpatch {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please remove the unused fetchpatch from inputs.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@nixinator
Copy link
Member

nixinator commented Nov 13, 2021

there are quite a few of these out of tree wireless drivers that exhibit this problem. (due 5.15 kernel maintainers removing ipx for ever :-( )

https://hydra.nixos.org/build/157498591/nixlog/1

so all these drivers need bumping/patching too :-).

how do we handle older kernels like 5.10.x , or is a 'nixos release' always tied to a 'supported kernel' and out of tree drivers need to follow the 'nixos release kernel version'????

@nixinator
Copy link
Member

linuxPackages_5_15.rtl8812au

is source from https://github.com/gordboy/rtl8812au-5.9.3.2

however that doesn't compile of 5.15 , however

https://github.com/morrownr/8812au-20210629

apparently does, however , do we swap over or make a new package for this driver?

I've not got the hardware so hard to test, or leave it for some poor soul to fix later?

@Mic92
Copy link
Member

Mic92 commented Nov 15, 2021

linuxPackages_5_15.rtl8812au

is source from https://github.com/gordboy/rtl8812au-5.9.3.2

Are both projects still maintained? Does the new project only build with new kernels?

however that doesn't compile of 5.15 , however

https://github.com/morrownr/8812au-20210629

apparently does, however , do we swap over or make a new package for this driver?

I've not got the hardware so hard to test, or leave it for some poor soul to fix later?

If the original kernel maintainer has tested the code and it compiles we might not need to test it ourself.

@nixinator
Copy link
Member

nixinator commented Nov 15, 2021

having looked at a few of these out of tree wireless drivers, the are quite a few repo's that seem to support the same things, so it's hard to get one source of truth. Many have patches to get them into the more 'exotic modes' of operation, like this driver for example.

Although we have fairly good integration testing via Nixos tests , we need a test farm of real machines, connected to real hardware, to do bare metal nixos testing in realtime!... :-). I hear of projects in the embedded space, that do exactly that.

You can but dream(2nix) of something like this one day.

again, this is a just a build error (ipx.h removed from 5.15), which is a clear show stopper, but to verify it actually works... you need hardware, and someone that cares about nixos+5.15+rtl cards!

@Mic92
Copy link
Member

Mic92 commented Nov 15, 2021

If you don't have any real users to rely on, than we can also disable those drivers for newer kernel versions and wait until someone can try one of the forks.

@tomberek
Copy link
Contributor

Although we have fairly good integration testing via Nixos tests , we need a test farm of real machines, connected to real hardware, to do bare metal nixos testing in realtime!... :-). I hear of projects in the embedded space, that do exactly that.

Random comment/response: I did this a few years ago with some ARM boards and software-defined radios, using Hydra for orchestration. It was glorious!

@nixinator
Copy link
Member

the force is strong with you @tomberek , this needs to happen again, and become glorious once more! :-)

'Nixos Certified Hardware' 🥇 🥇

@MatthewCroughan
Copy link
Contributor Author

@nixinator @tomberek

Thankfully, I've been testing my Intel wireless card by using the NixOS Hardware Certification test suite. Looks like it passed on 5.14.15.

nixos-hardware-certification

@nixos-discourse
Copy link

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:

https://discourse.nixos.org/t/nix-iso-unable-to-boot-in-uefi-mode-but-other-distros-can/16473/6

@nixos-discourse
Copy link

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:

https://discourse.nixos.org/t/wifi-failing-since-recent-nixos-update-looking-for-debug-or-bisect-help/17638/5

@MatthewCroughan MatthewCroughan deleted the fix-rtl88xxau-aircrack branch April 17, 2022 22:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants