Re: [Intel-gfx] [PATCH v2] drm/i915: Make contexts non-snooped on non-LLC platforms

2014-04-04 Thread Daniel Vetter
On Fri, Apr 04, 2014 at 02:41:00PM +0100, Chris Wilson wrote: > On Fri, Apr 04, 2014 at 04:36:10PM +0300, ville.syrj...@linux.intel.com wrote: > > From: Ville Syrjälä > > > > We don't do CPU access to GPU contexts so making the GPU access snoop > > the CPU caches seems silly, and potentially expe

Re: [Intel-gfx] [PATCH v2] drm/i915: Make contexts non-snooped on non-LLC platforms

2014-04-04 Thread Chris Wilson
On Fri, Apr 04, 2014 at 04:36:10PM +0300, ville.syrj...@linux.intel.com wrote: > From: Ville Syrjälä > > We don't do CPU access to GPU contexts so making the GPU access snoop > the CPU caches seems silly, and potentially expensive. > > v2: Use !IS_VALLEYVIEW instead of HAS_LLC as this is really

[Intel-gfx] [PATCH v2] drm/i915: Make contexts non-snooped on non-LLC platforms

2014-04-04 Thread ville . syrjala
From: Ville Syrjälä We don't do CPU access to GPU contexts so making the GPU access snoop the CPU caches seems silly, and potentially expensive. v2: Use !IS_VALLEYVIEW instead of HAS_LLC as this is really about what the PTEs can represent. Add a comment clarifying the situation. Signed-