Skip to content

Conversation

robn
Copy link
Member

@robn robn commented Aug 12, 2025

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.

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:

@amotin amotin added the Status: Code Review Needed Ready for review and testing label Aug 12, 2025
@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
Accidentally removed calls in ed048fd.

Sponsored-by: https://despairlabs.com/sponsor/
Signed-off-by: Rob Norris <[email protected]>
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 <[email protected]>
@robn robn force-pushed the linux-6.17-compat branch from 003825b to c3ead1f Compare September 8, 2025 23:33
@robn
Copy link
Member Author

robn commented Sep 8, 2025

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
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 <[email protected]>
Reviewed-by: Tony Hutter <[email protected]>
Signed-off-by: Rob Norris <[email protected]>
Closes #17621
behlendorf pushed a commit to behlendorf/zfs that referenced this pull request Sep 10, 2025
Accidentally removed calls in ed048fd.

Sponsored-by: https://despairlabs.com/sponsor/
Reviewed-by: Brian Behlendorf <[email protected]>
Reviewed-by: Tony Hutter <[email protected]>
Signed-off-by: Rob Norris <[email protected]>
Closes openzfs#17621
behlendorf pushed a commit to behlendorf/zfs that referenced this pull request Sep 10, 2025
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 <[email protected]>
Reviewed-by: Tony Hutter <[email protected]>
Signed-off-by: Rob Norris <[email protected]>
Closes openzfs#17621
@KrzysztofHajdamowicz
Copy link

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 :)

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)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants