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.

Sign up for GitHub

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

Jump to bottom

FreeBSD: add missing vop_fplookup assignments #14788

Merged
merged 1 commit into from
Apr 24, 2023
Merged

FreeBSD: add missing vop_fplookup assignments #14788

merged 1 commit into from
Apr 24, 2023

Conversation

mjguzik
Copy link
Contributor

@mjguzik mjguzik commented Apr 24, 2023

It became illegale to not have them as of
5f6df177758b9dff88e4b6069aeb2359e8b0c493 ("vfs: validate that vop vectors provide all or none fplookup vops") upstream.

Motivation and Context

Description

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)
  • Library ABI change (libzfs, libzfs_core, libnvpair, libuutil and libzfsbootenv)
  • Documentation (a change to man pages or other documentation)

Checklist:

@mjguzik
FreeBSD: add missing vop_fplookup assignments
4349ab4 
It became illegale to not have them as of
5f6df177758b9dff88e4b6069aeb2359e8b0c493 ("vfs: validate that vop
vectors provide all or none fplookup vops") upstream.

Signed-off-by: Mateusz Guzik <mjguzik@gmail.com>
@mjguzik
Copy link
Contributor Author

mjguzik commented Apr 24, 2023

@freqlabs @amotin

@behlendorf behlendorf added the Status: Accepted Ready to integrate (reviewed, tested) label Apr 24, 2023
@behlendorf behlendorf merged commit 81a2b2e into openzfs:master Apr 24, 2023
andrewc12 pushed a commit to andrewc12/openzfs that referenced this pull request May 1, 2023
@mjguzik @andrewc12
FreeBSD: add missing vop_fplookup assignments
8c7736b 
It became illegal to not have them as of
5f6df177758b9dff88e4b6069aeb2359e8b0c493 ("vfs: validate that vop
vectors provide all or none fplookup vops") upstream.

Reviewed-by: Alexander Motin <mav@FreeBSD.org>
Reviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>
Signed-off-by: Mateusz Guzik <mjguzik@gmail.com>
Closes openzfs#14788
behlendorf pushed a commit to behlendorf/zfs that referenced this pull request May 28, 2023
@mjguzik @behlendorf
FreeBSD: add missing vop_fplookup assignments
f514050 
It became illegal to not have them as of
5f6df177758b9dff88e4b6069aeb2359e8b0c493 ("vfs: validate that vop
vectors provide all or none fplookup vops") upstream.

Reviewed-by: Alexander Motin <mav@FreeBSD.org>
Reviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>
Signed-off-by: Mateusz Guzik <mjguzik@gmail.com>
Closes openzfs#14788
behlendorf pushed a commit that referenced this pull request May 30, 2023
@mjguzik @behlendorf
FreeBSD: add missing vop_fplookup assignments
07a2ba5 
It became illegal to not have them as of
5f6df177758b9dff88e4b6069aeb2359e8b0c493 ("vfs: validate that vop
vectors provide all or none fplookup vops") upstream.

Reviewed-by: Alexander Motin <mav@FreeBSD.org>
Reviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>
Signed-off-by: Mateusz Guzik <mjguzik@gmail.com>
Closes #14788
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@amotin amotin amotin approved these changes

@behlendorf behlendorf behlendorf 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.
3 participants
@mjguzik @behlendorf @amotin