On 07/30/2015 04:51 PM, Predut, Marius wrote: > Still not upstream. > > -----Original Message----- > From: Predut, Marius > Sent: Thursday, June 04, 2015 11:53 PM > To: [email protected] > Cc: Predut, Marius > Subject: [PATCH] aa: too sensitive at the line end points for PNV platform > > Adjust middles of the segments sample so that now the test pass also on PNV > platform. > > Bspec documentation G45: Volume 2: 3D/Media Last Pixel Enable bit of > SF_STATE controls whether the last pixel of the last line in a LINESTRIP_xxx > primitive or the last pixel of each line in a LINELIST_xxx primitive is > rendered. > > Signed-off-by: Marius Predut <[email protected]> > --- > tests/general/line-aa-width.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/tests/general/line-aa-width.c b/tests/general/line-aa-width.c > index c0fd7c1..e7998c4 100644 > --- a/tests/general/line-aa-width.c > +++ b/tests/general/line-aa-width.c > @@ -136,7 +136,7 @@ piglit_display(void) > * coverage in each column > */ > for (x1 = 2; x1 < piglit_width; x1 += seg_width) { > - int x2 = x1 + seg_width - 4; > + int x2 = x1 + seg_width - 5; > int sample_x; > float y1, y2; > float avg = 0.0; > @@ -144,8 +144,8 @@ piglit_display(void) > float max = -100.0; > char *err = NULL; > > - if (x2 > piglit_width - 4) > - x2 = piglit_width - 4; > + if (x2 > piglit_width - 5) > + x2 = piglit_width - 5; > > /* If we don't have a couple of pixels to sample because we've > * hit the edge of the window, we're done. > -- > 1.9.1 >
Looks fine, though I don't have available HW to confirm the test passes after applying corresponding Mesa patch. If somebody can confirm it, this patch is: Reviewed-By: Eduardo Lima Mitev <[email protected]> _______________________________________________ Piglit mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/piglit
