[gvt-linux:gvt-fixes 13/13] drivers/gpu/drm/i915/gvt/sched_policy.c:311:39: error: unused variable 'scheduler'

[Du, Changbin]

Good catching, will fix it. thanks.

-----Original Message-----
From: intel-gvt-dev [mailto:intel-gvt-dev-bounces at lists.freedesktop.org] On Behalf Of Zhenyu Wang
Sent: Thursday, September 21, 2017 8:20 AM
To: Wu, Fengguang <fengguang.wu at intel.com>
Cc: Du, Changbin <changbin.du at intel.com>; Zhu, Libo <libo.zhu at intel.com>; Xu, Terrence <terrence.xu at intel.com>; intel-gvt-dev at lists.freedesktop.org; kbuild-all at 01.org
Subject: Re: [gvt-linux:gvt-fixes 13/13] drivers/gpu/drm/i915/gvt/sched_policy.c:311:39: error: unused variable 'scheduler'


Thanks for catching this!

Changbin, pls help to fix this.

On 2017.09.21 08:23:28 +0800, kbuild test robot wrote:
> tree:   https://github.com/01org/gvt-linux.git gvt-fixes
> head:   002e00c55d2db3709b6fac336eae0c302ce8f455
> commit: 002e00c55d2db3709b6fac336eae0c302ce8f455 [13/13] drm/i915/gvt: Fix GPU hang after reusing vGPU instance across different guest OS
> config: x86_64-randconfig-x018-201738 (attached as .config)
> compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901
> reproduce:
>         git checkout 002e00c55d2db3709b6fac336eae0c302ce8f455
>         # save the attached .config to linux build tree
>         make ARCH=x86_64 
> 
> All errors (new ones prefixed by >>):
> 
>    drivers/gpu/drm/i915/gvt/sched_policy.c: In function 'tbs_sched_clean_vgpu':
> >> drivers/gpu/drm/i915/gvt/sched_policy.c:311:39: error: unused variable 'scheduler' [-Werror=unused-variable]
>      struct intel_gvt_workload_scheduler *scheduler = &vgpu->gvt->scheduler;
>                                           ^~~~~~~~~
>    cc1: all warnings being treated as errors
> 
> vim +/scheduler +311 drivers/gpu/drm/i915/gvt/sched_policy.c
> 
> 4b63960e Zhi Wang    2016-05-01  308  
> 4b63960e Zhi Wang    2016-05-01  309  static void tbs_sched_clean_vgpu(struct intel_vgpu *vgpu)
> 4b63960e Zhi Wang    2016-05-01  310  {
> 0e86cc9c Changbin Du 2017-05-04 @311  	struct intel_gvt_workload_scheduler *scheduler = &vgpu->gvt->scheduler;
> 0e86cc9c Changbin Du 2017-05-04  312  
> 4b63960e Zhi Wang    2016-05-01  313  	kfree(vgpu->sched_data);
> 4b63960e Zhi Wang    2016-05-01  314  	vgpu->sched_data = NULL;
> 4b63960e Zhi Wang    2016-05-01  315  }
> 4b63960e Zhi Wang    2016-05-01  316  
> 
> :::::: The code at line 311 was first introduced by commit
> :::::: 0e86cc9ccc3bf557348befaddf5cb613cf3c4458 drm/i915/gvt: implement per-vm mmio switching optimization
> 
> :::::: TO: Changbin Du <changbin.du at intel.com>
> :::::: CC: Zhenyu Wang <zhenyuw at linux.intel.com>
> 
> ---
> 0-DAY kernel test infrastructure                Open Source Technology Center
> https://lists.01.org/pipermail/kbuild-all                   Intel Corporation



-- 
Open Source Technology Center, Intel ltd.

$gpg --keyserver wwwkeys.pgp.net --recv-keys 4D781827