Hi Gabriel, On 31 August 2017 at 07:18, Gabriel Krisman Bertazi <krisman@xxxxxxxxxxxxxxx> wrote: > Two scenarios tested: > - unaligned stripe > - Stripe too small 'stride' in the commit message please. ;) But it is fine everywhere through the code. > @@ -323,7 +326,14 @@ static void generate_fb(data_t *data, struct igt_fb *fb, > */ > int ccs_width = ALIGN(width * 4, 32) / 32; > int ccs_height = ALIGN(height, 16) / 16; > - f.pitches[1] = ALIGN(ccs_width * 1, 128); > + int aux_stride = ALIGN(ccs_width * 1, 128); > + > + if (fb_flags & FB_MISALIGN_AUX_STRIDE) > + aux_stride = ccs_width; aux_stride -= 64 perhaps? > + else if (fb_flags & FB_SMALL_AUX_STRIDE) > + aux_stride = ALIGN(ccs_width/2, 128); This one seems OK, but maybe want to skip it in the unlikely case that (w <= 1024), since that already has the smallest possible valid stride at 128. Cheers, Daniel _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
