X-Git-Url: http://rtime.felk.cvut.cz/gitweb/can-benchmark.git/blobdiff_plain/19359cd66ecd54ece07987f80b98ab5f12a5e82d..839937d2c18ad240ff87ba33d6ce144d5d8793a0:/latester/latester.c

diff --git a/latester/latester.c b/latester/latester.c
index 9f0a466..bdd539c 100644
--- a/latester/latester.c
+++ b/latester/latester.c
@@ -59,10 +59,15 @@ struct options {
 	unsigned timeout_ms;
 	unsigned count;
 	unsigned oneattime;
-	FILE *file;
-	FILE *histogram;
+	char *name;
 	int length;
 	int userhist;
+	int quiet;
+
+	/* Temporary variables */
+	FILE *f_msgs;
+	FILE *f_hist;
+	FILE *f_stat;
 };
 
 struct options opt = {
@@ -72,8 +77,17 @@ struct options opt = {
 	.length = 2,
 };
 
+struct {
+	unsigned enobufs;
+	unsigned overrun;
+	unsigned lost;
+	struct timespec tic, tac;
+	unsigned timeouts;
+} stats;
+
 int num_interfaces = 0;
 int count = 0;			/* Number of sent messages */
+unsigned msg_in_progress = 0;
 int completion_pipe[2];
 
 struct msg_info {
@@ -126,7 +140,7 @@ static inline struct msg_info *frame2info(struct can_frame *frame)
 		if (idx >= MAX_INFOS)
 			error(1, 0, "%s idx too high", __FUNCTION__);
 	} else {
-		
+
 		error(1, 0, "%s error", __FUNCTION__);
 	}
 	return &msg_infos[idx];
@@ -151,12 +165,14 @@ void msg_info_print(FILE *f, struct msg_info *mi)
 #define S(ts) tstamp_str(local, &ts)
 #define DIFF(a, b) (timespec_subtract(&diff, &b, &a), S(diff))
 
-	if (num_interfaces == 2)
+	switch (num_interfaces) {
+	case 2:
 		fprintf(f, "%ld: %s %s -> %s (%s) %s = %s (%s)\n",
 			num, S(mi->ts_sent), sent, S(mi->ts_rx_final_kern), S(mi->ts_rx_final), received,
 		       DIFF(mi->ts_sent, mi->ts_rx_final_kern),
 		       DIFF(mi->ts_sent, mi->ts_rx_final));
-	else
+		break;
+	case 3:
 		fprintf(f, "%ld: %s %s -> %s (%s) -> %s (%s) %s = %s (%s), %s (%s)\n",
 			num, S(mi->ts_sent), sent,
 			S(mi->ts_rx_onwire_kern), S(mi->ts_rx_onwire),
@@ -165,35 +181,14 @@ void msg_info_print(FILE *f, struct msg_info *mi)
 			DIFF(mi->ts_sent, mi->ts_rx_onwire),
 			DIFF(mi->ts_rx_onwire_kern, mi->ts_rx_final_kern),
 			DIFF(mi->ts_rx_onwire, mi->ts_rx_final));
+		break;
+	}
 #undef S
 #undef DIFF
 	num++;
 	talloc_free (local);
 }
 
-int msg_info_store(FILE *f, struct msg_info *mi)
-{
-	struct timespec diff;
-	void *local = talloc_new (NULL);
-	static long num = 0;
-
-#define S(ts) tstamp_str(local, &ts)
-#define DIFF(a, b) (timespec_subtract(&diff, &b, &a), S(diff))
-
-	if (num_interfaces == 2)
-		fprintf(f, "%ld %d %d %s\n",
-			num, mi->id, mi->length,
-			DIFF(mi->ts_sent, mi->ts_rx_final_kern));
-	else
-		fprintf(f, "%ld %d %d %s\n",
-			num, mi->id, mi->length,
-			DIFF(mi->ts_rx_onwire_kern, mi->ts_rx_final_kern));
-#undef S
-#undef DIFF
-	talloc_free (local);
-}
-
-
 /* Subtract the `struct timespec' values X and Y, storing the result in
    RESULT.  Return 1 if the difference is negative, otherwise 0.  */
 
