• Chris Wilson's avatar
    drm/i915: Use spinlocks for checking when to waitboost · 8d3afd7d
    Chris Wilson authored
    In commit 1854d5ca
    Author: Chris Wilson <chris@chris-wilson.co.uk>
    Date:   Tue Apr 7 16:20:32 2015 +0100
    
        drm/i915: Deminish contribution of wait-boosting from clients
    
    we removed an atomic timer based check for allowing waitboosting and
    moved it below the mutex taken during RPS. However, that mutex can be
    held for long periods of time on Vallyview/Cherryview as communication
    with the PCU is slow. As clients may frequently wait for results (e.g.
    such as tranform feedback) we introduced contention between the client
    and the RPS worker. We can take advantage of the RPS worker, by
    switching the wait boost decision to use spin locks and defer the
    actual reclocking to the worker.
    
    Fixes a regression of up to 45% on Baytrail and Baswell!
    
    v2 (Daniel):
    - Use max_freq_softlimit instead of the not-yet-merged boost
      frequency.
    - Don't inject a fake irq into the boost work, instead treat
      client_boost as just another legit waker.
    
    v3: Drop the now unused mask (Chris).
    
    Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=90112
    Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> (v1)
    Signed-off-by: default avatarDaniel Vetter <daniel.vetter@ffwll.ch>
    8d3afd7d
i915_irq.c 127 KB