----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.csiden.org/r/133/#review332 -----------------------------------------------------------
Ship it! Ship It! - Steven Hartland On Nov. 16, 2014, 3:54 a.m., Matthew Ahrens wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.csiden.org/r/133/ > ----------------------------------------------------------- > > (Updated Nov. 16, 2014, 3:54 a.m.) > > > Review request for OpenZFS Developer Mailing List and Rich Lowe. > > > Bugs: 5348 > https://www.illumos.org/projects/illumos-gate//issues/5348 > > > Repository: illumos-gate > > > Description > ------- > > 5348 zio_checksum_error() only fills in info if ECKSUM > Reviewed by: George Wilson <[email protected]> > Reviewed by: Alex Reece <[email protected]> > Reviewed by: Christopher Siden <[email protected]> > > If zio_checksum_error() returns other than ECKSUM (e.g. EINVAL), it does not > fill in the "zio_bad_cksum_t *info" parameter. Caller should not attempt to > use it in this case. > > Original author: Matthew Ahrens > > > Diffs > ----- > > usr/src/uts/common/fs/zfs/zio.c 7cb6953845eb6cbd6346c22cd3a0cf9a311e463c > > Diff: https://reviews.csiden.org/r/133/diff/ > > > Testing > ------- > > ztest > zfs test suite > > (internal link: http://jenkins/job/zfs-precommit/1137/) > > > Thanks, > > Matthew Ahrens > >
_______________________________________________ developer mailing list [email protected] http://lists.open-zfs.org/mailman/listinfo/developer