@@ -230,16 +225,22 @@ void dbg_print_timespec(char *msg, struct timespec *tv)
 static inline unsigned get_msg_latency_us(struct msg_info *mi)
 {
 	struct timespec diff;
-	if (num_interfaces == 3)
+	switch (num_interfaces) {
+	case 3:
 		if (opt.userhist)
-			timespec_subtract(&diff, &mi->ts_rx_final_kern, &mi->ts_rx_onwire_kern);
-		else
 			timespec_subtract(&diff, &mi->ts_rx_final, &mi->ts_rx_onwire);
-	else
+		else
+			timespec_subtract(&diff, &mi->ts_rx_final_kern, &mi->ts_rx_onwire_kern);
+		break;
+	case 2:
 		if (opt.userhist)
 			timespec_subtract(&diff, &mi->ts_rx_final, &mi->ts_sent);
 		else
 			timespec_subtract(&diff, &mi->ts_rx_final_kern, &mi->ts_sent);
+		break;
+	default:
+		return 0;
+	}
 	return diff.tv_sec * 1000000 + diff.tv_nsec/1000;
 }
 
@@ -290,7 +291,7 @@ int init_ftrace()
 	char *debugfs;
 	char path[256];
 	FILE *f;
-	
+
 	debugfs = "/sys/kernel/debug";
 	if (debugfs) {
 		strcpy(path, debugfs);
@@ -316,6 +317,28 @@ int init_ftrace()
 #endif	/* FTRACE */
 }
 
+static inline void trace_on()
+{
+	if (trace_fd >= 0)
+		write(trace_fd, "1", 1);
+}
+
+static inline void trace_off(int ret)
+{
+	if (marker_fd >= 0) {
+		char marker[100];
+		sprintf(marker, "write returned %d\n", ret);
+		write(marker_fd, marker, strlen(marker));
+	}
+	if (trace_fd >= 0)
+		write(trace_fd, "0", 1);
+}
+
+void msg_info_free(struct msg_info *mi)
+{
+	mi->id = -1;
+}
+
 int send_frame(int socket)
 {
 	struct can_frame frame;
@@ -349,23 +372,30 @@ int send_frame(int socket)
 	mi->length = frame.can_dlc;
 	get_tstamp(&mi->ts_sent);
 	mi->sent = frame;
-	
-	if (trace_fd >= 0)
-		write(trace_fd, "1", 1);
+
+	trace_on();
 	ret = write(socket, &frame, sizeof(frame));
-	if (marker_fd >= 0) {
-		char marker[100];
-		sprintf(marker, "write returned %d\n", ret);
-		write(marker_fd, marker, strlen(marker));
-	}
-	if (trace_fd >= 0)
-		write(trace_fd, "0", 1);
+	trace_off(ret);
+
+	if (ret == -1 || num_interfaces == 1)
+		msg_info_free(mi);
 	return ret;
 }
 
-void msg_info_free(struct msg_info *mi)
+static inline send_and_check(int s)
 {
-	mi->id = -1;
+	int ret;
+	ret = send_frame(s);
+	if (ret != sizeof(struct can_frame)) {
+/* 		if (ret == -1 && errno == ENOBUFS && opt.period_us == 0 && !opt.oneattime) { */
+/* 			stats.enobufs++; */
+/* 			/\* Ignore this error - pfifo_fast qeuue is full *\/ */
+/* 		} else */
+			error(1, errno, "send_frame (line %d)", __LINE__);
+	} else {
+		count++;
+		msg_in_progress++;
+	}
 }
 
 static inline void get_next_timeout(struct timespec *timeout)
@@ -384,7 +414,10 @@ static inline void get_next_timeout(struct timespec *timeout)
 			last.tv_nsec -= 1000000000;
 			last.tv_sec++;
 		}
