N-way mirror read speedup in zfsonlinux

Will Andrews will at firepipe.net
Fri Aug 16 22:15:02 UTC 2013


On Fri, Aug 16, 2013 at 11:36 AM, Steven Hartland
<killing at multiplay.co.uk> wrote:
>> We have a vdev_geom_attrchanged() in our tree as well, to support
>> physical path management (an early version of which is in the zfsd
>> project branch; commits to HEAD are still pending).  So, my request
>> would be that you change vdev_geom_attrchanged() to execute
>> vdev_geom_set_rotating() explicitly only if the attribute matches, to
>> ease merging.
>
>
> Sure no problem done.

Thanks, I've made a similar change locally.

>> Does the change pass a STF run on a DEBUG build?
>
>
> I'll look at getting STF installed on this test box to confirm, do you have
> an updated version with the various issues I raised when testing it
> a while back?

I thought most if not all the issues you raised were due to
differences in the ZFS implementation, not test case issues.  We do
have some updates for the test suite itself, but mostly in response to
more fixes that have not been pushed out yet.

I'm currently working to push out some of the fixes.

Thanks.
--Will.


More information about the zfs-devel mailing list