diff --git a/.gitignore b/.gitignore index 950c1b4..2dcf8d2 100644 --- a/.gitignore +++ b/.gitignore @@ -61,4 +61,5 @@ p-server server client slowclient -slowserver \ No newline at end of file +slowserver +p-slowclient \ No newline at end of file diff --git a/p-client.c b/p-client.c index 8d69f66..fd40e70 100644 --- a/p-client.c +++ b/p-client.c @@ -16,21 +16,26 @@ #include #include "socket_wrapper.h" #include "timerhelper.h" +#include "display_bar.h" #ifndef DEFAULT_TIMEOUT static const int TIMEOUT = 5; #else static const int TIMEOUT = DEFAULT_TIMEOUT; #endif -#ifndef DEFAULT_PROGRESS_BAR_WIDTH -static const int PROGRESS_BAR_WIDTH = 30; -#else -static const int PROGRESS_BAR_WIDTH = DEFAULT_PROGRESS_BAR_WIDTH; -#endif enum{ FILENAME_BUF_SIZE = 1024 }; + +static bool DisplayProgress = true; + +static inline void server_perror(const char * msg){ + if(DisplayProgress) + myd_perror(msg); + else + perror(msg); +} /*======== *Operation *========*/ @@ -40,93 +45,63 @@ int sendReadOp(int sock,const char * filename){ op.file_url_size = strlen(filename); op.padding0 = 0; if(send(sock,&op,sizeof(op),0)<0){ - perror("readop send fail"); + server_perror("readop send fail"); return -1; } #ifdef MUL_CLIENT sleep(MUL_CLIENT); #endif if(send(sock,filename,op.file_url_size,0)<0){ - perror("readop filename send fail"); + server_perror("readop filename send fail"); return -1; } return 0; } -/** - * @arg cur_progress : it is percentage. -*/ -bool isProgressBarNeedUpdate(size_t offset,size_t total,double cur_progress){ - return ((double)offset / (double)total) > (cur_progress / 100.0); -} -void DisplayProgressBar(size_t offset,size_t total,double cur_progress){ - char buf[PROGRESS_BAR_WIDTH]; - size_t i; - size_t cur_pos = (cur_progress / 100.0 * PROGRESS_BAR_WIDTH); //must be less than SIZE_MAX. other value is undefined behavior. - for (i = 0; i < PROGRESS_BAR_WIDTH; i++) - { - if (i < cur_pos) - buf[i] = '='; - else if(i == cur_pos) - buf[i] = '>'; - else buf[i] = '.'; - } - printf("\r[%s]: %.2f%% bytes: %ld/%ld bytes",buf,cur_progress,total,offset); -} -void DisplayProgressBar100Percent(size_t total){ - size_t i; - char buf[PROGRESS_BAR_WIDTH+1]; - for (i = 0; i < PROGRESS_BAR_WIDTH; i++){ - buf[i] = '='; - } - buf[PROGRESS_BAR_WIDTH] = '\0'; - printf("\r[%s]: 100%% bytes: %ld/%ld bytes\n",buf,total,total); -} -static bool DisplayProgress = true; + int recvFile(int sock, const char * filename,size_t file_size){ int fd; size_t count = 0; int i; - double cur_progress = 1; int return_value = 0; int buf_sz = getBufferSizeFrom(sock); + progress_bar_t pbar; uint8_t * buf = malloc(buf_sz*sizeof(*buf)); if (buf == NULL){ return -1; } fd = open(filename,O_WRONLY|O_CREAT|O_TRUNC,S_IRWXU|S_IRWXG|S_IROTH); - if (fd < 0) - { - perror("file open fail"); + if (fd < 0){ + server_perror("file open fail"); return -1; } - + if(DisplayProgress){ + init_progress_bar(&pbar,10); + } while (file_size - count > 0) { int readed = buf_sz < file_size - count ? buf_sz : file_size - count; + if(DisplayProgress){ + DisplayProgressBar(&pbar,count,file_size,filename,false); + } if((i = recv_until_byte(sock,buf,readed,TIMEOUT)) < 0){ if(i == -2) fprintf(stderr,"recv file failed : timeout connetion lost\n"); - else perror("recv file failed"); + else server_perror("recv file failed"); return_value = -1; goto END; } if(write(fd,buf,readed)<0){ - perror("file write failed"); + server_perror("file write failed"); return_value = -1; goto END; } - if( DisplayProgress && isProgressBarNeedUpdate(count,file_size,cur_progress) ){ - DisplayProgressBar(count,file_size,cur_progress); - cur_progress = (int)((((double)count / (double)file_size)) * 100.0 + 1.0); - fflush(stdout); - } count += readed; #ifdef SLOW_CLIENT usleep(SLOW_CLIENT); #endif } if(DisplayProgress) - DisplayProgressBar100Percent(file_size); + DisplayProgressBar(&pbar,count,file_size,filename,true); END: free(buf); close(fd); @@ -137,29 +112,56 @@ int recvData(int sock,const char * filename){ struct TransferResult res; int i=0; if((i=recv_until_byte(sock,&res,sizeof(res),TIMEOUT)) < 0){ - if (i == -2) fprintf(stderr,"timeout\n"); - else perror("recv fail"); + if (i == -2) { + if(DisplayProgress && isatty_file(stderr)){ + lock_scrolled(); + add_scrolled_unlocked(1); + fprintf(stderr,"timeout\n"); + unlock_scrolled(); + } + else fprintf(stderr,"timeout\n"); + } + else server_perror("recv fail"); return -1; } - char error_meesage_buf[80] = ""; + char error_meesage_buf[256] = ""; switch(res.res){ case RES_ERR: - fprintf(stderr,"Server Fail: %s\n", strerror(res.err_number)); + strerror_r(res.err_number,error_meesage_buf,sizeof(error_meesage_buf)); + if(DisplayProgress && isatty_file(stderr)){ + lock_scrolled(); + add_scrolled_unlocked(1); + fprintf(stderr,"Server Fail: %s\n", error_meesage_buf); + unlock_scrolled(); + } + else fprintf(stderr,"Server Fail: %s\n", error_meesage_buf); return -1; case RES_USR_ERR: assert(res.error_msg_size < 80);/*todo : fix*/ if((i=recv_until_byte(sock,error_meesage_buf,res.error_msg_size,TIMEOUT)) < 0){ if (i == -2) fprintf(stderr,"timeout\n"); - else perror("recv fail"); + else server_perror("recv fail"); return -1; } - fprintf(stderr,"Error Message From Server: %s\n",error_meesage_buf); + if(DisplayProgress && isatty_file(stderr)){ + lock_scrolled(); + add_scrolled_unlocked(1); + fprintf(stderr,"Error Message From Server: %s\n",error_meesage_buf); + unlock_scrolled(); + } + else fprintf(stderr,"Error Message From Server: %s\n",error_meesage_buf); return -1; case RES_OK: return recvFile(sock,filename,res.file_size); break; default: - fprintf(stderr,"unknown value!\n"); + if(DisplayProgress && isatty_file(stderr)){ + lock_scrolled(); + add_scrolled_unlocked(1); + fprintf(stderr,"unknown value!\n"); + unlock_scrolled(); + } + else fprintf(stderr,"unknown value!\n"); return -1; } @@ -170,17 +172,23 @@ static bool quiet_mode = false; int SendOpAndReceiveFile(const char * filename, struct sockaddr const * addr){ int sock; int ret = -1; - if(!quiet_mode) - fprintf(stdout,"request %s\n",filename); - + if(!quiet_mode){ + if(DisplayProgress && isatty_file(stdout)){ + lock_scrolled(); + add_scrolled_unlocked(1); + fprintf(stdout,"request %s\n",filename); + unlock_scrolled(); + } + else fprintf(stdout,"request %s\n",filename); + } sock = socket(AF_INET,SOCK_STREAM,0); if(sock < 0){ - perror("sock create fail"); + server_perror("sock create fail"); return -1; } if(connect(sock,(struct sockaddr *)addr,sizeof(*addr)) < 0){ - perror("connect failed"); + server_perror("connect failed"); return -1; } if(sendReadOp(sock,filename) == 0){ @@ -343,6 +351,7 @@ static const char * show_help_message(FILE * file,const char * argv0){ "Options and arguments: \n" "-b or --benchmark\t:benchmark mode\n" "-nv or --no-verbose\t:no progress bar\n" + "-q or --quiet\t:show no message\n" "-h\t:print help message.\n"; fprintf(file,msg,argv0); return msg; @@ -389,6 +398,7 @@ static int parse_arg(int argc,const char *argv[]){ else if(strcmp("-q",argv[cur]) == 0 || strcmp("--quiet",argv[cur]) == 0){ cmd_args.quiet_mode = true; quiet_mode = true; + DisplayProgress = false; } else break; } @@ -409,11 +419,18 @@ int main(int argc, const char *argv[]){ init_bench_data(); if(parse_arg(argc,argv) < 0) return EXIT_FAILURE; + ready_progress_bar(); err = getsockaddrbyname(AF_INET,SOCK_STREAM,0,cmd_args.server_name,(struct sockaddr *)&addr); if (err != 0){ int check; - fprintf(stderr,"netdb fail: %s\n",gai_strerror(err)); + if(DisplayProgress && isatty_file(stderr)){ + lock_scrolled(); + add_scrolled_unlocked(1); + fprintf(stderr,"netdb fail: %s\n",gai_strerror(err)); + unlock_scrolled(); + } + else fprintf(stderr,"netdb fail: %s\n",gai_strerror(err)); //assume that sernmae is *.*.*.* and try to parse addr check = inet_pton(AF_INET,cmd_args.server_name,&addr.sin_addr); assert(check != -1); @@ -464,6 +481,6 @@ int main(int argc, const char *argv[]){ fprintf(stdout,"operation: %lf us/op\n",avg / 1000.0); fprintf(stdout,"resolution: %ld sec %ld ns\n",bench.resolution.tv_sec,bench.resolution.tv_nsec); } - + quit_progress_bar(); return retval; } \ No newline at end of file