Build failed in Jenkins: master-rtl-sdr » a1=default,a2=default,a3=default,osmocom-master-debian9 #6

2018-04-21 Thread jenkins
See 


--
[...truncated 93.24 KB...]
make[2]: Entering directory 
'
make[2]: Nothing to be done for 'dvi'.
make[2]: Leaving directory 
'
Making dvi in src
make[2]: Entering directory 
'
make[2]: Nothing to be done for 'dvi'.
make[2]: Leaving directory 
'
make[2]: Entering directory 
'
make[2]: Nothing to be done for 'dvi-am'.
make[2]: Leaving directory 
'
make[1]: Leaving directory 
'
make[1]: Entering directory 
'
make  check-recursive
make[2]: Entering directory 
'
Making check in include
make[3]: Entering directory 
'
make[3]: Nothing to be done for 'check'.
make[3]: Leaving directory 
'
Making check in src
make[3]: Entering directory 
'
make[3]: Nothing to be done for 'check'.
make[3]: Leaving directory 
'
make[3]: Entering directory 
'
make[3]: Leaving directory 
'
make[2]: Leaving directory 
'
make[1]: Leaving directory 
'
make[1]: Entering directory 
'
make  install-recursive
make[2]: Entering directory 
'
Making install in include
make[3]: Entering directory 
'
make[4]: Entering directory 
'
make[4]: Nothing to be done for 'install-exec-am'.
 /bin/mkdir -p 
'
 /usr/bin/install -c -m 644 ../../../include/rtl-sdr.h 
../../../include/rtl-sdr_export.h 

libosmocore[master]: gsm_08_58.h: Add Ericsson vendor-specific RSL message types

2018-04-21 Thread lynxis lazus

Patch Set 1: Code-Review+1

-- 
To view, visit https://gerrit.osmocom.org/7893
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I7cb65f3ff1cfdbe4eee97b7545bcd13a38c72e25
Gerrit-PatchSet: 1
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Owner: Pau Espin Pedrol 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: lynxis lazus 
Gerrit-HasComments: No


osmo-trx[master]: doc: examples: Add umtrx sample config

2018-04-21 Thread Alexander Chemeris

Patch Set 1:

(1 comment)

https://gerrit.osmocom.org/#/c/7890/1/doc/examples/osmo-trx-umtrx.cfg
File doc/examples/osmo-trx-umtrx.cfg:

Line 22:  chan 1
Am I correct that this enables dual-channel mode same as the old "-c 2" command 
line option?


-- 
To view, visit https://gerrit.osmocom.org/7890
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Id38de0bbbe75e5e6bbb0de2eecb7d1984786d528
Gerrit-PatchSet: 1
Gerrit-Project: osmo-trx
Gerrit-Branch: master
Gerrit-Owner: Pau Espin Pedrol 
Gerrit-Reviewer: Alexander Chemeris 
Gerrit-Reviewer: Jenkins Builder
Gerrit-HasComments: Yes


[PATCH] osmo-bsc[master]: abis_rsl.c: Clean ericsson specific imm assign code

2018-04-21 Thread Pau Espin Pedrol

Review at  https://gerrit.osmocom.org/7894

abis_rsl.c: Clean ericsson specific imm assign code

Move to its own function, store pointer to proper header format and use
the already defined IE define from libosmocore instead of using
hardcoded values.

Depends on: Change-Id I7cb65f3ff1cfdbe4eee97b7545bcd13a38c72e25

Change-Id: I845fd3f0c6ff31f268f68a31e1d55981f7ec6129
---
M src/libbsc/abis_rsl.c
1 file changed, 25 insertions(+), 14 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/osmo-bsc refs/changes/94/7894/1

diff --git a/src/libbsc/abis_rsl.c b/src/libbsc/abis_rsl.c
index 9081bf1..230016c 100644
--- a/src/libbsc/abis_rsl.c
+++ b/src/libbsc/abis_rsl.c
@@ -2048,12 +2048,34 @@
return 0;
 }
 
