Re: CVS commit: src/sys

["Arnaud Lacombe" <lacombar%gmail.com@localhost>]

Hi,

On Wed, Dec 3, 2008 at 1:52 PM, Tobias Nygren <tnn%netbsd.org@localhost> wrote:
> On Sat, 29 Nov 2008 23:48:12 +0000 (UTC)
> Christos Zoulas <christos%netbsd.org@localhost> wrote:
>
>> Modified Files:
>>       src/sys/arch/x86/pci: pchb.c
>>       src/sys/dev/pci: agp.c agp_i810.c agpreg.h pcidevs
>>
>> Log Message:
>> Add support for the Intel G45 AGP. From Arnaud Lacombe
>
> Hi,
>
> This commit seems to have broken AGP on i945GME. X.org no longer starts
> for me. Reverting agp_i810.c to 1.57 makes it work again.
>
> Any ideas?
>
As it reworks large part for the code, this is the reason I didn't
commit it and waited for report/review. I'll do an explicit "call for
testing" next time :-) Sorry for the inconvenience.

> dmesg now says:
> agp_i810_bind_memory: PGTBL_CTL is 0x3ee7001 - fixing
>
> Xorg.0.log:
> (WW) intel(0): ESR is 0x00000010, page table error
> (WW) intel(0): PGTBL_ER is 0x00500000, CS instruction GTT PTE, BIN 
> instruction GTT PTE
> (WW) intel(0): PRB0_CTL (0x000ee001) indicates ring buffer enabled
> (WW) intel(0): PRB0_HEAD (0x00000000) and PRB0_TAIL (0x000eb000) indicate 
> ring buffer not flushed
> (WW) intel(0): Existing errors found in hardware state.
> Error in I830WaitLpRing(), timeout for 2 seconds
> pgetbl_ctl: 0x3ffc0001 getbl_err: 0x00500000
> ipeir: 0x00000000 iphdr: 0x00000000
> LP ring tail: 0x0000b008 head: 0x00000000 len: 0x000ee001 start 0x03eed000
> eir: 0x0010 esr: 0x0010 emr: 0xffc5
> instdone: 0xffc7 instpm: 0x0000
> memmode: 0x00000001 instps: 0x400f08c0
> hwstam: 0x5001 ier: 0x7001 imr: 0x9001 iir: 0x0020
>
I'll have a look.

 - Arnaud