关于多核启动的hartid获取的讨论

[hzcx998]

一般情况下，都是SBI执行完成后跳转到内核执行，此时a0会保存hartid，也就是OpenSBI->kernel。
但是，在u740中，其它启动流程是SPL->OpenSBI->u-boot->kernel。因此，现在跳转到内核的是u-boot,那么他就可能不会给a0保存hartid。那么在内核中依然使用hartid去获取，这样不就会出错吗？

[TianhuaTao]

I haven't thought about this before. Yet no problem has occurred so far.
As far as I know, it's a convention for risc-v kernels to assume hartid in a0, and boot parameters (e.g., device tree pointer) in a1. And I believe this convention is followed by Linux, opensbi, and very probably u-boot as well. I believe a0 holds hartid unless it's documented otherwise.
I'm not familiar with the u-boot code, but this may be relevant (not very sure):
https://github.com/u-boot/u-boot/blob/f7bd9e4936b6e36c2443b9b2ef761e7593511521/arch/riscv/cpu/start.S#L441
Please check and correct me if I am wrong.

[hzcx998]

对于其它核，应该是直接从opensbi跳转过来的，因为在启动其它核的时候指定了跳转地址为kernel，可以正常通过a0获取到hartid。但是对于引导核，这个可能就不太确定了，得研究u-boot才行，不过一般都是使用hart0，应该问题不大，可以照常使用，后面有机会我再细细研究一下这个地方。

[TianhuaTao]

For the first core to boot, it's observed that it will have random hartid.

[hzcx998]

是的，我之前调试qemu的时候发现有这个问题。

[hzcx998]

我发现一个比较好的方式，就是在第一个核启动的时候用sbi的功能去获取当前启动的hartid，就可以说明它是当前的hartid了。通过这种方式就可以兼容u-boot引导的内核了。

int __GetBootHartid(int a0)
{
#if defined(CONFIG_NX_PLATFORM_HIFIVE_UNMACHED)
    int i;
    for (i = 0; i < 5; i++)
    {
        if (sbi_hsm_hart_status(i) == SBI_HSM_STATUS_STARTED)
        {
            return i;
        }
    }
#endif
    return a0;
}

[TianhuaTao]

你说的方法是可行的。不过现在的实现有问题吗？你是在担心 u-boot 可能不在 a0 保存 hartid 的情况吗？

[hzcx998]

是这样的，我担心的是uboot跳转进入kernel的时候，没有设置a0为hartid。而且，通过我的测试，发现跳转后，hartid为0，但实际上为1或者2。当然，也可能和我的启动方式有关。我是通过uboot的bootelf启动的。

[TianhuaTao]

通过我的测试，发现跳转后，hartid为0，但实际上为1或者2

这个情况我不清楚是怎么回事，我也不知道和 bootelf 有没有关系，如果确定有这个问题的话，你提的方法或许可以解决。另外是bootelf还是bootefi？我注意到有一些相关讨论：
riscvarchive/riscv-acpi#5

如果确实有必要在不使用 a0 的情况下获得 hartid，我建议加入device tree的解析功能。目前的kernel实现没有考虑这些复杂的功能。

[hzcx998]

是bootelf，就是我的kernel.elf，通过tftp下载到内存地址，通过这个命令他会自动去解析并跳转运行。
的确，设备树解析的方式更加优雅，但是现在还没到那种程度，我也是写小内核，后续会考虑试下。