bssgp: Add VTY command to Limit the bucket size by time

Currently the bucket size is by default being computed based on the
leak rate and the expected life time of LLC frames. The latter is
either taken from 'queue lifetime' (if given) or a fixed value is
used. Using 'queue lifetime' has the drawback that it sets a 'hard'
limit, since frames will be dropped if they stay in the queue
for a longer time.

This commit adds a VTY command to specifically set the time used for
the computation of the bucket size advertised to the SGSN. It does
not affect the PCUs queue handling in any way. If the bucket time is
not set (or if the 'no' command has been used), the old behaviour
(see above) is applied.

The following VTY commands are added (config-pcu node):

- flow-control bucket-time <1-65534>  Sets the time in centisecs
- no flow-control bucket-time         Don't use this time

Ticket: OW#1432
Sponsored-by: On-Waves ehf

1 files changed, 7 insertions, 3 deletions

diff --git a/src/gprs_bssgp_pcu.cpp b/src/gprs_bssgp_pcu.cpp
index 3737438c..bb6f7c53 100644
--- a/src/gprs_bssgp_pcu.cpp
+++ b/src/gprs_bssgp_pcu.cpp
@@ -530,12 +530,16 @@ static uint32_t compute_bucket_size(struct gprs_rlcmac_bts *bts,
 	uint32_t leak_rate, uint32_t fallback)
 {
 	uint32_t bucket_size = 0;
+	uint16_t bucket_time = bts->fc_bucket_time;
 
-	if (bts->force_llc_lifetime == 0xffff)
+	if (bucket_time == 0)
+		bucket_time = bts->force_llc_lifetime;
+
+	if (bucket_time == 0xffff)
 		bucket_size = FC_MAX_BUCKET_SIZE;
 
-	if (bucket_size == 0 && bts->force_llc_lifetime && leak_rate)
-		bucket_size = (uint64_t)leak_rate * bts->force_llc_lifetime / 100;
+	if (bucket_size == 0 && bucket_time && leak_rate)
+		bucket_size = (uint64_t)leak_rate * bucket_time / 100;
 
 	if (bucket_size == 0 && leak_rate)
 		bucket_size = leak_rate * FC_DEFAULT_LIFE_TIME_SECS;