Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 8 Jun 2008 23:57:47 +0200
From:      Roland Smith <rsmith@xs4all.nl>
To:        Robert Noland <rnoland@2hip.net>
Cc:        freebsd-x11@freebsd.org
Subject:   Re: 7-STABLE and Intel G33
Message-ID:  <20080608215747.GA1500@slackbox.xs4all.nl>
In-Reply-To: <1212937757.1471.23.camel@wombat.2hip.net>
References:  <20080608104234.GA73057@slackbox.xs4all.nl> <1212937757.1471.23.camel@wombat.2hip.net>

next in thread | previous in thread | raw e-mail | index | archive | help

--H+4ONPRPur6+Ovig
Content-Type: multipart/mixed; boundary="ReaqsoxgOBHFXBhH"
Content-Disposition: inline


--ReaqsoxgOBHFXBhH
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Sun, Jun 08, 2008 at 11:09:17AM -0400, Robert Noland wrote:
> On Sun, 2008-06-08 at 12:42 +0200, Roland Smith wrote:
> > My PC has built-in intel G33 graphics, which I'm trying to get to work
> > in something better then vesa.
> >=20
> > Following the instructions in
> > http://lists.freebsd.org/pipermail/freebsd-stable/2008-January/039638.h=
tml
> > I have compiled and installed the driver and kernel modules from the git
> > trees for drm and the xf86-video-intel driver from June 5th.
> >=20
> > I also patched agp_i810.c to remove the comments from the chipset
> > identifiers and rebuilt the kernel.
> >=20
> > After loading the i915.ko kernel module from the git tree, and starting
> > X with a config file using the intel driver, I still get;
> >=20
> >  (II) intel(0): xf86BindGARTMemory: bind key 1 at 0x006ff000 (pgoffset =
1791)
> >  (WW) intel(0): xf86BindGARTMemory: binding of gart memory with key 1
> >          at offset 0x6ff000 failed (Invalid argument)
> >=20
> >  Fatal server error:
> >  Couldn't bind memory for front buffer
> >=20
> > In dmesg output I see:
> >=20
> >  agp0: trying to bind into stolen memory

> I can try to look into it, but my time is a bit scattered for the next
> few weeks.  If I'm parsing Eric's analysis correctly, the attached patch
> along with the other items he pointed out, may do it.  Apply the patch
> to agp_i810.c and see if it makes a difference.

Unfortunately, the patch made no difference.

Eric said in above message: "The gtt_size detection is broken, though.
We should have gtt_size ... equal to zero on this hardware"

Your patch set the gtt_size to zero for the i965, not for the G33. So I
modified the file to set the gtt_size to 0 for the G33 instead. This did
have an effect. Instead of=20

agp0: <Intel G33 SVGA controller> on vgapci0
agp0: detected 7676k stolen memory
agp0: aperture size is 256M

I now get

agp0: <Intel G33 SVGA controller> on vgapci0
agp0: detected 8188k stolen memory
agp0: aperture size is 256M

Unfortunately the result is still the same:

(II) intel(0): xf86BindGARTMemory: bind key 1 at 0x006ff000 (pgoffset 1791)
(WW) intel(0): xf86BindGARTMemory: binding of gart memory with key 1
	at offset 0x6ff000 failed (Invalid argument)

Fatal server error:
Couldn't bind memory for front buffer

I'm attaching my patch in case you find it usefull.

Roland
--=20
R.F.Smith                                   http://www.xs4all.nl/~rsmith/
[plain text _non-HTML_ PGP/GnuPG encrypted/signed email much appreciated]
pgp: 1A2B 477F 9970 BA3C 2914  B7CE 1277 EFB0 C321 A725 (KeyID: C321A725)

--ReaqsoxgOBHFXBhH
Content-Type: text/plain; charset=us-ascii
Content-Disposition: attachment; filename=patch-RFS

--- /usr/src/sys/pci/agp_i810.c.orig	2008-03-20 22:13:14.000000000 +0100
+++ /usr/src/sys/pci/agp_i810.c	2008-06-08 23:16:56.000000000 +0200
@@ -156,14 +156,12 @@
 	    "Intel Q965 SVGA controller"},
 	{0x29a28086, CHIP_I965, 0x00020000,
 	    "Intel G965 SVGA controller"},
-/*
 	{0x29b28086, CHIP_G33, 0x00020000,
 	    "Intel Q35 SVGA controller"},
 	{0x29c28086, CHIP_G33, 0x00020000,
 	    "Intel G33 SVGA controller"},
 	{0x29d28086, CHIP_G33, 0x00020000,
 	    "Intel Q33 SVGA controller"},
-*/
 	{0x2a028086, CHIP_I965, 0x00020000,
 	    "Intel GM965 SVGA controller"},
 	{0x2a128086, CHIP_I965, 0x00020000,
@@ -492,8 +490,10 @@
 		case CHIP_I915:
 			gtt_size = 256;
 			break;
-		case CHIP_I965:
 		case CHIP_G33:
+			gtt_size = 0;
+			break;
+		case CHIP_I965:
 			switch (bus_read_4(sc->sc_res[0], AGP_I810_PGTBL_CTL) &
 			    AGP_I810_PGTBL_SIZE_MASK) {
 			case AGP_I810_PGTBL_SIZE_128KB:

--ReaqsoxgOBHFXBhH--

--H+4ONPRPur6+Ovig
Content-Type: application/pgp-signature
Content-Disposition: inline

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.9 (FreeBSD)

iEYEARECAAYFAkhMVdsACgkQEnfvsMMhpyV9RwCglJw39eqAPPUwNqOfU9E7O3j3
JeYAoJBxjKb7ysY6fbB4BuilEqtsnwof
=NOdg
-----END PGP SIGNATURE-----

--H+4ONPRPur6+Ovig--



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20080608215747.GA1500>