svn commit: r260713 - head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs

[Slawa Olhovchenkov]

On Mon, Sep 08, 2014 at 12:09:37AM +0300, Andriy Gapon wrote:

> on 07/09/2014 18:56 Slawa Olhovchenkov said the following:
> > On Thu, Jan 16, 2014 at 01:24:10PM +0000, Andriy Gapon wrote:
> > 
> >> Author: avg
> >> Date: Thu Jan 16 13:24:10 2014
> >> New Revision: 260713
> >> URL: http://svnweb.freebsd.org/changeset/base/260713
> >>
> >> Log:
> >>   fix a bug in ZFS mirror code for handling multiple DVAa
> >>   
> >>   The bug was introduced in r256956 "Improve ZFS N-way mirror read
> >>   performance".
> >>   The code in vdev_mirror_dva_select erroneously considers already
> >>   tried DVAs for the next attempt.  Thus, it is possible that a failing DVA
> >>   would be retried forever.
> >>   As a secondary effect, if the attempts fail with checksum error, then
> >>   checksum error reports are accumulated until the original request
> >>   ultimately fails or succeeds.  But because retrying is going on indefinitely
> >>   the cheksum reports accumulation will effectively be a memory leak.
> >>   
> >>   Reviewed by:	gibbs
> >>   MFC after:	13 days
> >>   Sponsored by:	HybridCluster
> > 
> > No NFC yet
> > 
> 
> Is r256956 MFC-ed yet?

Yes, I already see r256956 not MFC too.