]> git.proxmox.com Git - mirror_ubuntu-jammy-kernel.git/commitdiff
drm/i915/gvt: remove some debug messages in scheduler timer handler
authorZhenyu Wang <zhenyuw@linux.intel.com>
Wed, 12 Apr 2017 06:22:50 +0000 (14:22 +0800)
committerZhenyu Wang <zhenyuw@linux.intel.com>
Thu, 13 Apr 2017 05:49:25 +0000 (13:49 +0800)
As those debug messages might appear in every timer call for scheduler,
it's too noisy, eat too much log and aren't meaningful. So remove them.

Cc: Ping Gao <ping.a.gao@intel.com>
Signed-off-by: Zhenyu Wang <zhenyuw@linux.intel.com>
drivers/gpu/drm/i915/gvt/sched_policy.c
drivers/gpu/drm/i915/gvt/scheduler.c

index f459ec8b06a1103b2f2109d5bc7e23cd945b41d7..79ba4b3440aafd9537f287028d1e23a6186109a1 100644 (file)
@@ -133,9 +133,6 @@ static void try_to_schedule_next_vgpu(struct intel_gvt *gvt)
        if (!scheduler->next_vgpu)
                return;
 
-       gvt_dbg_sched("try to schedule next vgpu %d\n",
-                       scheduler->next_vgpu->id);
-
        /*
         * after the flag is set, workload dispatch thread will
         * stop dispatching workload for current vgpu
@@ -144,10 +141,8 @@ static void try_to_schedule_next_vgpu(struct intel_gvt *gvt)
 
        /* still have uncompleted workload? */
        for_each_engine(engine, gvt->dev_priv, i) {
-               if (scheduler->current_workload[i]) {
-                       gvt_dbg_sched("still have running workload\n");
+               if (scheduler->current_workload[i])
                        return;
-               }
        }
 
        cur_time = ktime_get();
index 37503825276129d2e3d0d5638d7572e0fdc0a8bb..0b685dd26cb3227b5a718b86a6fd7cda865f72d9 100644 (file)
@@ -274,11 +274,8 @@ static struct intel_vgpu_workload *pick_next_workload(
                goto out;
        }
 
-       if (list_empty(workload_q_head(scheduler->current_vgpu, ring_id))) {
-               gvt_dbg_sched("ring id %d stop - no available workload\n",
-                               ring_id);
+       if (list_empty(workload_q_head(scheduler->current_vgpu, ring_id)))
                goto out;
-       }
 
        /*
         * still have current workload, maybe the workload disptacher