Commit a7262ed4 authored by Stefano Garzarella's avatar Stefano Garzarella Committed by Jakub Kicinski
Browse files

vsock/test: add timeout_usleep() to allow sleeping in timeout sections 



The timeout API uses signals, so we have documented not to use sleep(),
but we can use nanosleep(2) since POSIX.1 explicitly specifies that it
does not interact with signals.

Let's provide timeout_usleep() for that.

Signed-off-by: default avatarStefano Garzarella <sgarzare@redhat.com>
Link: https://patch.msgid.link/20250514141927.159456-2-sgarzare@redhat.com


Signed-off-by: default avatarJakub Kicinski <kuba@kernel.org>
parent 9e1f7a31

tools/testing/vsock/timeout.c

+18 −0
Original line number Diff line number Diff line
@@ -21,6 +21,7 @@ 
#include <stdbool.h>

#include <unistd.h>

#include <stdio.h>

#include <time.h>

#include "timeout.h"



static volatile bool timeout;
@@ -28,6 +29,8 @@ static volatile bool timeout; 
/* SIGALRM handler function.  Do not use sleep(2), alarm(2), or

 * setitimer(2) while using this API - they may interfere with each

 * other.

 *

 * If you need to sleep, please use timeout_sleep() provided by this API.

 */

void sigalrm(int signo)

{
@@ -58,3 +61,18 @@ void timeout_end(void) 
	alarm(0);

	timeout = false;

}



/* Sleep in a timeout section.

 *

 * nanosleep(2) can be used with this API since POSIX.1 explicitly

 * specifies that it does not interact with signals.

 */

int timeout_usleep(useconds_t usec)

{

	struct timespec ts = {

		.tv_sec = usec / 1000000,

		.tv_nsec = (usec % 1000000) * 1000,

	};



	return nanosleep(&ts, NULL);

}

tools/testing/vsock/timeout.h

+1 −0
Original line number Diff line number Diff line
@@ -11,5 +11,6 @@ void sigalrm(int signo); 
void timeout_begin(unsigned int seconds);

void timeout_check(const char *operation);

void timeout_end(void);

int timeout_usleep(useconds_t usec);



#endif /* TIMEOUT_H */