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

Backport gcc-7 compilation fix from mainline linux tree #300

Merged
merged 1 commit into from
Mar 3, 2018
Merged

Backport gcc-7 compilation fix from mainline linux tree #300

merged 1 commit into from
Mar 3, 2018

Conversation

mk-fg
Copy link

@mk-fg mk-fg commented Jun 3, 2017

Original commit: 474c901
Author: Linus Torvalds [email protected]
Summary: give up on gcc ilog2() constant optimizations

This fixes build issue with gcc-7.

How failure looks: https://sourceforge.net/p/aufs/mailman/message/35865397/
Issue in kernel bugzilla: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=72785
Link to commit addressing it in mainline: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/patch/?id=474c90156c8dcc2fa815e6716cc9394d7930cb9c

Original commit: 474c901
Author: Linus Torvalds <[email protected]>
Summary: give up on gcc ilog2() constant optimizations
@Taijian
Copy link

Taijian commented Sep 17, 2017

Could we maybe get this merged, because currently compiling the odroid-3.14.y branch with gcc 7.x breaks hw accelleration on the Odroid C2 as can be seen here.

@mk-fg
Copy link
Author

mk-fg commented Sep 17, 2017

Does recompiling with gcc7 and this patch actually help (as opposed to no-patch or gcc6)?

I'm a bit skeptical, because it's a fix for build error, which afaict affects only aufs module, so if you can build with up-to-date gcc7, doesn't seem to be relevant to your issue and maybe not needed anymore.

@mdrjr mdrjr merged commit c3e4c73 into hardkernel:odroidc2-3.14.y Mar 3, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants