dri/i915: Fix off-by-one in i830 clip region size.
authorAlban Browaeys <prahal@yahoo.com>
Thu, 2 Feb 2012 18:20:22 +0000 (19:20 +0100)
committerEric Anholt <eric@anholt.net>
Sat, 3 Mar 2012 01:16:12 +0000 (17:16 -0800)
commit7d13a6e64bf88566875a8f68e0aac9b937e30feb
tree6c5774408ce75b4c12901d20a0239b1c8040093c
parentae92180d724f9b743a088050cdc4ff7dad20c28b
dri/i915: Fix off-by-one in i830 clip region size.

The hardware, like i915, uses an inclusive bounds on min and max for
the drawing rectangle, but we were providing a number for exclusive.
The number of bits used by the hardware only covers this value going
up to the maximum size, so when we programmed 2048 as the maximum
inclusive X, it saw a maximum X of 0 and clipped all rendering.  This
caused rendering failures in gnome-shell.

Fixes piglit fbo-maxsize.

v2: dropped changes to the blitter, which does use an exclusive x2, y2.
    [change by anholt]

Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=45558
Reviewed-by: Eric Anholt <eric@anholt.net>
NOTE: This is a candidate for release branches.
src/mesa/drivers/dri/i915/i830_vtbl.c