drm/xe/guc: Fix handling of GUC_HXG_TYPE_NO_RESPONSE_BUSY
authorMichal Wajdeczko <michal.wajdeczko@intel.com>
Thu, 16 Nov 2023 15:12:41 +0000 (16:12 +0100)
committerRodrigo Vivi <rodrigo.vivi@intel.com>
Thu, 21 Dec 2023 16:44:38 +0000 (11:44 -0500)
commit1d087cb7d81f9a17760154eef5ac8b894428cdbe
tree8f384bc51203d8e7fe142c9f3dd28f77fb461314
parentcd1c9c54c34b3a2540fdf49eafd49a61747a6342
drm/xe/guc: Fix handling of GUC_HXG_TYPE_NO_RESPONSE_BUSY

If GuC responds with the NO_RESPONSE_BUSY message, we extend
our timeout while waiting for the actual response, but we wrongly
assumed that the next message will be RESPONSE_SUCCESS, missing
that we still can get RESPONSE_FAILURE.

Change the condition for the expected message type, using only
common bits from RESPONSE_SUCCESS and RESPONSE_FAILURE (as they
differ, by ABI design, only by the last bit).

v2: add comment/checks to the code (Matt)

Signed-off-by: Michal Wajdeczko <michal.wajdeczko@intel.com>
Reviewed-by: Matthew Brost <matthew.brost@intel.com>
Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
drivers/gpu/drm/xe/xe_guc.c