Hi Matt,
> On Sep 15, 2014, at 10:24 PM, Matthew Ahrens <[email protected]> wrote: > >> 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? ok, the sweet spot seems to be 1MB -- richard > > --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