+/* Ericsson specific: Immediate Assign Sent */
+static int rsl_rx_ericsson_imm_assign_sent(struct msgb *msg)
+{
+   struct e1inp_sign_link *sign_link = msg->dst;
+   struct abis_rsl_dchan_hdr *dh = msgb_l2(msg);
+   uint32_t tlli;
+
+   LOGP(DRSL, LOGL_INFO, "IMM.ass sent\n");
+   msgb_pull(msg, sizeof(*dh));
+
+   /* FIXME: Move to TLV once we support defining TV types with V having 
len != 1 byte */
+   if(msg->len < 5)
+   LOGP(DRSL, LOGL_ERROR, "short IMM.ass sent message!\n");
+   else if(msg->data[0] != RSL_IE_ERIC_MOBILE_ID)
+   LOGP(DRSL, LOGL_ERROR, "unsupported IMM.ass message format! 
(please fix)\n");
+   else {
+   msgb_pull(msg, 1); /* drop previous data to use msg_pull_u32 */
+   tlli = msgb_pull_u32(msg);
+   pcu_tx_imm_ass_sent(sign_link->trx->bts, tlli);
+   }
+   return 0;
+}
+
 static int abis_rsl_rx_cchan(struct msgb *msg)
 {
struct e1inp_sign_link *sign_link = msg->dst;
struct abis_rsl_dchan_hdr *rslh = msgb_l2(msg);
int rc = 0;
-   uint32_t tlli;
 
msg->lchan = lchan_lookup(sign_link->trx, rslh->chan_nr,
  "Abis RSL rx CCHAN: ");
@@ -2075,19 +2097,8 @@
"type %s (0x%02x)\n",
rsl_msg_name(rslh->c.msg_type), rslh->c.msg_type);
break;
-   case 0x10: /* Ericsson specific: Immediate Assign Sent */
-   /* FIXME: Replace the messy message parsing below
-* with proper TV parser */
-   LOGP(DRSL, LOGL_INFO, "IMM.ass sent\n");
-   if(msg->len < 9)
-   LOGP(DRSL, LOGL_ERROR, "short IMM.ass sent message!\n");
-   else if(msg->data[4] != 0xf1)
-   LOGP(DRSL, LOGL_ERROR, "unsupported IMM.ass message 
format! (please fix)\n");
-   else {
-   msgb_pull(msg, 5); /* drop previous data to use 
msg_pull_u32 */
-   tlli = msgb_pull_u32(msg);
-   pcu_tx_imm_ass_sent(sign_link->trx->bts, tlli);
-   }
+   case RSL_MT_ERICSSON_IMM_ASS_SENT:
+   rc = rsl_rx_ericsson_imm_assign_sent(msg);
break;
default:
LOGP(DRSL, LOGL_NOTICE, "Unknown Abis RSL TRX message type "

-- 
To view, visit https://gerrit.osmocom.org/7894
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I845fd3f0c6ff31f268f68a31e1d55981f7ec6129
Gerrit-PatchSet: 1
Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Owner: Pau Espin Pedrol 


[PATCH] libosmocore[master]: gsm_08_58.h: Add Ericsson vendor-specific RSL message types

2018-04-21 Thread Pau Espin Pedrol

Review at  https://gerrit.osmocom.org/7893

gsm_08_58.h: Add Ericsson vendor-specific RSL message types

We are alredy doing the same way for ip.access and siemens ones, and
this way we avoid using the hardcoded value in osmo-bsc.

Change-Id: I7cb65f3ff1cfdbe4eee97b7545bcd13a38c72e25
---
M include/osmocom/gsm/protocol/gsm_08_58.h
1 file changed, 5 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/libosmocore refs/changes/93/7893/1

diff --git a/include/osmocom/gsm/protocol/gsm_08_58.h 
b/include/osmocom/gsm/protocol/gsm_08_58.h
index f0094a3..5e97b0d 100644
--- a/include/osmocom/gsm/protocol/gsm_08_58.h
+++ b/include/osmocom/gsm/protocol/gsm_08_58.h
@@ -204,6 +204,11 @@
RSL_MT_SIEMENS_MODIF_COND_IND   = 0x50,
 };
 
+/*! Ericsson vendor-specific RSL message types */
+enum abis_rsl_msgtype_ericsson {
+   RSL_MT_ERICSSON_IMM_ASS_SENT= 0x10,
+};
+
 /*! RSL Information Element Identifiers (Chapter 9.3) */
 enum abis_rsl_ie {
RSL_IE_CHAN_NR  = 0x01,

-- 
To view, visit https://gerrit.osmocom.org/7893
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I7cb65f3ff1cfdbe4eee97b7545bcd13a38c72e25
Gerrit-PatchSet: 1
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Owner: Pau Espin Pedrol 


osmo-bsc[master]: abis-rsl: Send imm.ass messages via PCH

2018-04-21 Thread lynxis lazus

Patch Set 2:

i've just copied. this is of course not ready for submit.
I would split off the abis-rsl message from the pcu socket modifications

-- 
To view, visit https://gerrit.osmocom.org/6203
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I0a899d9c866ed09dc301694dbbcad304b1ed49e5
Gerrit-PatchSet: 2
Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Owner: lynxis lazus 
Gerrit-Reviewer: Harald Welte 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: dexter 
Gerrit-Reviewer: lynxis lazus 
Gerrit-HasComments: No


[PATCH] libosmocore[master]: gsm_08_58.h: introduce RSL_IE_ERIC_PAGING_GROUP

2018-04-21 Thread lynxis lazus

Review at  https://gerrit.osmocom.org/7892

gsm_08_58.h: introduce RSL_IE_ERIC_PAGING_GROUP

Ericsson supports a RSL command to page and immediate assign
as single command. For paging a MS the BTS must know the
paging group.

Change-Id: I9194500e307ad69f8da07510bc965a7a5cd82a2a
---
M include/osmocom/gsm/protocol/gsm_08_58.h
1 file changed, 1 insertion(+), 0 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/libosmocore refs/changes/92/7892/1

diff --git a/include/osmocom/gsm/protocol/gsm_08_58.h 
b/include/osmocom/gsm/protocol/gsm_08_58.h
index f0094a3..97f8062 100644
--- a/include/osmocom/gsm/protocol/gsm_08_58.h
+++ b/include/osmocom/gsm/protocol/gsm_08_58.h
@@ -299,6 +299,7 @@
 
 /* Ericsson specific IEs, clash with above partially, so they're not
  * part of the enum */
+#define RSL_IE_ERIC_PAGING_GROUP   0x0e
 #define RSL_IE_ERIC_INST_NR0x48
 #define RSL_IE_ERIC_PGSL_TIMERS0x49
 #define RSL_IE_ERIC_REPEAT_DL_FACCH0x4a

-- 
To view, visit https://gerrit.osmocom.org/7892
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I9194500e307ad69f8da07510bc965a7a5cd82a2a
Gerrit-PatchSet: 1
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Owner: lynxis lazus 


[PATCH] osmo-bsc[master]: abis-rsl: Send imm.ass messages via PCH

2018-04-21 Thread lynxis lazus
Hello Jenkins Builder,

I'd like you to reexamine a change.  Please visit

https://gerrit.osmocom.org/6203

to look at the new patch set (#2).

abis-rsl: Send imm.ass messages via PCH

It is possible to send immidiate assign messages through the
paging channel. This commit adds the required functionality to
the pcu socket interface and to the abis_rsl api

Change-Id: I0a899d9c866ed09dc301694dbbcad304b1ed49e5
---
M include/osmocom/bsc/abis_rsl.h
M include/osmocom/bsc/pcuif_proto.h
M src/libbsc/abis_rsl.c
M src/libbsc/pcu_sock.c
4 files changed, 55 insertions(+), 17 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/osmo-bsc refs/changes/03/6203/2

diff --git a/include/osmocom/bsc/abis_rsl.h b/include/osmocom/bsc/abis_rsl.h
index b30702f..705b692 100644
--- a/include/osmocom/bsc/abis_rsl.h
+++ b/include/osmocom/bsc/abis_rsl.h
@@ -61,6 +61,8 @@
 
 /* Ericcson vendor specific RSL extensions */
 int rsl_ericsson_imm_assign_cmd(struct gsm_bts *bts, uint32_t tlli, uint8_t 
len, uint8_t *val);
+int rsl_ericsson_imm_assign_cmd_via_pch(struct gsm_bts *bts, uint32_t tlli,
+uint8_t len, uint8_t *val, uint8_t pag_grp);
 
 /* Siemens vendor-specific RSL extensions */
 int rsl_siemens_mrpci(struct gsm_lchan *lchan, struct rsl_mrpci *mrpci);
diff --git a/include/osmocom/bsc/pcuif_proto.h 
b/include/osmocom/bsc/pcuif_proto.h
index b9f61b6..8f4a3f9 100644
--- a/include/osmocom/bsc/pcuif_proto.h
+++ b/include/osmocom/bsc/pcuif_proto.h
@@ -3,7 +3,7 @@
 
 #include 
 
-#define PCU_IF_VERSION 0x09
+#define PCU_IF_VERSION 0x10
 #define TXT_MAX_LEN128
 
 /* msg_type */
@@ -27,7 +27,7 @@
 #define PCU_IF_SAPI_PDTCH  0x05/* packet data/control/ccch block */
 #define PCU_IF_SAPI_PRACH  0x06/* packet random access channel */
 #define PCU_IF_SAPI_PTCCH  0x07/* packet TA control channel */
-#define PCU_IF_SAPI_AGCH_DT0x08/* assignment on AGCH but with 
additional TLLI */
+#define PCU_IF_SAPI_AGCH_DT0x08/* assignment on PCH or AGCH but with 
additional TLLI */
 
 /* flags */
 #define PCU_IF_FLAG_ACTIVE (1 << 0)/* BTS is active */
@@ -75,6 +75,7 @@
 struct gsm_pcu_if_data_cnf_dt {
uint8_t sapi;
uint32_ttlli;
+   uint8_t imsi[3];
uint32_tfn;
uint16_tarfcn;
uint8_t trx_nr;
diff --git a/src/libbsc/abis_rsl.c b/src/libbsc/abis_rsl.c
index 2017d2c..656fc84 100644
--- a/src/libbsc/abis_rsl.c
+++ b/src/libbsc/abis_rsl.c
@@ -1096,6 +1096,15 @@
return abis_rsl_sendmsg(msg);
 }
 
+/* Append mobile idenitiy (tlli) to message buffer */
+static void rsl_ericsson_put_mi(struct msgb *msg, uint32_t tlli)
+{
+   /* ericsson can handle a reference at the end of the message which is 
used in
+* the confirm message. The confirm message is only sent if the trailer 
is present */
+   msgb_put_u8(msg, RSL_IE_ERIC_MOBILE_ID);
+   msgb_put_u32(msg, tlli);
+}
+
 /* Chapter 8.5.6 */
 int rsl_ericsson_imm_assign_cmd(struct gsm_bts *bts, uint32_t tlli, uint8_t 
len, uint8_t *val)
 {
@@ -1103,10 +1112,24 @@
if (!msg)
return 1;
 
-   /* ericsson can handle a reference at the end of the message which is 
used in
-* the confirm message. The confirm message is only sent if the trailer 
is present */
-   msgb_put_u8(msg, RSL_IE_ERIC_MOBILE_ID);
-   msgb_put_u32(msg, tlli);
+   /* Append ericsson propritary mobile identity field */
+   rsl_ericsson_put_mi(msg, tlli);
+
+   return abis_rsl_sendmsg(msg);
+}
+
+/* Chapter 8.5.6 (Ericcson vendor specific RSL extension) */
+int rsl_ericsson_imm_assign_cmd_via_pch(struct gsm_bts *bts, uint32_t tlli,
+uint8_t len, uint8_t *val, uint8_t pag_grp)
+{
+   struct msgb *msg = rsl_imm_assign_cmd_common(bts, len, val);
+
+   /* Append ericsson propritary paging group field */
+   msgb_put_u8(msg, RSL_IE_ERIC_PAGING_GROUP);
+   msgb_put_u8(msg, pag_grp);
+
+   /* Append ericsson propritary mobile identity field */
+   rsl_ericsson_put_mi(msg, tlli);
 
return abis_rsl_sendmsg(msg);
 }
diff --git a/src/libbsc/pcu_sock.c b/src/libbsc/pcu_sock.c
index 64422c7..2c477e1 100644
--- a/src/libbsc/pcu_sock.c
+++ b/src/libbsc/pcu_sock.c
@@ -326,9 +326,9 @@
mi_len = p1->data[0];
mi = p1->data+1;
LOGP(DPCU, LOGL_ERROR, "PCU Sends paging "
-"request type %02x (chan_needed=%02x, mi_len=%u, mi=%s)\n",
+"request type %02x (chan_needed=0x%02x, mi_len=%u, mi=%s, 
paging_group=0x%02x)\n",
 p1->msg_type, chan_needed, mi_len,
-osmo_hexdump_nospc(mi,mi_len));
+osmo_hexdump_nospc(mi,mi_len), paging_group);
/* NOTE: We will have to add 2 to mi_len and subtract 2 from
 * the mi pointer because rsl_paging_cmd() will perform the
 * reverse operations. This is because 

osmo-mgw[master]: Revert "stats: use libosmocore rate counter for in/out_strea...

2018-04-21 Thread lynxis lazus

Patch Set 1:

OBS is failing

-- 
To view, visit https://gerrit.osmocom.org/7891
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I15f3c78f8410d709733ed5692ba94ba17559d7e1
Gerrit-PatchSet: 1
Gerrit-Project: osmo-mgw
Gerrit-Branch: master
Gerrit-Owner: lynxis lazus 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: dexter 
Gerrit-Reviewer: lynxis lazus 
Gerrit-HasComments: No


osmo-mgw[master]: Revert "stats: use libosmocore rate counter for in/out_strea...

2018-04-21 Thread lynxis lazus

Patch Set 1:

https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/osmo-mgw/xUbuntu_17.04/i586

-- 
To view, visit https://gerrit.osmocom.org/7891
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I15f3c78f8410d709733ed5692ba94ba17559d7e1
Gerrit-PatchSet: 1
Gerrit-Project: osmo-mgw
Gerrit-Branch: master
Gerrit-Owner: lynxis lazus 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: dexter 
Gerrit-Reviewer: lynxis lazus 
Gerrit-HasComments: No


[PATCH] osmo-mgw[master]: Revert "stats: use libosmocore rate counter for in/out_strea...

2018-04-21 Thread lynxis lazus

Review at  https://gerrit.osmocom.org/7891

Revert "stats: use libosmocore rate counter for in/out_stream.err_ts_counter"

This reverts commit 7181cc1f026a0b63a061296aba4e10a9cadaf2c8.
The tests are broken on i686, arm (non 64bit systems).

Change-Id: I15f3c78f8410d709733ed5692ba94ba17559d7e1
---
M include/osmocom/mgcp/mgcp_internal.h
M src/libosmo-mgcp/mgcp_conn.c
M src/libosmo-mgcp/mgcp_network.c
M src/libosmo-mgcp/mgcp_stat.c
M src/libosmo-mgcp/mgcp_vty.c
M tests/mgcp/mgcp_test.c
6 files changed, 15 insertions(+), 63 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/osmo-mgw refs/changes/91/7891/1

diff --git a/include/osmocom/mgcp/mgcp_internal.h 
b/include/osmocom/mgcp/mgcp_internal.h
index ff02768..0da2c56 100644
--- a/include/osmocom/mgcp/mgcp_internal.h
+++ b/include/osmocom/mgcp/mgcp_internal.h
@@ -28,7 +28,6 @@
 #include 
 #include 
 #include 
-#include 
 
 #define CI_UNUSED 0
 
@@ -46,7 +45,7 @@
uint32_t ssrc;
uint16_t last_seq;
uint32_t last_timestamp;
-   struct rate_ctr *err_ts_ctr;
+   uint32_t err_ts_counter;
int32_t last_tsdelta;
uint32_t last_arrival_time;
 };
@@ -203,8 +202,6 @@
uint32_t octets;
} stats;
} osmux;
-
-   struct rate_ctr_group *rate_ctr_group;
 };
 
 /*! Connection type, specifies which member of the union "u" in mgcp_conn
diff --git a/src/libosmo-mgcp/mgcp_conn.c b/src/libosmo-mgcp/mgcp_conn.c
index 0055049..998dbc5 100644
--- a/src/libosmo-mgcp/mgcp_conn.c
+++ b/src/libosmo-mgcp/mgcp_conn.c
@@ -26,27 +26,7 @@
 #include 
 #include 
 #include 
-#include 
 #include 
-
-const static struct rate_ctr_desc rate_ctr_desc[] = {
-   {
-   .name = "in_stream_err_ts_ctr",
-   .description = "inbound rtp-stream timestamp errors",
-   },{
-   .name = "out_stream_err_ts_ctr",
-   .description = "outbound rtp-stream timestamp errors",
-   }
-};
-
-const static struct rate_ctr_group_desc rate_ctr_group_desc = {
-   .group_name_prefix = "conn_rtp",
-   .group_description = "rtp connection statistics",
-   .class_id = 1,
-   .num_ctr = 2,
-   .ctr_desc = rate_ctr_desc
-};
-
 
 /* Allocate a new connection identifier. According to RFC3435, they must
  * be unique only within the scope of the endpoint. (Caller must provide
@@ -107,10 +87,6 @@
 static void mgcp_rtp_conn_init(struct mgcp_conn_rtp *conn_rtp, struct 
mgcp_conn *conn)
 {
struct mgcp_rtp_end *end = _rtp->end;
-   /* FIXME: Each new rate counter group requires an unique index. At the
-* moment we generate this index using this counter, but perhaps there
-* is a more concious way to assign the indexes. */
-   static unsigned int rate_ctr_index = 0;
 
conn_rtp->type = MGCP_RTP_DEFAULT;
conn_rtp->osmux.allocated_cid = -1;
@@ -132,15 +108,6 @@
 
mgcp_rtp_codec_init(>codec);
mgcp_rtp_codec_init(>alt_codec);
-
-   conn_rtp->rate_ctr_group =
-   rate_ctr_group_alloc(conn, _ctr_group_desc,
-rate_ctr_index);
-   conn_rtp->state.in_stream.err_ts_ctr =
-   _rtp->rate_ctr_group->ctr[0];
-   conn_rtp->state.out_stream.err_ts_ctr =
-   _rtp->rate_ctr_group->ctr[1];
-   rate_ctr_index++;
 }
 
 /* Cleanup rtp connection struct */
@@ -149,7 +116,6 @@
osmux_disable_conn(conn_rtp);
osmux_release_cid(conn_rtp);
mgcp_free_rtp_port(_rtp->end);
-   rate_ctr_group_free(conn_rtp->rate_ctr_group);
 }
 
 /*! allocate a new connection list entry.
diff --git a/src/libosmo-mgcp/mgcp_network.c b/src/libosmo-mgcp/mgcp_network.c
index c56e433..6923b97 100644
--- a/src/libosmo-mgcp/mgcp_network.c
+++ b/src/libosmo-mgcp/mgcp_network.c
@@ -222,7 +222,7 @@
 
if (seq == sstate->last_seq) {
if (timestamp != sstate->last_timestamp) {
-   rate_ctr_inc(sstate->err_ts_ctr);
+   sstate->err_ts_counter += 1;
LOGP(DRTP, LOGL_ERROR,
 "The %s timestamp delta is != 0 but the sequence "
 "number %d is the same, "
@@ -272,7 +272,7 @@
ts_alignment_error(sstate, state->packet_duration, timestamp);
 
if (timestamp_error) {
-   rate_ctr_inc(sstate->err_ts_ctr);
+   sstate->err_ts_counter += 1;
LOGP(DRTP, LOGL_NOTICE,
 "The %s timestamp has an alignment error of %d "
 "on 0x%x SSRC: %u "
@@ -505,16 +505,13 @@
mgcp_rtp_annex_count(endp, state, seq, transit, ssrc);
 
if (!state->initialized) {
-   /* FIXME: Move this initialization to mgcp.conn.c */
state->initialized = 1;
state->in_stream.last_seq = seq - 1;
state->in_stream.ssrc = state->patch.orig_ssrc = ssrc;
state->in_stream.last_tsdelta = 0;

