diff options
| author | Ofir Bitton <[email protected]> | 2021-09-13 10:55:02 +0300 |
|---|---|---|
| committer | Oded Gabbay <[email protected]> | 2021-10-18 12:05:46 +0300 |
| commit | 027d53b03ca1fe55ea9356ffdb7296432d236e59 (patch) | |
| tree | 2c4bc4a98c7c30058f01906e12af77cbdf2711fc | |
| parent | 2b28485d0a3bf8aa220af4644eb186b5034ff830 (diff) | |
habanalabs: remove redundant cs validity checks
During TDR handling, we check multiple times if CS is valid.
No need to perform this check as CS must be valid at all time
during the TDR handling.
Signed-off-by: Ofir Bitton <[email protected]>
Reviewed-by: Oded Gabbay <[email protected]>
Signed-off-by: Oded Gabbay <[email protected]>
| -rw-r--r-- | drivers/misc/habanalabs/common/command_submission.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/drivers/misc/habanalabs/common/command_submission.c b/drivers/misc/habanalabs/common/command_submission.c index 6dafff375f1c..a344928363c8 100644 --- a/drivers/misc/habanalabs/common/command_submission.c +++ b/drivers/misc/habanalabs/common/command_submission.c @@ -431,11 +431,10 @@ static void cs_handle_tdr(struct hl_device *hdev, struct hl_cs *cs) /* Don't cancel TDR in case this CS was timedout because we might be * running from the TDR context */ - if (cs && (cs->timedout || - hdev->timeout_jiffies == MAX_SCHEDULE_TIMEOUT)) + if (cs->timedout || hdev->timeout_jiffies == MAX_SCHEDULE_TIMEOUT) return; - if (cs && cs->tdr_active) + if (cs->tdr_active) cancel_delayed_work_sync(&cs->work_tdr); spin_lock(&hdev->cs_mirror_lock); |