Skip to contentSkip to conversation

drm/i915/gvt: Fix gvt scheduler interval time

PATCH In review Zhenyu Wang 2 participants 2 messages Mar 17, 2017

Subsystems (4)
lore
Share & tools
lore
#0 lore raw

PATCH 1/1 drm/i915/gvt: Fix gvt scheduler interval time

Fix to correctly assign 1ms for gvt scheduler interval time, as previous code using HZ is pretty broken. And use no delay for start gvt scheduler function.

Fixes 4b63960ebd3f ("drm/i915/gvt: vGPU schedule policy framework")
Cc Zhi Wang <[email protected]>
Cc [email protected] # v4.10+
---
 drivers/gpu/drm/i915/gvt/sched_policy.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
drivers/gpu/drm/i915/gvt/sched_policy.c +2 -2
index 06c9584ac5f0..34b9acdf3479 100644
@@ -101,7 +101,7 @@ struct tbs_sched_data {
 	struct list_head runq_head;
 };
 
-#define GVT_DEFAULT_TIME_SLICE (1 * HZ / 1000)
+#define GVT_DEFAULT_TIME_SLICE (msecs_to_jiffies(1))
 
 static void tbs_sched_func(struct work_struct *work)
 {
@@ -223,7 +223,7 @@ static void tbs_sched_start_schedule(struct intel_vgpu *vgpu)
 		return;
 
 	list_add_tail(&vgpu_data->list, &sched_data->runq_head);
-	schedule_delayed_work(&sched_data->work, sched_data->period);
+	schedule_delayed_work(&sched_data->work, 0);
 }
 
 static void tbs_sched_stop_schedule(struct intel_vgpu *vgpu)
Signature · 1 line
2.11.0
Acked-by on v1 1/1
#1 lore raw
Quoted 11 lines

-----Original Message-----
From: intel-gvt-dev [mailto:[email protected]] On Behalf Of Zhenyu Wang
Sent: Friday, March 17, 2017 12:19 PM
To: [email protected]
Cc: [email protected]; #@freedesktop.org; [email protected]; Wang, Zhi A <[email protected]>
Subject: [PATCH] drm/i915/gvt: Fix gvt scheduler interval time

Quoted 7 lines

Fix to correctly assign 1ms for gvt scheduler interval time, as previous code using HZ is pretty broken. And use no delay for start gvt scheduler function.

Fixes 4b63960ebd3f ("drm/i915/gvt: vGPU schedule policy framework")
Cc Zhi Wang <[email protected]>
Cc [email protected] # v4.10+
Quoted 33 lines
---
 drivers/gpu/drm/i915/gvt/sched_policy.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
drivers/gpu/drm/i915/gvt/sched_policy.c +1 -1
index 06c9584ac5f0..34b9acdf3479 100644
@@ -101,7 +101,7 @@ struct tbs_sched_data {
 	struct list_head runq_head;
 };
 
-#define GVT_DEFAULT_TIME_SLICE (1 * HZ / 1000)
+#define GVT_DEFAULT_TIME_SLICE (msecs_to_jiffies(1))
 
 static void tbs_sched_func(struct work_struct *work)  { @@ -223,7 +223,7
 @@ static void tbs_sched_start_schedule(struct intel_vgpu *vgpu)
drivers/gpu/drm/i915/gvt/sched_policy.c +1 -2
 		return;
 
 	list_add_tail(&vgpu_data->list, &sched_data->runq_head);
-	schedule_delayed_work(&sched_data->work, sched_data->period);
+	schedule_delayed_work(&sched_data->work, 0);
 }
 
 static void tbs_sched_stop_schedule(struct intel_vgpu *vgpu)
--

2.11.0

_______________________________________________
intel-gvt-dev mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/intel-gvt-dev