Re: pkg/35416: agp driver hangs on ALi/ULi M1689 chipset

[Gary Duzan <gary%duzan.org@localhost>]

The following reply was made to PR kern/35416; it has been noted by GNATS.

From: Gary Duzan <gary%duzan.org@localhost>
To: Nicolas Joly <njoly%pasteur.fr@localhost>
Cc: gnats-bugs%NetBSD.org@localhost, kern-bug-people%NetBSD.org@localhost, 
gnats-admin%NetBSD.org@localhost,
        netbsd-bugs%NetBSD.org@localhost
Subject: Re: pkg/35416: agp driver hangs on ALi/ULi M1689 chipset 
Date: Sun, 06 Apr 2008 11:50:04 -0400

 In Message <20080405131758.GA476761%medusa.sis.pasteur.fr@localhost> ,
    Nicolas Joly <njoly%pasteur.fr@localhost> wrote:
 
 =>On Sat, Apr 05, 2008 at 11:45:01AM +0000, Martin Husemann wrote:
 =>>  On Sat, Apr 05, 2008 at 07:23:40AM -0400, Gary Duzan wrote:
 =>>  >    I believe so. However, the problem isn't the size, it is that
 =>>  > the wrong driver attaches, and I assume the aperture size is being
 =>>  > pulled from the wrong place.
 =>>  
 =>>  The check for zero size should be added there, however the right fix is
 =>>  to add an agp_match function in agp.c:
 =>[...]
 =>
 =>Can you try this patch ?
 
    It certainly looks better:
 
 agp0 at pchb1: 1 Miscellaneous Control unit(s) found.
 agp0: aperture at 0xdc000000, size 0x4000000
 
 Unfortunately, I can't test the functionality since I also have
 a jmide issue which is keeping me from completing a boot. I'll
 work on that a bit more and post my findings on current-users
 and/or send a PR.
 
    Thanks.
 
                                        Gary Duzan