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: Use NDFREE_PNBUF if available #13277

Merged
merged 1 commit into from
Apr 2, 2022
Merged

FreeBSD: Use NDFREE_PNBUF if available #13277

merged 1 commit into from
Apr 2, 2022

Conversation

ghost
Copy link

@ghost ghost commented Apr 1, 2022 •
edited by ghost
Loading

Motivation and Context

NDF_ONLY_PNBUF has been removed from FreeBSD in favor of NDFREE_PNBUF, breaking the build on FreeBSD main.

Description

Use NDFREE_PNBUF if available.

How Has This Been Tested?

Built on FreeBSD main.

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:

FreeBSD: Use NDFREE_PNBUF if available
2e26a0a 
NDF_ONLY_PNBUF has been removed from FreeBSD in favor of NDFREE_PNBUF.

Signed-off-by: Ryan Moeller <[email protected]>
@ghost ghost force-pushed the ndfree-fix branch from cca5655 to 2e26a0a Compare April 1, 2022 22:11
@behlendorf behlendorf added Status: Code Review Needed Ready for review and testing Status: Accepted Ready to integrate (reviewed, tested) and removed Status: Code Review Needed Ready for review and testing labels Apr 2, 2022
@behlendorf behlendorf merged commit b61507e into openzfs:master Apr 2, 2022
@ghost ghost deleted the ndfree-fix branch April 2, 2022 19:28
usaleem-ix pushed a commit to truenas/zfs that referenced this pull request Apr 5, 2022
@usaleem-ix
FreeBSD: Use NDFREE_PNBUF if available
adc5ca9 
NDF_ONLY_PNBUF has been removed from FreeBSD in favor of NDFREE_PNBUF.

Reviewed-by: Brian Behlendorf <[email protected]>
Signed-off-by: Ryan Moeller <[email protected]>
Closes openzfs#13277
@usaleem-ix usaleem-ix mentioned this pull request Apr 5, 2022
Merged
13 tasks
behlendorf pushed a commit that referenced this pull request Apr 6, 2022
@behlendorf
FreeBSD: Use NDFREE_PNBUF if available
a5a2872 
NDF_ONLY_PNBUF has been removed from FreeBSD in favor of NDFREE_PNBUF.

Reviewed-by: Brian Behlendorf <[email protected]>
Signed-off-by: Ryan Moeller <[email protected]>
Closes #13277
BrainSlayer pushed a commit to BrainSlayer/zfs that referenced this pull request Apr 11, 2022
@BrainSlayer
FreeBSD: Use NDFREE_PNBUF if available
7b97211 
NDF_ONLY_PNBUF has been removed from FreeBSD in favor of NDFREE_PNBUF.

Reviewed-by: Brian Behlendorf <[email protected]>
Signed-off-by: Ryan Moeller <[email protected]>
Closes openzfs#13277
nicman23 pushed a commit to nicman23/zfs that referenced this pull request Aug 22, 2022
@nicman23
FreeBSD: Use NDFREE_PNBUF if available
106b2df 
NDF_ONLY_PNBUF has been removed from FreeBSD in favor of NDFREE_PNBUF.

Reviewed-by: Brian Behlendorf <[email protected]>
Signed-off-by: Ryan Moeller <[email protected]>
Closes openzfs#13277
nicman23 pushed a commit to nicman23/zfs that referenced this pull request Aug 22, 2022
@nicman23
FreeBSD: Use NDFREE_PNBUF if available
1c997c0 
NDF_ONLY_PNBUF has been removed from FreeBSD in favor of NDFREE_PNBUF.

Reviewed-by: Brian Behlendorf <[email protected]>
Signed-off-by: Ryan Moeller <[email protected]>
Closes openzfs#13277
lundman pushed a commit to openzfsonwindows/openzfs that referenced this pull request Sep 2, 2022
@lundman
FreeBSD: Use NDFREE_PNBUF if available
f2eb421 
NDF_ONLY_PNBUF has been removed from FreeBSD in favor of NDFREE_PNBUF.

