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

Fix driver build for linux 6.13 #269

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

okt-sergeyn
Copy link
Contributor

Fix issue #264 and couple of others related to linux-6.13.


Checked by running HAVE_SFC=0 make on a few kernels: 5.15, 6.1, 6.13.
Build and some basic tests pass.

@okt-sergeyn okt-sergeyn requested a review from a team as a code owner March 3, 2025 15:51
Copy link
Contributor

@pemberso-xilinx pemberso-xilinx left a comment

Choose a reason for hiding this comment

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

I had recently reproduced this kbuild issue on 6.13 and was looking at same set of kernel changes.

Fix to Makefile looks good to me and confirmed build is ok on Ubuntu 24.04 with 6.13.0-061300-generic kernel

@okt-sergeyn okt-sergeyn force-pushed the pr-linux-6.13 branch 2 times, most recently from f987067 to 827e900 Compare March 5, 2025 17:34
Fix path to kernel sources for autocompat scripts. CURDIR cannot be used
since it now points to Onload build directory. Use objtree instead.
Fix expression, that sets 'src' variable, to use absolute 'obj' path.

See Linux commit 13b25489b6f8

Signed-off-by: Sergey Nikitin <[email protected]>
Current compat code assumes that get_file_rcu() is a macro. It's
not true since linux-6.7.
Additionally, in linux-6.13, after removing file::f_count field,
this code no longer compiles.
See Linux commits 0ede61d8589c and 90ee6ed776c0.

Update compat code to support new version of get_file_rcu().

Signed-off-by: Sergey Nikitin <[email protected]>
poll_does_not_wait() was removed in linux-6.13. Replace it
with direct access to poll_table structure.

Signed-off-by: Sergey Nikitin <[email protected]>
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.

4 participants