On Thu, Mar 13, 2008 at 10:09 PM,  <<a href="mailto:joe@smittys.pointclark.net">joe@smittys.pointclark.net</a>> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="Ih2E3d">Quoting Uwe Hermann <<a href="mailto:uwe@hermann-uwe.de">uwe@hermann-uwe.de</a>>:<br>
<br>
> On Thu, Mar 13, 2008 at 04:34:02PM -0400, <a href="mailto:joe@smittys.pointclark.net">joe@smittys.pointclark.net</a> wrote:<br>
>>> Compilation of rca:rm4100 is still broken<br>
>>> See the error log at<br>
>>> <a href="http://qa.coreboot.org/log_buildbrd.php?revision=3138&device=rm4100&vendor=rca" target="_blank">http://qa.coreboot.org/log_buildbrd.php?revision=3138&device=rm4100&vendor=rca</a><br>

>>><br>
>>><br>
>>> If something broke during this checkin please be a pain<br>
>>> in uwe's neck until the issue is fixed.<br>
>>><br>
>> Why is this failing?? The log looks like there is an error in<br>
>> rom_stream.c?? And it is in the "normal" directory? But I don't have a<br>
>> "normal" build just "fallback"?? Could this be one of the changes you<br>
>> made Uwe?<br>
><br>
> Don't think so. I've seen this before, I just don't remember what the<br>
> fix was. Will look into it.<br>
><br>
> The symptom is that -DPAYLOAD_SIZE='0x-17000' is passed somewhere, but I<br>
> haven't yet found out why exactly this happens and what the root cause is.<br>
><br>
</div>Yeh that is very strange. According to the log it is happening in the<br>
beginning, even before romcc.c, and it is the oppasite of<br>
-DROM_IMAGE_SIZE='0x17000' -DPAYLOAD_SIZE='0x-17000'<br>
<br>
Very strange. It built locally just fine for me....<br>
<br>
Thanks - Joe</blockquote><div><br>The problem is right here:<br><br>from src/rca/rm4100/Config.lb:<br>if USE_FALLBACK_IMAGE<br>    default ROM_SECTION_SIZE = FALLBACK_SIZE<br>    default ROM_SECTION_OFFSET = (ROM_SIZE - FALLBACK_SIZE)<br>
else<br>    default ROM_SECTION_SIZE = (ROM_SIZE - FALLBACK_SIZE)<br>    default ROM_SECTION_OFFSET = 0<br>end<br>default CONFIG_ROM_PAYLOAD_START = (0xffffffff - ROM_SIZE + ROM_SECTION_OFFSET + 1)<br>default PAYLOAD_SIZE = (ROM_SECTION_SIZE - ROM_IMAGE_SIZE)<br>
<br>and from src/rca/rm4100/Options.lb:<br>default ROM_SIZE = 512 * 1024<br>default ROM_IMAGE_SIZE = 128 * 1024<br>default HAVE_FALLBACK_BOOT = 1<br>default FALLBACK_SIZE = 512 * 1024<br> <br></div></div>abuild ignores the target/*/Config.lb and makes its own, with both a normal and a fallback image. Since the normal image has USE_FALLBACK_IMAGE = 0, it uses ROM_SECTION_SIZE = (ROM_SIZE - FALLBACK_SIZE), and since they're both 512k, it comes out to 0. PAYLOAD_SIZE is then (0 - ROM_IMAGE_SIZE). There are two possible fixes, the first to set FALLBACK_SIZE here to 256k and then set it to ROM_SIZE in target/'s <a href="http://Config.lb">Config.lb</a>, the second to create a <a href="http://Config-abuild.lb">Config-abuild.lb</a> with just the fallback image. I'm leaning towards the latter, because this system doesn't have a battery-backed CMOS, so it'd never boot a normal image anyways.<br>
<br>-Corey<br>