drm/xe/pf: Take RPM during calls to SR-IOV attr.store()

We expect that all SR-IOV attr.store() handlers will require active
runtime PM reference. To simplify implementation of those handlers,
take an implicit RPM reference on their behalf. Also wait until PF
completes its restart.

Signed-off-by: Michal Wajdeczko <michal.wajdeczko@intel.com>
Reviewed-by: Lucas De Marchi <lucas.demarchi@intel.com>
Link: https://patch.msgid.link/20251030222348.186658-3-michal.wajdeczko@intel.com

diff --git a/drivers/gpu/drm/xe/xe_sriov_pf_sysfs.c b/drivers/gpu/drm/xe/xe_sriov_pf_sysfs.c
index 0f2b19ca873ebfdeea8a340dfa7416ff806f8509..439a0cd02a864d6746f024e5bdd813329d8fbb20 100644 (file)
--- a/drivers/gpu/drm/xe/xe_sriov_pf_sysfs.c
+++ b/drivers/gpu/drm/xe/xe_sriov_pf_sysfs.c
@@ -9,7 +9,9 @@
 #include <drm/drm_managed.h>
 
 #include "xe_assert.h"
+#include "xe_pm.h"
 #include "xe_sriov.h"
+#include "xe_sriov_pf.h"
 #include "xe_sriov_pf_helpers.h"
 #include "xe_sriov_pf_sysfs.h"
 #include "xe_sriov_printk.h"
@@ -129,11 +131,16 @@ static ssize_t xe_sriov_dev_attr_store(struct kobject *kobj, struct attribute *a
        struct xe_sriov_dev_attr *vattr = to_xe_sriov_dev_attr(attr);
        struct xe_sriov_kobj *vkobj = to_xe_sriov_kobj(kobj);
        struct xe_device *xe = vkobj->xe;
+       ssize_t ret;
 
        if (!vattr->store)
                return -EPERM;
 
-       return vattr->store(xe, buf, count);
+       xe_pm_runtime_get(xe);
+       ret = xe_sriov_pf_wait_ready(xe) ?: vattr->store(xe, buf, count);
+       xe_pm_runtime_put(xe);
+
+       return ret;
 }
 
 static ssize_t xe_sriov_vf_attr_show(struct kobject *kobj, struct attribute *attr, char *buf)
@@ -158,13 +165,18 @@ static ssize_t xe_sriov_vf_attr_store(struct kobject *kobj, struct attribute *at
        struct xe_sriov_kobj *vkobj = to_xe_sriov_kobj(kobj);
        struct xe_device *xe = vkobj->xe;
        unsigned int vfid = vkobj->vfid;
+       ssize_t ret;
 
        xe_sriov_pf_assert_vfid(xe, vfid);
 
        if (!vattr->store)
                return -EPERM;
 
-       return vattr->store(xe, vfid, buf, count);
+       xe_pm_runtime_get(xe);
+       ret = xe_sriov_pf_wait_ready(xe) ?: vattr->store(xe, vfid, buf, count);
+       xe_pm_runtime_get(xe);
+
+       return ret;
 }
 
 static const struct sysfs_ops xe_sriov_dev_sysfs_ops = {