[sheepdog] [PATCH] logger: remove unnecessary lock for log rotation

Liu Yuan namei.unix at gmail.com
Mon Aug 26 07:11:45 CEST 2013


On Thu, Aug 22, 2013 at 02:01:35PM +0900, MORITA Kazutaka wrote:
> From: MORITA Kazutaka <morita.kazutaka at lab.ntt.co.jp>
> 
> The logger process runs with one thread, so no need for
> pthread_mutex_lock.
> 
> Signed-off-by: MORITA Kazutaka <morita.kazutaka at lab.ntt.co.jp>
> ---
>  lib/logger.c |    4 ----
>  1 file changed, 4 deletions(-)
> 
> diff --git a/lib/logger.c b/lib/logger.c
> index 42a405d..13d2aa1 100644
> --- a/lib/logger.c
> +++ b/lib/logger.c
> @@ -120,8 +120,6 @@ static char *log_buff;
>  
>  static int64_t max_logsize = 500 * 1024 * 1024;  /*500MB*/
>  
> -static pthread_mutex_t logsize_lock = PTHREAD_MUTEX_INITIALIZER;
> -
>  static const char *format_thread_name(char *str, size_t size, const char *name,
>  				      int idx)
>  {
> @@ -556,7 +554,6 @@ static void logger(char *log_dir, char *outfile)
>  		if (max_logsize) {
>  			off_t offset;
>  
> -			pthread_mutex_lock(&logsize_lock);
>  			offset = lseek(log_fd, 0, SEEK_END);
>  			if (offset < 0) {
>  				syslog(LOG_ERR, "sheep log error\n");
> @@ -565,7 +562,6 @@ static void logger(char *log_dir, char *outfile)
>  				if (log_size >= max_logsize)
>  					rotate_log();
>  			}
> -			pthread_mutex_unlock(&logsize_lock);
>  		}
>  
>  		sleep(1);

Applied thanks

Yuan



More information about the sheepdog mailing list