lkml.org 
[lkml]   [2013]   [May]   [17]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    Date
    From
    Subject[ 113/136 ] drm/i915: Use MLC (l3$) for context objects
    3.6.11.4 stable review patch.
    If anyone has any objections, please let me know.

    ------------------

    From: Chris Wilson <chris@chris-wilson.co.uk>

    [ Upstream commit 4615d4c9e27eda42c3e965f208a4b4065841498c ]

    Enabling context support increases SwapBuffers latency by about 20%
    (measured on an i7-3720qm). We can offset that loss slightly by enabling
    faster caching for the contexts. As they are not backed by any
    particular cache (such as the sampler or render caches) our only option
    is to select the generic mid-level cache. This reduces the latency of
    the swap by about 5%.

    Oddly this effect can be observed running smokin-guns on IVB at
    1280x1024:
    Using BLT copies for swaps: 151.67 fps
    Using Render copies for swaps (unpatched): 141.70 fps
    With contexts disabled: 150.23 fps
    With contexts in L3$: 150.77 fps

    Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
    Cc: Ben Widawsky <ben@bwidawsk.net>
    Cc: Kenneth Graunke <kenneth@whitecape.org>
    Reviewed-by: Kenneth Graunke <kenneth@whitecape.org>
    Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
    Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
    ---
    drivers/gpu/drm/i915/i915_gem_context.c | 7 +++++++
    1 file changed, 7 insertions(+)

    diff --git a/drivers/gpu/drm/i915/i915_gem_context.c b/drivers/gpu/drm/i915/i915_gem_context.c
    index a9d58d7..dda0d35 100644
    --- a/drivers/gpu/drm/i915/i915_gem_context.c
    +++ b/drivers/gpu/drm/i915/i915_gem_context.c
    @@ -155,6 +155,13 @@ create_hw_context(struct drm_device *dev,
    return ERR_PTR(-ENOMEM);
    }

    + if (INTEL_INFO(dev)->gen >= 7) {
    + ret = i915_gem_object_set_cache_level(ctx->obj,
    + I915_CACHE_LLC_MLC);
    + if (ret)
    + goto err_out;
    + }
    +
    /* The ring associated with the context object is handled by the normal
    * object tracking code. We give an initial ring value simple to pass an
    * assertion in the context switch code.
    --
    1.7.10.4



    \
     
     \ /
      Last update: 2013-05-18 05:21    [W:4.265 / U:0.440 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site