69,371
社区成员
发帖
与我相关
我的任务
分享sleep引起内存泄漏？
在线程函数fun调用的fun1函数中加上sleep，就会有内存泄漏，大概40几秒多出8k。把sleep注释了就不会有这个问题。
为什么？
int main()
{
pthread_t td;
pthread_create(&td,NULL,fun,this);
pthread_detach(td);
return ;
}
void* fun()
{
char c;
while(1)
{
while(read(fd,&c,1)) //从设备fd中读取字节，fd已经打开
fun1();
}
return NULL;
}
bool fun1()
{
......
sleep(3); // 只要注释这句就没有内存问题
return true;
}
为什么？
int main()
{
pthread_t td;
pthread_create(&td,NULL,fun,this);
pthread_detach(td);
return ;
}
void* fun()
{
char c;
while(1)
{
while(read(fd,&c,1)) //从设备fd中读取字节，fd已经打开
fun1();
}
return NULL;
}
bool fun1()
{
......
sleep(3); // 只要注释这句就没有内存问题
return true;
}
...全文
请发表友善的回复…
发表回复
赵4老师 2014-04-21
- 打赏
- 举报
线程不是用来频繁创建和销毁的；是用来在线程池中用的。
//循环向a函数每次发送200个字节长度（这个是固定的）的buffer,
//a函数中需要将循环传进来的buffer，组成240字节（也是固定的）的新buffer进行处理，
//在处理的时候每次从新buffer中取两个字节打印
#ifdef WIN32
#pragma warning(disable:4996)
#endif
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#ifdef WIN32
#include <windows.h>
#include <process.h>
#include <io.h>
#define MYVOID void
#define vsnprintf _vsnprintf
#else
#include <unistd.h>
#include <sys/time.h>
#include <pthread.h>
#define CRITICAL_SECTION pthread_mutex_t
#define MYVOID void *
#endif
//Log{
#define MAXLOGSIZE 20000000
#define MAXLINSIZE 16000
#include <time.h>
#include <sys/timeb.h>
#include <stdarg.h>
char logfilename1[]="MyLog1.log";
char logfilename2[]="MyLog2.log";
static char logstr[MAXLINSIZE+1];
char datestr[16];
char timestr[16];
char mss[4];
CRITICAL_SECTION cs_log;
FILE *flog;
#ifdef WIN32
void Lock(CRITICAL_SECTION *l) {
EnterCriticalSection(l);
}
void Unlock(CRITICAL_SECTION *l) {
LeaveCriticalSection(l);
}
void sleep_ms(int ms) {
Sleep(ms);
}
#else
void Lock(CRITICAL_SECTION *l) {
pthread_mutex_lock(l);
}
void Unlock(CRITICAL_SECTION *l) {
pthread_mutex_unlock(l);
}
void sleep_ms(int ms) {
usleep(ms*1000);
}
#endif
void LogV(const char *pszFmt,va_list argp) {
struct tm *now;
struct timeb tb;
if (NULL==pszFmt||0==pszFmt[0]) return;
vsnprintf(logstr,MAXLINSIZE,pszFmt,argp);
ftime(&tb);
now=localtime(&tb.time);
sprintf(datestr,"%04d-%02d-%02d",now->tm_year+1900,now->tm_mon+1,now->tm_mday);
sprintf(timestr,"%02d:%02d:%02d",now->tm_hour ,now->tm_min ,now->tm_sec );
sprintf(mss,"%03d",tb.millitm);
printf("%s %s.%s %s",datestr,timestr,mss,logstr);
flog=fopen(logfilename1,"a");
if (NULL!=flog) {
fprintf(flog,"%s %s.%s %s",datestr,timestr,mss,logstr);
if (ftell(flog)>MAXLOGSIZE) {
fclose(flog);
if (rename(logfilename1,logfilename2)) {
remove(logfilename2);
rename(logfilename1,logfilename2);
}
} else {
fclose(flog);
}
}
}
void Log(const char *pszFmt,...) {
va_list argp;
Lock(&cs_log);
va_start(argp,pszFmt);
LogV(pszFmt,argp);
va_end(argp);
Unlock(&cs_log);
}
//Log}
#define ASIZE 200
#define BSIZE 240
#define CSIZE 2
char Abuf[ASIZE];
char Cbuf[CSIZE];
CRITICAL_SECTION cs_HEX ;
CRITICAL_SECTION cs_BBB ;
struct FIFO_BUFFER {
int head;
int tail;
int size;
char data[BSIZE];
} BBB;
int No_Loop=0;
void HexDump(int cn,char *buf,int len) {
int i,j,k;
char binstr[80];
Lock(&cs_HEX);
for (i=0;i<len;i++) {
if (0==(i%16)) {
sprintf(binstr,"%03d %04x -",cn,i);
sprintf(binstr,"%s %02x",binstr,(unsigned char)buf[i]);
} else if (15==(i%16)) {
sprintf(binstr,"%s %02x",binstr,(unsigned char)buf[i]);
sprintf(binstr,"%s ",binstr);
for (j=i-15;j<=i;j++) {
sprintf(binstr,"%s%c",binstr,('!'<buf[j]&&buf[j]<='~')?buf[j]:'.');
}
Log("%s\n",binstr);
} else {
sprintf(binstr,"%s %02x",binstr,(unsigned char)buf[i]);
}
}
if (0!=(i%16)) {
k=16-(i%16);
for (j=0;j<k;j++) {
sprintf(binstr,"%s ",binstr);
}
sprintf(binstr,"%s ",binstr);
k=16-k;
for (j=i-k;j<i;j++) {
sprintf(binstr,"%s%c",binstr,('!'<buf[j]&&buf[j]<='~')?buf[j]:'.');
}
Log("%s\n",binstr);
}
Unlock(&cs_HEX);
}
int GetFromRBuf(int cn,CRITICAL_SECTION *cs,FIFO_BUFFER *fbuf,char *buf,int len) {
int lent,len1,len2;
lent=0;
Lock(cs);
if (fbuf->size>=len) {
lent=len;
if (fbuf->head+lent>BSIZE) {
len1=BSIZE-fbuf->head;
memcpy(buf ,fbuf->data+fbuf->head,len1);
len2=lent-len1;
memcpy(buf+len1,fbuf->data ,len2);
fbuf->head=len2;
} else {
memcpy(buf ,fbuf->data+fbuf->head,lent);
fbuf->head+=lent;
}
fbuf->size-=lent;
}
Unlock(cs);
return lent;
}
MYVOID thdB(void *pcn) {
char *recv_buf;
int recv_nbytes;
int cn;
int wc;
int pb;
cn=(int)pcn;
Log("%03d thdB thread begin...\n",cn);
while (1) {
sleep_ms(10);
recv_buf=(char *)Cbuf;
recv_nbytes=CSIZE;
wc=0;
while (1) {
pb=GetFromRBuf(cn,&cs_BBB,&BBB,recv_buf,recv_nbytes);
if (pb) {
Log("%03d recv %d bytes\n",cn,pb);
HexDump(cn,recv_buf,pb);
sleep_ms(1);
} else {
sleep_ms(1000);
}
if (No_Loop) break;//
wc++;
if (wc>3600) Log("%03d %d==wc>3600!\n",cn,wc);
}
if (No_Loop) break;//
}
#ifndef WIN32
pthread_exit(NULL);
#endif
}
int PutToRBuf(int cn,CRITICAL_SECTION *cs,FIFO_BUFFER *fbuf,char *buf,int len) {
int lent,len1,len2;
Lock(cs);
lent=len;
if (fbuf->size+lent>BSIZE) {
lent=BSIZE-fbuf->size;
}
if (fbuf->tail+lent>BSIZE) {
len1=BSIZE-fbuf->tail;
memcpy(fbuf->data+fbuf->tail,buf ,len1);
len2=lent-len1;
memcpy(fbuf->data ,buf+len1,len2);
fbuf->tail=len2;
} else {
memcpy(fbuf->data+fbuf->tail,buf ,lent);
fbuf->tail+=lent;
}
fbuf->size+=lent;
Unlock(cs);
return lent;
}
MYVOID thdA(void *pcn) {
char *send_buf;
int send_nbytes;
int cn;
int wc;
int a;
int pa;
cn=(int)pcn;
Log("%03d thdA thread begin...\n",cn);
a=0;
while (1) {
sleep_ms(100);
memset(Abuf,a,ASIZE);
a=(a+1)%256;
if (16==a) {No_Loop=1;break;}//去掉这句可以让程序一直循环直到按Ctrl+C或Ctrl+Break或当前目录下存在文件No_Loop
send_buf=(char *)Abuf;
send_nbytes=ASIZE;
Log("%03d sending %d bytes\n",cn,send_nbytes);
HexDump(cn,send_buf,send_nbytes);
wc=0;
while (1) {
pa=PutToRBuf(cn,&cs_BBB,&BBB,send_buf,send_nbytes);
Log("%03d sent %d bytes\n",cn,pa);
HexDump(cn,send_buf,pa);
send_buf+=pa;
send_nbytes-=pa;
if (send_nbytes<=0) break;//
sleep_ms(1000);
if (No_Loop) break;//
wc++;
if (wc>3600) Log("%03d %d==wc>3600!\n",cn,wc);
}
if (No_Loop) break;//
}
#ifndef WIN32
pthread_exit(NULL);
#endif
}
int main() {
#ifdef WIN32
InitializeCriticalSection(&cs_log);
InitializeCriticalSection(&cs_HEX );
InitializeCriticalSection(&cs_BBB );
#else
pthread_t threads[2];
int threadsN;
int rc;
pthread_mutex_init(&cs_log,NULL);
pthread_mutex_init(&cs_HEX,NULL);
pthread_mutex_init(&cs_BBB,NULL);
#endif
Log("Start===========================================================\n");
BBB.head=0;
BBB.tail=0;
BBB.size=0;
#ifdef WIN32
_beginthread((void(__cdecl *)(void *))thdA,0,(void *)1);
_beginthread((void(__cdecl *)(void *))thdB,0,(void *)2);
#else
threadsN=0;
rc=pthread_create(&(threads[threadsN++]),NULL,thdA,(void *)1);if (rc) Log("%d=pthread_create %d error!\n",rc,threadsN-1);
rc=pthread_create(&(threads[threadsN++]),NULL,thdB,(void *)2);if (rc) Log("%d=pthread_create %d error!\n",rc,threadsN-1);
#endif
if (!access("No_Loop",0)) {
remove("No_Loop");
if (!access("No_Loop",0)) {
No_Loop=1;
}
}
while (1) {
sleep_ms(1000);
if (No_Loop) break;//
if (!access("No_Loop",0)) {
No_Loop=1;
}
}
sleep_ms(3000);
Log("End=============================================================\n");
#ifdef WIN32
DeleteCriticalSection(&cs_BBB );
DeleteCriticalSection(&cs_HEX );
DeleteCriticalSection(&cs_log);
#else
pthread_mutex_destroy(&cs_BBB);
pthread_mutex_destroy(&cs_HEX);
pthread_mutex_destroy(&cs_log);
#endif
return 0;
}
於黾 2014-04-21
- 打赏
- 举报
sleep为什么一定要放到子函数里去,
直接放到while循环里不行么
不要做咸鱼 2014-04-21
- 打赏
- 举报
最简化了，还是这个问题。
fd设备是源源不断有数据出来的，会不会是sleep那3秒期间，缓冲区多了很多未读数据，导致内存增加？[/quote]
你看看是不是无限增加呢？或者是你只是把文件内容读取到了内存中导致了内存增加呢？排除法啊大哥，一点点的排除才能找到问题所在、
CaptainHookx 2014-04-21
- 打赏
- 举报
最简化了，还是这个问题。
fd设备是源源不断有数据出来的，会不会是sleep那3秒期间，缓冲区多了很多未读数据，导致内存增加？
CaptainHookx 2014-04-21
- 打赏
- 举报
受教了，但未能解惑
赵4老师 2014-04-21
- 打赏
- 举报
有没有内存问题不是通过看程序是否崩溃来决定的。
进程意外退出会在当前目录下产生‘core’文件或形如‘core.数字’的文件比如‘core.1234’
使用命令
gdb 运行程序名 core或core.数字
进入gdb然后使用bt命令
可以查看进程意外退出前函数调用的堆栈，内容为从上到下列出对应从里层到外层的函数调用历史。
如果进程意外退出不产生core文件，参考“ulimit -c core文件最大块大小”命令
检查是否资源泄漏的办法之一：
在任务管理器 进程 查看 选择列 里面选择：内存使用、虚拟内存大小、句柄数、线程数、USER对象、GDI对象
让你的程序(进程)不退出,循环执行主流程很多遍，越多越好，比如1000000次甚至无限循环，记录以上各数值，再隔至少一小时，越长越好，比如一个月，再记录以上各数值。如果以上两组数值的差较大或随时间流逝不断增加，则铁定有对应资源的资源泄漏！
CaptainHookx 2014-04-21
- 打赏
- 举报
您的代码我会慢慢参考，但我还是想问一下，我的问题出在哪里？
zhouda829829 2014-04-21
- 打赏
- 举报
赵老师 有没有讲述多线程编程的文章和帖子啊 要写的比较好的。
赵4老师 2014-04-21
- 打赏
- 举报
只要你仿照我上面代码中的方法使用线程，就不会发生你问的问题。
CaptainHookx 2014-04-21
- 打赏
- 举报
我的程序里只是创建了一个线程，没有频繁创建啊。您的这大段代码看得我云里雾里的，能不能直白一点，不是很懂您想说明什么。
sen_lin8350 2014-04-19
- 打赏
- 举报
while循环一直调用func1函数，而func1函数由于sleep而不运行，不能返回无法释放占有的空间，因此占用的空间越来越多
碼上道 2014-04-19
- 打赏
- 举报
是一直在增长，40几秒增长8k。代码fun1有简化，“……”是处理读出来的设备字节流的语句，我注释掉之后问题一样，应该跟那个没有关系。
sleep不能这样用的吗，在线程函数里[/quote]
sleep使用，在多线程的时候注意进程间锁，互斥的问题，正常情况不会带来内存的泄漏。
以前倒是使用出过问题，那就是在两个锁之间使用了sleep。你最简化你的代码，看看是否有问题。
FeelTouch Labs 2014-04-19
- 打赏
- 举报
sleep方法不推荐使用，可用wait。
Wait()是等待别人释放资源
Sleep()是自己占用资源，去做等待～
PDD123 2014-04-19
- 打赏
- 举报
sleep函数绝对不会导致内存泄漏。
我的多线程程序，在需要一直运行的线程的循环中，都会加上sleep(1)，否则cpu利用率会100%的。
FeelTouch Labs 2014-04-19
- 打赏
- 举报
一知半解，新手求详解[/quote]
Thread.Sleep（） 函数来使线程挂起一段时间。单位ms。
#include <windows.h>
#include<stdio.h>
int main()
{
int a;
a=1000;
printf("你");
Sleep(a);/* VC 使用Sleep*/
printf("好"); /*输出“你”和“好”之间会间隔一千毫秒，即间隔一秒，sleep()的单位为毫秒*/
return 0;
}二极管丫丫丫丫 2014-04-19
- 打赏
- 举报
呃，看不明白....
CaptainHookx 2014-04-19
- 打赏
- 举报
是一直在增长，40几秒增长8k。代码fun1有简化，“……”是处理读出来的设备字节流的语句，我注释掉之后问题一样，应该跟那个没有关系。
sleep不能这样用的吗，在线程函数里
碼上道 2014-04-19
- 打赏
- 举报
看个几分钟，看看是否一直在增长，你的代码是不是就是这样简单的，还是原来复杂现在做了一个简化让大家看的。
CaptainHookx 2014-04-19
- 打赏
- 举报
帮忙看看这段代码还有其它什么缺陷吗，新手不是很懂里面的利害关系。谢谢了！
CaptainHookx 2014-04-19
- 打赏
- 举报
一知半解，新手求详解
加载更多回复（3）
