[bug][v4.1.0-beta] IIC 初始化出现断言 Function[rt_sem_take] shall not be used before scheduler start

[dongly]

• 重现步骤:
  • 版本 rt-thread v4.1.0-beta
  • 启用IIC
  • 测试BSP stm32f407-atk-explorer
  • 编译器 GCC v10.2.1
• 错误信息

Function[rt_sem_take] shall not be used before scheduler start
(0) assertion failed at function:rt_sem_take, line number:489

• BUG定位
  • 跟踪到出错点在

    rt-thread/components/drivers/i2c/i2c_core.c

    Line 33 in a0fadd4

    LOG_I("I2C bus [%s] registered", bus_name);

  • 注释该行后正常

[dongly]

分析代码,以下代码也会出错

• rt-thread/bsp/at32/Libraries/rt_drivers/drv_soft_i2c.c

  Line 209 in a0fadd4

  LOG_D("software simulation %s init done, pin scl: %d, pin sda %d",

• rt-thread/bsp/stm32/libraries/HAL_Drivers/drv_soft_i2c.c

  Line 211 in a0fadd4

  LOG_D("software simulation %s init done, pin scl: %d, pin sda %d",

• rt-thread/bsp/gd32/libraries/gd32_drivers/drv_soft_i2c.c

  Line 191 in a0fadd4

  LOG_D("software simulation %s init done, pin scl: %d, pin sda %d",

• 其他 BSP 没看

[mysterywolf]

最近爆出了好多这种断言，看来内核增强安全检查是正确的
https://club.rt-thread.org/ask/question/434519.html
https://club.rt-thread.org/ask/question/434558.html
https://gitee.com/rtthread/rt-thread/issues/I4RVIU

[dongly]

建议开启GCC 的编译参数"-W",开启后有不少的警告,有潜在的风险.而lwIP,即使开启"-Wall",一个警告也没有

[mysterywolf]

#4740 (comment)

[mysterywolf]

#5576

[mysterywolf]

https://club.rt-thread.org/ask/article/3374.html

[Guozhanxin]

没有复现这个问题。。。

[mysterywolf]

@Guozhanxin 问题的关键不是iic，是mem，出出在这里已经定位到问题的原因了#4740 (comment) ， iic出现断言只不过是个表象，看上面的链接，还有LED初始化的时候出现断言的。

[Guozhanxin]

我发现是没开ulog的问题，所以没复现，修了一个了。#5589

[thewon86]

最近爆出了好多这种断言，看来内核增强安全检查是正确的 https://club.rt-thread.org/ask/question/434519.html https://club.rt-thread.org/ask/question/434558.html https://gitee.com/rtthread/rt-thread/issues/I4RVIU

增加的那些检测，明面上告诉大家别这么用，但是，最终 rtt 带头违规使用，然后用 rt_thread_self() != RT_NULL 打补丁去纠正违规行为。
这个问题，我很久之前就考虑过，论坛上的第二篇文章。

个人理解，ulog 和 malloc 的问题都可以通过调整系统初始化流程得到完美解决。

[Guozhanxin]

最近爆出了好多这种断言，看来内核增强安全检查是正确的 https://club.rt-thread.org/ask/question/434519.html https://club.rt-thread.org/ask/question/434558.html https://gitee.com/rtthread/rt-thread/issues/I4RVIU

增加的那些检测，明面上告诉大家别这么用，但是，最终 rtt 带头违规使用，然后用 rt_thread_self() != RT_NULL 打补丁去纠正违规行为。 这个问题，我很久之前就考虑过，论坛上的第二篇文章。

个人理解，ulog 和 malloc 的问题都可以通过调整系统初始化流程得到完美解决。

问题确实是启动流程的问题，但是由idle线程初始化串口和堆，感觉不太好弄，也不好兼容之前的版本

[mysterywolf]

内核这块如果有大改动的话得动smart合并进来之后才能有大的改动，现在内核已经进入冻结状态了，尽量保持少量修改。否则并入smart会导致冲突。

[thewon86]

最近爆出了好多这种断言，看来内核增强安全检查是正确的 https://club.rt-thread.org/ask/question/434519.html https://club.rt-thread.org/ask/question/434558.html https://gitee.com/rtthread/rt-thread/issues/I4RVIU

增加的那些检测，明面上告诉大家别这么用，但是，最终 rtt 带头违规使用，然后用 rt_thread_self() != RT_NULL 打补丁去纠正违规行为。 这个问题，我很久之前就考虑过，论坛上的第二篇文章。
个人理解，ulog 和 malloc 的问题都可以通过调整系统初始化流程得到完美解决。

问题确实是启动流程的问题，但是由idle线程初始化串口和堆，感觉不太好弄，也不好兼容之前的版本

idle 可以只负责启动 main timer 俩线程，线程栈还是可以控制在某个范围内。
包括 board 初始化在内的绝大多数初始化过程都可以在 main 线程进行，这要求 main 线程栈要够用。
保证 idle 线程运行起来的要求很低，基本上时钟配置好了之后，堆，串口控制台，pin 这些都可以不要求的。用最少的资源把系统跑起来，其它的硬件资源初始化控制在线程上下文中进行初始化。这样可以减少很多麻烦。

兼容性问题，内核被 bsp 拖累了。