Matthew Ahrens
2014-09-16 20:24:19 UTC
Note: developer-***@public.gmane.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 messages will only go to developer-***@public.gmane.org, and not to
zfs-bGTZWsvkwUGXjz/***@public.gmane.org You can continue to give review feedback via
replies to this email or through ReviewBoard (you'll need to create an
account if you don't already have one).
Let me know if this is too much email. Otherwise I'll continue using this
code review notification mechanism for future reviews.
--matt
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 messages will only go to developer-***@public.gmane.org, and not to
zfs-bGTZWsvkwUGXjz/***@public.gmane.org You can continue to give review feedback via
replies to this email or through ReviewBoard (you'll need to create an
account if you don't already have one).
Let me know if this is too much email. Otherwise I'll continue using this
code review notification mechanism for future reviews.
--matt
https://reviews.csiden.org/r/100/
Review request for OpenZFS Developer Mailing List and Christopher Siden.
By Matthew Ahrens.
*Bugs: * 5169
<https://www.illumos.org/projects/illumos-gate//issues/5169>, 5170
<https://www.illumos.org/projects/illumos-gate//issues/5170>, 5171
<https://www.illumos.org/projects/illumos-gate//issues/5171>
*Repository: * illumos-gate
Description
5169 zdb should limit its ARC size
5170 zdb -c should create more scrub i/os by default
5171 zdb should print status while loading metaslabs for leak detection
Original author: Matthew Ahrens
Testing
ztest
zfs test suite
manual testing of zdb
Diffs
- usr/src/cmd/zdb/zdb.c (2def6241b7118dd84d7943721164f8f8b9dfe978)
View Diff <https://reviews.csiden.org/r/100/diff/>
Review request for OpenZFS Developer Mailing List and Christopher Siden.
By Matthew Ahrens.
*Bugs: * 5169
<https://www.illumos.org/projects/illumos-gate//issues/5169>, 5170
<https://www.illumos.org/projects/illumos-gate//issues/5170>, 5171
<https://www.illumos.org/projects/illumos-gate//issues/5171>
*Repository: * illumos-gate
Description
5169 zdb should limit its ARC size
5170 zdb -c should create more scrub i/os by default
5171 zdb should print status while loading metaslabs for leak detection
Original author: Matthew Ahrens
Testing
ztest
zfs test suite
manual testing of zdb
Diffs
- usr/src/cmd/zdb/zdb.c (2def6241b7118dd84d7943721164f8f8b9dfe978)
View Diff <https://reviews.csiden.org/r/100/diff/>