Bug 41372

Summary:	i830_state.c PBO crash
Product:	Mesa	Reporter:	Kurt Roeckx <kurt>
Component:	Drivers/DRI/i915	Assignee:	Ian Romanick <idr>
Status:	RESOLVED FIXED	QA Contact:
Severity:	normal
Priority:	medium
Version:	unspecified
Hardware:	Other
OS:	All
Whiteboard:
i915 platform:		i915 features:
Attachments:	i830_state.c:i830PolygonStipple(): Make polygon stipple use pre-unpacked pixel data.

Description Kurt Roeckx 2011-09-30 15:17:41 UTC

Created attachment 51814 [details] [review]
i830_state.c:i830PolygonStipple(): Make polygon stipple use pre-unpacked pixel data.

Hi,

The glean pbo tests crashes in case it gets a NULL pattern.  The attached patch does the same thing in i830_state.c as this one does for i915_state.c:
commit a856da63247a4b403f6350914f732e14d1530ed1
Author: Eric Anholt <eric@anholt.net>
Date:   Tue Dec 18 14:14:44 2007 -0800

    [915] Make polygon stipple use pre-unpacked pixel data.
    
    This fixes a crash when stippling using data from a PBO.

Comment 1 Eric Anholt 2012-03-02 18:31:20 UTC

Pushed your fix.  Thanks!

commit 489ac8e73a2432d29597a62cd83f09b9d1848533
Author: Kurt Roeckx <kurt@roeckx.be>
Date:   Fri Mar 2 17:31:23 2012 -0800

    i915: Fix i830 polygon stipple from PBOs.
    
    This is a direct port of the i915 patch in
    a856da63247a4b403f6350914f732e14d1530ed1.
    
    Fixes glean's pbo test.
    
    Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=41372
    Reviewed-by: Eric Anholt <eric@anholt.net>
    NOTE: This is a candidate for release branches.

(note: in the future, git format-patch -1 will produce your git commit formatted in a way that's nice and easy to apply)

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.