[PATCH] only try to find radeon IGP ROM on X86
David Miller
davem at davemloft.net
Thu Oct 10 19:02:01 CEST 2013
From: Meelis Roos <mroos at linux.ee>
Date: Thu, 10 Oct 2013 18:26:39 +0300 (EEST)
>> On Thu, Oct 10, 2013 at 7:51 AM, Meelis Roos <mroos at linux.ee> wrote:
>> > To prevent hangs on non-PC machines (e.g. sparc64), probe Radeon ROM
>> > from ATI IGP only on X86. Fixes hang in this place and allows PCI radeon
>> > detection to move on to next problem.
>>
>> NACK. All this function does it attempt to read the rom from the
>> framebuffer PCI BAR. If you get hangs reading the BAR, then you have
>> larger problems on your platform. Also, there are non-x86 platforms
>> that IGP chips which this may break.
>
> OK. But the code seems to just dereference addresses returned from
> ioremap, but to the best of my knowledge, these are not universally
> (like on each arch) safe to read without readb/readw/readl. Should I
> convert these to readb for test?
That's correct, ioremap() returned pointers must not be directly
dereferenced, and must be accessed through the appropriate accessors.
More information about the dri-devel
mailing list