[Intel-gfx] [PATCH v2] drm/i915: Combine write_domain flushes to a single function

Chris Wilson chris at chris-wilson.co.uk
Wed Apr 12 10:23:13 UTC 2017


On Wed, Apr 12, 2017 at 01:06:04PM +0300, Joonas Lahtinen wrote:
> On ke, 2017-04-12 at 10:42 +0100, Chris Wilson wrote:
> > In the next patch, we will introduce a new cache domain for
> > differentiating between GTT access and direct WC access. This will
> > require us to include WC in our write_domain flushes. Rather than
> > duplicate a third function, combine the existing two into one and
> > flushing WC writes will then be automatically handled as well.
> > 
> > v2: Be smarter and clearer by passing in the write domains to flush (Joonas)
> > 
> > Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
> > Cc: Joonas Lahtinen <joonas.lahtinen at linux.intel.com>
> 
> <SNIP>
> 
> > @@ -266,7 +266,7 @@ static int check_partial_mapping(struct drm_i915_gem_object *obj,
> >  		if (offset >= obj->base.size)
> >  			continue;
> >  
> > -		i915_gem_object_flush_gtt_write_domain(obj);
> > +		flush_write_domain(obj, I915_GEM_DOMAIN_CPU);
> 
> Forgot the tilde, with that;

Thanks. I even tried to proofread it to spot what I had missed. I should
just go and busy myself running some tests for the rest of morning.
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre


More information about the Intel-gfx mailing list