diff options
| author |   Vladimir Ratnikov <vratnikov@netgate.com> | 2022-10-17 07:34:14 +0000 |
|---|---|---|
| committer |   Matthew Smith <mgsmith@netgate.com> | 2022-10-20 22:13:34 +0000 |
| commit | e3618c1960c473f033dd8dcf2dfb0fb4ff661eb0 (patch) | |
| tree | 86d2a53a555d229e64b0b3ffa23cdeaaf5249637 /src | |
| parent | b807f08d87c1186027bb1266dbd41857d9fcb91a (diff) | |
crypto-sw-scheduler: fix queue iterator
When there are several workers, iterator can and will skip
head iterator and it will last until BARRIER_SYNC_TIMEOUT won't
expire and will cause SIGABRT with `worker thread deadlock`
Type: fix
Signed-off-by: Vladimir Ratnikov <vratnikov@netgate.com>
Change-Id: Id4def4d5894e077ae27592367b141ecd822e86af
Signed-off-by: Matthew Smith <mgsmith@netgate.com>
(cherry picked from commit 65bff88c3671ec6ee561e70f17c60ea9784a39dd)
Diffstat (limited to 'src')
| -rw-r--r-- | src/plugins/crypto_sw_scheduler/main.c | 10 |
1 files changed, 10 insertions, 0 deletions
diff --git a/src/plugins/crypto_sw_scheduler/main.c b/src/plugins/crypto_sw_scheduler/main.c index 47fa37d7251..09d4a0b9b94 100644 --- a/src/plugins/crypto_sw_scheduler/main.c +++ b/src/plugins/crypto_sw_scheduler/main.c @@ -471,6 +471,15 @@ crypto_sw_scheduler_process_aead (vlib_main_t *vm, tail = current_queue->tail; head = current_queue->head; + /* Skip this queue unless tail < head or head has overflowed + * and tail has not. At the point where tail overflows (== 0), + * the largest possible value of head is (queue size - 1). + * Prior to that, the largest possible value of head is + * (queue size - 2). + */ + if ((tail > head) && (head >= CRYPTO_SW_SCHEDULER_QUEUE_MASK)) + goto skip_queue; + for (j = tail; j != head; j++) { @@ -488,6 +497,7 @@ crypto_sw_scheduler_process_aead (vlib_main_t *vm, } } + skip_queue: if (found || i == ptd->last_serve_lcore_id) { CLIB_MEMORY_STORE_BARRIER (); |