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

Jump to bottom

FreeBSD: Catch up with recent VFS changes #12668

Merged
merged 1 commit into from Oct 25, 2021
Merged

FreeBSD: Catch up with recent VFS changes #12668

merged 1 commit into from Oct 25, 2021

Conversation

ghost
Copy link

@ghost ghost commented Oct 22, 2021

cn_thread is always curthread.

Signed-off-by: Ryan Moeller freqlabs@FreeBSD.org

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:

@ghost ghost added the Status: Code Review Needed Ready for review and testing label Oct 22, 2021
@ghost ghost added this to In progress in FreeBSD via automation Oct 22, 2021
@ghost ghost requested a review from amotin October 22, 2021 16:58
@behlendorf
Copy link
Contributor

@asomers would you mind taking a look at this.

@ghost
Copy link
Author

ghost commented Oct 22, 2021

Don't mind the stable/12 failure, the buildbot config needs to be updated to be able to fetch the sources now that the 12.3 has begun.

asomers
asomers reviewed Oct 22, 2021
View reviewed changes
Copy link
Contributor

@asomers asomers left a comment •
edited

Choose a reason for hiding this comment

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

The changes are correct, though they would be even better if @freqlabs included links to the two relevant FreeBSD commits in his commit message.

@ghost
Copy link
Author

ghost commented Oct 22, 2021

  • Added links to the FreeBSD commits in the commit message

@behlendorf behlendorf added Status: Accepted Ready to integrate (reviewed, tested) and removed Status: Code Review Needed Ready for review and testing labels Oct 23, 2021
@behlendorf behlendorf merged commit 14b69c0 into openzfs:master Oct 25, 2021
FreeBSD automation moved this from In progress to Done Oct 25, 2021
@ghost ghost deleted the cn_thread branch October 25, 2021 16:47
tonyhutter pushed a commit that referenced this pull request Nov 2, 2021
@tonyhutter
FreeBSD: Catch up with recent VFS changes
27d9c6a 
cn_thread is always curthread.

https://cgit.freebsd.org/src/commit?id=b4a58fbf640409a1e507d9f7b411c83a3f83a2f3
https://cgit.freebsd.org/src/commit?id=2b68eb8e1dbbdaf6a0df1c83b26f5403ca52d4c3

Reviewed-by: Alexander Motin <mav@FreeBSD.org>
Reviewed-by: Alan Somers <asomers@gmail.com>
Signed-off-by: Ryan Moeller <freqlabs@FreeBSD.org>
Closes #12668
ghost pushed a commit to truenas/zfs that referenced this pull request Nov 3, 2021
FreeBSD: Catch up with recent VFS changes
56b1810 
cn_thread is always curthread.

https://cgit.freebsd.org/src/commit?id=b4a58fbf640409a1e507d9f7b411c83a3f83a2f3
https://cgit.freebsd.org/src/commit?id=2b68eb8e1dbbdaf6a0df1c83b26f5403ca52d4c3

Reviewed-by: Alexander Motin <mav@FreeBSD.org>
Reviewed-by: Alan Somers <asomers@gmail.com>
Signed-off-by: Ryan Moeller <freqlabs@FreeBSD.org>
Closes openzfs#12668
tonyhutter pushed a commit to tonyhutter/zfs that referenced this pull request Nov 6, 2021
@tonyhutter
FreeBSD: Catch up with recent VFS changes
19a8180 
cn_thread is always curthread.

https://cgit.freebsd.org/src/commit?id=b4a58fbf640409a1e507d9f7b411c83a3f83a2f3
https://cgit.freebsd.org/src/commit?id=2b68eb8e1dbbdaf6a0df1c83b26f5403ca52d4c3

Reviewed-by: Alexander Motin <mav@FreeBSD.org>
Reviewed-by: Alan Somers <asomers@gmail.com>
Signed-off-by: Ryan Moeller <freqlabs@FreeBSD.org>
Closes openzfs#12668
tonyhutter pushed a commit to tonyhutter/zfs that referenced this pull request Nov 13, 2021
@tonyhutter
FreeBSD: Catch up with recent VFS changes
3270584 
cn_thread is always curthread.

https://cgit.freebsd.org/src/commit?id=b4a58fbf640409a1e507d9f7b411c83a3f83a2f3
https://cgit.freebsd.org/src/commit?id=2b68eb8e1dbbdaf6a0df1c83b26f5403ca52d4c3

Reviewed-by: Alexander Motin <mav@FreeBSD.org>
Reviewed-by: Alan Somers <asomers@gmail.com>
Signed-off-by: Ryan Moeller <freqlabs@FreeBSD.org>
Closes openzfs#12668
tonyhutter pushed a commit to tonyhutter/zfs that referenced this pull request Nov 13, 2021
@tonyhutter
FreeBSD: Catch up with recent VFS changes
3fcf17e 
cn_thread is always curthread.

https://cgit.freebsd.org/src/commit?id=b4a58fbf640409a1e507d9f7b411c83a3f83a2f3
https://cgit.freebsd.org/src/commit?id=2b68eb8e1dbbdaf6a0df1c83b26f5403ca52d4c3

Reviewed-by: Alexander Motin <mav@FreeBSD.org>
Reviewed-by: Alan Somers <asomers@gmail.com>
Signed-off-by: Ryan Moeller <freqlabs@FreeBSD.org>
Closes openzfs#12668
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@tonynguien tonynguien

Labels
Status: Accepted Ready to integrate (reviewed, tested)
Projects
FreeBSD
Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@behlendorf @asomers @amotin @tonynguien