Skip to content

Linux 6.17 compat - #17621

Closed
robn wants to merge 2 commits into
openzfs:masterfrom
robn:linux-6.17-compat
Closed

Linux 6.17 compat#17621
robn wants to merge 2 commits into
openzfs:masterfrom
robn:linux-6.17-compat

Conversation

@robn

@robn robn commented Aug 12, 2025 •
edited
Loading

Copy link
Copy Markdown
Member

Motivation and Context

And Yertle the Turtle, the king of the trees,
The king of the air and the birds and the bees,
The king of a house and a cow and a mule . . .
Well, that was the end of the Turtle King’s rule!
For Yertle, the King of all Sala-ma-Sond,
Fell off his high throne and fell Plunk! in the pond!

– Yertle the Turtle, Dr. Seuss, 1958.

16.7-rc1 is here; lets make sure there's a fancy filesystem to go with it.

Description

Only one change so far:

  • d_set_d_op() is no longer available, so replace it with just enough to swap in dentry ops in the snapdir automounter.

More might be needed, in particular, the change from mmap to mmap_prepare in struct file_operations. However, this a deprecation, not a removal, so it can possibly wait.

How Has This Been Tested?

ZTS run passed on 16.7-rc1, -rc2, -rc3, -rc4, -rc5, -rc6.

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)
  • Quality assurance (non-breaking change which makes the code more robust against bugs)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Library ABI change (libzfs, libzfs_core, libnvpair, libuutil and libzfsbootenv)
  • Documentation (a change to man pages or other documentation)

Checklist:

@hphilm hphilm mentioned this pull request Aug 12, 2025
Closed
@amotin amotin added the Status: Code Review Needed Ready for review and testing label Aug 12, 2025
This was referenced Aug 13, 2025
Merged
Closed
@robn
robn force-pushed the linux-6.17-compat branch 2 times, most recently from e89e773 to 9b83cfc Compare August 25, 2025 11:06
@robn
robn force-pushed the linux-6.17-compat branch from 9b83cfc to 003825b Compare September 2, 2025 22:43
robn added 2 commits September 9, 2025 09:17
@robn
config: restore ZFS_AC_KERNEL_DENTRY tests
566b74e 
Accidentally removed calls in ed048fd.

Sponsored-by: https://despairlabs.com/sponsor/
Signed-off-by: Rob Norris
@robn
Linux 6.17: d_set_d_op() is no longer available
c3ead1f 
We only have extremely narrow uses, so move it all into a single
function that does only what we need, with and without d_set_d_op().

Sponsored-by: https://despairlabs.com/sponsor/
Signed-off-by: Rob Norris
@robn
robn force-pushed the linux-6.17-compat branch from 003825b to c3ead1f Compare September 8, 2025 23:33
@robn

robn commented Sep 8, 2025

Copy link
Copy Markdown
Member Author

Passed all the way through to -rc5. I don't think any further changes will be necessary.

@behlendorf behlendorf added Status: Accepted Ready to integrate (reviewed, tested) and removed Status: Code Review Needed Ready for review and testing labels Sep 9, 2025
@behlendorf behlendorf closed this in 9e5e95c Sep 9, 2025
behlendorf pushed a commit that referenced this pull request Sep 9, 2025
@robn @behlendorf
Linux 6.17: d_set_d_op() is no longer available
7939bad 
We only have extremely narrow uses, so move it all into a single
function that does only what we need, with and without d_set_d_op().

Sponsored-by: https://despairlabs.com/sponsor/
Reviewed-by: Brian Behlendorf 
Reviewed-by: Tony Hutter 
Signed-off-by: Rob Norris 
Closes #17621
behlendorf pushed a commit to behlendorf/zfs that referenced this pull request Sep 10, 2025
@robn @behlendorf
config: restore ZFS_AC_KERNEL_DENTRY tests
d469371 
Accidentally removed calls in ed048fd.

