laforge has submitted this change and it was merged. ( 
https://gerrit.osmocom.org/c/osmo-pcu/+/15726 )

Change subject: VTY: refactor pcu_vty_show_ms_all(): use show_ms()
......................................................................

VTY: refactor pcu_vty_show_ms_all(): use show_ms()

Change-Id: I72aa1a1de22602a3ad2a4d19604ae0935c88c750
---
M src/pcu_vty_functions.cpp
1 file changed, 11 insertions(+), 24 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/pcu_vty_functions.cpp b/src/pcu_vty_functions.cpp
index 3008267..fd8474b 100644
--- a/src/pcu_vty_functions.cpp
+++ b/src/pcu_vty_functions.cpp
@@ -124,30 +124,6 @@
        return CMD_SUCCESS;
 }

-int pcu_vty_show_ms_all(struct vty *vty, struct gprs_rlcmac_bts *bts_data)
-{
-       BTS *bts = bts_data->bts;
-       LListHead<GprsMs> *ms_iter;
-
-       llist_for_each(ms_iter, &bts->ms_store().ms_list()) {
-               GprsMs *ms = ms_iter->entry();
-
-               vty_out(vty, "MS TLLI=%08x, TA=%d, CS-UL=%s, CS-DL=%s, LLC=%zd, 
Cl=%d, E-Cl=%d,"
-                       " TBF-UL=%s, TBF-DL=%s, IMSI=%s%s",
-                       ms->tlli(),
-                       ms->ta(), mcs_name(ms->current_cs_ul()),
-                       mcs_name(ms->current_cs_dl()),
-                       ms->llc_queue()->size(),
-                       ms->ms_class(),
-                       ms->egprs_ms_class(),
-                       ms->ul_tbf() ? ms->ul_tbf()->state_name() : "NA",
-                       ms->dl_tbf() ? ms->dl_tbf()->state_name() : "NA",
-                       ms->imsi(),
-                       VTY_NEWLINE);
-       }
-       return CMD_SUCCESS;
-}
-
 static int show_ms(struct vty *vty, GprsMs *ms)
 {
        unsigned i;
@@ -230,6 +206,17 @@
        return CMD_SUCCESS;
 }

+int pcu_vty_show_ms_all(struct vty *vty, struct gprs_rlcmac_bts *bts_data)
+{
+       BTS *bts = bts_data->bts;
+       LListHead<GprsMs> *ms_iter;
+
+       llist_for_each(ms_iter, &bts->ms_store().ms_list())
+               show_ms(vty, ms_iter->entry());
+
+       return CMD_SUCCESS;
+}
+
 int pcu_vty_show_ms_by_tlli(struct vty *vty, struct gprs_rlcmac_bts *bts_data,
        uint32_t tlli)
 {

--
To view, visit https://gerrit.osmocom.org/c/osmo-pcu/+/15726
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-pcu
Gerrit-Branch: master
Gerrit-Change-Id: I72aa1a1de22602a3ad2a4d19604ae0935c88c750
Gerrit-Change-Number: 15726
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <axilira...@gmail.com>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <lafo...@osmocom.org>
Gerrit-Reviewer: pespin <pes...@sysmocom.de>
Gerrit-MessageType: merged

Reply via email to