]> git.ipfire.org Git - thirdparty/kernel/linux.git/commit
drm/xe/pf: Fix EAGAIN sign in pf_migration_consume()
authorShuicheng Lin <shuicheng.lin@intel.com>
Tue, 28 Apr 2026 20:14:48 +0000 (20:14 +0000)
committerMatthew Brost <matthew.brost@intel.com>
Tue, 5 May 2026 23:14:56 +0000 (16:14 -0700)
commitd01012c740bbb298b957e30cc0848e482c6f486f
treef1bbff4428186ec7494d1c4398e1eeffe50eca04
parent60a1e131a811b68703da58fd805ab359b704ab03
drm/xe/pf: Fix EAGAIN sign in pf_migration_consume()

PTR_ERR() returns a negative value, so comparing against the positive
EAGAIN is always true for ERR_PTR(-EAGAIN), causing pf_migration_consume()
to bail out instead of continuing to the remaining GTs. On multi-GT
platforms this can skip GTs that already have data ready.

Compare against -EAGAIN to match the intent (and the following line
that correctly uses -EAGAIN). While at it, gate PTR_ERR() with
IS_ERR().

v2: add IS_ERR() guard before PTR_ERR(). (Gustavo)

Fixes: 67df4a5cbc58 ("drm/xe/pf: Add data structures and handlers for migration rings")
Cc: MichaƂ Winiarski <michal.winiarski@intel.com>
Reviewed-by: Gustavo Sousa <gustavo.sousa@intel.com>
Link: https://patch.msgid.link/20260428201448.3999428-1-shuicheng.lin@intel.com
Signed-off-by: Shuicheng Lin <shuicheng.lin@intel.com>
(cherry picked from commit 9d770e72e1edb54beacfce5f402edb51632811e3)
Signed-off-by: Matthew Brost <matthew.brost@intel.com>
drivers/gpu/drm/xe/xe_sriov_pf_migration.c