laforge has submitted this change. ( https://gerrit.osmocom.org/c/osmo-remsim/+/27950 )
Change subject: bankd: Differentiate log levels, not everything is DEBUG ......................................................................
bankd: Differentiate log levels, not everything is DEBUG
Change-Id: If3699642fee2d5d92fb164f7f9c41ccff02007cc --- M src/bankd/bankd_main.c 1 file changed, 4 insertions(+), 4 deletions(-)
Approvals: Jenkins Builder: Verified pespin: Looks good to me, but someone else must approve laforge: Looks good to me, approved
diff --git a/src/bankd/bankd_main.c b/src/bankd/bankd_main.c index 9e2d8f8..36fd651 100644 --- a/src/bankd/bankd_main.c +++ b/src/bankd/bankd_main.c @@ -170,7 +170,7 @@ struct client_slot cs; RsproPDU_t *resp;
- LOGPFSM(srvc->fi, "Rx RSPRO %s\n", rspro_msgt_name(pdu)); + LOGPFSML(srvc->fi, LOGL_DEBUG, "Rx RSPRO %s\n", rspro_msgt_name(pdu));
switch (pdu->msg.present) { case RsproPDUchoice_PR_connectBankRes: @@ -206,7 +206,7 @@ resp = rspro_gen_CreateMappingRes(ResultCode_ok); goto send_resp; } else { - LOGPFSM(srvc->fi, "implicitly removing slotmap\n"); + LOGPFSML(srvc->fi, LOGL_NOTICE, "implicitly removing slotmap\n"); bankd_srvc_remove_mapping(map); } } @@ -243,10 +243,10 @@ } else { rspro2client_slot(&cs, &rreq->client); if (!client_slot_equals(&map->client, &cs)) { - LOGPFSM(srvc->fi, "ClientId in removeMappingReq != map\n"); + LOGPFSML(srvc->fi, LOGL_NOTICE, "ClientId in removeMappingReq != map\n"); resp = rspro_gen_RemoveMappingRes(ResultCode_unknownSlotmap); } else { - LOGPFSM(srvc->fi, "removing slotmap\n"); + LOGPFSML(srvc->fi, LOGL_INFO, "removing slotmap\n"); bankd_srvc_remove_mapping(map); resp = rspro_gen_RemoveMappingRes(ResultCode_ok); }