HI Matt, I don't see anything of concern in the approach or the code. But it is unclear to me if it has been tested to 16MB in ztest or the zfs test suite?
-- richard On Jul 20, 2014, at 9:34 PM, Matthew Ahrens <[email protected]> wrote: > https://reviews.csiden.org/r/51/diff/# > > Note: this change is not part of the Delphix product, and therefore has had > less rigorous testing than most of the changes I submit. I would appreciate > a thorough code review and also any testing others can offer. Diffs can be > downloaded via the link above, or from > https://github.com/ahrens/illumos/tree/largeblock > > --matt > _______________________________________________ > developer mailing list > [email protected] > http://lists.open-zfs.org/mailman/listinfo/developer -- [email protected] +1-760-896-4422
_______________________________________________ developer mailing list [email protected] http://lists.open-zfs.org/mailman/listinfo/developer