Sponsored-by: https://despairlabs.com/sponsor/
Reviewed-by: Brian Behlendorf 
Reviewed-by: Tony Hutter 
Signed-off-by: Rob Norris 
Closes openzfs#17621
behlendorf pushed a commit to behlendorf/zfs that referenced this pull request Sep 10, 2025
@robn @behlendorf
Linux 6.17: d_set_d_op() is no longer available
0df91ab 
We only have extremely narrow uses, so move it all into a single
function that does only what we need, with and without d_set_d_op().

Sponsored-by: https://despairlabs.com/sponsor/
Reviewed-by: Brian Behlendorf 
Reviewed-by: Tony Hutter 
Signed-off-by: Rob Norris 
Closes openzfs#17621
@KrzysztofHajdamowicz

KrzysztofHajdamowicz commented Sep 10, 2025

Copy link
Copy Markdown

Hello!

What is needed to run zfs 2.3.3 against kernel 6.17?
From what I understant from #17623 and https://gitlab.manjaro.org/packages/extra/zfs-dkms/-/commit/1c8279c8dc91aad57cb6974c7e776f7984147a0b
I deducted that I need to apply changes from this pull request minus kernel-dentry-operations.m4-related changes

Please confirm or straighten my thinking :)

@mabod mabod mentioned this pull request Oct 1, 2025
Closed
tonyhutter pushed a commit to tonyhutter/zfs that referenced this pull request Oct 17, 2025
@robn @tonyhutter
Linux 6.17: d_set_d_op() is no longer available
7d7ae89 
We only have extremely narrow uses, so move it all into a single
function that does only what we need, with and without d_set_d_op().

Sponsored-by: https://despairlabs.com/sponsor/
Reviewed-by: Brian Behlendorf 
Reviewed-by: Tony Hutter 
Signed-off-by: Rob Norris 
Closes openzfs#17621
stgraber pushed a commit to zabbly/zfs that referenced this pull request Oct 17, 2025
@robn @stgraber
Linux 6.17: d_set_d_op() is no longer available
64c0640 
We only have extremely narrow uses, so move it all into a single
function that does only what we need, with and without d_set_d_op().

Sponsored-by: https://despairlabs.com/sponsor/
Reviewed-by: Brian Behlendorf 
Reviewed-by: Tony Hutter 
Signed-off-by: Rob Norris 
Closes openzfs#17621
tonyhutter pushed a commit to tonyhutter/zfs that referenced this pull request Oct 20, 2025
@robn @tonyhutter
Linux 6.17: d_set_d_op() is no longer available
bfcbf9c 
We only have extremely narrow uses, so move it all into a single
function that does only what we need, with and without d_set_d_op().

Sponsored-by: https://despairlabs.com/sponsor/
Reviewed-by: Brian Behlendorf 
Reviewed-by: Tony Hutter 
Signed-off-by: Rob Norris 
Closes openzfs#17621
tonyhutter pushed a commit to tonyhutter/zfs that referenced this pull request Oct 21, 2025
@robn @tonyhutter
Linux 6.17: d_set_d_op() is no longer available
365926a 
We only have extremely narrow uses, so move it all into a single
function that does only what we need, with and without d_set_d_op().

Sponsored-by: https://despairlabs.com/sponsor/
Reviewed-by: Brian Behlendorf 
Reviewed-by: Tony Hutter 
Signed-off-by: Rob Norris 
Closes openzfs#17621
MingcongBai pushed a commit to AOSC-Tracking/zfs that referenced this pull request Oct 27, 2025
@robn @MingcongBai
Linux 6.17: d_set_d_op() is no longer available
c78b9c5 
We only have extremely narrow uses, so move it all into a single
function that does only what we need, with and without d_set_d_op().

