Toggle navigation
首页
问答
文章
积分商城
专家
专区
更多专区...
文档中心
返回主站
搜索
提问
会员
中心
登录
注册
RT-Thread一般讨论
关于rt-thread/components/drivers/src/ringbuffer.c的bug
发布于 2015-01-10 12:23:21 浏览:2543
订阅该版
之前用rtt1.1.0在串口接收部分上加了一个dma。现在用1.2.3 。也想要加一个dma。看到1.2.3的串口驱动里加入了ringbuffer。 由于dma也是大批写入。 stm32103x的串口驱动里只有rt_ringbuffer_putchar_force,没有 rt_ringbuffer_put_force。 就想把rt_ringbuffer_put_force移植到串口驱动里。 看到rt_ringbuffer_put_force函数的这里 ``` if (rb->buffer_size - rb->write_index > length) { memcpy(&rb->buffer_ptr[rb->write_index], ptr, length); rb->write_index += length; if (old_state == RT_RINGBUFFER_FULL) rb->read_index = rb->write_index; return length; }``` 感觉 if (old_state == RT_RINGBUFFER_FULL)这里的判断有问题。是否应该改为if(length > rt_ringbuffer_space_len(rb))。 还有这里 ``` memcpy(&rb->buffer_ptr[rb->write_index], &ptr[0], rb->buffer_size - rb->write_index); memcpy(&rb->buffer_ptr[0], &ptr[rb->buffer_size - rb->write_index], length - (rb->buffer_size - rb->write_index)); rb->write_mirror = ~rb->write_mirror; rb->write_index = length - (rb->buffer_size - rb->write_index); if (old_state == RT_RINGBUFFER_FULL) { rb->read_mirror = ~rb->read_mirror; rb->read_index = rb->write_index; } if (old_state == RT_RINGBUFFER_FULL) { rb->read_mirror = ~rb->read_mirror; rb->read_index = rb->write_index; }``` 是否应该改为 ```if (length > rt_ringbuffer_space_len(rb)) { rb->read_mirror = ~rb->write_mirror; rb->read_index = rb->write_index; }``` 因为即使之前环形队列不为空,如果剩余空间不够,也有覆盖的可能。
查看更多
5
个回答
默认排序
按发布时间排序
grissiom
2015-01-10
这家伙很懒,什么也没写!
嗯,说的有理~ 如果方便的话可以起 pull request。这样就能有你的 commit 了。;)
pangwei
2015-01-11
这家伙很懒,什么也没写!
**if (rb->buffer_size - rb->write_index > length)** ``` { memcpy(&rb->buffer_ptr[rb->write_index], ptr, length); rb->write_index += length; if (old_state == RT_RINGBUFFER_FULL) rb->read_index = rb->write_index; return length; } ```
grissiom
2015-01-11
这家伙很懒,什么也没写!
``` if (rb->buffer_size - rb->write_index > length) ``` 是判断需不需要分开拷贝的,没有判断有没有覆写旧数据。
OXape
2015-01-12
这家伙很懒,什么也没写!
如果覆盖了旧的数据就应该修改read_index.不然再次读数据时就会造成错误。
撰写答案
登录
注册新账号
关注者
0
被浏览
2.5k
关于作者
OXape
这家伙很懒,什么也没写!
提问
12
回答
21
被采纳
0
关注TA
发私信
相关问题
1
有关动态模块加载的一篇论文
2
最近的调程序总结
3
晕掉了,这么久都不见layer2的踪影啊
4
继续K9ii的历程
5
[GUI相关] FreeType 2
6
[GUI相关]嵌入式系统中文输入法的设计
7
20081101 RT-Thread开发者聚会总结
8
嵌入式系统基础
9
linux2.4.19在at91rm9200 上的寄存器设置
10
[转]基于嵌入式Linux的通用触摸屏校准程序
推荐文章
1
RT-Thread应用项目汇总
2
玩转RT-Thread系列教程
3
机器人操作系统 (ROS2) 和 RT-Thread 通信
4
五分钟玩转RT-Thread新社区
5
国产MCU移植系列教程汇总,欢迎查看!
6
【技术三千问】之《玩转ART-Pi》,看这篇就够了!干货汇总
7
关于STM32H7开发板上使用SDIO接口驱动SD卡挂载文件系统的问题总结
8
STM32的“GPU”——DMA2D实例详解
9
RT-Thread隐藏的宝藏之completion
10
【ART-PI】RT-Thread 开启RTC 与 Alarm组件
最新文章
1
FRDM-MCXN947 & RT-Thread的CAN实践——控制GM6020云台电机
2
[Vision Board创客营]WDT实践之配置、issue、PR一条龙
3
恩智浦FRDM-MCXN947上的RW007联网实践
4
[Vision Board创客营]学习片上Flash移植FAL
5
MCXN947测评系列(二)串口设备测评
热门标签
RT-Thread Studio
串口
LWIP
Env
SPI
Bootloader
AT
ART-Pi
CAN总线
Hardfault
FinSH
USB
文件系统
DMA
RT-Thread
SCons
线程
RT-Thread Nano
MQTT
STM32
RTC
FAL
rt-smart
ESP8266
ota在线升级
WIZnet_W5500
I2C_IIC
UART
flash
cubemx
packages_软件包
freemodbus
潘多拉开发板_Pandora
PWM
定时器
ADC
BSP
中断
keil_MDK
编译报错
socket
MicroPython
GD32
msh
Debug
ulog
SDIO总线
SFUD
rt_mq_消息队列_msg_queue
C++_cpp
本月问答贡献
xiaorui
6
个答案
3
次被采纳
sakumisu
4
个答案
3
次被采纳
三世执戟
31
个答案
1
次被采纳
梦笑真美
17
个答案
1
次被采纳
踩姑娘的小蘑菇
14
个答案
1
次被采纳
本月文章贡献
Z_Y
2
篇文章
5
次点赞
Alipay
2
篇文章
2
次点赞
2345vor
2
篇文章
1
次点赞
比特饼干
1
篇文章
6
次点赞
Lu_盼盼
1
篇文章
3
次点赞
回到
顶部
发布
问题
分享
好友
手机
浏览
扫码手机浏览
投诉
建议
回到
底部