On 02/22/2019 01:39 PM, Eric Farman wrote:
Per the discussion [1] about a problem with how vfio-ccw calculates the length of a channel program (specifically when using the forthcoming QEMU BIOS code for DASD IPL), I present this fix. Patch 1 fixes the problem, and is over-engineered for readability sake. Patch 2 takes the functions from Patch 1, and refactors the existing code to make other areas a little easier to understand. (I hope.) I've been running fio for over 24 hours now, and have seen zero hours. Previously, I would have probably seen "a few" errors by now, where prior to the original fix I would've seen "many" errors. Further tests are still ongoing. [1] https://marc.info/?l=linux-s390&m=155063096321940&w=2 Eric Farman (2): s390/cio: Fix vfio-ccw handling of recursive TICs s390/cio: Use cpa range elsewhere within vfio-ccw drivers/s390/cio/vfio_ccw_cp.c | 55 ++++++++++++++++++++++++++++++++---------- 1 file changed, 42 insertions(+), 13 deletions(-)
Reviewed-by: Farhan Ali <alifm@xxxxxxxxxxxxx> for the series.