grwilson approved this pull request.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/openzfs/openzfs/pull/422#pullrequestreview-58790150
--
openzfs-developer
Dear reviewers: my most recent commit is just a comment change, so no retest is
necessary.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/openzfs/openzfs/pull/422#issuecomment-325055352
@asomers pushed 1 commit.
3c5ae23 Fix an out-of-date comment as suggested by grwilson
--
You are receiving this because you are subscribed to this thread.
View it on GitHub:
asomers commented on this pull request.
> c = vd->vdev_children;
mm = kmem_zalloc(offsetof(mirror_map_t, mm_child[c]), KM_SLEEP);
mm->mm_children = c;
- mm->mm_replacing = (vd->vdev_ops == _replacing_ops ||
+ /*
+
I discovered the problem using our version of the ZFS test suite. I patched it
in our product back in January 2013, but I have never pushed the patch upstream
(to OpenZFS or FreeBSD) until now.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly
ahrens approved this pull request.
This looks good to me but I'd also like @grwilson to take a look.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/openzfs/openzfs/pull/422#pullrequestreview-58514036
What else do we need to do for this PR?
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/openzfs/openzfs/pull/422#issuecomment-324754857
--
openzfs-developer
LGTM.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/openzfs/openzfs/pull/422#issuecomment-318613316
--
openzfs-developer
Archives:
Can I please get some review on this? Without this change, ZFS will fail to
detect corruption on an active spare, even when you scrub it.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub: