Re: [PATCH v6 02/18] s390/dasd: Propagate partial completion length across ERP recovery

> dasd_default_erp_postaction() copies the timing and device state from
> the finished ERP request back to the original request but drops
> proc_bytes. A request that was partially completed, an ESE read of a
> not-yet-allocated track returns fewer bytes than requested, and then
> recovered through the ERP chain loses its partial-completion length.
> __dasd_cleanup_cqr() then sees proc_bytes == 0 and completes the whole
> request instead of requeueing the remainder, silently returning zeroed
> data for the part that was never read.
> 
> Carry proc_bytes over to the original request like the other
> per-request state.
> 
> Fixes: 5e6bdd37c552 ("s390/dasd: fix data corruption for thin provisioned devices")
> Cc: [email protected]
> Reviewed-by: Jan Höppner <[email protected]>
> Signed-off-by: Stefan Haberland <[email protected]>

Sashiko has reviewed this patch and found no issues. It looks great!

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/[email protected]?part=2