Re: [OpenZFS Developer] [review request] 3693 restore_object uses at least two transactions to restore an object

2014-09-16 Thread Matthew Ahrens
Any reviewers? --matt On Thu, Sep 11, 2014 at 6:02 PM, Matthew Ahrens wrote: > 3693 restore_object uses at least two transactions to restore an object > > https://reviews.csiden.org/r/90/ > > Thanks to Andriy Gapon for co-developing this fix with me > > --matt >

Re: [OpenZFS Developer] [review request] 5150 zfs clone of a defer_destroy snapshot causes strangeness

2014-09-16 Thread Matthew Ahrens
Any takers? --matt On Wed, Sep 10, 2014 at 9:51 PM, Matthew Ahrens wrote: > 5150 zfs clone of a defer_destroy snapshot causes strangeness > > https://reviews.csiden.org/r/93/ > > --matt > ___ developer mailing list developer@open-zfs.org http://lists.

[OpenZFS Developer] Review Request 105: 5178 zdb -vvvvv on old-format pool fails in dump_deadlist()

2014-09-16 Thread Matthew Ahrens
--- This is an automatically generated e-mail. To reply, visit: https://reviews.csiden.org/r/105/ --- Review request for OpenZFS Developer Mailing List and Christopher Siden. Bugs: 51

[OpenZFS Developer] Review Request 104: 5177 remove dead code from dsl_scan.c

2014-09-16 Thread Matthew Ahrens
--- This is an automatically generated e-mail. To reply, visit: https://reviews.csiden.org/r/104/ --- Review request for OpenZFS Developer Mailing List and Christopher Siden. Bugs: 51

[OpenZFS Developer] Review Request 103: 5176 lock contention on godfather zio

2014-09-16 Thread Matthew Ahrens
--- This is an automatically generated e-mail. To reply, visit: https://reviews.csiden.org/r/103/ --- Review request for OpenZFS Developer Mailing List and Christopher Siden. Bugs: 51

[OpenZFS Developer] Review Request 102: 5175 implement dmu_read_uio_dbuf() to improve cached read performance

2014-09-16 Thread Matthew Ahrens
--- This is an automatically generated e-mail. To reply, visit: https://reviews.csiden.org/r/102/ --- Review request for OpenZFS Developer Mailing List and Christopher Siden. Bugs: 51

[OpenZFS Developer] Review Request 101: 5174 add sdt probe for blocked read in dbuf_read()

2014-09-16 Thread Matthew Ahrens
--- This is an automatically generated e-mail. To reply, visit: https://reviews.csiden.org/r/101/ --- Review request for OpenZFS Developer Mailing List and Christopher Siden. Bugs: 51

Re: [OpenZFS Developer] Review Request 100: zdb fixes

2014-09-16 Thread Matthew Ahrens
Note: developer@open-zfs.org will receive notifications of updates to this review request via ReviewBoard. This should make it easier for folks following along to see review feedback, and how it is addressed. (Previously, you had to periodically check the ReviewBoard site.) Note that these messa

[OpenZFS Developer] Review Request 100: zdb fixes

2014-09-16 Thread Matthew Ahrens
--- This is an automatically generated e-mail. To reply, visit: https://reviews.csiden.org/r/100/ --- Review request for OpenZFS Developer Mailing List and Christopher Siden. Bugs: 51

Re: [OpenZFS Developer] review request: 5027 zfs large block support

2014-09-16 Thread Richard Elling
Hi Matt, > On Sep 15, 2014, at 10:24 PM, Matthew Ahrens wrote: > >> On Mon, Sep 8, 2014 at 8:08 AM, Richard Elling >> 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?