Skip to content

Commit 76903b2

Browse files
dceraolorodrigovivi
authored andcommitted
drm/xe/pxp: Clear restart flag in pxp_start after jumping back
If we don't clear the flag we'll keep jumping back at the beginning of the function once we reach the end. Fixes: ccd3c68 ("drm/xe/pxp: Decouple queue addition from PXP start") Signed-off-by: Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com> Cc: Julia Filipchuk <julia.filipchuk@intel.com> Reviewed-by: Julia Filipchuk <julia.filipchuk@intel.com> Link: https://patch.msgid.link/20260324153718.3155504-9-daniele.ceraolospurio@intel.com (cherry picked from commit 0850ec7bb2459602351639dccf7a68a03c9d1ee0) Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
1 parent 4fed244 commit 76903b2

1 file changed

Lines changed: 3 additions & 1 deletion

File tree

drivers/gpu/drm/xe/xe_pxp.c

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -512,7 +512,7 @@ static int __exec_queue_add(struct xe_pxp *pxp, struct xe_exec_queue *q)
512512
static int pxp_start(struct xe_pxp *pxp, u8 type)
513513
{
514514
int ret = 0;
515-
bool restart = false;
515+
bool restart;
516516

517517
if (!xe_pxp_is_enabled(pxp))
518518
return -ENODEV;
@@ -541,6 +541,8 @@ static int pxp_start(struct xe_pxp *pxp, u8 type)
541541
msecs_to_jiffies(PXP_ACTIVATION_TIMEOUT_MS)))
542542
return -ETIMEDOUT;
543543

544+
restart = false;
545+
544546
mutex_lock(&pxp->mutex);
545547

546548
/* If PXP is not already active, turn it on */

0 commit comments

Comments
 (0)