Re: [OpenZFS Developer] outstanding ZFS review requests

2014-10-26 Thread Steven Hartland
All looks good Matt. I committed something very similar in FreeBSD a while back as it was breaking our TRIM support, so yes count me as a reviewer :) Regards Steve On 26/10/2014 15:54, Matthew Ahrens wrote: I don't see it. Did you have feedback or should I just count you as a review

Re: [OpenZFS Developer] Review Request 118: 5243 zdb -b could be much faster

2014-10-26 Thread Richard Elling
--- This is an automatically generated e-mail. To reply, visit: https://reviews.csiden.org/r/118/#review290 --- Ship it! for HDDs, you'll find diminishing returns for zfs_vdev_async_r

Re: [OpenZFS Developer] Review Request 119: 5244 zio pipeline callers should explicitly invoke next stage

2014-10-26 Thread Richard Elling
--- This is an automatically generated e-mail. To reply, visit: https://reviews.csiden.org/r/119/#review289 --- Ship it! Ship It! - Richard Elling On Oct. 20, 2014, 10:07 p.m., Mat

Re: [OpenZFS Developer] outstanding ZFS review requests

2014-10-26 Thread Matthew Ahrens
I don't see it. Did you have feedback or should I just count you as a reviewer? --matt On Sat, Oct 25, 2014 at 9:25 PM, Steven Hartland wrote: > I replied to 119 did it get through? > > > On 26/10/2014 01:43, Matthew Ahrens wrote: > >> Anyone interested in reviewing these changes? >> >> https: