On Thu, Jan 15, 2015 at 08:36:15PM +0100, Daniel Vetter wrote: > On Wed, Jan 14, 2015 at 9:34 PM, Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> wrote: > > This (partially) reverts > > > > commit 5537252b6b6d71fb1a8ed7395a8e5babf91953fd > > Author: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> > > Date: Tue Mar 25 13:23:06 2014 +0000 > > > > drm/i915: Invalidate our pages under memory pressure > > Shouldn't we also revert the hunk in i915_gem_free_objects? Without > the truncate vs. invalidate disdinction it seems to have lost it's > reason for existence ... No, setting MADV_DONTNEED has other nice properties during put_pages() - I think it is useful in its own right, for example that is where my page stealing code goes... -Chris -- Chris Wilson, Intel Open Source Technology Centre -- To unsubscribe from this list: send the line "unsubscribe stable" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html