✓ CI.checkpatch: success for drm/xe/guc_pc: Remove duplicated pc_start call
Patchwork
patchwork at emeril.freedesktop.org
Thu Mar 6 22:29:02 UTC 2025
== Series Details ==
Series: drm/xe/guc_pc: Remove duplicated pc_start call
URL : https://patchwork.freedesktop.org/series/145947/
State : success
== Summary ==
+ KERNEL=/kernel
+ git clone https://gitlab.freedesktop.org/drm/maintainer-tools mt
Cloning into 'mt'...
warning: redirecting to https://gitlab.freedesktop.org/drm/maintainer-tools.git/
+ git -C mt rev-list -n1 origin/master
cbb4e4a079d89106c2736adc3c7de6f9dc56da07
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit 145cd2d7d9c69d71a61822d3a0ba1b1733c39f15
Author: Rodrigo Vivi <rodrigo.vivi at intel.com>
Date: Thu Mar 6 17:06:43 2025 -0500
drm/xe/guc_pc: Remove duplicated pc_start call
xe_guc_pc_start() was getting called from both
xe_uc_init_hw() and from xe_guc_start().
But both are called from do_gt_restart() and only
xe_uc_init_hw() is called at initialization.
So, let's remove the duplication in the regular gt_restart
path.
The only place where xe_guc_pc_start() won't get called now
is on the gt_reset failure path. However, if gt_reset has
failed, it is really unlikely that the PC start will work
or is desired.
Cc: Vinay Belgaumkar <vinay.belgaumkar at intel.com>
Reviewed-by: Jonathan Cavitt <jonathan.cavitt at intel.com>
Reviewed-by: Vinay Belgaumkar <vinay.belgaumkar at intel.com>
Signed-off-by: Rodrigo Vivi <rodrigo.vivi at intel.com>
+ /mt/dim checkpatch 6f2e5afc45e3ca8bf46427ae21a9c1029ea6cdb3 drm-intel
145cd2d7d9c6 drm/xe/guc_pc: Remove duplicated pc_start call
More information about the Intel-xe
mailing list