On Fri, Feb 24, 2017 at 5:04 AM, i1w5d7gf38keg@tutanota.com wrote:
Yes, this die() is what i mean. Try to get maybe some functionality is i think better then just stopping there and providing zero functionality.
I also know, that there is a message when the CPUID is not known. Its about the die() afterwards.
Patches always welcome. Did you try removing the die() and seeing if things actually booted?
- Feb 2017 14:31 by coreboot@coreboot.org:
On Thu, Feb 23, 2017 at 2:39 AM, Nico Huber nico.h@gmx.de wrote:
On 23.02.2017 00:07, i1w5d7gf38keg@tutanota.com wrote:
There is a Filter to stop booting when the CPUID is not in a list of supported CPUs. This filter does not make sense in the real world usage.
It's not a filter. It's a measure to know which code to run for which CPU. Please dig a little deeper before making such useless complaints.
To add to Nico's point: the cpuid list is a way to bind code code to run for certain devices -- including CPUs. If the cpuid is not listed then the match on device->code to run is not met. Therefore, the code necessary to make that CPU work won't ever be ran. src/arch/x86/cpu.c has the cpu driver binding. And there already is message printed. See the callers of set_cpu_ops() in that file. The issue is that we die() when no match is found. We could attempt to boot further, but there's no guarantee it'd actually succeed.
Nico
-- coreboot mailing list: coreboot@coreboot.org https://www.coreboot.org/mailman/listinfo/coreboot
-- coreboot mailing list: coreboot@coreboot.org https://www.coreboot.org/mailman/listinfo/coreboot