Re: [OpenZFS Developer] Review Request 136: 5351, 5352 scrub time fixes

2014-11-16 Thread Matthew Ahrens
> On Nov. 16, 2014, 4:02 p.m., Richard Elling wrote: > > usr/src/uts/common/fs/zfs/dsl_scan.c, line 432 > > > > > > since we don't have to be very precise here, rather than divide, is it > > ok to shift >> 30? > > Matthe

Re: [OpenZFS Developer] Review Request 136: 5351, 5352 scrub time fixes

2014-11-16 Thread Matthew Ahrens
> On Nov. 16, 2014, 4:02 p.m., Richard Elling wrote: > > usr/src/uts/common/fs/zfs/dsl_scan.c, line 432 > > > > > > since we don't have to be very precise here, rather than divide, is it > > ok to shift >> 30? I think a

Re: [OpenZFS Developer] Review Request 136: 5351, 5352 scrub time fixes

2014-11-16 Thread Richard Elling
--- This is an automatically generated e-mail. To reply, visit: https://reviews.csiden.org/r/136/#review328 --- LGTM FWIW, we see the same extra second on Solaris 11 in the field usr

[OpenZFS Developer] Review Request 136: 5351, 5352 scrub time fixes

2014-11-16 Thread Matthew Ahrens
--- This is an automatically generated e-mail. To reply, visit: https://reviews.csiden.org/r/136/ --- Review request for OpenZFS Developer Mailing List. Bugs: 5351 and 5352 https: