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 [email protected] http://lists.freedesktop.org/mailman/listinfo/intel-gfx
