[PATCH] drm/xe/gsc: do not flush the GSC worker from the reset path
Julia Filipchuk
julia.filipchuk at intel.com
Tue Apr 29 00:27:49 UTC 2025
Changes look good. As intended flush is only on suspend path and reset path
awaits register change.
Only minor suggestions below.
- Correct commit spelling
- Rename xe_gsc_wait_for_proxy_done to xe_gsc_wait_for_proxy_init_done for
clarity that we await for init completion.
On 4/17/2025 3:37 PM, Daniele Ceraolo Spurio wrote:
> The workqueue used for the reset worker is marked as WQ_MEM_RECLAIM,
> while the GSC one isn't (and can't be as we need to do memory
> allocations in the gsc worker). Therefore, we can't flush the latter
> from the former.
>
> The reason why we had such a flush was to avoid interrupting either
> the GSC FW load or in progress GSC proxy operations. GSC proxy
> operations fall into 2 categories:
>
> 1) GSC proxy init: this only happen once immediately after GSC FW laod
Resolve in this line to: happens, load
Reviewed-by: Julia Filipchuk <julia.filipchuk at intel.com>
More information about the Intel-xe
mailing list