drm/i915/gvt: Fix gvt scheduler interval time

zhenyw · zhenyw · commit 2958b9013fcb · 2017-03-17T16:46:45.000+08:00
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: 4b63960 ("drm/i915/gvt: vGPU schedule policy framework") Cc: Zhi Wang <zhi.a.wang@intel.com> Cc: stable@vger.kernel.org # v4.10+ Acked-by: Chuanxiao Dong <chuanxiao.dong@intel.com> Signed-off-by: Zhenyu Wang <zhenyuw@linux.intel.com>
diff --git a/drivers/gpu/drm/i915/gvt/sched_policy.c b/drivers/gpu/drm/i915/gvt/sched_policy.c
@@ -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)

Original file line number	Diff line number	Diff line change
`@@ -101,7 +101,7 @@ struct tbs_sched_data {`
`101`	`101`	`struct list_head runq_head;`
`102`	`102`	`};`
`103`	`103`
`104`		`-#define GVT_DEFAULT_TIME_SLICE (1 * HZ / 1000)`
	`104`	`+#define GVT_DEFAULT_TIME_SLICE (msecs_to_jiffies(1))`
`105`	`105`
`106`	`106`	`static void tbs_sched_func(struct work_struct *work)`
`107`	`107`	`{`
`@@ -223,7 +223,7 @@ static void tbs_sched_start_schedule(struct intel_vgpu *vgpu)`
`223`	`223`	`return;`
`224`	`224`
`225`	`225`	`list_add_tail(&vgpu_data->list, &sched_data->runq_head);`
`226`		`- schedule_delayed_work(&sched_data->work, sched_data->period);`
	`226`	`+ schedule_delayed_work(&sched_data->work, 0);`
`227`	`227`	`}`
`228`	`228`
`229`	`229`	`static void tbs_sched_stop_schedule(struct intel_vgpu *vgpu)`