diff options
| author | David Lamparter <equinox@opensourcerouting.org> | 2018-09-12 21:25:49 +0200 |
|---|---|---|
| committer | David Lamparter <equinox@opensourcerouting.org> | 2018-09-19 22:01:50 +0200 |
| commit | 64777f5d87e5e6a1b58e555f1e15849746808400 (patch) | |
| tree | 6bd9c29387d689f0faf18f4998837510db8b0ac2 /lib/frr_pthread.c | |
| parent | 1ac267a2d9b422c26d3aba0a1b1cab91fbc3394d (diff) | |
lib: remove unused frr_pthread_yield()
OS-level yield is generally a bad and possibly dangerous idea. If the
thread should be suspended, there should always be something to wait on,
or it turns into busy waiting. And if it's "just giving something else
the chance to run" - that's the kernel's job to determine, and the
kernel will do so while considering priorities, cgroups, and whatnot.
Let it do its job.
Signed-off-by: David Lamparter <equinox@diac24.net>
Diffstat (limited to 'lib/frr_pthread.c')
| -rw-r--r-- | lib/frr_pthread.c | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/lib/frr_pthread.c b/lib/frr_pthread.c index 6541437f55..c1ce57e24e 100644 --- a/lib/frr_pthread.c +++ b/lib/frr_pthread.c @@ -205,11 +205,6 @@ void frr_pthread_stop_all() pthread_mutex_unlock(&frr_pthread_list_mtx); } -void frr_pthread_yield(void) -{ - (void)sched_yield(); -} - /* * ---------------------------------------------------------------------------- * Default Event Loop |