crypto: qat - change SLAs cleanup flow at shutdown
authorDamian Muszynski <damian.muszynski@intel.com>
Fri, 9 Feb 2024 12:42:07 +0000 (13:42 +0100)
committerHerbert Xu <herbert@gondor.apana.org.au>
Sat, 17 Feb 2024 01:09:16 +0000 (09:09 +0800)
commitc2304e1a0b8051a60d4eb9c99a1c509d90380ae5
tree228c7d38d664807e661ec224a114bcdc9851dc67
parent3ee2cee56c5e399424bb6d18de3b63be7126f28a
crypto: qat - change SLAs cleanup flow at shutdown

The implementation of the Rate Limiting (RL) feature includes the cleanup
of all SLAs during device shutdown. For each SLA, the firmware is notified
of the removal through an admin message, the data structures that take
into account the budgets are updated and the memory is freed.
However, this explicit cleanup is not necessary as (1) the device is
reset, and the firmware state is lost and (2) all RL data structures
are freed anyway.

In addition, if the device is unresponsive, for example after a PCI
AER error is detected, the admin interface might not be available.
This might slow down the shutdown sequence and cause a timeout in
the recovery flows which in turn makes the driver believe that the
device is not recoverable.

Fix by replacing the explicit SLAs removal with just a free of the
SLA data structures.

Fixes: d9fb8408376e ("crypto: qat - add rate limiting feature to qat_4xxx")
Cc: <stable@vger.kernel.org>
Signed-off-by: Damian Muszynski <damian.muszynski@intel.com>
Reviewed-by: Giovanni Cabiddu <giovanni.cabiddu@intel.com>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
drivers/crypto/intel/qat/qat_common/adf_rl.c