Hello Matt, This looks good to me.
-Alek On Mon, Sep 8, 2014 at 8:08 AM, Richard Elling <[email protected]> wrote: > 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 > _______________________________________________ developer mailing list [email protected] http://lists.open-zfs.org/mailman/listinfo/developer
