Cellular Network Infrastructure » Radio Access Network » OsmoPCU

I submitted following patch which fixes one of the issues:
https://gerrit.osmocom.org/c/osmo-pcu/+/17282 llc_queue::{dequeue,enqueue}() refactor

However, there's still a new one:

 dequeued msg, length 11 (expected 11), data 4c 4c 43 20 6d 65 73 73 61 67 65
 dequeued msg, length 17 (expected 17), data 6f 74 68 65 72 20 4c 4c 43 20 6d 65 73 73 61 67 65
+../../../src/llc.cpp:216:29: runtime error: signed integer overflow: -864197545 * 1000 cannot be represented in type 'long int'
 dequeued msg, length 13 (expected 13), data 4c 4c 43 20 6d 65 73 73 61 67 65 20 31
 dequeued msg, length 13 (expected 13), data 4c 4c 43 20 6d 65 73 73 61 67 65 20 32
 dequeued msg, length 3 (expected 3), data 2a 41 2a

Forget the negative number, that because it was running with the wrong attempt to fix it. Still it gives a big number which basically comes from test_llc_meta():

info1.recv_time.tv_sec = 123456777;
info2.recv_time.tv_sec = 987654321;

When substracting both in gprs_llc_queue::dequeue():

timersub(&tv_now, &meta_storage->recv_time, &tv_result);
lifetime = tv_result.tv_sec*1000 + tv_result.tv_usec/1000;

So the tv_now (987654321) is waaaay in the future regarding meta_storage->recv_time (123456777), so apparently it cannot be stored (864197544*1000) in "uint32_t lifetime;" which interestingly according to the runtime error seems to be a "long int" in raspberrypi4 architecture...

So we need to check whether it makes sense for 2 enqueues to happen so different in time and then a dequeue for the first one. It probably doesn't make sense and we should decrease the time between both.

also TODO: move osmo_gettimeofday and recv_time + expire_time of MetaInfo and llc_queue to use a Monotonic clock (osmo_clock_gettime) so avoid running into weird stuff (like going to the past).

runtime error fixed by:
https://gerrit.osmocom.org/c/osmo-pcu/+/17459 tests/llc: Change unrealistic time jump to avoid runtime error under ARM

use monotonic clock everywhere submitted here:
https://gerrit.osmocom.org/c/osmo-pcu/+/17460 Use clock_gettime(CLOCK_MONOTONIC) and timespec everywhere

Once merged, ticket can be closed.

Merged, closing.