[PATCH] osmo-trx[master]: doc: examples: Add umtrx sample config

2018-04-21 Thread Pau Espin Pedrol

Review at  https://gerrit.osmocom.org/7890

doc: examples: Add umtrx sample config

Change-Id: Id38de0bbbe75e5e6bbb0de2eecb7d1984786d528
---
A doc/examples/osmo-trx-umtrx.cfg
1 file changed, 22 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/osmo-trx refs/changes/90/7890/1

diff --git a/doc/examples/osmo-trx-umtrx.cfg b/doc/examples/osmo-trx-umtrx.cfg
new file mode 100644
index 000..b7a73b0
--- /dev/null
+++ b/doc/examples/osmo-trx-umtrx.cfg
@@ -0,0 +1,22 @@
+log stderr
+  logging filter all 1
+  logging color 1
+  logging print category 1
+  logging timestamp 1
+  logging level all info
+!
+line vty
+ no login
+!
+trx
+ bind-ip 127.0.0.1
+ remote-ip 127.0.0.1
+ base-port 5700
+ dev-args 
addr=192.168.10.2,pa=NONE,pa_power_max_dbm=23,fifo_ctrl_window=0,status_port=12345
+ egprs disable
+ tx-sps 4
+ rx-sps 4
+ rssi-offset 38
+ rt-prio 18
+ chan 0
+ chan 1

