Hi Minh,
first of all, thanks for helping out in terms of OsmoPCU/OsmoBTS bug hunting, it is much appreciated. It is a collaborative project, after all :)
On Mon, May 02, 2016 at 09:32:12AM -0400, Minh-Quang Nguyen wrote:
After an investigation, we observe that the 'pcu_rx_rts_req' function in pcu_l1_if.cpp receives invalid block number from BTS.
Looking at the BTS side, the calculation of L1SAP_FN2PTCCHBLOCK defined as '#define L1SAP_FN2PTCCHBLOCK(fn) ((fn / 52) & 7)' does not seem to be valid for expected PTCCH block numbers B0, B1,B2,B3 defined in Table 6 in TS 05.02 version 8.11.0.
What do you think about re-define the calculation of the L1SAP_FN2PTCCHBLOCK?
If you find the existing calculation is not agreeing with the specification (I'd have to re-read the spec, but I trust you when you say that), I would appreciate if you could send a patch (or patch series) that changes it to be within the spec. If the calculation is more complex, better use a separate function than the simplistic macro.
Thanks in advance!
Regards, Harald