请教1个内存释放的问题。

zydwh

请教1个问题，在“例11-1 UCOSIII消息传递”中，有个函数：
//定时器1的回调函数
void tmr1_callback(void *p_tmr,void *p_arg)
{
u8 *pbuf;
static u8 msg_num;
OS_ERR err;
pbuf = mymalloc(SRAMIN,10); //申请10个字节
if(pbuf) //申请内存成功
{
msg_num++;
sprintf((char*)pbuf,"ALIENTEK %d",msg_num);
//发送消息
OSQPost((OS_Q* )&DATA_Msg,
(void* )pbuf,
(OS_MSG_SIZE)10,
(OS_OPT )OS_OPT_POST_FIFO,
(OS_ERR* )&err);
if(err != OS_ERR_NONE) //消息队列已满！
{
myfree(SRAMIN,pbuf); //释放内存
OSTmrStop(&tmr1,OS_OPT_TMR_NONE,0,&err); //停止定时器1
tmr1sta = !tmr1sta;
LCD_ShowString(10,150,100,16,16,"TMR1 STOP! ");
}
}
}

为何需要 释放内存呀？去掉后，程序也运行正常！

myfree(SRAMIN,pbuf); //释放内存

jermy_z

那是因为你的内存池没被用光

等你把内存池管理的内存全部申请光，你就知道正常不正常了

zuozhongkai

额，大哥啊，谁告诉你内存申请和释放必须在一个任务中成对出现的！我就不能在任务1中malloc()，在任务2中free()吗？
你看到的定时器回调函数tmr1_callback中一直在申请内存(只有当发送失败的时候才释放内存)，函数msgdis_task()却只有释放内存，把这两个函数结合起来看不就对上了！tmr1_callback()函数里面调用OSQPost()发送消息之前肯定要malloc一段内存，msgdis_task()处理掉消息 队列中的消息以后就可以释放掉内存了。

zydwh

说错了，是另一个函数：

zydwh

//显示消息队列中的消息
void msgdis_task(void *p_arg)
{
u8 *p;
OS_MSG_SIZE size;
OS_ERR err; 
while(1)
{
//请求消息
p=OSQPend((OS_Q* )&DATA_Msg,   
  (OS_TICK )0,
                  (OS_OPT )OS_OPT_PEND_BLOCKING,
                  (OS_MSG_SIZE* )&size,
                  (CPU_TS* )0,
                  (OS_ERR* )&err);
LCD_ShowString(5,270,100,16,16,p);
myfree(SRAMIN,p); //释放内存
OSTimeDlyHMSM(0,0,1,0,OS_OPT_TIME_PERIODIC,&err); //延时1s
}
}

zydwh

// myfree(SRAMIN,p); //释放内存
注释掉，也工作正常。关键是，不知为何需要释放内存。

jermy_z

回复【5楼】zydwh:
---------------------------------
那你应改好好百度内存管理的原理

jermy_z

回复【5楼】zydwh:
---------------------------------
而不是在那异想天开

zydwh

因为这个函数没有申请内存！只看到释放内存，感觉有点特别！

zydwh

//定时器1的回调函数
void tmr1_callback(void *p_tmr,void *p_arg)
{
u8 *pbuf;
static u8 msg_num;
OS_ERR err;
pbuf = mymalloc(SRAMIN,16); //申请16个字节
if(pbuf) //申请内存成功
{
msg_num++;
sprintf((char*)pbuf,"ALIENTEK %d",msg_num);
//发送消息
OSQPost((OS_Q* )&DATA_Msg,
(void* )pbuf,
(OS_MSG_SIZE)16,
(OS_OPT )OS_OPT_POST_FIFO,
(OS_ERR* )&err);
if(err != OS_ERR_NONE) //消息队列已满！
{
myfree(SRAMIN,pbuf); //释放内存
OSTmrStop(&tmr1,OS_OPT_TMR_NONE,0,&err);  //停止定时器1
tmr1sta = !tmr1sta;
LCD_ShowString(10,150,100,16,16,"TMR1 STOP! ");
}
}
}

zydwh

定时器1的回调函数 ，好像也有问题，500mS定时到，就申请1次内存，4个500mS定时到，就申请4次内存。但是，第5次，因为消息队列满了，所以发送消息失败，就关闭定时器。总的来说，申请了5次内存，却只释放1次内存，是否有问题呀？

zydwh

我是初学者，请大家指教！

zydwh

我明白了，定时器1的回调函数 多申请的内存，在“显示消息队列中的消息”任务中释放了。

zydwh

//显示消息队列中的消息 
void msgdis_task(void *p_arg) 
{ 
u8 *p; 
OS_MSG_SIZE size; 
OS_ERR err;  
while(1) 
{ 
//请求消息 
p=OSQPend((OS_Q*  )&DATA_Msg,    
  (OS_TICK  )0, 
                  (OS_OPT  )OS_OPT_PEND_BLOCKING, 
                  (OS_MSG_SIZE* )&size,
                  (CPU_TS*  )0, 
                  (OS_ERR*  )&err); 
LCD_ShowString(5,270,100,16,16,p); 
myfree(SRAMIN,p); //释放内存 
OSTimeDlyHMSM(0,0,1,0,OS_OPT_TIME_PERIODIC,&err); //延时1s 
} 
}

zydwh

myfree(SRAMIN,p); //释放内存
这行程序不能去掉！，它释放的是“定时器1的回调函数”中申请的内存。

joe_knight

在定时器里每发送一条消息，分配一次内存，然后将其加到消息队列里， 正常读消息队列的时候会释放这一块内存，如此循环，不断地生产和消费有何问题？
如果发送失败，那就不应该由读消息队列来释放内存了，而是在本地错误处理处释放掉内存。
逻辑要搞清楚啊。

joe_knight

回复【14楼】zydwh:
---------------------------------
再说，频繁地分配和释放内存，造成大量的内存碎片，这个习惯可不太好