drm/xe/oa: Check managed mutex initialization errors

drmm_mutex_init() can fail while registering its managed cleanup action.
On failure, the reset path destroys the mutex, so continuing OA setup
leaves an unusable lock that later paths may acquire.

Return the error from per-GT OA initialization and abort device-wide OA
initialization if the metrics lock cannot be initialized.

Fixes: a9f905ae7b6f ("drm/xe/oa/uapi: Initialize OA units")
Fixes: cdf02fe1a94a ("drm/xe/oa/uapi: Add/remove OA config perf ops")
Signed-off-by: Linmao Li <lilinmao@kylinos.cn>
Reviewed-by: Umesh Nerlige Ramappa <umesh.nerlige.ramappa@intel.com>
Signed-off-by: Umesh Nerlige Ramappa <umesh.nerlige.ramappa@intel.com>
Link: https://patch.msgid.link/20260713083053.321091-1-lilinmao@kylinos.cn
(cherry picked from commit 360b293de27bfdd0d07047f8efd5ba8e91fa90b7)
Signed-off-by: Thomas Hellström <thomas.hellstrom@linux.intel.com>

diff --git a/drivers/gpu/drm/xe/xe_oa.c b/drivers/gpu/drm/xe/xe_oa.c
index 9cf962b71cb6593c7050c3df510ec9a030eabfc6..18d990c5d4ec42c396bb6720aac1965c77c50aef 100644 (file)
--- a/drivers/gpu/drm/xe/xe_oa.c
+++ b/drivers/gpu/drm/xe/xe_oa.c
@@ -2717,9 +2717,7 @@ static int xe_oa_init_gt(struct xe_gt *gt)
 
        __xe_oa_init_oa_units(gt);
 
-       drmm_mutex_init(&gt_to_xe(gt)->drm, &gt->oa.gt_lock);
-
-       return 0;
+       return drmm_mutex_init(&gt_to_xe(gt)->drm, &gt->oa.gt_lock);
 }
 
 static void xe_oa_print_gt_oa_units(struct xe_gt *gt)
@@ -2859,7 +2857,10 @@ int xe_oa_init(struct xe_device *xe)
        oa->xe = xe;
        oa->oa_formats = oa_formats;
 
-       drmm_mutex_init(&oa->xe->drm, &oa->metrics_lock);
+       ret = drmm_mutex_init(&oa->xe->drm, &oa->metrics_lock);
+       if (ret)
+               goto exit;
+
        idr_init_base(&oa->metrics_idr, 1);
 
        ret = xe_oa_init_oa_units(oa);