[coreboot] [PATCH]The resource allocator does not like zero-sized fixed resources

Myles Watson mylesgw at gmail.com
Wed Aug 19 19:59:20 CEST 2009


On Wed, Aug 19, 2009 at 11:32 AM, Patrick Georgi<patrick at georgi-clan.de> wrote:
> Am 19.08.2009 19:08, schrieb Myles Watson:
>>
>> The example you gave in src/devices/pci_device.c was for the old way of
>> specifying PCI ROM locations.  Now that CBFS is going mainstream, that
>> should disappear.  If it doesn't, I think it should have a size.  Even 1
>> would work, since no Option ROM can be smaller than that.
>>
>
> I'd love to get rid of the rom_address thing once we drop the pre-cbfs way
> of doing things (which will take quite some time, still), but I'm not sure
> if there isn't some need for it left.
OK.

>> What other fixed resources have 0 size, especially on Kontron?  I'd like
>> to
>> fix the root cause.
>>
>
> That's the only one on Kontron. I just wanted to have proper behaviour even
> with such weird entries.
Let's add something like this patch.  I'm happy to have the wording
changed, but I think it's important that we don't silently fail to
avoid fixed resources.

Signed-off-by: Myles Watson <mylesgw at gmail.com>
Thanks,
Myles
-------------- next part --------------
A non-text attachment was scrubbed...
Name: zero_fixed.diff
Type: text/x-patch
Size: 1224 bytes
Desc: not available
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20090819/b0b72f99/attachment.diff>


More information about the coreboot mailing list