Reviewed-by: Brian Behlendorf <[email protected]>
Signed-off-by: Ryan Moeller <[email protected]>
Closes openzfs#13277
beren12 pushed a commit to beren12/zfs that referenced this pull request Sep 19, 2022
@beren12
FreeBSD: Use NDFREE_PNBUF if available
0a7b6f5 
NDF_ONLY_PNBUF has been removed from FreeBSD in favor of NDFREE_PNBUF.

Reviewed-by: Brian Behlendorf <[email protected]>
Signed-off-by: Ryan Moeller <[email protected]>
Closes openzfs#13277
andrewc12 pushed a commit to andrewc12/openzfs that referenced this pull request Sep 23, 2022
@andrewc12
FreeBSD: Use NDFREE_PNBUF if available
52d6dac 
NDF_ONLY_PNBUF has been removed from FreeBSD in favor of NDFREE_PNBUF.

Reviewed-by: Brian Behlendorf <[email protected]>
Signed-off-by: Ryan Moeller <[email protected]>
Closes openzfs#13277
andrewc12 pushed a commit to andrewc12/openzfs that referenced this pull request Sep 23, 2022
@andrewc12
FreeBSD: Use NDFREE_PNBUF if available
8940bec 
NDF_ONLY_PNBUF has been removed from FreeBSD in favor of NDFREE_PNBUF.

Reviewed-by: Brian Behlendorf <[email protected]>
Signed-off-by: Ryan Moeller <[email protected]>
Closes openzfs#13277
andrewc12 pushed a commit to andrewc12/openzfs that referenced this pull request Sep 23, 2022
@andrewc12
FreeBSD: Use NDFREE_PNBUF if available
aef4b98 
NDF_ONLY_PNBUF has been removed from FreeBSD in favor of NDFREE_PNBUF.

Reviewed-by: Brian Behlendorf <[email protected]>
Signed-off-by: Ryan Moeller <[email protected]>
Closes openzfs#13277
andrewc12 pushed a commit to andrewc12/openzfs that referenced this pull request Sep 23, 2022
@andrewc12
FreeBSD: Use NDFREE_PNBUF if available
44d310f 
NDF_ONLY_PNBUF has been removed from FreeBSD in favor of NDFREE_PNBUF.

Reviewed-by: Brian Behlendorf <[email protected]>
Signed-off-by: Ryan Moeller <[email protected]>
Closes openzfs#13277
andrewc12 pushed a commit to andrewc12/openzfs that referenced this pull request Sep 23, 2022
@andrewc12
FreeBSD: Use NDFREE_PNBUF if available
457f7ec 
NDF_ONLY_PNBUF has been removed from FreeBSD in favor of NDFREE_PNBUF.

Reviewed-by: Brian Behlendorf <[email protected]>
Signed-off-by: Ryan Moeller <[email protected]>
Closes openzfs#13277
andrewc12 pushed a commit to andrewc12/openzfs that referenced this pull request Sep 23, 2022
@andrewc12
FreeBSD: Use NDFREE_PNBUF if available
56eae85 
NDF_ONLY_PNBUF has been removed from FreeBSD in favor of NDFREE_PNBUF.

Reviewed-by: Brian Behlendorf <[email protected]>
Signed-off-by: Ryan Moeller <[email protected]>
Closes openzfs#13277
andrewc12 pushed a commit to andrewc12/openzfs that referenced this pull request Sep 23, 2022
@andrewc12
FreeBSD: Use NDFREE_PNBUF if available
9d2672c 
NDF_ONLY_PNBUF has been removed from FreeBSD in favor of NDFREE_PNBUF.

Reviewed-by: Brian Behlendorf <[email protected]>
Signed-off-by: Ryan Moeller <[email protected]>
Closes openzfs#13277
andrewc12 pushed a commit to andrewc12/openzfs that referenced this pull request Sep 23, 2022
@andrewc12
FreeBSD: Use NDFREE_PNBUF if available
01b8647 
NDF_ONLY_PNBUF has been removed from FreeBSD in favor of NDFREE_PNBUF.

