[Bug] bsp/cvitek/cv18xx_aarch64 无法 boot 进入控制台界面

[unicornx]

RT-Thread Version

master

Hardware Type/Architectures

bsp/cvitek/cv18xx_aarch64

Develop Toolchain

GCC

Describe the bug

目前 master 上的代码 duo 的 aarch64 大核无法启动进入控制台界面，只能打印出 RT flag 就 hold 住了。

bisect 看了一下，最近一个可以 work 的 commit 是 ae6a328 Add psoc62、61 config
加入 754c59a [Feature] DFS mount auto by kernel parameters (#8989) 后就会出现该现象。

继续跟踪了一下，发现卡住是因为 components/drivers/core/mnt.c 文件中的 rootfs_mnt_init() 进入了死循环。死循环位置大致在下面的 for 中

static int rootfs_mnt_init(void)
{
......
        for (; rootdelay || rootwait; --timeout)
        {
            if (!rootwait && timeout == 0)
            {
                LOG_E("Wait for /dev/%s init time out", dev);

                /*
                 * We don't return at once because the device driver may init OK
                 * when we break from this point, might as well give it another
                 * try.
                 */
                break;
            }

            if (rt_device_find(dev))
            {
                break;
            }

            rt_thread_mdelay(1);
        }
......
}
INIT_ENV_EXPORT(rootfs_mnt_init);

这个函数，包括整个文件都是 754c59a[Feature] DFS mount auto by kernel parameters (#8989) 新引入的。所以怀疑是引入整个 feature 后 bsp/cvitek/cv18xx_aarch 需要做一些对应的改动但没有做导致。

所以想问问 754c59a 的作者 @GuEe-GUI 是否可以提供一些帮助？

Other additional context

No response

[GuEe-GUI]

设备树启动参数没有 rootwait，这里是不会死等的：

rt-thread/components/drivers/core/mnt.c

Lines 73 to 86 in 8dc166b

	rootwait = bootargs_select("rootwait", 0);
	/* Maybe it is undefined or 'rootwaitABC' */
	if (!rootwait || *rootwait)
	{
	rootdelay = bootargs_select("rootdelay=", 0);
	if (rootdelay)
	{
	timeout = rt_tick_from_millisecond(atoi(rootdelay));
	}
	rootwait = RT_NULL;
	}

[unicornx]

@liYony FYI, 是不是设备树上有什么要改动？

[unicornx]

设备树启动参数没有 rootwait，这里是不会死等的：

rt-thread/components/drivers/core/mnt.c

Lines 73 to 86 in 8dc166b

	rootwait = bootargs_select("rootwait", 0);
	/* Maybe it is undefined or 'rootwaitABC' */
	if (!rootwait || *rootwait)
	{
	rootdelay = bootargs_select("rootdelay=", 0);
	if (rootdelay)
	{
	timeout = rt_tick_from_millisecond(atoi(rootdelay));
	}
	rootwait = RT_NULL;
	}

@GuEe-GUI 请问您的意思是我们的 DTS 里少了 rootwait 这个参数吗，这些参数是必需的吗？

另外有关 rootfs_mnt_init·() 函数的代码我有个疑问。

我发现由于我们提供的 dtb 中没有给出 rootwait 和 rootdelay，

rootwait = bootargs_select("rootwait", 0);

这个调用会返回一个空字符串(rootwait 的值也不是 NULL)。 这导致 for 循环中无法满足 break 的条件。

请问这是不是一个bug？

[GuEe-GUI]

设备树启动参数没有 rootwait，这里是不会死等的：

rt-thread/components/drivers/core/mnt.c

Lines 73 to 86 in 8dc166b

	rootwait = bootargs_select("rootwait", 0);
	/* Maybe it is undefined or 'rootwaitABC' */
	if (!rootwait || *rootwait)
	{
	rootdelay = bootargs_select("rootdelay=", 0);
	if (rootdelay)
	{
	timeout = rt_tick_from_millisecond(atoi(rootdelay));
	}
	rootwait = RT_NULL;
	}

@GuEe-GUI 请问您的意思是我们的 DTS 里少了 rootwait 这个参数吗，这些参数是必需的吗？

另外有关 rootfs_mnt_init·() 函数的代码我有个疑问。

我发现由于我们提供的 dtb 中没有给出 rootwait 和 rootdelay，

rootwait = bootargs_select("rootwait", 0);

这个调用会返回一个空字符串(rootwait 的值也不是 NULL)。 这导致 for 循环中无法满足 break 的条件。

请问这是不是一个bug？

如果 bootargs 没有这个参数，依然返回了了非 NULL 的空字符串，那就是有 BUG，不妨把这个字符串打印出来看看。

[unicornx]

设备树启动参数没有 rootwait，这里是不会死等的：

rt-thread/components/drivers/core/mnt.c

Lines 73 to 86 in 8dc166b

	rootwait = bootargs_select("rootwait", 0);
	/* Maybe it is undefined or 'rootwaitABC' */
	if (!rootwait || *rootwait)
	{
	rootdelay = bootargs_select("rootdelay=", 0);
	if (rootdelay)
	{
	timeout = rt_tick_from_millisecond(atoi(rootdelay));
	}
	rootwait = RT_NULL;
	}

@GuEe-GUI 请问您的意思是我们的 DTS 里少了 rootwait 这个参数吗，这些参数是必需的吗？
另外有关 rootfs_mnt_init·() 函数的代码我有个疑问。
我发现由于我们提供的 dtb 中没有给出 rootwait 和 rootdelay，

rootwait = bootargs_select("rootwait", 0);

这个调用会返回一个空字符串(rootwait 的值也不是 NULL)。 这导致 for 循环中无法满足 break 的条件。
请问这是不是一个bug？

如果 bootargs 没有这个参数，依然返回了了非 NULL 的空字符串，那就是有 BUG，不妨把这个字符串打印出来看看。

打印出来是空的，rootwait 的值也打印了，和初始值（目前随机）不同，感觉是分配了一个内存了，但内容为空。

[unicornx]

sg2002_milkv_duo256m_arm_sd.txt

上传反汇编的 dts 文件，txt 后缀改成 dts 即可

@GuEe-GUI 供参考

[GuEe-GUI]

设备树启动参数没有 rootwait，这里是不会死等的：

rt-thread/components/drivers/core/mnt.c

Lines 73 to 86 in 8dc166b

	rootwait = bootargs_select("rootwait", 0);
	/* Maybe it is undefined or 'rootwaitABC' */
	if (!rootwait || *rootwait)
	{
	rootdelay = bootargs_select("rootdelay=", 0);
	if (rootdelay)
	{
	timeout = rt_tick_from_millisecond(atoi(rootdelay));
	}
	rootwait = RT_NULL;
	}

@GuEe-GUI 请问您的意思是我们的 DTS 里少了 rootwait 这个参数吗，这些参数是必需的吗？
另外有关 rootfs_mnt_init·() 函数的代码我有个疑问。
我发现由于我们提供的 dtb 中没有给出 rootwait 和 rootdelay，

rootwait = bootargs_select("rootwait", 0);

这个调用会返回一个空字符串(rootwait 的值也不是 NULL)。 这导致 for 循环中无法满足 break 的条件。
请问这是不是一个bug？

如果 bootargs 没有这个参数，依然返回了了非 NULL 的空字符串，那就是有 BUG，不妨把这个字符串打印出来看看。

打印出来是空的，rootwait 的值也打印了，和初始值（目前随机）不同，感觉是分配了一个内存了，但内容为空。

抱歉，没描述清楚，因为 rootwait 后面已经没有字符串了（如果 bootargs 有这个参数的话），那 *rootwait 就是一个开头为 '\0' 的字符串，也就是说，只要 select 的 key 存在，返回值就肯定不是空，而是这个 key 后面的字符串（空格之前的）。

[GuEe-GUI]

sg2002_milkv_duo256m_arm_sd.txt

上传反汇编的 dts 文件，txt 后缀改成 dts 即可

@GuEe-GUI 供参考

请问这个是 Bootloader 运行时修改的设备树还是在 Linux source 下构建的设备树？

[unicornx]

sg2002_milkv_duo256m_arm_sd.txt
上传反汇编的 dts 文件，txt 后缀改成 dts 即可
@GuEe-GUI 供参考

请问这个是 Bootloader 运行时修改的设备树还是在 Linux source 下构建的设备树？

@liYony 请回复一下。谢谢

[unicornx]

抱歉，没描述清楚，因为 rootwait 后面已经没有字符串了（如果 bootargs 有这个参数的话），那 *rootwait 就是一个开头为 '\0' 的字符串，也就是说，只要 select 的 key 存在，返回值就肯定不是空，而是这个 key 后面的字符串（空格之前的）。

没看懂您这里说的意思，我们的 dts 里没有 bootargs 呢。

[GuEe-GUI]

sg2002_milkv_duo256m_arm_sd.txt
上传反汇编的 dts 文件，txt 后缀改成 dts 即可
@GuEe-GUI 供参考

请问这个是 Bootloader 运行时修改的设备树还是在 Linux source 下构建的设备树？

@liYony 请回复一下。谢谢

最直接的做法，启动后把 bootargs 都打印出来就可以，如果串口可以早期打印：

    const char *bootargs;
    struct rt_ofw_node *ofw_chosen_node = rt_ofw_find_node_by_path("/chosen");

    if (!rt_ofw_prop_read_string(ofw_chosen_node, "bootargs", &bootargs))
    {
        rt_kputs("Command line: ");
        // 可能会很长被截断，就不使用 printf 了
        rt_kputs(bootargs);
        rt_kputs("\n");
    }

[unicornx]

sg2002_milkv_duo256m_arm_sd.txt
上传反汇编的 dts 文件，txt 后缀改成 dts 即可
@GuEe-GUI 供参考

请问这个是 Bootloader 运行时修改的设备树还是在 Linux source 下构建的设备树？

@liYony 请回复一下。谢谢

最直接的做法，启动后把 bootargs 都打印出来就可以，如果串口可以早期打印：

    const char *bootargs;
    struct rt_ofw_node *ofw_chosen_node = rt_ofw_find_node_by_path("/chosen");

    if (!rt_ofw_prop_read_string(ofw_chosen_node, "bootargs", &bootargs))
    {
        rt_kputs("Command line: ");
        // 可能会很长被截断，就不使用 printf 了
        rt_kputs(bootargs);
        rt_kputs("\n");
    }

@GuEe-GUI 我把这段代码就加在 rootfs_mnt_init 的入口处得到如下打印：

 \ | /
- RT -     Thread Operating System
 / | \     5.2.0 build Jul 24 2024 17:04:11
 2006 - 2024 Copyright by RT-Thread team
Command line: root=/dev/mmcblk0p2 rootwait rw console=ttyS0,115200 earlycon=sbi riscv.fwsz=0x80000 loglevel=9