Re: [PATCH 04/22] block: Abstract out bvec iterator

Kent Overstreet
2013-08-13T18:51:57Z

On Tue, Aug 13, 2013 at 10:03:04AM -0400, Ed Cashin wrote:
> On Aug 9, 2013, Ed Cashin wrote:
> > On Aug 8, 2013, at 9:05 PM, Kent Overstreet wrote:
> > ...
> > > It's in the for-jens branch now.
> >
> >
> > Just examining the patches, I like the way it cleans up the aoe code. I
> > had a question about a new BUG added by the for-jens branch the
> > read-response handling path of the aoe driver.
>
> The aoe driver in linux-bcache/for-jens commit 4c36c973a8f45 is
> passing my tests.
>
> Here is a patch against that branch illustrating my suggestion for
> handling bad target responses gracefully.

Thanks - shall I just fold that into the aoe immutable bvec patch?