Bug 14440

Have you changed any bios setting?

G33 issue is not relevant here.

Could you try to test by revert this commit or change it by hand?
(ignore G33 line above it, which was reverted later.) I don't know
if I had something to confuse Dave, I'll check the doc any way.

commit e67aa27a6179c287983c6c525beb5320f5cd1672
Author: Dave Airlie <airlied@linux.ie>
Date:   Tue Sep 18 22:46:35 2007 -0700

    intel-agp: Fix i830 mask variable that changed with G33 support

    The mask on i830 should be 0x70 always, later chips 0xF0 should be okay.

    Signed-off-by: Dave Airlie <airlied@linux.ie>
    Acked-by: Zhenyu Wang <zhenyu.z.wang@intel.com>
    Cc: Michael Haas <laga@laga.ath.cx>
    Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
    Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>

diff --git a/drivers/char/agp/intel-agp.c b/drivers/char/agp/intel-agp.c
index 7c69bf2..a5d0e95 100644
--- a/drivers/char/agp/intel-agp.c
+++ b/drivers/char/agp/intel-agp.c
@@ -511,7 +511,7 @@ static void intel_i830_init_gtt_entries(void)
                 */
                if (IS_G33)
                        size = 0;
-               switch (gmch_ctrl & I830_GMCH_GMS_MASK) {
+               switch (gmch_ctrl & I855_GMCH_GMS_MASK) {
                case I855_GMCH_GMS_STOLEN_1M:
                        gtt_entries = MB(1) - KB(size);
                        break;

I changed in back to
                }
        } else {
                switch (gmch_ctrl & I830_GMCH_GMS_MASK) {
                case I855_GMCH_GMS_STOLEN_1M:
                        gtt_entries = MB(1) - KB(size);

but this does not help. X does not start and the error message is still the same. 

I did not change any BIOS settings.

So it seems not an xf86-video-intel issue, but a kernel issue. Anyway, we can continue to track it here.

(In reply to comment #2)
> I changed in back to
>                 }
>         } else {
>                 switch (gmch_ctrl & I830_GMCH_GMS_MASK) {
>                 case I855_GMCH_GMS_STOLEN_1M:
>                         gtt_entries = MB(1) - KB(size);
> 
> but this does not help. X does not start and the error message is still the
> same. 

yeah, I misread the code, that line doesn't apply to 830M chipset.

So the problem seems with 830M stolen memory size mask, with the I830_GMCH_GMS_MASK change in 2.6.24, kernel agpgart now should detect real stolen mem size which fixed the origin check error. But this change hasn't been applied to intel video driver yet.

I don't have 830M GMCH data sheet by hand, will double-check later. Could you try below patch against xf86-video-intel driver?

Created attachment 14308 [details] [review]
830m stolen mem mask fix

I applied the patch to the current debian sources (version 2.2.0.90-3). The patch fixes the problem. X now starts again without an error message! Thank you very much!

Could you attach your current fine X log? And could you help me to check your bios setting to see if your stolen memory size config is 8M really?

I'd like to check with other chips to fully fix this stolen size detect problem. Thanks.

Created attachment 14325 [details]
BIOS unchanged

The BIOS settings were unchanged (factory default) for this config.

Created attachment 14326 [details]
BIOS settings

Created attachment 14327 [details]
Xorg log when setting "VGA fram buffer size" to 8 M

ok, thanks for the info. 

The patch has been pushed upstream.