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

linux-hardkernel: 4.14.47-139 -> 4.14.55-146. Additionally, use vendo… #43637

Merged
merged 1 commit into from Aug 1, 2018

Conversation

qolii
Copy link
Contributor

@qolii qolii commented Jul 16, 2018

…r defconfig.

Motivation for this change

Update this vendor kernel to the latest release, and adopt the defconfig provided by them.

In order to make this build, I had to disable the gator module via extraConfig. The build error was of the form of not being able to find a header file related to the mali graphics driver. I've left in (commented out) my attempt to address the problem, but it doesn't work. I can provide more info if anyone is interested, and if anyone knows how to fix this properly, I'm all ears.

In any case, disabling the building of gator seems to work fine, and I've had this kernel running on my HC1 for the last few days without incident.

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option 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/)
  • Determined the impact on package closure size (by running nix path-info -S before and after)
  • Fits CONTRIBUTING.md.

@qolii
Copy link
Contributor Author

qolii commented Jul 17, 2018

Oh, I didn't expect this to cause so many rebuilds. Whomever gets to it, feel free to merge into staging instead, I guess?

@qolii
Copy link
Contributor Author

qolii commented Jul 18, 2018

Cc: @andrew-d (if you're interested!)

@andrew-d
Copy link
Contributor

👍 yeah, I remember seeing this myself at some point, though I think I ended up disabling something more specific? I'm not near my XU4 to check right now, but I might have just disabled CONFIG_MALI_GATOR_SUPPORT to get everything to build? I can test tomorrow evening and report back.

@qolii
Copy link
Contributor Author

qolii commented Jul 19, 2018

Oh, cool. I have zero idea what Gator does or how this build works, so apparently I went for a larger hammer :)

I'll also give your idea a try if I get to it by then.

Thanks!

@qolii
Copy link
Contributor Author

qolii commented Jul 19, 2018

Well, adding MALI_GATOR_SUPPORT n to the extraConfig didn't do it. The failure is:

../drivers/gator/gator_events_mali_midgard.c:25:10: fatal error: linux/mali_linux_trace.h: No such file or directory
 #include "linux/mali_linux_trace.h"
          ^~~~~~~~~~~~~~~~~~~~~~~~~~
compilation terminated.

which seems like a problem with gator, not with gator support in mali.

Is there a GATOR_MALI_SUPPORT option instead?

@andrew-d
Copy link
Contributor

Okay, so, I did some digging and I think I figured out the issue.

  • The error is happening here because the header file isn't found
  • That file is included in the Makefile here if the CONFIG_GATOR_MALI_MIDGARD flag is set to y.
  • The odroidxu4_defconfig file sets it to y here.

So, you should be able to set GATOR_MALI_MIDGARD n to solve the problem, I think? There's some question of what happens with this block of the Makefile, but I'd try out the GATOR_MALI_MIDGARD n first.

@qolii
Copy link
Contributor Author

qolii commented Jul 20, 2018

Just tried GATOR_MALI_MIDGARD n. Same error :(

How important do you feel this is?

@andrew-d
Copy link
Contributor

Gah, that's annoying. Honestly, I think GATOR n is okay for this PR.

Separately, do you think it's worth opening an issue at https://github.com/hardkernel/linux/ to report this upstream?

@qolii
Copy link
Contributor Author

qolii commented Jul 20, 2018

It can't hurt! I have been guessing they would say they don't have the resources to chase this, since it clearly works for them and the failure is somewhat NixOS-specific.

This page implies to me that it's really a problem with the makefile in the upstream kernel. Another page I found (but can't find again) suggested it only goes wrong when you set source and destination directories for the kernel build as different, which does sound like something NixOS would do.

Maybe Hardkernel would be interested in escalating... I'll file a bug with them, maybe later today, if you don't get to it first :)

@qolii
Copy link
Contributor Author

qolii commented Jul 20, 2018

I think, to anyone who has the power/desire to merge this: please go ahead. This works ok as it is.

I'll follow up with HK about fixing the build problem properly, and we can remove the kludge in this package at that time.

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

4 participants