Sponsored-by: https://despairlabs.com/sponsor/
Reviewed-by: Brian Behlendorf 
Reviewed-by: Tony Hutter 
Signed-off-by: Rob Norris 
Closes openzfs#17621
MingcongBai pushed a commit to AOSC-Tracking/zfs that referenced this pull request Oct 27, 2025
@robn @MingcongBai
Linux 6.17: d_set_d_op() is no longer available
f722a7e 
We only have extremely narrow uses, so move it all into a single
function that does only what we need, with and without d_set_d_op().

Sponsored-by: https://despairlabs.com/sponsor/
Reviewed-by: Brian Behlendorf 
Reviewed-by: Tony Hutter 
Signed-off-by: Rob Norris 
Closes openzfs#17621
@robn
robn deleted the linux-6.17-compat branch January 9, 2026 02:37
lundman pushed a commit to openzfsonosx/openzfs-fork that referenced this pull request Jan 30, 2026
@robn @lundman
config: restore ZFS_AC_KERNEL_DENTRY tests
cd80b56 
Accidentally removed calls in ed048fd.

Sponsored-by: https://despairlabs.com/sponsor/
Reviewed-by: Brian Behlendorf 
Reviewed-by: Tony Hutter 
Signed-off-by: Rob Norris 
Closes openzfs#17621
lundman pushed a commit to openzfsonosx/openzfs-fork that referenced this pull request Jan 30, 2026
@robn @lundman
Linux 6.17: d_set_d_op() is no longer available
aca6967 
We only have extremely narrow uses, so move it all into a single
function that does only what we need, with and without d_set_d_op().

Sponsored-by: https://despairlabs.com/sponsor/
Reviewed-by: Brian Behlendorf 
Reviewed-by: Tony Hutter 
Signed-off-by: Rob Norris 
Closes openzfs#17621
tonyhutter pushed a commit to tonyhutter/zfs that referenced this pull request Feb 12, 2026
@robn @tonyhutter
config: restore ZFS_AC_KERNEL_DENTRY tests
29bda86 
Accidentally removed calls in ed048fd.

Sponsored-by: https://despairlabs.com/sponsor/
Reviewed-by: Brian Behlendorf 
Reviewed-by: Tony Hutter 
Signed-off-by: Rob Norris 
Closes openzfs#17621
lundman pushed a commit to openzfsonwindows/openzfs that referenced this pull request Feb 23, 2026
@robn @lundman
config: restore ZFS_AC_KERNEL_DENTRY tests
62715d4 
Accidentally removed calls in ed048fd.

Sponsored-by: https://despairlabs.com/sponsor/
Reviewed-by: Brian Behlendorf 
Reviewed-by: Tony Hutter 
Signed-off-by: Rob Norris 
Closes openzfs#17621
lundman pushed a commit to openzfsonwindows/openzfs that referenced this pull request Feb 23, 2026
@robn @lundman
Linux 6.17: d_set_d_op() is no longer available
a8dddd1 
We only have extremely narrow uses, so move it all into a single
function that does only what we need, with and without d_set_d_op().

Sponsored-by: https://despairlabs.com/sponsor/
Reviewed-by: Brian Behlendorf 
Reviewed-by: Tony Hutter 
Signed-off-by: Rob Norris 
Closes openzfs#17621
bugclerk pushed a commit to truenas/zfs that referenced this pull request Aug 19, 2026
@robn @bugclerk
Linux 6.17: d_set_d_op() is no longer available
fc571e0 
We only have extremely narrow uses, so move it all into a single
function that does only what we need, with and without d_set_d_op().

Sponsored-by: https://despairlabs.com/sponsor/
Reviewed-by: Brian Behlendorf 
Reviewed-by: Tony Hutter 
Signed-off-by: Rob Norris 
Closes openzfs#17621
(cherry picked from commit 365926a)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@behlendorf behlendorf behlendorf approved these changes
@tonyhutter tonyhutter tonyhutter approved these changes

Assignees

No one assigned

Labels

Status: Accepted Ready to integrate (reviewed, tested)

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@robn @KrzysztofHajdamowicz @behlendorf @tonyhutter @amotin