Reserve WRR service ring size using real count

Balancer service ring size use only <Ñservice real count> * <max real
weight> items in size so we definitely should not use one common
constant for all the services.
Also fix some boundary violations.

Author: GeorgyKirichenko

common/actions.h

@@ -161,10 +161,10 @@ struct DumpAction final
 	tCounterId counter_id;
 
 	DumpAction(const acl::dump_t& dump_action) :
-	        dump_id(dump_action.dump_id), counter_id(dump_action.counter_id){};
+	        dump_id(dump_action.dump_id), counter_id(dump_action.counter_id) {};
 
 	DumpAction() :
-	        dump_id(0), counter_id(0){};
+	        dump_id(0), counter_id(0) {};
 
 	[[nodiscard]] bool terminating() const { return false; }
 
@@ -193,7 +193,7 @@ struct FlowAction final
 	std::optional<uint32_t> timeout;
 
 	FlowAction(const globalBase::tFlow& flow) :
-	        flow(flow){};
+	        flow(flow) {};
 
 	FlowAction(globalBase::tFlow&& flow) :
 	        flow(std::move(flow)) {}
@@ -228,7 +228,7 @@ struct CheckStateAction final
 	tCounterId counter_id{0};
 
 	CheckStateAction(const acl::check_state_t& check_state_action) :
-	        counter_id(check_state_action.counter_id){};
+	        counter_id(check_state_action.counter_id) {};
 	CheckStateAction() = default;
 
 	[[nodiscard]] bool terminating() const { return false; }
@@ -255,10 +255,10 @@ struct StateTimeoutAction final
 	uint32_t timeout;
 
 	StateTimeoutAction(const acl::state_timeout_t& timeout_action) :
-	        timeout(timeout_action.timeout){};
+	        timeout(timeout_action.timeout) {};
 
 	StateTimeoutAction() :
-	        timeout(0){};
+	        timeout(0) {};
 
 	[[nodiscard]] bool terminating() const { return false; }
 
@@ -287,11 +287,11 @@ struct HitCountAction final
 
 	HitCountAction(const acl::hit_count_t& hitcount_action) :
 	        id(std::move(hitcount_action.id)),
-	        counter_id(hitcount_action.counter_id){};
+	        counter_id(hitcount_action.counter_id) {};
 
 	HitCountAction() :
 	        id(""),
-	        counter_id(0){};
+	        counter_id(0) {};
 
 	[[nodiscard]] bool terminating() const { return false; }
 

dataplane/globalbase.cpp

@@ -1854,7 +1854,11 @@ generation::ServiceSize generation::rebuild_service_ring_one_wrr(
         const balancer_service_t& service)
 {
 	balancer_real_id_t* end = start;
-	balancer_real_id_t* reserve = start + YANET_BALANCER_WRR_SERVICE_SIZE;
+	balancer_real_id_t* reserve = start + YANET_CONFIG_BALANCER_REAL_WEIGHT_MAX * service.real_size;
+	if (reserve > do_not_exceed)
+	{
+		reserve = (balancer_real_id_t*)do_not_exceed;
+	}
 	for (uint32_t real_idx = service.real_start;
 	     real_idx < service.real_start + service.real_size;
 	     ++real_idx)
@@ -1867,6 +1871,8 @@ generation::ServiceSize generation::rebuild_service_ring_one_wrr(
 		if (end + weight > reserve)
 		{
 			YANET_LOG_ERROR("Balancer service exceeded ring chunk bounds. Some reals skipped.\n");
+			std::fill_n(end, reserve - end, real_id);
+			end = reserve;
 			break;
 		}