[Intel-gfx] [drm-intel:drm-intel-next-queued 2/2] drivers/gpu/drm/i915/i915_gem_stolen.c:516:20: warning: unused variable 'ggtt'

kbuild test robot fengguang.wu at intel.com
Thu Nov 17 15:49:47 UTC 2016


tree:   git://anongit.freedesktop.org/drm-intel drm-intel-next-queued
head:   95a2e2be952c3c3a643b8e0504f2ceef15294d4d
commit: 95a2e2be952c3c3a643b8e0504f2ceef15294d4d [2/2] drm/i915: Remove stolen object spam
config: i386-defconfig (attached as .config)
compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901
reproduce:
        git checkout 95a2e2be952c3c3a643b8e0504f2ceef15294d4d
        # save the attached .config to linux build tree
        make ARCH=i386 

All warnings (new ones prefixed by >>):

   drivers/gpu/drm/i915/i915_gem_stolen.c: In function 'i915_pages_create_for_stolen':
>> drivers/gpu/drm/i915/i915_gem_stolen.c:516:20: warning: unused variable 'ggtt' [-Wunused-variable]
     struct i915_ggtt *ggtt = &dev_priv->ggtt;
                       ^~~~

vim +/ggtt +516 drivers/gpu/drm/i915/i915_gem_stolen.c

1ca36d4cb Paulo Zanoni    2015-09-23  500  	 * on the first page. So we don't reserve this page for now because of
1ca36d4cb Paulo Zanoni    2015-09-23  501  	 * that. Our current solution is to just prevent new nodes from being
1ca36d4cb Paulo Zanoni    2015-09-23  502  	 * inserted on the first page - see the check we have at
1ca36d4cb Paulo Zanoni    2015-09-23  503  	 * i915_gem_stolen_insert_node_in_range(). We may want to fix the fbcon
1ca36d4cb Paulo Zanoni    2015-09-23  504  	 * problem later.
1ca36d4cb Paulo Zanoni    2015-09-23  505  	 */
72e96d645 Joonas Lahtinen 2016-03-30  506  	drm_mm_init(&dev_priv->mm.stolen, 0, ggtt->stolen_usable_size);
9797fbfbc Chris Wilson    2012-04-24  507  
9797fbfbc Chris Wilson    2012-04-24  508  	return 0;
9797fbfbc Chris Wilson    2012-04-24  509  }
0104fdbb8 Chris Wilson    2012-11-15  510  
0104fdbb8 Chris Wilson    2012-11-15  511  static struct sg_table *
0104fdbb8 Chris Wilson    2012-11-15  512  i915_pages_create_for_stolen(struct drm_device *dev,
0104fdbb8 Chris Wilson    2012-11-15  513  			     u32 offset, u32 size)
0104fdbb8 Chris Wilson    2012-11-15  514  {
72e96d645 Joonas Lahtinen 2016-03-30  515  	struct drm_i915_private *dev_priv = to_i915(dev);
72e96d645 Joonas Lahtinen 2016-03-30 @516  	struct i915_ggtt *ggtt = &dev_priv->ggtt;
0104fdbb8 Chris Wilson    2012-11-15  517  	struct sg_table *st;
0104fdbb8 Chris Wilson    2012-11-15  518  	struct scatterlist *sg;
0104fdbb8 Chris Wilson    2012-11-15  519  
95a2e2be9 Chris Wilson    2016-11-16  520  	GEM_BUG_ON(offset > ggtt->stolen_size - size);
0104fdbb8 Chris Wilson    2012-11-15  521  
0104fdbb8 Chris Wilson    2012-11-15  522  	/* We hide that we have no struct page backing our stolen object
0104fdbb8 Chris Wilson    2012-11-15  523  	 * by wrapping the contiguous physical allocation with a fake
0104fdbb8 Chris Wilson    2012-11-15  524  	 * dma mapping in a single scatterlist.

:::::: The code at line 516 was first introduced by commit
:::::: 72e96d6450c067f58b65224bb5e73914e2cc43ab drm/i915: Refer to GGTT {,VM} consistently

:::::: TO: Joonas Lahtinen <joonas.lahtinen at linux.intel.com>
:::::: CC: Joonas Lahtinen <joonas.lahtinen at linux.intel.com>

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation
-------------- next part --------------
A non-text attachment was scrubbed...
Name: .config.gz
Type: application/gzip
Size: 25157 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/intel-gfx/attachments/20161117/6f938c34/attachment-0001.gz>


More information about the Intel-gfx mailing list