-		timespec_subtract(timeout, &last, &now);
+		if (timespec_subtract(timeout, &last, &now) /* is negative */) {
+			stats.overrun++;
+			memset(timeout, 0, sizeof(*timeout));
+		}
 	} else if (opt.timeout_ms != 0) {
 		timeout->tv_sec = opt.timeout_ms/1000;
 		timeout->tv_nsec = (opt.timeout_ms%1000)*1000000;
@@ -457,15 +490,14 @@ void process_final_rx(int s)
 	struct can_frame frame;
 	struct msg_info *mi;
 	int ret;
-	
+
 	receive(s, &frame, &ts_kern, &ts_user);
 	mi = frame2info(&frame);
 	mi->ts_rx_final_kern = ts_kern;
 	mi->ts_rx_final = ts_user;
 	mi->received = frame;
 
-	if (opt.histogram)
-		histogram_add(&histogram, get_msg_latency_us(mi));
+	histogram_add(&histogram, get_msg_latency_us(mi));
 
 	ret = write(completion_pipe[1], &mi, sizeof(mi));
 	if (ret == -1)
@@ -479,7 +511,7 @@ void *measure_thread(void *arg)
 	struct timespec timeout;
 	struct sockaddr_can addr;
 	sigset_t set;
-	unsigned msg_in_progress = 0;
+	int consecutive_timeouts = 0;
 
 	MEMSET_ZERO(pfd);
 
@@ -519,21 +551,14 @@ void *measure_thread(void *arg)
 			pfd[i].events = POLLIN;
 	}
 
-	set_sched_policy_and_prio(SCHED_FIFO, 99);
+	set_sched_policy_and_prio(SCHED_FIFO, 40);
 
-#define SEND()						\
-	do {						\
-		ret = send_frame(pfd[0].fd);		\
-		if (ret != sizeof(struct can_frame))	\
-			error(1, errno, "send_frame (line %d)", __LINE__); \
-		count++;				\
-		msg_in_progress++;			\
-	} while (0)
+#define SEND() send_and_check(pfd[0].fd)
 
-	if (opt.oneattime) {
+	if (opt.oneattime)
 		SEND();
-		count = 1;
-	}
+
+	get_tstamp(&stats.tic);
 
 	while (!finish_flag &&
 	       (opt.count == 0 || count < opt.count || msg_in_progress != 0)) {
@@ -553,7 +578,13 @@ void *measure_thread(void *arg)
 					SEND();
 				}
 			} else {
-				error(1, 0, "poll timeout");
+				/* Lost message - send a new one */
+				stats.timeouts++;
+				consecutive_timeouts++;
+				if (consecutive_timeouts < 10)
+					SEND();
+				else /* Something is really broken */
+					finish_flag = 1;
 			}
 			break;
 		default: // Event
@@ -573,6 +604,7 @@ void *measure_thread(void *arg)
 
 			i = (num_interfaces == 2) ? 1 : 2;
 			if (pfd[i].revents != 0) {
+				consecutive_timeouts = 0;
 				process_final_rx(pfd[i].fd);
 				msg_in_progress--;
 				pfd[i].revents = 0;
@@ -584,6 +616,8 @@ void *measure_thread(void *arg)
 		}
 	}
 
+	get_tstamp(&stats.tac);
+
 	for (i=0; i<num_interfaces; i++)
 		close(pfd[i].fd);
 
