Re: integrity regression fixes for 4.13-rc

2017-08-09 Thread Martin K. Petersen

Christoph,

> this series fixes regressions in the integrity handling update in
> 4.13-rc.
>
> The first one was sent earlier by Milan and while both Martin and I
> aren't exactly happy about the way dm uses the integrity code to cause
> this regression this minimal fix gets us back to the status quo.
>
> The second one makes sure that we only verify the DIF checksums on the
> lowest layer where we attach the integrity information.

These look OK to me.

Reviewed-by: Martin K. Petersen 

-- 
Martin K. Petersen  Oracle Linux Engineering


Re: integrity regression fixes for 4.13-rc

2017-08-09 Thread Jens Axboe
On 08/09/2017 09:47 AM, Christoph Hellwig wrote:
> Hi Jens,
> 
> this series fixes regressions in the integrity handling update in 4.13-rc.
> 
> The first one was sent earlier by Milan and while both Martin and I aren't
> exactly happy about the way dm uses the integrity code to cause this
> regression this minimal fix gets us back to the status quo.
> 
> The second one makes sure that we only verify the DIF checksums on the
> lowest layer where we attach the integrity information.

Thanks, I was waiting on these. Applied for 4.13. I did just ship off
a pull request, but I'll get this in for the next -rc.

-- 
Jens Axboe