]> www.infradead.org Git - users/jedix/linux-maple.git/commit
drm/xe/eustall: Fix a possible pointer dereference after free
authorHarish Chegondi <harish.chegondi@intel.com>
Wed, 12 Mar 2025 17:31:20 +0000 (10:31 -0700)
committerLucas De Marchi <lucas.demarchi@intel.com>
Thu, 20 Mar 2025 20:01:07 +0000 (13:01 -0700)
commit488975c2d3e171bd07ec5caaf3c9cbc6a0746e2d
tree8129f59f29ba5378b80350168261a63fcc688733
parent5da39dce1fa3c81dc6552a16a9f748ba2980d630
drm/xe/eustall: Fix a possible pointer dereference after free

If devm_add_action_or_reset() isn't successful, xe_eu_stall_fini()
is invoked. So, unsuccessful return from devm_add_action_or_reset()
shouldn't dereference gt->eu_stall as xe_eu_stall_fini() already
frees it. Fix this issue.

Fixes: 9a0b11d4cf3b ("drm/xe/eustall: Add support to init, enable and disable EU stall sampling")
Signed-off-by: Harish Chegondi <harish.chegondi@intel.com>
Reviewed-by: Ashutosh Dixit <ashutosh.dixit@intel.com>
Signed-off-by: Ashutosh Dixit <ashutosh.dixit@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/eae49a414a7314921108e0388810aaee6261ad92.1741800396.git.harish.chegondi@intel.com
(cherry picked from commit 278469ff569e1082d56b4a7af26fbaecef9fbf3b)
Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
drivers/gpu/drm/xe/xe_eu_stall.c