@@ -598,10 +632,10 @@ struct poptOption optionsTable[] = {
 	{ "timeout",'t', POPT_ARG_INT|POPT_ARGFLAG_SHOW_DEFAULT,  &opt.timeout_ms,0, "Timeout when period is zero", "ms"},
 	{ "oneattime",'o', POPT_ARG_NONE,			  &opt.oneattime,0,  "Send the next message only when the previous was finally received"},
 	{ "verbose",'v', POPT_ARG_NONE,			  	  NULL, 'v',   	     "Send the next message only when the previous was finally received"},
-	{ "file",   'f', POPT_ARG_STRING, 			  NULL, 'f',   	     "File where to store results", "filename"},
-	{ "histogram", 'h', POPT_ARG_STRING, 			  NULL, 'h',   	     "Store histogram in file", "filename"},
+	{ "name",   'n', POPT_ARG_STRING,			  &opt.name, 0,	     "Prefix of the generated files"},
 	{ "length", 'l', POPT_ARG_INT|POPT_ARGFLAG_SHOW_DEFAULT,  &opt.length, 0,    "The length of generated messages", "bytes"},
 	{ "userhist", 'u', POPT_ARG_NONE,  			  &opt.userhist, 0,  "Generate histogram from userspace timestamps"},
+	{ "quiet",  'q', POPT_ARG_NONE,  			  &opt.quiet, 0,     "Do not print progress and statistics"},
 	POPT_AUTOHELP
 	{ NULL, 0, 0, NULL, 0 }
 };
@@ -610,6 +644,7 @@ int parse_options(int argc, const char *argv[])
 {
 	int c;
 	poptContext optCon;   /* context for parsing command-line options */
+	void *local = talloc_new (NULL);
 
 	optCon = poptGetContext(NULL, argc, argv, optionsTable, 0);
 	//poptSetOtherOptionHelp(optCon, "[OPTIONS]* <port>");
@@ -620,16 +655,6 @@ int parse_options(int argc, const char *argv[])
 		case 'd':
 			num_interfaces++;
 			break;
-		case 'f':
-			opt.file = fopen(poptGetOptArg(optCon), "w");
-			if (!opt.file)
-				error(1, errno, "fopen: %s", poptGetOptArg(optCon));
-			break;
-		case 'h':
-			opt.histogram = fopen(poptGetOptArg(optCon), "w");
-			if (!opt.histogram)
-				error(1, errno, "fopen: %s", poptGetOptArg(optCon));
-			break;
 		}
 	}
 	if (c < -1)
@@ -637,17 +662,48 @@ int parse_options(int argc, const char *argv[])
 		      poptBadOption(optCon, POPT_BADOPTION_NOALIAS),
 		      poptStrerror(c));
 
-	if (num_interfaces < 2 || num_interfaces > 3)
-		error(1, 0, "-d option must be given exactly 2 or 3 times");
+	if (num_interfaces < 1 || num_interfaces > 3)
+		error(1, 0, "-d option must only be given one, two or three times");
 
 	if (opt.oneattime && opt.period_us)
 		error(1, 0, "oneattime and period cannot be specified at the same time");
 
-	poptFreeContext(optCon);
+	if (opt.name) {
+		char *f = talloc_asprintf(local, "%s-msgs.txt", opt.name);
+		opt.f_msgs = fopen(f, "w");
+		if (!opt.f_msgs)
+			error(1, errno, "fopen: %s", f);
+	}
+
+	if (opt.name) {
+		char *f = talloc_asprintf(local, "%s-hist.txt", opt.name);
+		opt.f_hist = fopen(f, "w");
+		if (!opt.f_hist)
+			error(1, errno, "fopen: %s", f);
+	}
+
+	if (opt.name) {
+		char *f = talloc_asprintf(local, "%s-stat.txt", opt.name);
+		opt.f_stat = fopen(f, "w");
+		if (!opt.f_stat)
+			error(1, errno, "fopen: %s", f);
+	}
 
+	poptFreeContext(optCon);
+	talloc_free(local);
 	return 0;
 }
 
