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 reviewer?

--matt

On Sat, Oct 25, 2014 at 9:25 PM, Steven Hartland <[email protected] <mailto:[email protected]>> 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://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 much faster

        https://reviews.csiden.org/r/110/
        5213 panic in metaslab_init due to space_map_open returning ENXIO
        (Bayard, are you happy with the responses in this review?)

        --matt




_______________________________________________
developer mailing list
[email protected]
http://lists.open-zfs.org/mailman/listinfo/developer

Reply via email to