Re: [Intel-gfx] [PATCH 2/2 v2] drm/i915: mark GEM objects dirty after overwriting their contents

2015-12-11 Thread Daniel Vetter
On Thu, Dec 10, 2015 at 02:52:57PM +, Chris Wilson wrote: > On Thu, Dec 10, 2015 at 03:06:55PM +0100, Daniel Vetter wrote: > > On Wed, Dec 09, 2015 at 03:52:52PM +, Dave Gordon wrote: > > > In a few places, we fill a GEM object with data, or overwrite some > > > portion of its contents othe

Re: [Intel-gfx] [PATCH 2/2 v2] drm/i915: mark GEM objects dirty after overwriting their contents

2015-12-10 Thread Dave Gordon
On 10/12/15 14:06, Daniel Vetter wrote: On Wed, Dec 09, 2015 at 03:52:52PM +, Dave Gordon wrote: In a few places, we fill a GEM object with data, or overwrite some portion of its contents other than a single page. In such cases, we should mark the object dirty so that its pages in the pageca

Re: [Intel-gfx] [PATCH 2/2 v2] drm/i915: mark GEM objects dirty after overwriting their contents

2015-12-10 Thread Chris Wilson
On Thu, Dec 10, 2015 at 03:06:55PM +0100, Daniel Vetter wrote: > On Wed, Dec 09, 2015 at 03:52:52PM +, Dave Gordon wrote: > > In a few places, we fill a GEM object with data, or overwrite some > > portion of its contents other than a single page. In such cases, we > > should mark the object dir

Re: [Intel-gfx] [PATCH 2/2 v2] drm/i915: mark GEM objects dirty after overwriting their contents

2015-12-10 Thread Daniel Vetter
On Wed, Dec 09, 2015 at 03:52:52PM +, Dave Gordon wrote: > In a few places, we fill a GEM object with data, or overwrite some > portion of its contents other than a single page. In such cases, we > should mark the object dirty so that its pages in the pagecache are > written to backing store (r

Re: [Intel-gfx] [PATCH 2/2 v2] drm/i915: mark GEM objects dirty after overwriting their contents

2015-12-10 Thread Chris Wilson
On Wed, Dec 09, 2015 at 03:52:52PM +, Dave Gordon wrote: > In a few places, we fill a GEM object with data, or overwrite some > portion of its contents other than a single page. In such cases, we > should mark the object dirty so that its pages in the pagecache are > written to backing store (r

[Intel-gfx] [PATCH 2/2 v2] drm/i915: mark GEM objects dirty after overwriting their contents

2015-12-09 Thread Dave Gordon
In a few places, we fill a GEM object with data, or overwrite some portion of its contents other than a single page. In such cases, we should mark the object dirty so that its pages in the pagecache are written to backing store (rather than discarded) if the object is evicted due to memory pressure