On Wed, 13 Nov 2013 21:29:23 +0100 Jean-Sébastien Pédron wrote:
> Le 10/11/2013 18:20, d...@gmx.com a écrit :
>> drmn0: info: GTT: 0M 0xF0000000 - 0xEFFFFFFF
> 
> Tijl Coosemans is right, the problem is this line.
> 
> As I don't really now how AGP works and have no AGP hardware to 
> reproduce the problem, can you post the output of the following commands 
> as a start?
>      pciconf -lvbce
>      devinfo -vr

The attached patch should fix it, but I haven't been able to test it
yet.  The ai_aperture_size field is in bytes.
Index: sys/dev/drm2/radeon/radeon_agp.c
===================================================================
--- sys/dev/drm2/radeon/radeon_agp.c	(revision 258128)
+++ sys/dev/drm2/radeon/radeon_agp.c	(working copy)
@@ -153,11 +153,11 @@ int radeon_agp_init(struct radeon_device
 		return ret;
 	}
 
-	if (rdev->ddev->agp->info.ai_aperture_size < 32) {
+	if (rdev->ddev->agp->info.ai_aperture_size < (32 << 20)) {
 		drm_agp_release(rdev->ddev);
 		dev_warn(rdev->dev, "AGP aperture too small (%zuM) "
 			"need at least 32M, disabling AGP\n",
-			rdev->ddev->agp->info.ai_aperture_size);
+			rdev->ddev->agp->info.ai_aperture_size >> 20);
 		return -EINVAL;
 	}
 
@@ -246,7 +246,7 @@ int radeon_agp_init(struct radeon_device
 	}
 
 	rdev->mc.agp_base = rdev->ddev->agp->info.ai_aperture_base;
-	rdev->mc.gtt_size = rdev->ddev->agp->info.ai_aperture_size << 20;
+	rdev->mc.gtt_size = rdev->ddev->agp->info.ai_aperture_size;
 	rdev->mc.gtt_start = rdev->mc.agp_base;
 	rdev->mc.gtt_end = rdev->mc.gtt_start + rdev->mc.gtt_size - 1;
 	dev_info(rdev->dev, "GTT: %juM 0x%08jX - 0x%08jX\n",
_______________________________________________
freebsd-current@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-current
To unsubscribe, send any mail to "freebsd-current-unsubscr...@freebsd.org"

Reply via email to