Discussion:
[OpenZFS Developer] Review Request 105: 5178 zdb -vvvvv on old-format pool fails in dump_deadlist()
Matthew Ahrens
2014-09-17 03:15:51 UTC
Permalink
-----------------------------------------------------------
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: 5178
https://www.illumos.org/projects/illumos-gate//issues/5178


Repository: illumos-gate


Description
-------

5178 zdb -vvvvv on old-format pool fails in dump_deadlist()
Reviewed by: Christopher Siden <christopher.siden-IrO+***@public.gmane.org>
Reviewed by: George Wilson <george.wilson-IrO+***@public.gmane.org>

Original author: Matthew Ahrens


Diffs
-----

usr/src/cmd/zdb/zdb.c 2def6241b7118dd84d7943721164f8f8b9dfe978

Diff: https://reviews.csiden.org/r/105/diff/


Testing
-------

ztest
zfs test suite
manual testing of zdb on old pool


Thanks,

Matthew Ahrens
Richard Elling
2014-09-28 20:28:51 UTC
Permalink
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.csiden.org/r/105/#review262
-----------------------------------------------------------

Ship it!


Ship It!

- Richard Elling
Post by Matthew Ahrens
-----------------------------------------------------------
https://reviews.csiden.org/r/105/
-----------------------------------------------------------
(Updated Sept. 17, 2014, 3:15 a.m.)
Review request for OpenZFS Developer Mailing List and Christopher Siden.
Bugs: 5178
https://www.illumos.org/projects/illumos-gate//issues/5178
Repository: illumos-gate
Description
-------
5178 zdb -vvvvv on old-format pool fails in dump_deadlist()
Original author: Matthew Ahrens
Diffs
-----
usr/src/cmd/zdb/zdb.c 2def6241b7118dd84d7943721164f8f8b9dfe978
Diff: https://reviews.csiden.org/r/105/diff/
Testing
-------
ztest
zfs test suite
manual testing of zdb on old pool
Thanks,
Matthew Ahrens
Alek Pinchuk
2014-09-29 17:05:41 UTC
Permalink
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.csiden.org/r/105/#review271
-----------------------------------------------------------

Ship it!


Ship It!

- Alek Pinchuk
Post by Matthew Ahrens
-----------------------------------------------------------
https://reviews.csiden.org/r/105/
-----------------------------------------------------------
(Updated Sept. 17, 2014, 3:15 a.m.)
Review request for OpenZFS Developer Mailing List and Christopher Siden.
Bugs: 5178
https://www.illumos.org/projects/illumos-gate//issues/5178
Repository: illumos-gate
Description
-------
5178 zdb -vvvvv on old-format pool fails in dump_deadlist()
Original author: Matthew Ahrens
Diffs
-----
usr/src/cmd/zdb/zdb.c 2def6241b7118dd84d7943721164f8f8b9dfe978
Diff: https://reviews.csiden.org/r/105/diff/
Testing
-------
ztest
zfs test suite
manual testing of zdb on old pool
Thanks,
Matthew Ahrens
Rich Lowe
2014-09-27 02:18:57 UTC
Permalink
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.csiden.org/r/105/#review260
-----------------------------------------------------------

Ship it!


Ship It!

- Rich Lowe
Post by Matthew Ahrens
-----------------------------------------------------------
https://reviews.csiden.org/r/105/
-----------------------------------------------------------
(Updated Sept. 16, 2014, 11:15 p.m.)
Review request for OpenZFS Developer Mailing List and Christopher Siden.
Bugs: 5178
https://www.illumos.org/projects/illumos-gate//issues/5178
Repository: illumos-gate
Description
-------
5178 zdb -vvvvv on old-format pool fails in dump_deadlist()
Original author: Matthew Ahrens
Diffs
-----
usr/src/cmd/zdb/zdb.c 2def6241b7118dd84d7943721164f8f8b9dfe978
Diff: https://reviews.csiden.org/r/105/diff/
Testing
-------
ztest
zfs test suite
manual testing of zdb on old pool
Thanks,
Matthew Ahrens
Loading...