libbb: make msleep() result in only one syscall instead of looping

function                                             old     new   delta
msleep                                                45      52      +7

Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
This commit is contained in:
Denys Vlasenko 2020-12-11 16:48:47 +01:00
parent 56ee576507
commit 030fe31760
3 changed files with 21 additions and 4 deletions

View File

@ -36,6 +36,7 @@ void FAST_FUNC sleep1(void)
void FAST_FUNC msleep(unsigned ms) void FAST_FUNC msleep(unsigned ms)
{ {
#if 0
/* 1. usleep(n) is not guaranteed by standards to accept n >= 1000000 /* 1. usleep(n) is not guaranteed by standards to accept n >= 1000000
* 2. multiplication in usleep(ms * 1000) can overflow if ms > 4294967 * 2. multiplication in usleep(ms * 1000) can overflow if ms > 4294967
* (sleep of ~71.5 minutes) * (sleep of ~71.5 minutes)
@ -46,4 +47,21 @@ void FAST_FUNC msleep(unsigned ms)
ms -= 500; ms -= 500;
} }
usleep(ms * 1000); usleep(ms * 1000);
#else
//usleep is often implemented as a call to nanosleep.
//Simply do the same to implement msleep.
//it's marginally larger, but wakes your CPU less often:
//function old new delta
//msleep 45 52 +7
struct timespec ts;
ts.tv_sec = ms / 1000;
ts.tv_nsec = (ms % 1000) * 1000000;
/*
* If a signal has non-default handler, nanosleep returns early.
* Our version of msleep doesn't return early
* if interrupted by such signals:
*/
while (nanosleep(&ts, &ts) != 0)
continue;
#endif
} }

View File

@ -27,7 +27,6 @@ int FAST_FUNC usleep(unsigned usec)
* If a signal has non-default handler, nanosleep returns early. * If a signal has non-default handler, nanosleep returns early.
* Our version of usleep doesn't return early * Our version of usleep doesn't return early
* if interrupted by such signals: * if interrupted by such signals:
*
*/ */
while (nanosleep(&ts, &ts) != 0) while (nanosleep(&ts, &ts) != 0)
continue; continue;

View File

@ -1357,15 +1357,15 @@ static FILE *open_new_state_file(void)
IFSTATE_FILE_PATH".new"); IFSTATE_FILE_PATH".new");
} }
/* Someone else created the .new file */ /* Someone else created the .new file */
if (cnt > 30 * 1000) { if (cnt > 30) {
/* Waited for 30*30/2 = 450 milliseconds, still EEXIST. /* Waited for 30*30/2 = 450 milliseconds, still EEXIST.
* Assuming a stale file, rewriting it. * Assuming a stale file, rewriting it.
*/ */
flags = (O_WRONLY | O_CREAT | O_TRUNC); flags = (O_WRONLY | O_CREAT | O_TRUNC);
continue; continue;
} }
usleep(cnt); msleep(cnt);
cnt += 1000; cnt++;
} }
return xfdopen_for_write(fd); return xfdopen_for_write(fd);