| Message ID | 20230118203111.529766-1-akrowiak@linux.ibm.com (mailing list archive) |
|---|---|
| Headers | show |
| Series | improve AP queue reset processing | expand |
Am 18.01.23 um 21:31 schrieb Tony Krowiak: > This series introduces several improvements to the function that performs > AP queue resets: > > * Breaks up reset processing into multiple smaller, more concise functions. > > * Use TAPQ to verify completion of a reset in progress rather than mulitple > invocations of ZAPQ. > > * Check TAPQ response codes when verifying successful completion of ZAPQ. > > * Fix erroneous handling of some error response codes. > > * Increase the maximum amount of time to wait for successful completion of > ZAPQ. > > Change log v1 => v2: > ------------------- > Remove patch 7/7 to restore original behavior since we don't know whether > interrupts are disabled when an unexpected response code is returned from > ZAPQ. (Halil) > > Tony Krowiak (6): > s390/vfio-ap: verify reset complete in separate function > s390/vfio_ap: check TAPQ response code when waiting for queue reset > s390/vfio_ap: use TAPQ to verify reset in progress completes > s390/vfio_ap: verify ZAPQ completion after return of response code > zero > s390/vfio_ap: fix handling of error response codes > s390/vfio_ap: increase max wait time for reset verification > > drivers/s390/crypto/vfio_ap_ops.c | 104 +++++++++++++++++++++--------- > 1 file changed, 72 insertions(+), 32 deletions(-) > Thanks applied and queued for CI and regression runs. Will likely go via s390 tree.
On 1/20/23 5:25 AM, Christian Borntraeger wrote: > Am 18.01.23 um 21:31 schrieb Tony Krowiak: >> This series introduces several improvements to the function that >> performs >> AP queue resets: >> >> * Breaks up reset processing into multiple smaller, more concise >> functions. >> >> * Use TAPQ to verify completion of a reset in progress rather than >> mulitple >> invocations of ZAPQ. >> >> * Check TAPQ response codes when verifying successful completion of >> ZAPQ. >> >> * Fix erroneous handling of some error response codes. >> >> * Increase the maximum amount of time to wait for successful >> completion of >> ZAPQ. >> Change log v1 => v2: >> ------------------- >> Remove patch 7/7 to restore original behavior since we don't know >> whether >> interrupts are disabled when an unexpected response code is returned >> from >> ZAPQ. (Halil) >> >> Tony Krowiak (6): >> s390/vfio-ap: verify reset complete in separate function >> s390/vfio_ap: check TAPQ response code when waiting for queue reset >> s390/vfio_ap: use TAPQ to verify reset in progress completes >> s390/vfio_ap: verify ZAPQ completion after return of response code >> zero >> s390/vfio_ap: fix handling of error response codes >> s390/vfio_ap: increase max wait time for reset verification >> >> drivers/s390/crypto/vfio_ap_ops.c | 104 +++++++++++++++++++++--------- >> 1 file changed, 72 insertions(+), 32 deletions(-) >> > > Thanks applied and queued for CI and regression runs. Will likely go > via s390 tree. Got it, thanks.
This series introduces several improvements to the function that performs AP queue resets: * Breaks up reset processing into multiple smaller, more concise functions. * Use TAPQ to verify completion of a reset in progress rather than mulitple invocations of ZAPQ. * Check TAPQ response codes when verifying successful completion of ZAPQ. * Fix erroneous handling of some error response codes. * Increase the maximum amount of time to wait for successful completion of ZAPQ. Change log v1 => v2: ------------------- Remove patch 7/7 to restore original behavior since we don't know whether interrupts are disabled when an unexpected response code is returned from ZAPQ. (Halil) Tony Krowiak (6): s390/vfio-ap: verify reset complete in separate function s390/vfio_ap: check TAPQ response code when waiting for queue reset s390/vfio_ap: use TAPQ to verify reset in progress completes s390/vfio_ap: verify ZAPQ completion after return of response code zero s390/vfio_ap: fix handling of error response codes s390/vfio_ap: increase max wait time for reset verification drivers/s390/crypto/vfio_ap_ops.c | 104 +++++++++++++++++++++--------- 1 file changed, 72 insertions(+), 32 deletions(-)