Attachment 4774 Details for Bug 6011 – Add rv280 workaround and limit vram to 128M again

[patch] Add rv280 workaround and limit vram to 128M again

radeon-more-mem-fix.diff (text/plain), 1.69 KB, created by Benjamin Herrenschmidt on 2006-02-28 16:45:53 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Benjamin Herrenschmidt

Created: 2006-02-28 16:45:53 UTC

Size: 1.69 KB

patch

obsolete

>diff -urN xf86-video-ati-COMMIT/src/radeon_driver.c xf86-video-ati/src/radeon_driver.c
>--- xf86-video-ati-COMMIT/src/radeon_driver.c	2006-02-26 21:01:32.000000000 +1100
>+++ xf86-video-ati/src/radeon_driver.c	2006-02-28 16:42:56.000000000 +1100
>@@ -1,5 +1,5 @@
> /* $XFree86: xc/programs/Xserver/hw/xfree86/drivers/ati/radeon_driver.c,v 1.117 2004/02/19 22:38:12 tsi Exp $ */
>-/* $XdotOrg: driver/xf86-video-ati/src/radeon_driver.c,v 1.88 2006/02/26 10:01:32 benh Exp $ */
>+/* $XdotOrg: driver/xf86-video-ati/src/radeon_driver.c,v 1.87 2006/02/25 01:57:05 sroland Exp $ */
> /*
>  * Copyright 2000 ATI Technologies Inc., Markham, Ontario, and
>  *                VA Linux Systems Inc., Fremont, California.
>@@ -2300,6 +2300,14 @@
> 	{
> 	    CARD32 aper0_base = INREG(RADEON_CONFIG_APER_0_BASE);
> 
>+	    /* The rv280 has an "issue" with it's memory controller, the
>+	     * location must be aligned to the size. We just align it down,
>+	     * too bad if we walk over the top of system memory, we don't
>+	     * use DMA without a remapped anyway
>+	     */
>+	    if (info->ChipFamily == CHIP_FAMILY_RV280)
>+		    aper0_base &= ~(mem_size - 1);
>+
> 	    info->mc_fb_location = (aper0_base >> 16) |
> 		    ((aper0_base + mem_size - 1) & 0xffff0000U);
> 	}
>@@ -2758,6 +2766,14 @@
> 
> 	/* Get accessible memory */
> 	accessible = RADEONGetAccessibleVRAM(pScrn);
>+
>+	/* XXX There are still some unresolved issues with card splitting
>+	 * over two PCI functions, so we have to limit accessible RAM to
>+	 * 128Mb for the time being
>+	 */
>+	if (accessible > 0x08000000)
>+	    accessible = 0x08000000;
>+
> 	xf86DrvMsg(pScrn->scrnIndex, X_INFO,
> 	       "Detected total video RAM: %dl, max accessible: %dK\n",
> 	       pScrn->videoRam, accessible);

Actions: View | Splinter Review

Attachments on bug 6011: 4729 | 4730 | 4732 | 4750 | 4751 | 4773 | 4774 | 4775

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.