<div dir="ltr">For those of us working on boards that don't ship in a product, the die() is probably not appropriate. But  if you intend to ship a real product then you definitely want to die() if someone tries to use a CPU that's not tested on the board. <div><br></div><div>Or at least that's the way it seems to me.</div></div><br><div class="gmail_quote"><div dir="ltr">On Fri, Feb 24, 2017 at 7:42 AM Aaron Durbin via coreboot <<a href="mailto:coreboot@coreboot.org">coreboot@coreboot.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Fri, Feb 24, 2017 at 5:04 AM,  <<a href="mailto:i1w5d7gf38keg@tutanota.com" class="gmail_msg" target="_blank">i1w5d7gf38keg@tutanota.com</a>> wrote:<br class="gmail_msg">
> Yes, this die() is what i mean. Try to get maybe some functionality is i<br class="gmail_msg">
> think better then just stopping there and providing zero functionality.<br class="gmail_msg">
><br class="gmail_msg">
> I also know, that there is a message when the CPUID is not known. Its about<br class="gmail_msg">
> the die() afterwards.<br class="gmail_msg">
<br class="gmail_msg">
Patches always welcome. Did you try removing the die() and seeing if<br class="gmail_msg">
things actually booted?<br class="gmail_msg">
<br class="gmail_msg">
><br class="gmail_msg">
><br class="gmail_msg">
> 23. Feb 2017 14:31 by <a href="mailto:coreboot@coreboot.org" class="gmail_msg" target="_blank">coreboot@coreboot.org</a>:<br class="gmail_msg">
><br class="gmail_msg">
> On Thu, Feb 23, 2017 at 2:39 AM, Nico Huber <<a href="mailto:nico.h@gmx.de" class="gmail_msg" target="_blank">nico.h@gmx.de</a>> wrote:<br class="gmail_msg">
><br class="gmail_msg">
> On 23.02.2017 00:07, <a href="mailto:i1w5d7gf38keg@tutanota.com" class="gmail_msg" target="_blank">i1w5d7gf38keg@tutanota.com</a> wrote:<br class="gmail_msg">
><br class="gmail_msg">
> There is a Filter to stop booting when the CPUID is not in a list of<br class="gmail_msg">
> supported CPUs. This filter does not make sense in the real world usage.<br class="gmail_msg">
><br class="gmail_msg">
><br class="gmail_msg">
> It's not a filter. It's a measure to know which code to run for which<br class="gmail_msg">
> CPU. Please dig a little deeper before making such useless complaints.<br class="gmail_msg">
><br class="gmail_msg">
><br class="gmail_msg">
> To add to Nico's point: the cpuid list is a way to bind code code to<br class="gmail_msg">
> run for certain devices -- including CPUs. If the cpuid is not listed<br class="gmail_msg">
> then the match on device->code to run is not met. Therefore, the code<br class="gmail_msg">
> necessary to make that CPU work won't ever be ran. src/arch/x86/cpu.c<br class="gmail_msg">
> has the cpu driver binding. And there already is message printed. See<br class="gmail_msg">
> the callers of set_cpu_ops() in that file. The issue is that we die()<br class="gmail_msg">
> when no match is found. We could attempt to boot further, but there's<br class="gmail_msg">
> no guarantee it'd actually succeed.<br class="gmail_msg">
><br class="gmail_msg">
><br class="gmail_msg">
> Nico<br class="gmail_msg">
><br class="gmail_msg">
> --<br class="gmail_msg">
> coreboot mailing list: <a href="mailto:coreboot@coreboot.org" class="gmail_msg" target="_blank">coreboot@coreboot.org</a><br class="gmail_msg">
> <a href="https://www.coreboot.org/mailman/listinfo/coreboot" rel="noreferrer" class="gmail_msg" target="_blank">https://www.coreboot.org/mailman/listinfo/coreboot</a><br class="gmail_msg">
><br class="gmail_msg">
><br class="gmail_msg">
> --<br class="gmail_msg">
> coreboot mailing list: <a href="mailto:coreboot@coreboot.org" class="gmail_msg" target="_blank">coreboot@coreboot.org</a><br class="gmail_msg">
> <a href="https://www.coreboot.org/mailman/listinfo/coreboot" rel="noreferrer" class="gmail_msg" target="_blank">https://www.coreboot.org/mailman/listinfo/coreboot</a><br class="gmail_msg">
<br class="gmail_msg">
--<br class="gmail_msg">
coreboot mailing list: <a href="mailto:coreboot@coreboot.org" class="gmail_msg" target="_blank">coreboot@coreboot.org</a><br class="gmail_msg">
<a href="https://www.coreboot.org/mailman/listinfo/coreboot" rel="noreferrer" class="gmail_msg" target="_blank">https://www.coreboot.org/mailman/listinfo/coreboot</a><br class="gmail_msg">
</blockquote></div>