[Asterisk-code-review] serializer: set high/low alert levels on whole pool (asterisk[13])

Friendly Automation asteriskteam at digium.com
Tue Nov 19 10:10:29 CST 2019


Friendly Automation has submitted this change. ( https://gerrit.asterisk.org/c/asterisk/+/13214 )

Change subject: serializer: set high/low alert levels on whole pool
......................................................................

serializer: set high/low alert levels on whole pool

The current code sets alert levels starting from index 1.
Need to set on whole pool starting from index 0.

Change-Id: I5decbb43160954fb9a512f04302637fc666b6f5d
---
M main/serializer.c
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Kevin Harwell: Looks good to me, but someone else must approve
  George Joseph: Looks good to me, approved
  Friendly Automation: Approved for Submit



diff --git a/main/serializer.c b/main/serializer.c
index 280ada0..a10dc49 100644
--- a/main/serializer.c
+++ b/main/serializer.c
@@ -177,7 +177,7 @@
 		tps_queue_low = -1;
 	}
 
-	for (idx = 1; idx < AST_VECTOR_SIZE(&pool->serializers); ++idx) {
+	for (idx = 0; idx < AST_VECTOR_SIZE(&pool->serializers); ++idx) {
 		struct ast_taskprocessor *cur = AST_VECTOR_GET(&pool->serializers, idx);
 		if (ast_taskprocessor_alert_set_levels(cur, tps_queue_low, tps_queue_high)) {
 			ast_log(AST_LOG_WARNING, "Failed to set alert levels for serializer '%s'.\n",

-- 
To view, visit https://gerrit.asterisk.org/c/asterisk/+/13214
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-Change-Id: I5decbb43160954fb9a512f04302637fc666b6f5d
Gerrit-Change-Number: 13214
Gerrit-PatchSet: 1
Gerrit-Owner: Alexei Gradinari <alex2grad at gmail.com>
Gerrit-Reviewer: Friendly Automation
Gerrit-Reviewer: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Kevin Harwell <kharwell at digium.com>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20191119/716d991c/attachment-0001.html>


More information about the asterisk-code-review mailing list