When another discrete video card(SM750 or AST1400) is mounted into the mini
PCIe slot of my ASRock AD2550B-ITX board, the gma500 drivers fails to work.
It probably because the UEFI firmware of that board forget to initialize
the PSB_PGETBL_CTL reg, therefore the value of dev_priv->pge_ctl is 0, then
the value of gtt_phys_start is also 0.
On normal case, the value of the dev_priv->stolen_base is 0xbf800000 for
this board, so the value of the vram_stolen_size will be negative in this
case, the calculation of the stolen vram size of drm/gma500 is pasted at
below:
vram_stolen_size = pg->gtt_phys_start - dev_priv->stolen_base - PAGE_SIZE;
= 0 - 0xbf800000 - 4096 = 1056764 KiB
Which is so large, so this cause the ioremap_wc() fail, see below dmesg for
more information.
gma500 0000:00:02.0: enabling device (0000 -> 0003)
gma500 0000:00:02.0: GPU: power management timed out.
gma500 0000:00:02.0: [drm] Phys start of GTT: 0x0
gma500 0000:00:02.0: [drm] Stolen memory base 0xbf800000, size 1056764KiB
x86/PAT: systemd-udevd:386 conflicting memory types bf800000-fffff000 write-combining<->uncached-minus
x86/PAT: memtype_reserve failed [mem 0xbf800000-0xffffefff], track write-combining, req write-combining
ioremap memtype_reserve failed -16
gma500 0000:00:02.0: Failure to map stolen base.
gma500: probe of 0000:00:02.0 failed with error -12
Regardless, we want this driver works even there have another video card
mounted. This patch solve this problem by given 8M stolen memory if the
value of pg->gtt_phys_start is zero. And after apply this patch, it works
fine.
$ dmesg | grep drm
gma500 0000:00:02.0: [drm] Phys start of GTT: 0x0
gma500 0000:00:02.0: [drm] Stolen memory base 0xbf800000, size 8192KiB
[drm] Initialized gma500 1.0.0 20140314 for 0000:00:02.0 on minor 0
gma500 0000:00:02.0: [drm] fb1: gma500drmfb frame buffer device
Sui Jingfeng (3):
drm/gma500: Print the stolen memory base and size with drm_info()
drm/gma500: Print the start address of the GTT
drm/gma500: Fix the failure to map the stolen memory
drivers/gpu/drm/gma500/gem.c | 10 +++++++---
drivers/gpu/drm/gma500/gtt.c | 2 ++
2 files changed, 9 insertions(+), 3 deletions(-)
base-commit: bb585492db95d4cc7fe3797523ed2bbf5c621d37
--
2.34.1
while at it, also replace the 'K' with 'KiB'.
Signed-off-by: Sui Jingfeng <[email protected]>
---
drivers/gpu/drm/gma500/gem.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/gma500/gem.c b/drivers/gpu/drm/gma500/gem.c
index 4b7627a72637..6fe78f61e127 100644
--- a/drivers/gpu/drm/gma500/gem.c
+++ b/drivers/gpu/drm/gma500/gem.c
@@ -344,8 +344,8 @@ int psb_gem_mm_init(struct drm_device *dev)
stolen_size = vram_stolen_size;
- dev_dbg(dev->dev, "Stolen memory base 0x%x, size %luK\n",
- dev_priv->stolen_base, vram_stolen_size / 1024);
+ drm_info(dev, "Stolen memory base 0x%x, size %luKiB\n",
+ dev_priv->stolen_base, vram_stolen_size / 1024);
pg->stolen_size = stolen_size;
dev_priv->vram_stolen_size = vram_stolen_size;
--
2.34.1
When another discrete video card(SM750 or AST1400) is mounted into the mini
PCIe slot of my ASRock AD2550B-ITX board, the gma500 drivers fails to work.
It probably because the UEFI firmware of that board forget to initialize
the PSB_PGETBL_CTL reg, therefore the value of dev_priv->pge_ctl is 0, then
the value of gtt_phys_start is also 0.
On normal case, the value of the dev_priv->stolen_base is 0xbf800000 for
this board, so the value of the vram_stolen_size will be negative in this
case, the calculation of the stolen vram size of drm/gma500 is pasted at
below:
vram_stolen_size = pg->gtt_phys_start - dev_priv->stolen_base - PAGE_SIZE;
= 0 - 0xbf800000 - 4096 = 1056764 KiB
Which is so large, so this cause the ioremap_wc() fail, see below dmesg for
more information.
gma500 0000:00:02.0: enabling device (0000 -> 0003)
gma500 0000:00:02.0: GPU: power management timed out.
gma500 0000:00:02.0: [drm] Phys start of GTT: 0x0
gma500 0000:00:02.0: [drm] Stolen memory base 0xbf800000, size 1056764KiB
x86/PAT: systemd-udevd:386 conflicting memory types bf800000-fffff000 write-combining<->uncached-minus
x86/PAT: memtype_reserve failed [mem 0xbf800000-0xffffefff], track write-combining, req write-combining
ioremap memtype_reserve failed -16
gma500 0000:00:02.0: Failure to map stolen base.
gma500: probe of 0000:00:02.0 failed with error -12
Regardless, we want this driver works even there have another video card
mounted. This patch solve this problem by given 8M stolen memory if the
value of pg->gtt_phys_start is zero. And after apply this patch, it works
fine.
$ dmesg | grep drm
gma500 0000:00:02.0: [drm] Phys start of GTT: 0x0
gma500 0000:00:02.0: [drm] Stolen memory base 0xbf800000, size 8192KiB
[drm] Initialized gma500 1.0.0 20140314 for 0000:00:02.0 on minor 0
gma500 0000:00:02.0: [drm] fb1: gma500drmfb frame buffer device
Signed-off-by: Sui Jingfeng <[email protected]>
---
drivers/gpu/drm/gma500/gem.c | 6 +++++-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/gma500/gem.c b/drivers/gpu/drm/gma500/gem.c
index 6fe78f61e127..0e9971bb24fa 100644
--- a/drivers/gpu/drm/gma500/gem.c
+++ b/drivers/gpu/drm/gma500/gem.c
@@ -340,7 +340,11 @@ int psb_gem_mm_init(struct drm_device *dev)
pg = &dev_priv->gtt;
pci_read_config_dword(pdev, PSB_BSM, &dev_priv->stolen_base);
- vram_stolen_size = pg->gtt_phys_start - dev_priv->stolen_base - PAGE_SIZE;
+
+ if (pg->gtt_phys_start)
+ vram_stolen_size = pg->gtt_phys_start - dev_priv->stolen_base - PAGE_SIZE;
+ else
+ vram_stolen_size = 8 * 1024 * 1024;
stolen_size = vram_stolen_size;
--
2.34.1