多线程同时写一个日志文件

totopper

一个旧项目的日志系统，没有按照常规的思路用一个线程单独写日志，而是使用多个线程同时写一个日志文件，简化如下：

1. #include <stdio.h>
   
2. #include <stdlib.h>
   
3. #include <pthread.h>
   
4. #include <unistd.h>
   
5. static FILE* logfile;
   
6. static char* logfilename = "test.log";
   
7. void* thr_func(void* arg);
   
8. int main()
   
9. {
   
10.     logfile = fopen(logfilename, "a");
    
11.     //setvbuf(logfile,NULL,(int)_IOLBF,0);
    
12.     int thread_num = 3;
    
13.     pthread_t a[3];
    
14.     printf("%ld\n",sizeof(pthread_t));
    
15.     for ( int i = 0; i< thread_num ; i++ )
    
16.     {
    
17.         pthread_create(a+i, NULL, thr_func,NULL)
    
18.     }
    
19.     sleep(10);
    
20.     return 0;
    
21. }
    
22. 
    
23. void* thr_func(void* arg)
    
24. {
    
25.     pthread_t tid = pthread_self();
    
26.     fprintf(logfile,"threadid = %lu--%s\n",tid,"writing file");
    
27.     fprintf(logfile,"threadid = %lu--%s\n",tid,"writing file");
    
28.     fprintf(logfile,"threadid = %lu--%s\n",tid,"writing file");
    
29.     return (void*)0;
    
30. }

复制代码

测试结果如下：
threadid = 139753191249664--writing file
threadid = 139753191249664--writing file
threadid = 139753191249664--writing file
threadid = 139753182856960--writing file
threadid = 139753182856960--writing file
threadid = 139753182856960--writing file
threadid = 139753174464256--writing file
threadid = 139753174464256--writing file
threadid = 139753174464256--writing file
也就是看起来似乎还可以，日志内容虽然每行发生交叉，但是没有出现行内交叉。

请教如下问题：
1、fprintf需要加锁？实际测试时，若加锁，同时开启多个线程，则CPU占用异常高，应该就是抢占锁造成的，大大影响系统的效率
2、不加锁，那么如何保证一个线程的log不会被另一个线程的log打乱

timespace

如果是多个线程调用，想要保证三个fprintf连续，必须加锁。或者合并三个fprintf为一个。
PS：这个demo，不加锁，难道CPU就不高了？

linux_c_py_php

fprintf是否有锁与glibc编译参数有关, 正确的做法应该是内存里snprintf格式化后write O_APPEND写出。

dspecialtwo

同时写一个文件 都要上锁吧 ？？

totopper

回复 4# dspecialtwo


    嗯，理论上都应该上锁

qiaoshui

追加写一个文件，不加锁，会出现 coredump

栏外人27

你已追加方式打开文件，本身这导致文件的操作时原子操作，不需要加锁的的吧！你也说“日志内容虽然每行发生交叉，但是没有出现行内交叉”，这就已经达到了要求了啊！不然怎么会那么巧行内不发生交叉？明显是原子操作嘛！

totopper

回复 7# 栏外人27


    这个回答和楼上的回答冲突了..