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? > I have updated ztest to test up to 1MB. I've manually tested up to 16MB, by changing the tunable. I haven't added anything to the test suite. Not ideal, but better than nothing? --matt > > -- 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
