[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