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@xxxxxxxxxxxxxxx>
:::::: CC: Joonas Lahtinen <joonas.lahtinen@xxxxxxxxxxxxxxx>
---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation
Attachment:
.config.gz
Description: application/gzip
_______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
