HomeFreeBSD

Merge biodone_finish() back into biodone(). The primary purpose is

Description

Merge biodone_finish() back into biodone(). The primary purpose is
to make the order of operations clearer to avoid the race condition
that was fixed in r328914. In particular, this commit corrects a
similar race that existed in the soft updates callback.

Doing some sleuthing through the SVN repository, it appears that
bufdone_finish() was added to support XFS:


r153192 | rodrigc | 2005-12-06 19:39:08 -0800 (Tue, 06 Dec 2005) | 13 lines

Changes imported from XFS for FreeBSD project:

  • add fields to struct buf (needed by XFS)
    • 3 private fields: b_fsprivate1, b_fsprivate2, b_fsprivate3
    • b_pin_count, count of pinned buffer
  • add new B_MANAGED flag
  • add breada() function to initiate asynchronous I/O on read-ahead blocks.
  • add bufdone_finish(), bpin(), bunpin_wait() functions

Patches provided by: kan
Reviewed by: phk
Silence on: arch@


It does not appear to ever have been used for anything else. XFS was
disconnected in r241607:


r241607 | attilio | 2012-10-16 03:04:00 -0700 (Tue, 16 Oct 2012) | 5 lines

Disconnect non-MPSAFE XFS from the build in preparation for dropping
GIANT from VFS.

This is not targeted for MFC.


and removed entirely in r247631:


r247631 | attilio | 2013-03-02 07:33:54 -0800 (Sat, 02 Mar 2013) | 5 lines

Garbage collect XFS bits which are now already completely disconnected
from the tree since few months.

This is not targeted for MFC.


Since XFS support is gone, there is no reason to retain biodone_finish().

Suggested by: Warner Losh (imp)
Discussed with: cem, kib
Tested by: Peter Holm (pho)

Details

Provenance
mckusickAuthored on
Reviewer
cem
Parents
rS329077: tftp(1): Fix libedit state corruption involving signals
Branches
Unknown
Tags
Unknown