[PATCH] radeonfb: If no video memory, exit with error [repost]

STALE7954d

3 messages, 2 authors, 2004-10-27 · open the first message on its own page

[PATCH] radeonfb: If no video memory, exit with error [repost]

From: Bjorn Helgaas <hidden>
Date: 2004-10-26 22:03:49

Posted this last week (10/21) but haven't seen any response.
Would you consider this for the next -mm?  Also attached in
case kmail mangles the whitespace.


[PATCH] radeonfb: If no video memory, exit with error

Nothing good will happen if we try to ioremap and use a zero-sized
frame buffer.  I observed this problem on an ia64 sx1000 box, where
the BIOS doesn't run the option ROM.  If we try to continue, radeonfb
just gets hopelessly confused because the card isn't initialized
correctly.

Signed-off-by: Bjorn Helgaas <redacted>

===== drivers/video/aty/radeon_base.c 1.32 vs edited =====
--- 1.32/drivers/video/aty/radeon_base.c 2004-10-19 03:40:34 -06:00
+++ edited/drivers/video/aty/radeon_base.c 2004-10-21 11:50:51 -06:00
@@ -2186,7 +2186,9 @@
           rinfo->video_ram = 8192 * 1024;
           break;
          default:
-          break;
+   printk (KERN_ERR "radeonfb: no video RAM reported\n");
+   ret = -ENXIO;
+   goto err_unmap_rom;
   }
  }
 

Re: [PATCH] radeonfb: If no video memory, exit with error [repost]

From: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Date: 2004-10-26 23:54:47

Andrew, looks ok.

On Tue, 2004-10-26 at 16:03 -0600, Bjorn Helgaas wrote:
quoted hunk
Posted this last week (10/21) but haven't seen any response.
Would you consider this for the next -mm?  Also attached in
case kmail mangles the whitespace.


[PATCH] radeonfb: If no video memory, exit with error

Nothing good will happen if we try to ioremap and use a zero-sized
frame buffer.  I observed this problem on an ia64 sx1000 box, where
the BIOS doesn't run the option ROM.  If we try to continue, radeonfb
just gets hopelessly confused because the card isn't initialized
correctly.

Signed-off-by: Bjorn Helgaas <redacted>

===== drivers/video/aty/radeon_base.c 1.32 vs edited =====
--- 1.32/drivers/video/aty/radeon_base.c 2004-10-19 03:40:34 -06:00
+++ edited/drivers/video/aty/radeon_base.c 2004-10-21 11:50:51 -06:00
@@ -2186,7 +2186,9 @@
           rinfo->video_ram = 8192 * 1024;
           break;
          default:
-          break;
+   printk (KERN_ERR "radeonfb: no video RAM reported\n");
+   ret = -ENXIO;
+   goto err_unmap_rom;
   }
  }
 
-- 
Benjamin Herrenschmidt [off-list ref]



-------------------------------------------------------
This SF.Net email is sponsored by:
Sybase ASE Linux Express Edition - download now for FREE
LinuxWorld Reader's Choice Award Winner for best database on Linux.
http://ads.osdn.com/?ad_id=5588&alloc_id=12065&op=click

Re: [Linux-fbdev-devel] [PATCH] radeonfb: If no video memory, exit with error [repost]

From: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Date: 2004-10-27 09:47:45

On Tue, 2004-10-26 at 16:03 -0600, Bjorn Helgaas wrote:
Posted this last week (10/21) but haven't seen any response.
Would you consider this for the next -mm?  Also attached in
case kmail mangles the whitespace.
I did tell Andrew it was OK this morning.

Ben.




-------------------------------------------------------
This SF.Net email is sponsored by:
Sybase ASE Linux Express Edition - download now for FREE
LinuxWorld Reader's Choice Award Winner for best database on Linux.
http://ads.osdn.com/?ad_id=5588&alloc_id=12065&op=click
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help