sven
/
can-utils
mirror of https://github.com/linux-can/can-utils.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

canfdtest: remove usage of gettimeofday 

This patch fixes this bug
https://github.com/linux-can/can-utils/issues/13

Modified the existing millisleep function to use clock_nanosleep with
MONOTONIC clock and relative timeout.

Signed-off-by: Ramesh Shanmugasundaram <ramesh.shanmugasundaram@bp.renesas.com>
pull/15/head
Ramesh Shanmugasundaram 2016-11-21 12:06:40 +00:00
parent 4d0e6d5169
commit fa624eb2e6
1 changed files with 11 additions and 30 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

27
canfdtest.c
View File

@ -120,15 +120,13 @@ static void compare_frame(struct can_frame *exp, struct can_frame *rec)
static void millisleep(int msecs) static void millisleep(int msecs)
{ {
if (msecs <= 0) {
sched_yield();
} else {
struct timespec rqtp, rmtp; struct timespec rqtp, rmtp;
/* sleep in ms */ /* sleep in ms */
rqtp.tv_sec = msecs / 1000; rqtp.tv_sec = msecs / 1000;
rqtp.tv_nsec = (msecs % 1000) * 1000000; rqtp.tv_nsec = msecs % 1000 * 1000000;
while (nanosleep(&rqtp, &rmtp)) {
while (clock_nanosleep(CLOCK_MONOTONIC, 0, &rqtp, &rmtp)) {
if (errno != EINTR) { if (errno != EINTR) {
printf("t\n"); printf("t\n");
break; break;
@ -136,7 +134,6 @@ static void millisleep(int msecs)
rqtp = rmtp; rqtp = rmtp;
} }
} }
}
static void echo_progress(unsigned char data) static void echo_progress(unsigned char data)
{ {
@ -194,7 +191,6 @@ static int send_frame(struct can_frame *frame)
static int can_echo_dut(void) static int can_echo_dut(void)
{ {
unsigned int frame_count = 0; unsigned int frame_count = 0;
struct timeval tvn, tv_stop;
struct can_frame frame; struct can_frame frame;
int i; int i;
@ -227,22 +223,7 @@ static int can_echo_dut(void)
*/ */
if (frame_count == CAN_MSG_WAIT) { if (frame_count == CAN_MSG_WAIT) {
frame_count = 0; frame_count = 0;
if (gettimeofday(&tv_stop, NULL)) { millisleep(3);
perror("gettimeofday failed\n");
return -1;
} else {
tv_stop.tv_usec += 3000;
if (tv_stop.tv_usec > 999999) {
tv_stop.tv_sec++;
tv_stop.tv_usec =
tv_stop.tv_usec % 1000000;
}
gettimeofday(&tvn, NULL);
while ((tv_stop.tv_sec > tvn.tv_sec) ||
((tv_stop.tv_sec == tvn.tv_sec) &&
(tv_stop.tv_usec >= tvn.tv_usec)))
gettimeofday(&tvn, NULL);
}
} }
} }