9878 zio_free_sync() should recognize gang block size

Review Request #1225 - Created Oct. 7, 2018 and updated

Information
Toomas Soome
illumos-gate
9878
1224
7d520e1...
Reviewers
general
the zio_free_sync() in illumos tree does not recognize gang block size
while freebsd does via passing extra size argument (http://src.illumos.org/source/xref/freebsd-head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/zio.c#997).

Using extra argument there seems like unneeded overhead, we can use
BP_IS_GANG() to select between SPA_GANGBLOCKSIZE and BP_GET_PSIZE(bp) in
zio_free_sync() and avoid introducing size argument.

  
Toomas Soome
Review request changed

Change Summary:

attach issue #

Summary:

-zio_free_sync() should recognize gang block size
+9878 zio_free_sync() should recognize gang block size

Bugs:

+9878
Matthew Ahrens

Does this fix a bug, or is it essentially code cleanup? Seems like this would depend on if the free pipeline looks at io_size/orig_size/lsize?

Loading...