diff options
| author | Donald Sharp <sharpd@nvidia.com> | 2024-12-11 11:43:48 -0500 |
|---|---|---|
| committer | Donald Sharp <sharpd@nvidia.com> | 2024-12-11 11:43:48 -0500 |
| commit | 5d8bf74f0a5e28687630e9d1b35ac41db1af0efe (patch) | |
| tree | 00fe58e831f7cd80080611245d6f8c3339fa3a9b /zebra | |
| parent | c322b94124287acb7f190a96cbb725139c01a256 (diff) | |
zebra: Remove tests for allocation failure
This cannot happen. No need to test
Signed-off-by: Donald Sharp <sharpd@nvidia.com>
Diffstat (limited to 'zebra')
| -rw-r--r-- | zebra/zebra_rib.c | 18 |
1 files changed, 4 insertions, 14 deletions
diff --git a/zebra/zebra_rib.c b/zebra/zebra_rib.c index 574083ae02..44addf4dfe 100644 --- a/zebra/zebra_rib.c +++ b/zebra/zebra_rib.c @@ -3746,10 +3746,8 @@ static struct meta_queue *meta_queue_new(void) new = XCALLOC(MTYPE_WORK_QUEUE, sizeof(struct meta_queue)); - for (i = 0; i < MQ_SIZE; i++) { + for (i = 0; i < MQ_SIZE; i++) new->subq[i] = list_new(); - assert(new->subq[i]); - } return new; } @@ -3935,12 +3933,7 @@ void meta_queue_free(struct meta_queue *mq, struct zebra_vrf *zvrf) /* initialise zebra rib work queue */ static void rib_queue_init(void) { - if (!(zrouter.ribq = work_queue_new(zrouter.master, - "route_node processing"))) { - flog_err(EC_ZEBRA_WQ_NONEXISTENT, - "%s: could not initialise work queue!", __func__); - return; - } + zrouter.ribq = work_queue_new(zrouter.master, "route_node processing"); /* fill in the work queue spec */ zrouter.ribq->spec.workfunc = &meta_queue_process; @@ -3950,11 +3943,8 @@ static void rib_queue_init(void) zrouter.ribq->spec.hold = ZEBRA_RIB_PROCESS_HOLD_TIME; zrouter.ribq->spec.retry = ZEBRA_RIB_PROCESS_RETRY_TIME; - if (!(zrouter.mq = meta_queue_new())) { - flog_err(EC_ZEBRA_WQ_NONEXISTENT, - "%s: could not initialise meta queue!", __func__); - return; - } + zrouter.mq = meta_queue_new(); + return; } |