pespin has submitted this change. ( https://gerrit.osmocom.org/c/osmo-pcu/+/33157 ) Change subject: Use OSMO_UNLIKELY() in bts_rfn_to_fn() ...................................................................... Use OSMO_UNLIKELY() in bts_rfn_to_fn() That case should only happen under really rare conditions, like receiving a RACH.ind before having received any DATA.ind. Change-Id: I4c71f3481764b501a4441bc735a87725884a3e75 --- M src/bts.cpp 1 file changed, 13 insertions(+), 1 deletion(-) Approvals: Jenkins Builder: Verified fixeria: Looks good to me, approved diff --git a/src/bts.cpp b/src/bts.cpp index 5160fd5..44acbed 100644 --- a/src/bts.cpp +++ b/src/bts.cpp @@ -748,7 +748,7 @@ OSMO_ASSERT(rfn < RFN_MODULUS); m_cur_fn = bts_current_frame_number(bts); - if (m_cur_fn == FN_UNSET) { + if (OSMO_UNLIKELY(m_cur_fn == FN_UNSET)) { LOGP(DRLCMAC, LOGL_ERROR, "Unable to calculate full FN from RFN %u: Current FN not known!\n", rfn); return rfn; -- To view, visit https://gerrit.osmocom.org/c/osmo-pcu/+/33157 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-pcu Gerrit-Branch: master Gerrit-Change-Id: I4c71f3481764b501a4441bc735a87725884a3e75 Gerrit-Change-Number: 33157 Gerrit-PatchSet: 2 Gerrit-Owner: pespin <pespin@sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de> Gerrit-Reviewer: pespin <pespin@sysmocom.de> Gerrit-MessageType: merged