crypto: qat - avoid division by zero
commit f99fb7d660f7 ("crypto: qat - avoid division by zero") upstream
Check if delta_us is not zero and return -EINVAL if it is.
delta_us is unlikely to be zero as there is a sleep between the reads of
the two timestamps.
This is to fix the following warning when compiling the QAT driver
using clang scan-build:
drivers/crypto/intel/qat/qat_common/adf_clock.c:87:9: warning: Division by zero [core.DivideZero]
87 | temp = DIV_ROUND_CLOSEST_ULL(temp, delta_us);
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Intel-SIG: commit f99fb7d660f7 ("crypto: qat - avoid division by zero)"
Backport to support Intel QAT in-tree driver
Fixes: e2980ba57e
("crypto: qat - add measure clock frequency")
Signed-off-by: Adam Guerin <adam.guerin@intel.com>
Reviewed-by: Giovanni Cabiddu <giovanni.cabiddu@intel.com>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
[ Aichun Shi: amend commit log ]
Signed-off-by: Aichun Shi <aichun.shi@intel.com>
This commit is contained in:
parent
cd41e2604d
commit
0b9bf2d4e0
|
@ -83,6 +83,9 @@ static int measure_clock(struct adf_accel_dev *accel_dev, u32 *frequency)
|
|||
}
|
||||
|
||||
delta_us = timespec_to_us(&ts3) - timespec_to_us(&ts1);
|
||||
if (!delta_us)
|
||||
return -EINVAL;
|
||||
|
||||
temp = (timestamp2 - timestamp1) * ME_CLK_DIVIDER * 10;
|
||||
temp = DIV_ROUND_CLOSEST_ULL(temp, delta_us);
|
||||
/*
|
||||
|
|
Loading…
Reference in New Issue