Attention is currently required from: pespin, lynxis lazus, dexter. neels has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bsc/+/29704 )
Change subject: abis_rsl: when doing packet access, log t1,t3,t2 and fn ......................................................................
Patch Set 7:
(2 comments)
Commit Message:
https://gerrit.osmocom.org/c/osmo-bsc/+/29704/comment/66076f67_e4989521 PS7, Line 9: the .
File src/osmo-bsc/abis_rsl.c:
https://gerrit.osmocom.org/c/osmo-bsc/+/29704/comment/e011c545_c8a983bd PS7, Line 1822: LOG_BTS(rqd->bts, DRSL, LOGL_INFO, "CHAN RQD: fn(t1=%u,t3=%u,t2=%u) = %u\n", t1, t3, t2, fn); re: why not in the order t1, t2, t3? you hint at a comment, i assume you mean this one:
" The order T1, T2, T3 is intentional. That is the order in which the numbers are stored and also the order in which wireshark will display the values. When debuging it is easier when one does not have to convert between different display variants.
The fn is displayed at later log lines as well, no need to display it here once more. (it is important to see T1, T2, T3 since wireshark won't always do the conversion.) "
The confusion is not gone, your comment says 1,2,3 while the code says 1,3,2. Is wireshark showing 1,3,2?