[Intel-xe] [PATCH V11 6/6] drm/xe: Add min/max cap for engine scheduler properties
Niranjana Vishwanathapura
niranjana.vishwanathapura at intel.com
Tue Aug 1 17:11:58 UTC 2023
On Tue, Aug 01, 2023 at 12:47:42AM +0530, Tejas Upadhyay wrote:
>Add sysfs entries for the min, max, and defaults for each of
>engine scheduler controls for every hardware engine class.
>
>Non-elevated user IOCTLs to set these controls must be within
>the min-max ranges of the sysfs entries, elevated user can set
>these controls to any value. However, introduced compile time
>CONFIG min-max values which restricts elevated user to be in
>compile time min-max range if at all sysfs min/max are violated.
>
>Sysfs entries examples are,
>DUT# cat /sys/class/drm/cardX/device/tileN/gtN/engines/ccs/.defaults/
>job_timeout_max job_timeout_ms preempt_timeout_min timeslice_duration_max timeslice_duration_us
>job_timeout_min preempt_timeout_max preempt_timeout_us timeslice_duration_min
>
>DUT# cat /sys/class/drm/card1/device/tileN/gtN/engines/ccs/
>.defaults/ job_timeout_min preempt_timeout_max preempt_timeout_us timeslice_duration_min
>job_timeout_max job_timeout_ms preempt_timeout_min timeslice_duration_max timeslice_duration_us
>
>V11:
> - Make engine_get_prop_minmax and enforce_sched_limit static - Matt
> - use enum in place of string in engine_get_prop_minmax - Matt
> - no need to use enforce_sched_limit or no need to filter min/max
> per user type in sysfs - Matt
>V10:
> - Add kernel doc for non-static func
> - Make helper to get min/max for range validation - Matt
> - Filter min/max per user type
>V9 :
> - Rebase to use s/xe_engine/xe_hw_engine/ - Matt
>V8 :
> - fix enforce_sched_limit and avoid code duplication - Niranjana
> - Make sure min < max - Niranjana
>V7 :
> - Rebase to replace hw engine with eclass interface
> - return EINVAL in place of EPERM
> - Use some APIs to avoid code duplication
>V6 :
> - Rebase changes to reflect per engine class props interface - MattB
> - Use #if ENABLED - MattB
> - Remove MAX_SCHED_TIMEOUT check as range validation is enough
>V5 :
> - Rebase to resolve conflicts - CI
>V4 :
> - Rebase
> - Update commit to reflect tile addition
> - Use XE_HW macro directly as they are already filtered
> for CONFIG checks - Niranjana
> - Add CONFIG for enable/disable min/max limitation
> on elevated user. Default is enable - Matt/Joonas
>V3 :
> - Resolve CI hooks warning for kernel-doc
>V2 :
> - Restric min/max setting to #define default min/max for
> elevated user - Himal
> - Remove unrelated changes from patch - Niranjana
>
>Signed-off-by: Tejas Upadhyay <tejas.upadhyay at intel.com>
>---
> drivers/gpu/drm/xe/Kconfig | 6 +
> drivers/gpu/drm/xe/Kconfig.profile | 46 +++
> drivers/gpu/drm/xe/xe_engine.c | 89 ++++-
> drivers/gpu/drm/xe/xe_hw_engine.c | 8 +
> drivers/gpu/drm/xe/xe_hw_engine.h | 31 ++
> drivers/gpu/drm/xe/xe_hw_engine_class_sysfs.c | 332 ++++++++++++++++++
> drivers/gpu/drm/xe/xe_hw_engine_class_sysfs.h | 1 +
> 7 files changed, 507 insertions(+), 6 deletions(-)
> create mode 100644 drivers/gpu/drm/xe/Kconfig.profile
>
>diff --git a/drivers/gpu/drm/xe/Kconfig b/drivers/gpu/drm/xe/Kconfig
>index d44794f99338..0a4ea965645b 100644
>--- a/drivers/gpu/drm/xe/Kconfig
>+++ b/drivers/gpu/drm/xe/Kconfig
>@@ -83,3 +83,9 @@ depends on DRM_XE
> depends on EXPERT
> source "drivers/gpu/drm/xe/Kconfig.debug"
> endmenu
>+
>+menu "drm/xe Profile Guided Optimisation"
>+ visible if EXPERT
>+ depends on DRM_XE
>+ source "drivers/gpu/drm/xe/Kconfig.profile"
>+endmenu
>diff --git a/drivers/gpu/drm/xe/Kconfig.profile b/drivers/gpu/drm/xe/Kconfig.profile
>new file mode 100644
>index 000000000000..e72f15ec4bf6
>--- /dev/null
>+++ b/drivers/gpu/drm/xe/Kconfig.profile
>@@ -0,0 +1,46 @@
>+config DRM_XE_JOB_TIMEOUT_MAX
>+ int "Default max job timeout (ms)"
>+ default 10000 # milliseconds
>+ help
>+ Configures the default max job timeout after which job will
>+ be forcefully taken away from scheduler.
>+config DRM_XE_JOB_TIMEOUT_MIN
>+ int "Default max job timeout (ms)"
>+ default 1 # milliseconds
>+ help
>+ Configures the default min job timeout after which job will
>+ be forcefully taken away from scheduler.
>+config DRM_XE_TIMESLICE_MAX
>+ int "Default max timeslice duration (us)"
>+ default 10000000 # microseconds
>+ help
>+ Configures the default max timeslice duration between multiple
>+ contexts by guc scheduling.
>+config DRM_XE_TIMESLICE_MIN
>+ int "Default min timeslice duration (us)"
>+ default 1 # microseconds
>+ help
>+ Configures the default min timeslice duration between multiple
>+ contexts by guc scheduling.
>+config DRM_XE_PREEMPT_TIMEOUT_MAX
>+ int "Default max preempt timeout (us)"
>+ default 10000000 # microseconds
>+ help
>+ Configures the default max preempt timeout after which context
>+ will be forcefully taken away and higher priority context will
>+ run.
>+config DRM_XE_PREEMPT_TIMEOUT_MIN
>+ int "Default min preempt timeout (us)"
>+ default 1 # microseconds
>+ help
>+ Configures the default min preempt timeout after which context
>+ will be forcefully taken away and higher priority context will
>+ run.
>+config DRM_XE_ENABLE_SCHEDTIMEOUT_LIMIT
>+ bool "Default configuration of limitation on scheduler timeout"
>+ default y
>+ help
>+ Configures the enablement of limitation on scheduler timeout
>+ to apply to applicable user. For elevated user, all above MIN
>+ and MAX values will apply when this configuration is enable to
>+ apply limitation. By default limitation is applied.
Looking at this patch, the requirement seems to be,
The sysfs limits (min/max) setting through sysfs is always subjected to config limits.
The sched_prop values set through sysfs is always subjected to sysfs limits.
The sched_prop values set through engine set_property ioctl can override limits
if cap_nice and DRM_XE_ENABLE_SCHEDTIMEOUT_LIMIT is not set.
In that case,
Reviewed-by: Niranjana Vishwanathapura <niranjana.vishwanathapura at intel.com>
Niranjana
>diff --git a/drivers/gpu/drm/xe/xe_engine.c b/drivers/gpu/drm/xe/xe_engine.c
>index f4eb67915bfc..d77dd42384eb 100644
>--- a/drivers/gpu/drm/xe/xe_engine.c
>+++ b/drivers/gpu/drm/xe/xe_engine.c
>@@ -13,6 +13,7 @@
>
> #include "xe_device.h"
> #include "xe_gt.h"
>+#include "xe_hw_engine_class_sysfs.h"
> #include "xe_hw_fence.h"
> #include "xe_lrc.h"
> #include "xe_macros.h"
>@@ -22,6 +23,13 @@
> #include "xe_trace.h"
> #include "xe_vm.h"
>
>+enum xe_engine_sched_prop {
>+ XE_ENGINE_JOB_TIMEOUT = 0,
>+ XE_ENGINE_TIMESLICE = 1,
>+ XE_ENGINE_PREEMPT_TIMEOUT = 2,
>+ XE_ENGINE_SCHED_PROP_MAX = 3,
>+};
>+
> static struct xe_engine *__xe_engine_create(struct xe_device *xe,
> struct xe_vm *vm,
> u32 logical_mask,
>@@ -201,11 +209,68 @@ static int engine_set_priority(struct xe_device *xe, struct xe_engine *e,
> return e->ops->set_priority(e, value);
> }
>
>+static bool xe_engine_enforce_schedule_limit(void)
>+{
>+#if IS_ENABLED(CONFIG_DRM_XE_ENABLE_SCHEDTIMEOUT_LIMIT)
>+ return true;
>+#else
>+ return !capable(CAP_SYS_NICE);
>+#endif
>+}
>+
>+static void xe_engine_get_prop_minmax(struct xe_hw_engine_class_intf *eclass,
>+ enum xe_engine_sched_prop prop,
>+ u32 *min, u32 *max)
>+{
>+ switch (prop) {
>+ case XE_ENGINE_JOB_TIMEOUT:
>+ *min = eclass->sched_props.job_timeout_min;
>+ *max = eclass->sched_props.job_timeout_max;
>+ break;
>+ case XE_ENGINE_TIMESLICE:
>+ *min = eclass->sched_props.timeslice_min;
>+ *max = eclass->sched_props.timeslice_max;
>+ break;
>+ case XE_ENGINE_PREEMPT_TIMEOUT:
>+ *min = eclass->sched_props.preempt_timeout_min;
>+ *max = eclass->sched_props.preempt_timeout_max;
>+ break;
>+ default:
>+ break;
>+ }
>+#if IS_ENABLED(CONFIG_DRM_XE_ENABLE_SCHEDTIMEOUT_LIMIT)
>+ if (capable(CAP_SYS_NICE)) {
>+ switch (prop) {
>+ case XE_ENGINE_JOB_TIMEOUT:
>+ *min = XE_HW_ENGINE_JOB_TIMEOUT_MIN;
>+ *max = XE_HW_ENGINE_JOB_TIMEOUT_MAX;
>+ break;
>+ case XE_ENGINE_TIMESLICE:
>+ *min = XE_HW_ENGINE_TIMESLICE_MIN;
>+ *max = XE_HW_ENGINE_TIMESLICE_MAX;
>+ break;
>+ case XE_ENGINE_PREEMPT_TIMEOUT:
>+ *min = XE_HW_ENGINE_PREEMPT_TIMEOUT_MIN;
>+ *max = XE_HW_ENGINE_PREEMPT_TIMEOUT_MAX;
>+ break;
>+ default:
>+ break;
>+ }
>+ }
>+#endif
>+}
>+
> static int engine_set_timeslice(struct xe_device *xe, struct xe_engine *e,
> u64 value, bool create)
> {
>- if (!capable(CAP_SYS_NICE))
>- return -EPERM;
>+ u32 min = 0, max = 0;
>+
>+ xe_engine_get_prop_minmax(e->hwe->eclass,
>+ XE_ENGINE_TIMESLICE, &min, &max);
>+
>+ if (xe_engine_enforce_schedule_limit() &&
>+ !xe_hw_engine_timeout_in_range(value, min, max))
>+ return -EINVAL;
>
> return e->ops->set_timeslice(e, value);
> }
>@@ -214,8 +279,14 @@ static int engine_set_preemption_timeout(struct xe_device *xe,
> struct xe_engine *e, u64 value,
> bool create)
> {
>- if (!capable(CAP_SYS_NICE))
>- return -EPERM;
>+ u32 min = 0, max = 0;
>+
>+ xe_engine_get_prop_minmax(e->hwe->eclass,
>+ XE_ENGINE_PREEMPT_TIMEOUT, &min, &max);
>+
>+ if (xe_engine_enforce_schedule_limit() &&
>+ !xe_hw_engine_timeout_in_range(value, min, max))
>+ return -EINVAL;
>
> return e->ops->set_preempt_timeout(e, value);
> }
>@@ -279,11 +350,17 @@ static int engine_set_persistence(struct xe_device *xe, struct xe_engine *e,
> static int engine_set_job_timeout(struct xe_device *xe, struct xe_engine *e,
> u64 value, bool create)
> {
>+ u32 min = 0, max = 0;
>+
> if (XE_IOCTL_DBG(xe, !create))
> return -EINVAL;
>
>- if (!capable(CAP_SYS_NICE))
>- return -EPERM;
>+ xe_engine_get_prop_minmax(e->hwe->eclass,
>+ XE_ENGINE_JOB_TIMEOUT, &min, &max);
>+
>+ if (xe_engine_enforce_schedule_limit() &&
>+ !xe_hw_engine_timeout_in_range(value, min, max))
>+ return -EINVAL;
>
> return e->ops->set_job_timeout(e, value);
> }
>diff --git a/drivers/gpu/drm/xe/xe_hw_engine.c b/drivers/gpu/drm/xe/xe_hw_engine.c
>index b7be7b0acb35..b8fcc6e985cf 100644
>--- a/drivers/gpu/drm/xe/xe_hw_engine.c
>+++ b/drivers/gpu/drm/xe/xe_hw_engine.c
>@@ -364,8 +364,16 @@ static void hw_engine_init_early(struct xe_gt *gt, struct xe_hw_engine *hwe,
>
> if (!gt->eclass[hwe->class].sched_props.job_timeout_ms) {
> gt->eclass[hwe->class].sched_props.job_timeout_ms = 5 * 1000;
>+ gt->eclass[hwe->class].sched_props.job_timeout_min = XE_HW_ENGINE_JOB_TIMEOUT_MIN;
>+ gt->eclass[hwe->class].sched_props.job_timeout_max = XE_HW_ENGINE_JOB_TIMEOUT_MAX;
> gt->eclass[hwe->class].sched_props.timeslice_us = 1 * 1000;
>+ gt->eclass[hwe->class].sched_props.timeslice_min = XE_HW_ENGINE_TIMESLICE_MIN;
>+ gt->eclass[hwe->class].sched_props.timeslice_max = XE_HW_ENGINE_TIMESLICE_MAX;
> gt->eclass[hwe->class].sched_props.preempt_timeout_us = 640 * 1000;
>+ gt->eclass[hwe->class].sched_props.preempt_timeout_min =
>+ XE_HW_ENGINE_PREEMPT_TIMEOUT_MIN;
>+ gt->eclass[hwe->class].sched_props.preempt_timeout_max =
>+ XE_HW_ENGINE_PREEMPT_TIMEOUT_MAX;
> /* Record default props */
> gt->eclass[hwe->class].defaults = gt->eclass[hwe->class].sched_props;
> }
>diff --git a/drivers/gpu/drm/xe/xe_hw_engine.h b/drivers/gpu/drm/xe/xe_hw_engine.h
>index 7eca9d53c7b1..3d37d6d44261 100644
>--- a/drivers/gpu/drm/xe/xe_hw_engine.h
>+++ b/drivers/gpu/drm/xe/xe_hw_engine.h
>@@ -10,6 +10,37 @@
>
> struct drm_printer;
>
>+#ifdef CONFIG_DRM_XE_JOB_TIMEOUT_MIN
>+#define XE_HW_ENGINE_JOB_TIMEOUT_MIN CONFIG_DRM_XE_JOB_TIMEOUT_MIN
>+#else
>+#define XE_HW_ENGINE_JOB_TIMEOUT_MIN 1
>+#endif
>+#ifdef CONFIG_DRM_XE_JOB_TIMEOUT_MAX
>+#define XE_HW_ENGINE_JOB_TIMEOUT_MAX CONFIG_DRM_XE_JOB_TIMEOUT_MAX
>+#else
>+#define XE_HW_ENGINE_JOB_TIMEOUT_MAX (10 * 1000)
>+#endif
>+#ifdef CONFIG_DRM_XE_TIMESLICE_MIN
>+#define XE_HW_ENGINE_TIMESLICE_MIN CONFIG_DRM_XE_TIMESLICE_MIN
>+#else
>+#define XE_HW_ENGINE_TIMESLICE_MIN 1
>+#endif
>+#ifdef CONFIG_DRM_XE_TIMESLICE_MAX
>+#define XE_HW_ENGINE_TIMESLICE_MAX CONFIG_DRM_XE_TIMESLICE_MAX
>+#else
>+#define XE_HW_ENGINE_TIMESLICE_MAX (10 * 1000 * 1000)
>+#endif
>+#ifdef CONFIG_DRM_XE_PREEMPT_TIMEOUT_MIN
>+#define XE_HW_ENGINE_PREEMPT_TIMEOUT_MIN CONFIG_DRM_XE_PREEMPT_TIMEOUT_MIN
>+#else
>+#define XE_HW_ENGINE_PREEMPT_TIMEOUT_MIN 1
>+#endif
>+#ifdef CONFIG_DRM_XE_PREEMPT_TIMEOUT_MAX
>+#define XE_HW_ENGINE_PREEMPT_TIMEOUT_MAX CONFIG_DRM_XE_PREEMPT_TIMEOUT_MAX
>+#else
>+#define XE_HW_ENGINE_PREEMPT_TIMEOUT_MAX (10 * 1000 * 1000)
>+#endif
>+
> int xe_hw_engines_init_early(struct xe_gt *gt);
> int xe_hw_engines_init(struct xe_gt *gt);
> void xe_hw_engine_handle_irq(struct xe_hw_engine *hwe, u16 intr_vec);
>diff --git a/drivers/gpu/drm/xe/xe_hw_engine_class_sysfs.c b/drivers/gpu/drm/xe/xe_hw_engine_class_sysfs.c
>index 238275675297..d9b662f22fc8 100644
>--- a/drivers/gpu/drm/xe/xe_hw_engine_class_sysfs.c
>+++ b/drivers/gpu/drm/xe/xe_hw_engine_class_sysfs.c
>@@ -13,6 +13,22 @@
> #define MAX_ENGINE_CLASS_NAME_LEN 16
> static int xe_add_hw_engine_class_defaults(struct kobject *parent);
>
>+/**
>+ * xe_hw_engine_timeout_in_range - Helper to check if timeout is in range
>+ * @timeout: timeout to validate
>+ * @min: min value of valid range
>+ * @max: max value of valid range
>+ *
>+ * This helper helps to validate if timeout is in min-max range of HW engine
>+ * scheduler.
>+ *
>+ * Returns: Returns false value for failure and true for success.
>+ */
>+bool xe_hw_engine_timeout_in_range(u64 timeout, u64 min, u64 max)
>+{
>+ return timeout >= min && timeout <= max;
>+}
>+
> static void kobj_xe_hw_engine_release(struct kobject *kobj)
> {
> kfree(kobj);
>@@ -41,11 +57,85 @@ kobj_xe_hw_engine_class(struct kobject *parent, char *name)
> return keclass;
> }
>
>+static ssize_t job_timeout_max_store(struct kobject *kobj,
>+ struct kobj_attribute *attr,
>+ const char *buf, size_t count)
>+{
>+ struct xe_hw_engine_class_intf *eclass = kobj_to_eclass(kobj);
>+ u32 timeout;
>+ int err;
>+
>+ err = kstrtou32(buf, 0, &timeout);
>+ if (err)
>+ return err;
>+
>+ if (timeout < eclass->sched_props.job_timeout_min)
>+ return -EINVAL;
>+
>+ if (!xe_hw_engine_timeout_in_range(timeout,
>+ XE_HW_ENGINE_JOB_TIMEOUT_MIN,
>+ XE_HW_ENGINE_JOB_TIMEOUT_MAX))
>+ return -EINVAL;
>+
>+ WRITE_ONCE(eclass->sched_props.job_timeout_max, timeout);
>+
>+ return count;
>+}
>+
>+static ssize_t job_timeout_max_show(struct kobject *kobj,
>+ struct kobj_attribute *attr, char *buf)
>+{
>+ struct xe_hw_engine_class_intf *eclass = kobj_to_eclass(kobj);
>+
>+ return sprintf(buf, "%u\n", eclass->sched_props.job_timeout_max);
>+}
>+
>+static struct kobj_attribute job_timeout_max_attr =
>+__ATTR(job_timeout_max, 0644, job_timeout_max_show, job_timeout_max_store);
>+
>+static ssize_t job_timeout_min_store(struct kobject *kobj,
>+ struct kobj_attribute *attr,
>+ const char *buf, size_t count)
>+{
>+ struct xe_hw_engine_class_intf *eclass = kobj_to_eclass(kobj);
>+ u32 timeout;
>+ int err;
>+
>+ err = kstrtou32(buf, 0, &timeout);
>+ if (err)
>+ return err;
>+
>+ if (timeout > eclass->sched_props.job_timeout_max)
>+ return -EINVAL;
>+
>+ if (!xe_hw_engine_timeout_in_range(timeout,
>+ XE_HW_ENGINE_JOB_TIMEOUT_MIN,
>+ XE_HW_ENGINE_JOB_TIMEOUT_MAX))
>+ return -EINVAL;
>+
>+ WRITE_ONCE(eclass->sched_props.job_timeout_min, timeout);
>+
>+ return count;
>+}
>+
>+static ssize_t job_timeout_min_show(struct kobject *kobj,
>+ struct kobj_attribute *attr, char *buf)
>+{
>+ struct xe_hw_engine_class_intf *eclass = kobj_to_eclass(kobj);
>+
>+ return sprintf(buf, "%u\n", eclass->sched_props.job_timeout_min);
>+}
>+
>+static struct kobj_attribute job_timeout_min_attr =
>+__ATTR(job_timeout_min, 0644, job_timeout_min_show, job_timeout_min_store);
>+
> static ssize_t job_timeout_store(struct kobject *kobj,
> struct kobj_attribute *attr,
> const char *buf, size_t count)
> {
> struct xe_hw_engine_class_intf *eclass = kobj_to_eclass(kobj);
>+ u32 min = eclass->sched_props.job_timeout_min;
>+ u32 max = eclass->sched_props.job_timeout_max;
> u32 timeout;
> int err;
>
>@@ -53,6 +143,9 @@ static ssize_t job_timeout_store(struct kobject *kobj,
> if (err)
> return err;
>
>+ if (!xe_hw_engine_timeout_in_range(timeout, min, max))
>+ return -EINVAL;
>+
> WRITE_ONCE(eclass->sched_props.job_timeout_ms, timeout);
>
> return count;
>@@ -80,11 +173,35 @@ static ssize_t job_timeout_default(struct kobject *kobj,
> static struct kobj_attribute job_timeout_def =
> __ATTR(job_timeout_ms, 0444, job_timeout_default, NULL);
>
>+static ssize_t job_timeout_min_default(struct kobject *kobj,
>+ struct kobj_attribute *attr, char *buf)
>+{
>+ struct xe_hw_engine_class_intf *eclass = kobj_to_eclass(kobj->parent);
>+
>+ return sprintf(buf, "%u\n", eclass->defaults.job_timeout_min);
>+}
>+
>+static struct kobj_attribute job_timeout_min_def =
>+__ATTR(job_timeout_min, 0444, job_timeout_min_default, NULL);
>+
>+static ssize_t job_timeout_max_default(struct kobject *kobj,
>+ struct kobj_attribute *attr, char *buf)
>+{
>+ struct xe_hw_engine_class_intf *eclass = kobj_to_eclass(kobj->parent);
>+
>+ return sprintf(buf, "%u\n", eclass->defaults.job_timeout_max);
>+}
>+
>+static struct kobj_attribute job_timeout_max_def =
>+__ATTR(job_timeout_max, 0444, job_timeout_max_default, NULL);
>+
> static ssize_t timeslice_duration_store(struct kobject *kobj,
> struct kobj_attribute *attr,
> const char *buf, size_t count)
> {
> struct xe_hw_engine_class_intf *eclass = kobj_to_eclass(kobj);
>+ u32 min = eclass->sched_props.timeslice_min;
>+ u32 max = eclass->sched_props.timeslice_max;
> u32 duration;
> int err;
>
>@@ -92,11 +209,89 @@ static ssize_t timeslice_duration_store(struct kobject *kobj,
> if (err)
> return err;
>
>+ if (!xe_hw_engine_timeout_in_range(duration, min, max))
>+ return -EINVAL;
>+
> WRITE_ONCE(eclass->sched_props.timeslice_us, duration);
>
> return count;
> }
>
>+static ssize_t timeslice_duration_max_store(struct kobject *kobj,
>+ struct kobj_attribute *attr,
>+ const char *buf, size_t count)
>+{
>+ struct xe_hw_engine_class_intf *eclass = kobj_to_eclass(kobj);
>+ u32 duration;
>+ int err;
>+
>+ err = kstrtou32(buf, 0, &duration);
>+ if (err)
>+ return err;
>+
>+ if (duration < eclass->sched_props.timeslice_min)
>+ return -EINVAL;
>+
>+ if (!xe_hw_engine_timeout_in_range(duration,
>+ XE_HW_ENGINE_TIMESLICE_MIN,
>+ XE_HW_ENGINE_TIMESLICE_MAX))
>+ return -EINVAL;
>+
>+ WRITE_ONCE(eclass->sched_props.timeslice_max, duration);
>+
>+ return count;
>+}
>+
>+static ssize_t timeslice_duration_max_show(struct kobject *kobj,
>+ struct kobj_attribute *attr,
>+ char *buf)
>+{
>+ struct xe_hw_engine_class_intf *eclass = kobj_to_eclass(kobj);
>+
>+ return sprintf(buf, "%u\n", eclass->sched_props.timeslice_max);
>+}
>+
>+static struct kobj_attribute timeslice_duration_max_attr =
>+ __ATTR(timeslice_duration_max, 0644, timeslice_duration_max_show,
>+ timeslice_duration_max_store);
>+
>+static ssize_t timeslice_duration_min_store(struct kobject *kobj,
>+ struct kobj_attribute *attr,
>+ const char *buf, size_t count)
>+{
>+ struct xe_hw_engine_class_intf *eclass = kobj_to_eclass(kobj);
>+ u32 duration;
>+ int err;
>+
>+ err = kstrtou32(buf, 0, &duration);
>+ if (err)
>+ return err;
>+
>+ if (duration > eclass->sched_props.timeslice_max)
>+ return -EINVAL;
>+
>+ if (!xe_hw_engine_timeout_in_range(duration,
>+ XE_HW_ENGINE_TIMESLICE_MIN,
>+ XE_HW_ENGINE_TIMESLICE_MAX))
>+ return -EINVAL;
>+
>+ WRITE_ONCE(eclass->sched_props.timeslice_min, duration);
>+
>+ return count;
>+}
>+
>+static ssize_t timeslice_duration_min_show(struct kobject *kobj,
>+ struct kobj_attribute *attr, char *buf)
>+{
>+ struct xe_hw_engine_class_intf *eclass = kobj_to_eclass(kobj);
>+
>+ return sprintf(buf, "%u\n", eclass->sched_props.timeslice_min);
>+}
>+
>+static struct kobj_attribute timeslice_duration_min_attr =
>+ __ATTR(timeslice_duration_min, 0644, timeslice_duration_min_show,
>+ timeslice_duration_min_store);
>+
> static ssize_t timeslice_duration_show(struct kobject *kobj,
> struct kobj_attribute *attr, char *buf)
> {
>@@ -120,11 +315,35 @@ static ssize_t timeslice_default(struct kobject *kobj,
> static struct kobj_attribute timeslice_duration_def =
> __ATTR(timeslice_duration_us, 0444, timeslice_default, NULL);
>
>+static ssize_t timeslice_min_default(struct kobject *kobj,
>+ struct kobj_attribute *attr, char *buf)
>+{
>+ struct xe_hw_engine_class_intf *eclass = kobj_to_eclass(kobj->parent);
>+
>+ return sprintf(buf, "%u\n", eclass->defaults.timeslice_min);
>+}
>+
>+static struct kobj_attribute timeslice_duration_min_def =
>+__ATTR(timeslice_duration_min, 0444, timeslice_min_default, NULL);
>+
>+static ssize_t timeslice_max_default(struct kobject *kobj,
>+ struct kobj_attribute *attr, char *buf)
>+{
>+ struct xe_hw_engine_class_intf *eclass = kobj_to_eclass(kobj->parent);
>+
>+ return sprintf(buf, "%u\n", eclass->defaults.timeslice_max);
>+}
>+
>+static struct kobj_attribute timeslice_duration_max_def =
>+__ATTR(timeslice_duration_max, 0444, timeslice_max_default, NULL);
>+
> static ssize_t preempt_timeout_store(struct kobject *kobj,
> struct kobj_attribute *attr,
> const char *buf, size_t count)
> {
> struct xe_hw_engine_class_intf *eclass = kobj_to_eclass(kobj);
>+ u32 min = eclass->sched_props.preempt_timeout_min;
>+ u32 max = eclass->sched_props.preempt_timeout_max;
> u32 timeout;
> int err;
>
>@@ -132,6 +351,9 @@ static ssize_t preempt_timeout_store(struct kobject *kobj,
> if (err)
> return err;
>
>+ if (!xe_hw_engine_timeout_in_range(timeout, min, max))
>+ return -EINVAL;
>+
> WRITE_ONCE(eclass->sched_props.preempt_timeout_us, timeout);
>
> return count;
>@@ -160,17 +382,127 @@ static ssize_t preempt_timeout_default(struct kobject *kobj,
> static struct kobj_attribute preempt_timeout_def =
> __ATTR(preempt_timeout_us, 0444, preempt_timeout_default, NULL);
>
>+static ssize_t preempt_timeout_min_default(struct kobject *kobj,
>+ struct kobj_attribute *attr,
>+ char *buf)
>+{
>+ struct xe_hw_engine_class_intf *eclass = kobj_to_eclass(kobj->parent);
>+
>+ return sprintf(buf, "%u\n", eclass->defaults.preempt_timeout_min);
>+}
>+
>+static struct kobj_attribute preempt_timeout_min_def =
>+__ATTR(preempt_timeout_min, 0444, preempt_timeout_min_default, NULL);
>+
>+static ssize_t preempt_timeout_max_default(struct kobject *kobj,
>+ struct kobj_attribute *attr,
>+ char *buf)
>+{
>+ struct xe_hw_engine_class_intf *eclass = kobj_to_eclass(kobj->parent);
>+
>+ return sprintf(buf, "%u\n", eclass->defaults.preempt_timeout_max);
>+}
>+
>+static struct kobj_attribute preempt_timeout_max_def =
>+__ATTR(preempt_timeout_max, 0444, preempt_timeout_max_default, NULL);
>+
>+static ssize_t preempt_timeout_max_store(struct kobject *kobj,
>+ struct kobj_attribute *attr,
>+ const char *buf, size_t count)
>+{
>+ struct xe_hw_engine_class_intf *eclass = kobj_to_eclass(kobj);
>+ u32 timeout;
>+ int err;
>+
>+ err = kstrtou32(buf, 0, &timeout);
>+ if (err)
>+ return err;
>+
>+ if (timeout < eclass->sched_props.preempt_timeout_min)
>+ return -EINVAL;
>+
>+ if (!xe_hw_engine_timeout_in_range(timeout,
>+ XE_HW_ENGINE_PREEMPT_TIMEOUT_MIN,
>+ XE_HW_ENGINE_PREEMPT_TIMEOUT_MAX))
>+ return -EINVAL;
>+
>+ WRITE_ONCE(eclass->sched_props.preempt_timeout_max, timeout);
>+
>+ return count;
>+}
>+
>+static ssize_t preempt_timeout_max_show(struct kobject *kobj,
>+ struct kobj_attribute *attr, char *buf)
>+{
>+ struct xe_hw_engine_class_intf *eclass = kobj_to_eclass(kobj);
>+
>+ return sprintf(buf, "%u\n", eclass->sched_props.preempt_timeout_max);
>+}
>+
>+static struct kobj_attribute preempt_timeout_max_attr =
>+ __ATTR(preempt_timeout_max, 0644, preempt_timeout_max_show,
>+ preempt_timeout_max_store);
>+
>+static ssize_t preempt_timeout_min_store(struct kobject *kobj,
>+ struct kobj_attribute *attr,
>+ const char *buf, size_t count)
>+{
>+ struct xe_hw_engine_class_intf *eclass = kobj_to_eclass(kobj);
>+ u32 timeout;
>+ int err;
>+
>+ err = kstrtou32(buf, 0, &timeout);
>+ if (err)
>+ return err;
>+
>+ if (timeout > eclass->sched_props.preempt_timeout_max)
>+ return -EINVAL;
>+
>+ if (!xe_hw_engine_timeout_in_range(timeout,
>+ XE_HW_ENGINE_PREEMPT_TIMEOUT_MIN,
>+ XE_HW_ENGINE_PREEMPT_TIMEOUT_MAX))
>+ return -EINVAL;
>+
>+ WRITE_ONCE(eclass->sched_props.preempt_timeout_min, timeout);
>+
>+ return count;
>+}
>+
>+static ssize_t preempt_timeout_min_show(struct kobject *kobj,
>+ struct kobj_attribute *attr, char *buf)
>+{
>+ struct xe_hw_engine_class_intf *eclass = kobj_to_eclass(kobj);
>+
>+ return sprintf(buf, "%u\n", eclass->sched_props.preempt_timeout_min);
>+}
>+
>+static struct kobj_attribute preempt_timeout_min_attr =
>+ __ATTR(preempt_timeout_min, 0644, preempt_timeout_min_show,
>+ preempt_timeout_min_store);
>+
> static const struct attribute *defaults[] = {
> &job_timeout_def.attr,
>+ &job_timeout_min_def.attr,
>+ &job_timeout_max_def.attr,
> ×lice_duration_def.attr,
>+ ×lice_duration_min_def.attr,
>+ ×lice_duration_max_def.attr,
> &preempt_timeout_def.attr,
>+ &preempt_timeout_min_def.attr,
>+ &preempt_timeout_max_def.attr,
> NULL
> };
>
> static const struct attribute *files[] = {
> &job_timeout_attr.attr,
>+ &job_timeout_min_attr.attr,
>+ &job_timeout_max_attr.attr,
> ×lice_duration_attr.attr,
>+ ×lice_duration_min_attr.attr,
>+ ×lice_duration_max_attr.attr,
> &preempt_timeout_attr.attr,
>+ &preempt_timeout_min_attr.attr,
>+ &preempt_timeout_max_attr.attr,
> NULL
> };
>
>diff --git a/drivers/gpu/drm/xe/xe_hw_engine_class_sysfs.h b/drivers/gpu/drm/xe/xe_hw_engine_class_sysfs.h
>index c093f381abbe..60469fde4147 100644
>--- a/drivers/gpu/drm/xe/xe_hw_engine_class_sysfs.h
>+++ b/drivers/gpu/drm/xe/xe_hw_engine_class_sysfs.h
>@@ -12,6 +12,7 @@ struct xe_gt;
> struct xe_hw_engine_class_intf;
>
> int xe_hw_engine_class_sysfs_init(struct xe_gt *gt);
>+bool xe_hw_engine_timeout_in_range(u64 timeout, u64 min, u64 max);
>
> /**
> * struct kobj_eclass - A eclass's kobject struct that connects the kobject and the
>--
>2.25.1
>
More information about the Intel-xe
mailing list