Attention is currently required from: lynxis lazus. Jenkins Builder has posted comments on this change by lynxis lazus. ( https://gerrit.osmocom.org/c/osmo-sgsn/+/39563?usp=email ) Change subject: Add Gn support to allow MME->SGSN, SGSN->MME cell reselection ...................................................................... Patch Set 3: (13 comments) File src/sgsn/gprs_gmm.c: Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21554): https://gerrit.osmocom.org/c/osmo-sgsn/+/39563/comment/c370157c_338be0e0?us… : PS3, Line 2568: if (mmctx) { braces {} are not necessary for single statement blocks Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21554): https://gerrit.osmocom.org/c/osmo-sgsn/+/39563/comment/136d5fbe_b9470e8d?us… : PS3, Line 2597: if (!mme) { braces {} are not necessary for single statement blocks Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21554): https://gerrit.osmocom.org/c/osmo-sgsn/+/39563/comment/c882e71b_558be22e?us… : PS3, Line 2627: } else { else is not generally useful after a break or return File src/sgsn/gprs_rau_fsm.c: Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21554): https://gerrit.osmocom.org/c/osmo-sgsn/+/39563/comment/38985099_e80396b3?us… : PS3, Line 365: .in_event_mask = S(GMM_RAU_E_UE_RAU_REQUEST) | S(GMM_RAU_E_GGSN_UPD_RESP) | S(GMM_RAU_E_VLR_RAU_REJECT) | S(GMM_RAU_E_VLR_TERM_SUCCESS)| S(GMM_RAU_E_VLR_TERM_FAIL), need consistent spacing around '|' (ctx:VxW) (or typedef missing in osmo-ci/lint/checkpatch/typedefs_osmo.txt?) File src/sgsn/sgsn_libgtp.c: Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21554): https://gerrit.osmocom.org/c/osmo-sgsn/+/39563/comment/a146b799_b3eed156?us… : PS3, Line 190: if (!ggsn->gsn) { braces {} are not necessary for single statement blocks Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21554): https://gerrit.osmocom.org/c/osmo-sgsn/+/39563/comment/a6a88f70_b7e6299c?us… : PS3, Line 913: #define CHECK_SPACE_ERR(bytes) \ Macros starting with if should be enclosed by a do - while loop to avoid possible if/else logic defects Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21554): https://gerrit.osmocom.org/c/osmo-sgsn/+/39563/comment/504db988_927fbfd2?us… : PS3, Line 918: #define MEMCPY_CHK(dst, src, len) \ Macros with multiple statements should be enclosed in a do - while loop Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21554): https://gerrit.osmocom.org/c/osmo-sgsn/+/39563/comment/a7b8ef7d_efa5cac2?us… : PS3, Line 918: #define MEMCPY_CHK(dst, src, len) \ macros should not use a trailing semicolon Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21554): https://gerrit.osmocom.org/c/osmo-sgsn/+/39563/comment/e5c71c16_921aa1ae?us… : PS3, Line 1085: LOGMMCTXP(LOGL_INFO, mmctx, "Ctx will be transfered to another SGSN/MME\n"); 'transfered' may be misspelled - perhaps 'transferred'? Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21554): https://gerrit.osmocom.org/c/osmo-sgsn/+/39563/comment/5345de74_548b39b6?us… : PS3, Line 1125: if (buf_len <= 0) { braces {} are not necessary for single statement blocks Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21554): https://gerrit.osmocom.org/c/osmo-sgsn/+/39563/comment/8022684a_0e747ce5?us… : PS3, Line 1139: // break; please, no space before tabs Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21554): https://gerrit.osmocom.org/c/osmo-sgsn/+/39563/comment/98acc286_73fbfc73?us… : PS3, Line 1378: strncpy(mmctx->imsi, imsi_str, sizeof(mmctx->imsi)); Instead of strncpy()/strcpy(), use osmo_strlcpy() or OSMO_STRLCPY_ARRAY() Robot Comment from checkpatch (run ID jenkins-gerrit-lint-21554): https://gerrit.osmocom.org/c/osmo-sgsn/+/39563/comment/017380e5_f0b233ce?us… : PS3, Line 1388: if (rc) { braces {} are not necessary for single statement blocks -- To view, visit https://gerrit.osmocom.org/c/osmo-sgsn/+/39563?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-sgsn Gerrit-Branch: master Gerrit-Change-Id: Iffe83b31db2b6e6869fedf2351375184096cff6f Gerrit-Change-Number: 39563 Gerrit-PatchSet: 3 Gerrit-Owner: lynxis lazus &lt;lynxis(a)fe80.eu&gt; Gerrit-Reviewer: Jenkins Builder Gerrit-Attention: lynxis lazus &lt;lynxis(a)fe80.eu&gt; Gerrit-Comment-Date: Sun, 16 Feb 2025 21:39:52 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No