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 5.2 compat: Fix config/kernel-shrink.m4 test failure #8776

Merged
merged 1 commit into from
May 25, 2019

Conversation

kusumi
Copy link
Member

@kusumi kusumi commented May 21, 2019

Motivation and Context

Fix ./configure failure on 5.2-rc1.
Note that ZoL still doesn't compile on 5.2 due to several other kernel side changes.

Description

whether ->count_objects callback exists test failed with
error: error message (at AC_MSG_ERROR(error)) for using
an incomplete function shrinker_cb().

Confirmed on Fedora 29 with 5.2-rc1 (but not 5.1 with same gcc).

How Has This Been Tested?

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Performance enhancement (non-breaking change which improves efficiency)
  • Code cleanup (non-breaking change which makes code smaller or more readable)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation (a change to man pages or other documentation)

Checklist:

@loli10K loli10K added the Status: Code Review Needed Ready for review and testing label May 21, 2019
@codecov
Copy link

codecov bot commented May 21, 2019

Codecov Report

Merging #8776 into master will decrease coverage by 0.03%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #8776      +/-   ##
==========================================
- Coverage   78.75%   78.71%   -0.04%     
==========================================
  Files         382      382              
  Lines      117809   117809              
==========================================
- Hits        92776    92730      -46     
- Misses      25033    25079      +46
Flag Coverage Δ
#kernel 79.27% <ø> (-0.04%) ⬇️
#user 67.32% <ø> (-0.18%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c3e5907...0040eae. Read the comment docs.

@behlendorf behlendorf requested a review from tonyhutter May 23, 2019 20:55
Copy link
Contributor

@behlendorf behlendorf left a comment

Choose a reason for hiding this comment

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

The updated check looks right, but could you clarify why this is specific to the 5.2 kernel.

@behlendorf behlendorf added the Type: Building Indicates an issue related to building binaries label May 23, 2019
@kusumi
Copy link
Member Author

kusumi commented May 24, 2019

I'm not sure, but there looks to be change(s) in kbuild in 5.2, e.g. torvalds/linux@83da1be.
I'll look into some more details, and update the commit log if possible.

@behlendorf behlendorf requested a review from loli10K May 25, 2019 00:14
Copy link
Contributor

@loli10K loli10K left a comment

Choose a reason for hiding this comment

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

I built the linux 5.2 compat PRs with a vanilla 5.2.0-rc1 kernel and they look good, thank you.

"whether ->count_objects callback exists" test failed with
"error: error" message for using an incomplete function shrinker_cb().

This is caused by torvalds/linux@83da1bed86. It's configurable,
but we would want to be able to compile with default kbuild setting.

Signed-off-by: Tomohiro Kusumi <kusumi.tomohiro@osnexus.com>
@kusumi
Copy link
Member Author

kusumi commented May 25, 2019

This was caused by torvalds/linux@83da1be.
Mentioned above commit in the commit log.

Definition of KBUILD_MODPOST_WARN is as follows.

--------------------------------------------------
KBUILD_MODPOST_WARN can be set to avoid errors in case of undefined
symbols in the final module linking stage. It changes such errors
into warnings.

@behlendorf behlendorf merged commit 36c110f into openzfs:master May 25, 2019
@behlendorf behlendorf added Status: Accepted Ready to integrate (reviewed, tested) and removed Status: Code Review Needed Ready for review and testing labels May 25, 2019
behlendorf pushed a commit that referenced this pull request Jun 7, 2019
"whether ->count_objects callback exists" test failed with
"error: error" message for using an incomplete function shrinker_cb().

This is caused by torvalds/linux@83da1bed86. It's configurable,
but we would want to be able to compile with default kbuild setting.

Reviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>
Reviewed-by: loli10K <ezomori.nozomu@gmail.com>
Signed-off-by: Tomohiro Kusumi <kusumi.tomohiro@osnexus.com>
Closes #8776
allanjude pushed a commit to allanjude/zfs that referenced this pull request Jun 7, 2019
"whether ->count_objects callback exists" test failed with
"error: error" message for using an incomplete function shrinker_cb().

This is caused by torvalds/linux@83da1bed86. It's configurable,
but we would want to be able to compile with default kbuild setting.

Reviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>
Reviewed-by: loli10K <ezomori.nozomu@gmail.com>
Signed-off-by: Tomohiro Kusumi <kusumi.tomohiro@osnexus.com>
Closes openzfs#8776
allanjude pushed a commit to allanjude/zfs that referenced this pull request Jun 15, 2019
"whether ->count_objects callback exists" test failed with
"error: error" message for using an incomplete function shrinker_cb().

This is caused by torvalds/linux@83da1bed86. It's configurable,
but we would want to be able to compile with default kbuild setting.

Reviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>
Reviewed-by: loli10K <ezomori.nozomu@gmail.com>
Signed-off-by: Tomohiro Kusumi <kusumi.tomohiro@osnexus.com>
Closes openzfs#8776
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Accepted Ready to integrate (reviewed, tested) Type: Building Indicates an issue related to building binaries
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants