[OpenZFS Developer] outstanding zfs review requests

2015-12-18 Thread Matthew Ahrens
I have 2 outstanding review requests that I could use feedback on. These have been open for quite a while so I'm probably going to RTI them this weekend. Provide mechanism to artificially limit disk performance https://github.com/openzfs/openzfs/pull/39 This patch hijacks the existing 'zinject -

[OpenZFS Developer] outstanding ZFS review requests

2015-03-21 Thread Matthew Ahrens
Any takers for these reviews? If not then I'll RTI with just the Delphix reviewers and see how that goes. (Thanks to Jeff Sipek for reviewing one of them!) https://reviews.csiden.org/r/172/ 5694 traverse_prefetcher does not prefetch enough https://reviews.csiden.org/r/174/ 5696 ::refcount -r do

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] 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:

Re: [OpenZFS Developer] outstanding ZFS review requests

2014-10-25 Thread Steven Hartland
I replied to 119 did it get through? On 26/10/2014 01:43, Matthew Ahrens wrote: Anyone interested in reviewing these changes? https://reviews.csiden.org/r/119/ 5244 zio pipeline callers should explicitly invoke next stage (Related to the thread "zfs zio reordering in zio_vdev_io_start causing

[OpenZFS Developer] outstanding ZFS review requests

2014-10-25 Thread Matthew Ahrens
Anyone interested in reviewing these changes? https://reviews.csiden.org/r/119/ 5244 zio pipeline callers should explicitly invoke next stage (Related to the thread "zfs zio reordering in zio_vdev_io_start causing panics" from April 2014) https://reviews.csiden.org/r/118/ 5243 zdb -b could be muc

[OpenZFS Developer] outstanding ZFS review requests

2014-09-26 Thread Matthew Ahrens
I have several review requests that have been outstanding for more than 7 days. If you would like to review these changes, please do so by Tuesday the 30th, or let me know if you need more time. 3693 restore_object uses at least two transactions to restore an object https://reviews.csiden.org/r/9