hi! Any comments please?
rgds, toomas > Begin forwarded message: > > From: Toomas Soome via Review Board <[email protected]> > Subject: Review Request 1225: zio_free_sync() should recognize gang block size > Date: 7 October 2018 at 10:26:03 EEST > To: Joshua Clulow <[email protected]>, Vitaliy Gusev > <[email protected]>, Marcel Telka <[email protected]>, Toomas Soome > <[email protected]>, Andy Fiddaman <[email protected]>, carlos neira > <[email protected]>, Juraj Lutter <[email protected]>, Dan Vatca > <[email protected]>, John Howard <[email protected]>, Norm Jacobs > <[email protected]>, Albert Lee <[email protected]> > Reply-To: Toomas Soome <[email protected]> > > This is an automatically generated e-mail. To reply, visit: > https://www.illumos.org/rb/r/1225/ <https://www.illumos.org/rb/r/1225/> > Review request for general. > By Toomas Soome. > Repository: illumos-gate > Description > > 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. > Diffs > > usr/src/uts/common/fs/zfs/zio.c (b7e11f922a9569fdbea185810afa2360375c299c) > View Diff <https://www.illumos.org/rb/r/1225/diff/1/> ------------------------------------------ openzfs: openzfs-developer Permalink: https://openzfs.topicbox.com/groups/developer/T398b23af1f0a2efe-Ma878fa4e146325ec5608e9e5 Delivery options: https://openzfs.topicbox.com/groups/developer/subscription
