Bug 20417 - [GM965] DRI broken by commit "Implement front buffer resize for KMS" on 2.6 branch
Summary: [GM965] DRI broken by commit "Implement front buffer resize for KMS" on 2.6 b...
Status: RESOLVED FIXED
Alias: None
Product: xorg
Classification: Unclassified
Component: Driver/intel (show other bugs)
Version: git
Hardware: x86-64 (AMD64) Linux (All)
: highest critical
Assignee: Eric Anholt
QA Contact: Xorg Project Team
URL:
Whiteboard:
Keywords: regression
Depends on:
Blocks:
 
Reported: 2009-03-01 20:18 UTC by David L.
Modified: 2009-03-02 12:25 UTC (History)
2 users (show)

See Also:
i915 platform:
i915 features:

Attachments

Description David L. 2009-03-01 20:18:29 UTC 
on the xf86-video-intel !STABLE! (2.6) branch, direct rendering on i965GM is broken by

commit 009085d314a1c2999e1408c9627e74201591fe68
Author: Kristian Høgsberg krh redhat com
Date:   Fri Jan 30 17:53:03 2009 -0500

    Implement front buffer resize for KMS.
    
    This adds back the resize hook so we can resize the front buffer under
    kernel mode setting as well.

    The patch also pulls the drmmode_* structs from drmmode_display.h into
    drmmode_display.c and eliminates the header file.
    
    (cherry picked from commit 0cb87ccfe97b0e016e47dcf236fd5ce78dddfc4b)


subsequently, the X server hangs on creation of a large pixmap (which probably is another bug in some less-tested codepath used only when DRI is disabled)

a working driver can be constructed by reverting the following patches:
36f3136... KMS: Hook up rotated shadow buffers.
cdcca95... Don't disable vga centering bit.
5a86b78... TV quirk for HP Compaq nx6310
79b0070... Fix front buffer memset() for non-KMS case.
133dd16... Un-revert the I915_SETPARAM_NUM_USED_FENCES commit reverted by accident.
d349f36... Fix last-minute "cleanup" that broke the patch.
009085d... Implement front buffer resize for KMS.

none of these patches fixes the problem.

more information available on request of course, just need to know what to add (no outstanding error messages...)

GPU is 00:02.0 VGA compatible controller [0300]: Intel Corporation Mobile GM965/GL960 Integrated Graphics Controller [8086:2a02] (rev 03) (prog-if 00 [VGA controller])
        Subsystem: Micro-Star International Co., Ltd. Device [1462:334b]


mesa is 5199774dc6a2a65d24ac26fe15715654f2568a02 (7.5 pre-gallium merge)
drm is a6dd0afa87558a670f970e61b023f45a396539eb (2.4.5)
kernel is 2.6.29-rc6-wl-dirty
Comment 1 David L. 2009-03-01 20:44:46 UTC 
same problem on git master, can be fixed by reverting

> 5018d0f... KMS: Hook up rotated shadow buffers.
> 5c37009... Fix front buffer memset() for non-KMS case.
> 2013799... Un-revert the I915_SETPARAM_NUM_USED_FENCES commit reverted by
> accident.
> 127330b... Fix last-minute "cleanup" that broke the patch.
> 0cb87cc... Implement front buffer resize for KMS.
> 

mesa is 8b8e954f9e67357b87dac487c838a01fa991d0f1 (7.5 HEAD --disable-gallium)
drm is a6dd0afa87558a670f970e61b023f45a396539eb (2.4.5)
kernel is 2.6.29-rc6-wl-dirty
Comment 2 Eric Anholt 2009-03-02 12:25:45 UTC 
commit 5441be42649e4f969ac16c323de2fb5ed93b271a
Author: Eric Anholt <eric@anholt.net>
Date:   Tue Feb 24 20:54:05 2009 -0800

    Disable fb resizing for DRI1-only server so that DRI1 can initialize.
    (cherry picked from commit 70e0261208654c6c875ad462da2734c6aa9eeb96)

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.