• Daniel Vetter's avatar
    drm/i915: prevent tiling changes on framebuffer backing storage · 80075d49
    Daniel Vetter authored
    Assuming that all framebuffer related metadata is invariant simplifies
    our userspace input data checking. And current userspace always first
    updates the tiling of an object before creating a framebuffer with it.
    
    This allows us to upconvert a check in pin_and_fence to a WARN.
    
    In the future it should also be helpful to know which buffer objects
    are potential scanout targets for e.g. frontbuffer rendering tracking
    and similar things.
    
    Note that SNA shipped for one prerelease with code which will be
    broken through this patch. But users shouldn't notice since it's
    purely an optimization and will transparently fall back to allocating
    a new fb. i-g-t also had offending code (now fixed), but we don't
    really care about breaking the test-suite.
    
    Cc: Ville Syrjälä <ville.syrjala@linux.intel.com>
    Reviewed-by: default avatarVille Syrjälä <ville.syrjala@linux.intel.com>
    Grumpily-reviewed-by: default avatarChris Wilson <chris@chris-wilson.co.uk>
    Signed-off-by: default avatarDaniel Vetter <daniel.vetter@ffwll.ch>
    80075d49
intel_display.c 304 KB