+void print_progress()
+{
+	if (! opt.quiet) {
+		if (num_interfaces > 1)
+			printf("\rSent %5d, in progress %5d", count, msg_in_progress);
+		else
+			printf("\rSent %5d", count);
+		fflush(stdout);
+	}
+}
 
 int main(int argc, const char *argv[])
 {
@@ -665,9 +721,7 @@ int main(int argc, const char *argv[])
 	for (i=0; i<MAX_INFOS; i++)
 		msg_infos[i].id = -1;
 
-	if (opt.histogram) {
-		histogram_init(&histogram, 5000000, 1);
-	}
+	histogram_init(&histogram, 5000000, 1);
 
 	ret = pipe(completion_pipe);
 	if (ret == -1)
@@ -677,10 +731,10 @@ int main(int argc, const char *argv[])
 		error(1, errno, "pipe fcntl");
 
 	init_ftrace();
-	
+
 	pthread_create(&thread, 0, measure_thread, NULL);
 
-	struct timespec next, now, diff;
+	struct timespec next, now, diff, allsent = {0,0};
 	clock_gettime(CLOCK_MONOTONIC, &next);
 	int completed = 0;
 	while (!finish_flag && (opt.count == 0 || completed < opt.count)) {
@@ -696,36 +750,67 @@ int main(int argc, const char *argv[])
 			ret = read(completion_pipe[0], &mi, sizeof(mi));
 			if (ret < sizeof(mi))
 				error(1, errno, "read completion returned %d", ret);
-			if (opt.file)
-				msg_info_print(opt.file, mi);
+			msg_info_print(opt.f_msgs, mi);
 			msg_info_free(mi);
 			completed++;
 		}
 
 		clock_gettime(CLOCK_MONOTONIC, &now);
 		if (timespec_subtract(&diff, &next, &now)) {
-			printf("\rMessage %d", count);
-			fflush(stdout);
-		        next.tv_nsec += 100000000;
+			print_progress();
+			next.tv_nsec += 100000000;
 			while (next.tv_nsec >= 1000000000) {
 				next.tv_nsec -= 1000000000;
 				next.tv_sec++;
 			}
 		}
+		if (opt.count != 0 && count >= opt.count) {
+			if (allsent.tv_sec == 0)
+				allsent = now;
+			timespec_subtract(&diff, &now, &allsent);
+			if (diff.tv_sec >= 1)
+				finish_flag = 1;
+		}
 	}
-	printf("\rMessage %d\n", count);
+	print_progress();
+	if (!opt.quiet)
+		printf("\n");
+
+	stats.lost = msg_in_progress;
 
 	pthread_join(thread, NULL);
 
 	close(completion_pipe[0]);
 	close(completion_pipe[1]);
 
-	if (opt.histogram) {
-		histogram_fprint(&histogram, opt.histogram);
-		fclose(opt.histogram);
-	}
-	if (opt.file)
-		fclose(opt.file);
+	histogram_fprint(&histogram, opt.f_hist);
+	fclose(opt.f_hist);
+	fclose(opt.f_msgs);
+
+
+	fprintf(opt.f_stat, "cmdline='");
+	for (i=0; i<argc; i++)
+		fprintf(opt.f_stat, "%s%s", argv[i], i < argc-1 ? " " : "");
+	fprintf(opt.f_stat, "'\n");
+
+	timespec_subtract(&diff, &stats.tac, &stats.tic);
+	fprintf(opt.f_stat, "duration=%s # seconds\n", tstamp_str(NULL, &diff));
+	
+	fprintf(opt.f_stat, "sent=%d\n", count);
+	fprintf(opt.f_stat, "overrun=%d\n", stats.overrun);
+	if (stats.overrun && !opt.quiet)
+		printf("overrun=%d\n", stats.overrun);
+	fprintf(opt.f_stat, "enobufs=%d\n", stats.enobufs);
+	if (stats.enobufs && !opt.quiet)
+		printf("enobufs=%d\n", stats.enobufs);
+	fprintf(opt.f_stat, "lost=%d\n", stats.lost);
+	if (stats.lost && !opt.quiet)
+		printf("lost=%d\n", stats.lost);
+	fprintf(opt.f_stat, "timeouts=%d\n", stats.timeouts);
+	if (stats.timeouts && !opt.quiet)
+		printf("timeouts=%d\n", stats.timeouts);
+
+	fclose(opt.f_stat);
 
 	return 0;
 }