msh shell 线程和 ulog asunc 线程优先级相同时会导致uart device open_flag 被篡改

发布于 2023-12-28 10:35:05 浏览：326 订阅该版

[tocm]

## 问题复现步骤如下
1. 开启 msh shell 线程和 ulog async 线程 并且优先级相同  main 线程优先级大于前两者
2. 在main线程中调用日志接口使其输出日志比如 `LOG_I("xxx")` (注意 这里的日志尽可能的长 保证在输出的第一条日志耗时大于ulog async线程时间片)
3. main 线程结束 如果此时先运行ulog async 由于刚刚main线程有一条日志需要输出 所以, 最终ulog 调用函数 ulog_console_backend_output() 开始输出日志
4. 注意这个函数会先保存当前的open_flag 然后调用 rt_device_write 后再写回 **问题就出在这里**
5. 在执行第 4 步的 rt_device_write 将耗时超过时间片, 所以 msh shell 线程 将得到运行
6. 在 msh shell 线程 运行后 又以新的oflag 标志打开了 `uart device` 代码如下(只截取了部分)
7. 此时 uart dev oflag 已经更新
8. 此时 线程msh 时间片结束, ulog 线程得到运行 继续第 4 步
9. 此时会将之前旧的 old_flag 写回给 `open_flag   dev->open_flag = old_flag;` 导致刚刚mesh 线程打开的oflag无效

### 第 3 步 代码
```c
void ulog_console_backend_output(struct ulog_backend *backend, rt_uint32_t level, const char *tag, rt_bool_t is_raw,
        const char *log, size_t len)
{
#ifdef RT_USING_DEVICE
    rt_device_t dev = rt_console_get_device();

if (dev == RT_NULL)
    {
        rt_hw_console_output(log);
    }
    else
    {
        rt_uint16_t old_flag = dev->open_flag;

dev->open_flag |= RT_DEVICE_FLAG_STREAM;
        rt_device_write(dev, 0, log, len);
        dev->open_flag = old_flag;
    }
#else
    rt_hw_console_output(log);
#endif

}
```

### 第6步代码
```c
void finsh_set_device(const char *device_name)
{
    rt_device_t dev = RT_NULL;

RT_ASSERT(shell != RT_NULL);
    dev = rt_device_find(device_name);
    if (dev == RT_NULL)
    {
        rt_kprintf("finsh: can not find device: %s\n", device_name);
        return;
    }

/* check whether it's a same device */
    if (dev == shell->device) return;
    /* open this device and set the new device in finsh shell */
    if (rt_device_open(dev, RT_DEVICE_OFLAG_RDWR | RT_DEVICE_FLAG_INT_RX | \
                       RT_DEVICE_FLAG_STREAM) == RT_EOK)
    {
```