On 12/15/2014 10:24 AM, Daniel Vetter wrote:
On Wed, Dec 10, 2014 at 05:27:58PM +0000, Tvrtko Ursulin wrote:
diff --git a/drivers/gpu/drm/i915/i915_gem.c b/drivers/gpu/drm/i915/i915_gem.c
index de241eb..c6610bc 100644
--- a/drivers/gpu/drm/i915/i915_gem.c
+++ b/drivers/gpu/drm/i915/i915_gem.c
@@ -2098,8 +2098,7 @@ i915_gem_shrink(struct drm_i915_private *dev_priv,
/* For the unbound phase, this should be a no-op! */
list_for_each_entry_safe(vma, v,
&obj->vma_list, vma_link)
- if (i915_vma_unbind(vma))
- break;
+ i915_vma_unbind(vma);
This hunk adds a new warning. It's the 2nd time a __must_check warning
from someone from vpg london crept through all the premerge testing. Can
you please double-check whether you have some crazy gcc flags settings or
something in your builders?
I've dropped the hunk locally.
Yes sorry, somehow I've lost the relevant flag from Kconfig. Doesn't
matter for this hunk I suppose since it was in the air anyway.
Regards,
Tvrtko
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
