On Mon, Jan 23, 2017 at 1:42 AM, Kenneth Graunke <kenneth at whitecape.org> wrote: > + *ymin = MIN2(ndc_gb_ymin, ndc_gb_ymax); Should this be limited on the negative end of the range? > + *ymax = MIN2(MAX2(ndc_gb_ymin, ndc_gb_ymax), 16383); And should this be a different value for gen6? Perhaps based on gb_size?