-- 
To view, visit https://gerrit.osmocom.org/7890
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Id38de0bbbe75e5e6bbb0de2eecb7d1984786d528
Gerrit-PatchSet: 1
Gerrit-Project: osmo-trx
Gerrit-Branch: master
Gerrit-Owner: Pau Espin Pedrol 


Build failed in Jenkins: master-asn1c » a1=default,a2=default,a3=default,osmocom-master-debian9 #99

2018-04-21 Thread jenkins
See 


--
[...truncated 2.15 KB...]
checking build system type... x86_64-unknown-linux-gnu
checking host system type... x86_64-unknown-linux-gnu
checking target system type... x86_64-unknown-linux-gnu
checking for a BSD-compatible install... /usr/bin/install -c
checking whether build environment is sane... yes
checking for gawk... gawk
checking whether make sets $(MAKE)... yes
checking whether to enable maintainer-specific portions of Makefiles... no
checking for style of include used by make... GNU
checking for gcc... gcc
checking for C compiler default output file name... a.out
checking whether the C compiler works... yes
checking whether we are cross compiling... no
checking for suffix of executables... 
checking for suffix of object files... o
checking whether we are using the GNU C compiler... yes
checking whether gcc accepts -g... yes
checking for gcc option to accept ISO C89... none needed
checking dependency style of gcc... gcc3
checking for a sed that does not truncate output... /bin/sed
checking for grep that handles long lines and -e... /bin/grep
checking for egrep... /bin/grep -E
checking for ld used by gcc... /usr/bin/ld
checking if the linker (/usr/bin/ld) is GNU ld... yes
checking for /usr/bin/ld option to reload object files... -r
checking for BSD-compatible nm... /usr/bin/nm -B
checking whether ln -s works... yes
checking how to recognise dependent libraries... pass_all
checking how to run the C preprocessor... gcc -E
checking for ANSI C header files... yes
checking for sys/types.h... yes
checking for sys/stat.h... yes
checking for stdlib.h... yes
checking for string.h... yes
checking for memory.h... yes
checking for strings.h... yes
checking for inttypes.h... yes
checking for stdint.h... yes
checking for unistd.h... yes
checking dlfcn.h usability... yes
checking dlfcn.h presence... yes
checking for dlfcn.h... yes
checking for g++... g++
checking whether we are using the GNU C++ compiler... yes
checking whether g++ accepts -g... yes
checking dependency style of g++... gcc3
checking how to run the C++ preprocessor... g++ -E
checking for g77... no
checking for f77... no
checking for xlf... no
checking for frt... no
checking for pgf77... no
checking for cf77... no
checking for fort77... no
checking for fl32... no
checking for af77... no
checking for f90... no
checking for xlf90... no
checking for pgf90... no
checking for pghpf... no
checking for epcf90... no
checking for gfortran... no
checking for g95... no
checking for f95... no
checking for fort... no
checking for xlf95... no
checking for ifort... no
checking for ifc... no
checking for efc... no
checking for pgf95... no
checking for lf95... no
checking for ftn... no
checking whether we are using the GNU Fortran 77 compiler... no
checking whether  accepts -g... no
checking the maximum length of command line arguments... 32768
checking command to parse /usr/bin/nm -B output from gcc object... ok
checking for objdir... .libs
checking for ar... ar
checking for ranlib... ranlib
checking for strip... strip
checking if gcc supports -fno-rtti -fno-exceptions... no
checking for gcc option to produce PIC... -fPIC
checking if gcc PIC flag -fPIC works... yes
checking if gcc static flag -static works... yes
checking if gcc supports -c -o file.o... yes
checking whether the gcc linker (/usr/bin/ld -m elf_x86_64) supports shared 
libraries... yes
checking whether -lc should be explicitly linked in... no
checking dynamic linker characteristics... GNU/Linux ld.so
checking how to hardcode library paths into programs... immediate
checking whether stripping libraries is possible... yes
checking if libtool supports shared libraries... yes
checking whether to build shared libraries... yes
checking whether to build static libraries... yes
configure: creating libtool
appending configuration tag "CXX" to libtool
checking for ld used by g++... /usr/bin/ld -m elf_x86_64
checking if the linker (/usr/bin/ld -m elf_x86_64) is GNU ld... yes
checking whether the g++ linker (/usr/bin/ld -m elf_x86_64) supports shared 
libraries... yes
checking for g++ option to produce PIC... -fPIC
checking if g++ PIC flag -fPIC works... yes
checking if g++ static flag -static works... yes
checking if g++ supports -c -o file.o... yes
checking whether the g++ linker (/usr/bin/ld -m elf_x86_64) supports shared 
libraries... yes
checking dynamic linker characteristics... GNU/Linux ld.so
checking how to hardcode library paths into programs... immediate
appending configuration tag "F77" to libtool
checking for autoconf... /usr/bin/autoconf
checking for autoheader... /usr/bin/autoheader
checking for gcc... (cached) gcc
checking whether we are using the GNU C compiler... (cached) yes
checking whether gcc accepts -g... (cached) yes
checking for gcc option to accept ISO C89... (cached) none needed
checking dependency style