Reviewed-by: Brian Behlendorf <[email protected]>
Signed-off-by: Ryan Moeller <[email protected]>
Closes openzfs#13277
andrewc12 pushed a commit to andrewc12/openzfs that referenced this pull request Sep 23, 2022
@andrewc12
FreeBSD: Use NDFREE_PNBUF if available
e413856 
NDF_ONLY_PNBUF has been removed from FreeBSD in favor of NDFREE_PNBUF.

Reviewed-by: Brian Behlendorf <[email protected]>
Signed-off-by: Ryan Moeller <[email protected]>
Closes openzfs#13277
andrewc12 pushed a commit to andrewc12/openzfs that referenced this pull request Sep 23, 2022
@andrewc12
FreeBSD: Use NDFREE_PNBUF if available
2e1b49d 
NDF_ONLY_PNBUF has been removed from FreeBSD in favor of NDFREE_PNBUF.

Reviewed-by: Brian Behlendorf <[email protected]>
Signed-off-by: Ryan Moeller <[email protected]>
Closes openzfs#13277
andrewc12 pushed a commit to andrewc12/openzfs that referenced this pull request Sep 23, 2022
@andrewc12
FreeBSD: Use NDFREE_PNBUF if available
248ab19 
NDF_ONLY_PNBUF has been removed from FreeBSD in favor of NDFREE_PNBUF.

Reviewed-by: Brian Behlendorf <[email protected]>
Signed-off-by: Ryan Moeller <[email protected]>
Closes openzfs#13277
andrewc12 pushed a commit to andrewc12/openzfs that referenced this pull request Sep 23, 2022
@andrewc12
FreeBSD: Use NDFREE_PNBUF if available
03cb42f 
NDF_ONLY_PNBUF has been removed from FreeBSD in favor of NDFREE_PNBUF.

Reviewed-by: Brian Behlendorf <[email protected]>
Signed-off-by: Ryan Moeller <[email protected]>
Closes openzfs#13277
andrewc12 pushed a commit to andrewc12/openzfs that referenced this pull request Sep 23, 2022
@andrewc12
FreeBSD: Use NDFREE_PNBUF if available
c39ae87 
NDF_ONLY_PNBUF has been removed from FreeBSD in favor of NDFREE_PNBUF.

Reviewed-by: Brian Behlendorf <[email protected]>
Signed-off-by: Ryan Moeller <[email protected]>
Closes openzfs#13277
andrewc12 pushed a commit to andrewc12/openzfs that referenced this pull request Sep 23, 2022
@andrewc12
FreeBSD: Use NDFREE_PNBUF if available
55583ad 
NDF_ONLY_PNBUF has been removed from FreeBSD in favor of NDFREE_PNBUF.

Reviewed-by: Brian Behlendorf <[email protected]>
Signed-off-by: Ryan Moeller <[email protected]>
Closes openzfs#13277
andrewc12 pushed a commit to andrewc12/openzfs that referenced this pull request Sep 23, 2022
@andrewc12
FreeBSD: Use NDFREE_PNBUF if available
9187d13 
NDF_ONLY_PNBUF has been removed from FreeBSD in favor of NDFREE_PNBUF.

Reviewed-by: Brian Behlendorf <[email protected]>
Signed-off-by: Ryan Moeller <[email protected]>
Closes openzfs#13277
andrewc12 pushed a commit to andrewc12/openzfs that referenced this pull request Sep 23, 2022
@andrewc12
FreeBSD: Use NDFREE_PNBUF if available
512798f 
NDF_ONLY_PNBUF has been removed from FreeBSD in favor of NDFREE_PNBUF.

Reviewed-by: Brian Behlendorf <[email protected]>
Signed-off-by: Ryan Moeller <[email protected]>
Closes openzfs#13277
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
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.
1 participant
@behlendorf