On Wed, Jan 04, 2017 at 05:36:22PM -0800, Ben Widawsky wrote:
> On 17-01-04 09:51:20, Topi Pohjolainen Topi Pohjolainen wrote:
> > On Mon, Jan 02, 2017 at 06:37:13PM -0800, Ben Widawsky wrote:
> > > v2: Leave "image+mod" (Topi)
> > >
> > > Signed-off-by: Ben Widawsky <b...@bwidawsk.net>
> > > Acked-by: Daniel Stone <dani...@collabora.com>
> > > ---
> > > src/mesa/drivers/dri/i965/intel_screen.c | 33
> > > ++++++++++++++++++++++++++++----
> > > 1 file changed, 29 insertions(+), 4 deletions(-)
> > >
> > > diff --git a/src/mesa/drivers/dri/i965/intel_screen.c
> > > b/src/mesa/drivers/dri/i965/intel_screen.c
> > > index 153542c1d1..805de5b461 100644
> > > --- a/src/mesa/drivers/dri/i965/intel_screen.c
> > > +++ b/src/mesa/drivers/dri/i965/intel_screen.c
> > > @@ -549,6 +549,7 @@ create_image_with_modifier(struct intel_screen
> > > *screen,
> > > uint32_t requested_tiling = 0, tiling = 0;
> > > unsigned long pitch;
> > > unsigned tiled_height = 0;
> > > + unsigned ccs_height = 0;
> >
> > You could add a short note into the commit message that ccs_height is left
> > zero, and that it will be hooked in later. Patch title makes one believe
> > that
> > something gets effective. Either way:
> >
> > Reviewed-by: Topi Pohjolainen <topi.pohjolai...@intel.com>
> >
>
> Yeah. This patch changed over time and the commit message doesn't make sense.
> How is this?
>
> i965: Add logic for allocating BO with CCS
>
> This patch provides the support (and comments) for allocating the BO with
> space for the CCS buffer just underneath it.
>
> This patch was originally titled:
> "i965: Create correctly sized mcs for an image"
>
> In order to make things more bisectable, reviewable, and to have the
> CCS_MODIFIER token saved for the last patch, this patch now does less so
> it was renamed.
>
> v2: Leave "image+mod" (Topi)
>
> Signed-off-by: Ben Widawsky <b...@bwidawsk.net>
> Acked-by: Daniel Stone <dani...@collabora.com>
> Reviewed-by: Topi Pohjolainen <topi.pohjolai...@intel.com>
Looks good. Thanks!
>
>
> > >
> > > switch (modifier) {
> > > case I915_FORMAT_MOD_Y_TILED:
> > > @@ -566,10 +567,33 @@ create_image_with_modifier(struct intel_screen
> > > *screen,
> > > /* For now, all modifiers require some tiling */
> > > assert(tiling);
> > >
> > > + /*
> > > + * CCS width is always going to be less than or equal to the image's
> > > width.
> > > + * All we need to do is make sure we add extra rows (height) for the
> > > CCS.
> > > + *
> > > + * A pair of CCS bits correspond to 8x4 pixels, and must be cacheline
> > > + * granularity. Each CCS tile is laid out in 8b strips, which
> > > corresponds to
> > > + * 1024x512 pixel region. In memory, it looks like the following:
> > > + *
> > > + * ?????????????????????????????????????????????????????????
> > > + * ??? ???
> > > + * ??? ???
> > > + * ??? ???
> > > + * ??? Image ???
> > > + * ??? ???
> > > + * ??? ???
> > > + * ???xxxxxxxxxxxxxxxxx???
> > > + * ?????????????????????????????????????????????????????????
> > > + * ??? ??? |
> > > + * ???ccs ??? unused |
> > > + * ?????????????????????-----------???
> > > + * <------pitch------>
> > > + */
> > > cpp = _mesa_get_format_bytes(image->format);
> > > - image->bo = drm_intel_bo_alloc_tiled(screen->bufmgr, "image+mod",
> > > - width, tiled_height, cpp,
> > > &tiling,
> > > - &pitch, 0);
> > > + image->bo = drm_intel_bo_alloc_tiled(screen->bufmgr,
> > > + ccs_height ? "image+ccs" :
> > > "image+mod",
> > > + width, tiled_height + ccs_height,
> > > + cpp, &tiling, &pitch, 0);
> > > if (image->bo == NULL)
> > > return false;
> > >
> > > @@ -587,7 +611,8 @@ create_image_with_modifier(struct intel_screen
> > > *screen,
> > > if (image->planar_format)
> > > assert(image->planar_format->nplanes == 1);
> > >
> > > - image->aux_offset = 0; /* y_tiled_height * pitch; */
> > > + if (ccs_height)
> > > + image->aux_offset = tiled_height * pitch /* + mt->offset */;
> > >
> > > return true;
> > > }
> > > --
> > > 2.11.0
> > >
> > > _______________________________________________
> > > mesa-dev mailing list
> > > mesa-dev@lists.freedesktop.org
> > > https://lists.freedesktop.org/mailman/listinfo/mesa-dev
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
