[Intel-gfx] [PATCH 08/12] drm/i915: Add CCS capability for sprites

[Ville Syrjälä]

On Mon, Dec 11, 2017 at 12:33:47PM +0000, Daniel Stone wrote:
> Hi,
> 
> On 11 December 2017 at 12:08, Mika Kahola <mika.kahola at intel.com> wrote:
> > On Mon, 2017-12-11 at 12:00 +0000, Daniel Stone wrote:
> >> Did you manage to test this? When I tried, the DDB/watermark
> >> allocation was too conservative for sprites, and never allowed enough
> >> blocks to be able to use anything but linear or X-tiled on sprite
> >> planes. The allocation was almost entirely taken up by the primary
> >> plane, even if that itself was later set to linear or X-tiled.
> >
> > I tried to test this but unfortunately this didn't apply on top of drm-
> > tip. The whole series dates back to August so a lot have changed since
> > then.
> >
> > I would need this patch for my CNL work so I would like to see this
> > merged.
> 
> I would like to see this merged, but I would also like to see it ever
> working first. :) When I tried it at the time, even a 256x256 plane
> could not use Y-tiling or CCS, due to DDB exhaustion.

It worked just fine when I tried it a few months ago.

22:45 < vsyrjala> my skl seems perfectly happy here with two 2560x1080
planes at same time scanning out yccs
22:47 < danvet> I think daniels had a huge screen or something like that
22:47 < vsyrjala> let me try my 4k
22:54 < vsyrjala> ok, so now i hit a wm limit
23:01 < vsyrjala> so two 3k x 2x planes both scanning out yccs works
23:02 < vsyrjala> 3.5k x 2k fails

-- 
Ville Syrjälä
Intel OTC