资源互斥问题,这种情况有没有必要加锁?
有一个全局变量,只有任务A会写该变量,其它任务都只会读该变量,请问像这种情况有没有必要加锁保护,或是其它的互斥手段。
[解决办法]
一般情况下,是需要的。
[解决办法]
把读封装成方法,然后加锁。
[解决办法]
如果其他线程之间没有特别的关联,可以不加锁
[解决办法]
《Windows核心编程》
[解决办法]
加锁是个好习惯吧,现在只有一个线程改写,那将来会不会有其他线程改写呢?
[解决办法]
一般来说要加,实际这是读写锁的典型应用,另外如果你全局变量是int类型的,并且修改时不需要读也不用加
[解决办法]
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#ifdef WIN32
#include <windows.h>
#include <io.h>
#include <process.h>
#define MYVOID void
#else
#include <unistd.h>
#include <sys/time.h>
#include <pthread.h>
#define CRITICAL_SECTION pthread_mutex_t
#define _vsnprintf vsnprintf
#define MYVOID void *
#endif
//Log{
#define MAXLOGSIZE 20000000
#define ARRSIZE(x) (sizeof(x)/sizeof(x[0]))
#include <time.h>
#include <sys/timeb.h>
#include <stdarg.h>
char logfilename1[]="MyLog1.log";
char logfilename2[]="MyLog2.log";
char logstr[16000];
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;
if (-1==_vsnprintf(logstr,ARRSIZE(logstr),pszFmt,argp)) logstr[ARRSIZE(logstr)-1]=0;
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);
}
flog=fopen(logfilename1,"a");
if (NULL==flog) return;
}
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}
static int volatile No_Loop=0;
static int volatile gLock = 0;
static int volatile gCounter = 0;
MYVOID testThread(void *pcn) {
int n,i;
n=(int)pcn;
while (1) {
if (No_Loop==2) {
for (i = 0; i < 1000000; ) {
//以下代码无效
// if ( 0 == gLock ) {
// gLock = n;
// if( gLock == n ) {
// ++i;
// ++gCounter;
// //if (i%100000==0) Log("%d %d\n",n,gCounter);
// gLock = 0;
// //if (i%100000==0) sleep_ms(10*(rand()%50));
// continue;
// } else sleep_ms(100);
// } else sleep_ms(100);
//应改为以下代码
InterlockedCompareExchange((void **)&gLock, (void *)n, 0);
if( gLock == n ) {
++i;
++gCounter;
if (i%100000==0) Log("%d %d\n",n,gCounter);
InterlockedExchange((long *)&gLock, 0);
if (i%100000==0) sleep_ms(10*(rand()%50));
continue;
} else sleep_ms(100);
}
sleep_ms(1000);
No_Loop=1;
}
}
}
int main(int argc,char * argv[]) {
int i;
srand(time(NULL));
#ifdef WIN32
InitializeCriticalSection(&cs_log);
#else
pthread_mutex_init(&cs_log,NULL);
pthread_t threads[4];
// void *thread_result;
int threadsN;
int rc;
#endif
Log("=========BEGIN==================\n");
#ifdef WIN32
_beginthread((void(__cdecl *)(void *))testThread,0,(void *)1);
_beginthread((void(__cdecl *)(void *))testThread,0,(void *)2);
_beginthread((void(__cdecl *)(void *))testThread,0,(void *)3);
_beginthread((void(__cdecl *)(void *))testThread,0,(void *)4);
#else
threadsN=0;
rc=pthread_create(&(threads[threadsN++]),NULL,testThread,(void *)1);if (rc) Log("%d=pthread_create %d error!\n",rc,threadsN-1);
rc=pthread_create(&(threads[threadsN++]),NULL,testThread,(void *)2);if (rc) Log("%d=pthread_create %d error!\n",rc,threadsN-1);
rc=pthread_create(&(threads[threadsN++]),NULL,testThread,(void *)3);if (rc) Log("%d=pthread_create %d error!\n",rc,threadsN-1);
rc=pthread_create(&(threads[threadsN++]),NULL,testThread,(void *)4);if (rc) Log("%d=pthread_create %d error!\n",rc,threadsN-1);
#endif
sleep_ms(1000);
No_Loop=2;
i=0;
while (1) {
sleep_ms(1000);
if (No_Loop==1) break;//
}
sleep_ms(1000);
Log("Result: %d\n",gCounter);
Log("=========END====================\n");
#ifdef WIN32
DeleteCriticalSection(&cs_log);
#else
pthread_mutex_destroy(&cs_log);
#endif
return 0;
}