aboutsummaryrefslogtreecommitdiffstats
path: root/tests/llc
diff options
context:
space:
mode:
authorPau Espin Pedrol <pespin@espeweb.net>2020-12-16 15:59:45 +0100
committerpespin <pespin@sysmocom.de>2021-01-05 10:34:25 +0000
commitda971ee5026479e869ed75d944404d398c548497 (patch)
tree30bf447c2d828264e51d09f5422ea05421c48d47 /tests/llc
parent86fad1ec4e246110b9d8ae66fd7bc30e1cedb5de (diff)
Convert GprsMS and helpers classes to C
As we integrate osmo-pcu more and more with libosmocore features, it becomes really hard to use them since libosmocore relies heavily on C specific compilation features, which are not available in old C++ compilers (such as designated initializers for complex types in FSMs). GprsMs is right now a quite simple object since initial design of osmo-pcu made it optional and most of the logic was placed and stored duplicated in TBF objects. However, that's changing as we introduce more features, with the GprsMS class getting more weight. Hence, let's move it now to be a C struct in order to be able to easily use libosmocore features there, such as FSMs. Some helper classes which GprsMs uses are also mostly move to C since they are mostly structs with methods, so there's no point in having duplicated APIs for C++ and C for such simple cases. For some more complex classes, like (ul_,dl_)tbf, C API bindings are added where needed so that GprsMs can use functionalitites from that class. Most of those APIs can be kept afterwards and drop the C++ ones since they provide no benefit in general. Change-Id: I0b50e3367aaad9dcada76da97b438e452c8b230c
Diffstat (limited to 'tests/llc')
-rw-r--r--tests/llc/LlcTest.cpp80
1 files changed, 40 insertions, 40 deletions
diff --git a/tests/llc/LlcTest.cpp b/tests/llc/LlcTest.cpp
index 10cd96bd..bfcac778 100644
--- a/tests/llc/LlcTest.cpp
+++ b/tests/llc/LlcTest.cpp
@@ -60,10 +60,10 @@ static void enqueue_data(gprs_llc_queue *queue, const uint8_t *data, size_t len,
}
static void dequeue_and_check(gprs_llc_queue *queue, const uint8_t *exp_data,
- size_t len, const gprs_llc_queue::MetaInfo *exp_info)
+ size_t len, const MetaInfo *exp_info)
{
struct msgb *llc_msg;
- const gprs_llc_queue::MetaInfo *info_res;
+ const MetaInfo *info_res;
llc_msg = queue->dequeue(&info_res);
OSMO_ASSERT(llc_msg != NULL);
@@ -88,7 +88,7 @@ static void enqueue_data(gprs_llc_queue *queue, const char *message,
}
static void dequeue_and_check(gprs_llc_queue *queue, const char *exp_message,
- const gprs_llc_queue::MetaInfo *exp_info = 0)
+ const MetaInfo *exp_info = 0)
{
dequeue_and_check(queue,
(uint8_t *)(exp_message), strlen(exp_message), exp_info);
@@ -101,33 +101,33 @@ static void test_llc_queue()
printf("=== start %s ===\n", __func__);
- queue.init();
- OSMO_ASSERT(queue.size() == 0);
- OSMO_ASSERT(queue.octets() == 0);
+ llc_queue_init(&queue);
+ OSMO_ASSERT(llc_queue_size(&queue) == 0);
+ OSMO_ASSERT(llc_queue_octets(&queue) == 0);
enqueue_data(&queue, "LLC message", &expire_time);
- OSMO_ASSERT(queue.size() == 1);
- OSMO_ASSERT(queue.octets() == 11);
+ OSMO_ASSERT(llc_queue_size(&queue) == 1);
+ OSMO_ASSERT(llc_queue_octets(&queue) == 11);
enqueue_data(&queue, "other LLC message", &expire_time);
- OSMO_ASSERT(queue.size() == 2);
- OSMO_ASSERT(queue.octets() == 28);
+ OSMO_ASSERT(llc_queue_size(&queue) == 2);
+ OSMO_ASSERT(llc_queue_octets(&queue) == 28);
dequeue_and_check(&queue, "LLC message");
- OSMO_ASSERT(queue.size() == 1);
- OSMO_ASSERT(queue.octets() == 17);
+ OSMO_ASSERT(llc_queue_size(&queue) == 1);
+ OSMO_ASSERT(llc_queue_octets(&queue) == 17);
dequeue_and_check(&queue, "other LLC message");
- OSMO_ASSERT(queue.size() == 0);
- OSMO_ASSERT(queue.octets() == 0);
+ OSMO_ASSERT(llc_queue_size(&queue) == 0);
+ OSMO_ASSERT(llc_queue_octets(&queue) == 0);
enqueue_data(&queue, "LLC", &expire_time);
- OSMO_ASSERT(queue.size() == 1);
- OSMO_ASSERT(queue.octets() == 3);
+ OSMO_ASSERT(llc_queue_size(&queue) == 1);
+ OSMO_ASSERT(llc_queue_octets(&queue) == 3);
- queue.clear(NULL);
- OSMO_ASSERT(queue.size() == 0);
- OSMO_ASSERT(queue.octets() == 0);
+ llc_queue_clear(&queue, NULL);
+ OSMO_ASSERT(llc_queue_size(&queue) == 0);
+ OSMO_ASSERT(llc_queue_octets(&queue) == 0);
printf("=== end %s ===\n", __func__);
}
@@ -135,14 +135,14 @@ static void test_llc_queue()
static void test_llc_meta()
{
gprs_llc_queue queue;
- gprs_llc_queue::MetaInfo info1 = {0};
- gprs_llc_queue::MetaInfo info2 = {0};
+ MetaInfo info1 = {0};
+ MetaInfo info2 = {0};
printf("=== start %s ===\n", __func__);
- queue.init();
- OSMO_ASSERT(queue.size() == 0);
- OSMO_ASSERT(queue.octets() == 0);
+ llc_queue_init(&queue);
+ OSMO_ASSERT(llc_queue_size(&queue) == 0);
+ OSMO_ASSERT(llc_queue_octets(&queue) == 0);
info1.recv_time.tv_sec = 123456777;
info1.recv_time.tv_nsec = 123456000;
@@ -161,9 +161,9 @@ static void test_llc_meta()
dequeue_and_check(&queue, "LLC message 1", &info1);
dequeue_and_check(&queue, "LLC message 2", &info2);
- queue.clear(NULL);
- OSMO_ASSERT(queue.size() == 0);
- OSMO_ASSERT(queue.octets() == 0);
+ llc_queue_clear(&queue, NULL);
+ OSMO_ASSERT(llc_queue_size(&queue) == 0);
+ OSMO_ASSERT(llc_queue_octets(&queue) == 0);
printf("=== end %s ===\n", __func__);
}
@@ -176,8 +176,8 @@ static void test_llc_merge()
printf("=== start %s ===\n", __func__);
- queue1.init();
- queue2.init();
+ llc_queue_init(&queue1);
+ llc_queue_init(&queue2);
clk_mono_override_time->tv_sec += 1;
enqueue_data(&queue1, "*A*", &expire_time);
@@ -194,17 +194,17 @@ static void test_llc_merge()
clk_mono_override_time->tv_sec += 1;
enqueue_data(&queue2, "*E*", &expire_time);
- OSMO_ASSERT(queue1.size() == 3);
- OSMO_ASSERT(queue1.octets() == 9);
- OSMO_ASSERT(queue2.size() == 2);
- OSMO_ASSERT(queue2.octets() == 6);
+ OSMO_ASSERT(llc_queue_size(&queue1) == 3);
+ OSMO_ASSERT(llc_queue_octets(&queue1) == 9);
+ OSMO_ASSERT(llc_queue_size(&queue2) == 2);
+ OSMO_ASSERT(llc_queue_octets(&queue2) == 6);
- queue2.move_and_merge(&queue1);
+ llc_queue_move_and_merge(&queue2, &queue1);
- OSMO_ASSERT(queue1.size() == 0);
- OSMO_ASSERT(queue1.octets() == 0);
- OSMO_ASSERT(queue2.size() == 5);
- OSMO_ASSERT(queue2.octets() == 15);
+ OSMO_ASSERT(llc_queue_size(&queue1) == 0);
+ OSMO_ASSERT(llc_queue_octets(&queue1) == 0);
+ OSMO_ASSERT(llc_queue_size(&queue2) == 5);
+ OSMO_ASSERT(llc_queue_octets(&queue2) == 15);
dequeue_and_check(&queue2, "*A*");
dequeue_and_check(&queue2, "*B*");
@@ -212,8 +212,8 @@ static void test_llc_merge()
dequeue_and_check(&queue2, "*D*");
dequeue_and_check(&queue2, "*E*");
- OSMO_ASSERT(queue2.size() == 0);
- OSMO_ASSERT(queue2.octets() == 0);
+ OSMO_ASSERT(llc_queue_size(&queue2) == 0);
+ OSMO_ASSERT(llc_queue_octets(&queue2) == 0);
printf("=== end %s ===\n", __func__);
}