Patch Set 1: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/2602
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I129f5189ce2bebd761bc752b158fb8778b31010e
Gerrit-PatchSet: 1
Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Patch Set 1:
(1 comment)
https://gerrit.osmocom.org/#/c/2606/1/src/osmo_gsm_tester/bts_sysmo.py
File src/osmo_gsm_tester/bts_sysmo.py:
Line 50: self.run_dir =
util.Dir(os.path.abspath(self.suite_run.trial.get_run_dir().new_dir(self.name(
I think it would be better to use abspat
Patch Set 2:
(1 comment)
https://gerrit.osmocom.org/#/c/2608/2/contrib/jenkins-run.sh
File contrib/jenkins-run.sh:
Line 20: exit_code="1"
Isn't the script be terminated before reaching the end due to set -e if
osmo-gsm-tester.py exits with != 0?
--
To view, visit https://gerrit.osmocom.org/
Patch Set 1: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/2609
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I42ab624f8dd8e91ee62b8eccfa451e01fd3694d0
Gerrit-PatchSet: 1
Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Patch Set 2: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/2598
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ie92f1879a35d8bb399ee916b7ef72f9ee7d47409
Gerrit-PatchSet: 2
Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Patch Set 1:
I saw you already did that in another commit afterwards. I guess we can drop
this one then.
--
To view, visit https://gerrit.osmocom.org/2606
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ib430657fd0d83a1a59b4b4b0ad6186c9
Patch Set 2: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/2607
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ia2ba876a42eef9122a92645edd1f28ddb56e6be3
Gerrit-PatchSet: 2
Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Max has submitted this change and it was merged.
Change subject: Prepare for BTS attribute reporting via OML
..
Prepare for BTS attribute reporting via OML
* move BTS model name resolution into separate function
* add convenien
Max has submitted this change and it was merged.
Change subject: Prepare for BTS attribute reporting via OML
..
Prepare for BTS attribute reporting via OML
* use define for number of attributes instead of magic number
* add sub
Hello Harald Welte, Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/2544
to look at the new patch set (#3).
Get basic BTS attributes
Request BTS attributes via OML on connection and parse the response:
request/parse incoming response as sw-conf
Hello Harald Welte, Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/2588
to look at the new patch set (#4).
Restructure SI2quater generation
In preparation for extended SI2q messages:
* add SI2q-specific accessor macro
* add *_offset variables
Patch Set 1: Code-Review+1
--
To view, visit https://gerrit.osmocom.org/2611
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic36eafbb9868b2ec9ae2173f8d1b651b976ccf88
Gerrit-PatchSet: 1
Gerrit-Project: libosmo-sccp
Gerrit-Branch: master
Ge
Patch Set 1:
How was this found? Is there specific test case we might add which fails with
old code but works with new one?
--
To view, visit https://gerrit.osmocom.org/2592
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Id52ddd8358fd4a
Patch Set 1: Code-Review+1
--
To view, visit https://gerrit.osmocom.org/2591
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic3d5da00f7ece6dbcd4c999187a5748c9331e60f
Gerrit-PatchSet: 1
Gerrit-Project: libosmocore
Gerrit-Branch: master
Ger
Patch Set 1:
> How was this found? Is there specific test case we might add which
> fails with old code but works with new one?
This was found by cppcheck: "unreachable part of code".
My assumption is that the TVLV_GROSS_LEN was merely
copy-pasted to VTVLV_GAN_GROSS_LEN, and return statements
w
Patch Set 1:
sorry, I missed that patch ... will comment there
--
To view, visit https://gerrit.osmocom.org/2595
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Icbb0f8d2058fa7ebb7f0f731645f9266cacdb120
Gerrit-PatchSet: 1
Gerrit-Project:
Patch Set 2:
(1 comment)
https://gerrit.osmocom.org/#/c/2608/2/contrib/jenkins-run.sh
File contrib/jenkins-run.sh:
Line 20: exit_code="1"
> Isn't the script be terminated before reaching the end due to set -e if osm
since 'if' evaluates the return code, it does not exit immediately on failure
Review at https://gerrit.osmocom.org/2612
SI2quater: fix EARFCN inclusion check
Previously only the existance of bts->si_common.si2quater_neigh_list was
checked but not the actual number of EARFCNs in it. Fix it by using
si2q_earfcn_count() and adjust tests accordingly. While at it - reformat
t
Patch Set 1:
You're probably right. I still wonder how can we test this for sure. Do we have
GAN implementation using those somewhere?
--
To view, visit https://gerrit.osmocom.org/2592
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Id52
Patch Set 1:
(1 comment)
https://gerrit.osmocom.org/#/c/2606/1/src/osmo_gsm_tester/bts_sysmo.py
File src/osmo_gsm_tester/bts_sysmo.py:
Line 50: self.run_dir =
util.Dir(os.path.abspath(self.suite_run.trial.get_run_dir().new_dir(self.name(
> I think it would be better to use absp
Patch Set 1:
(1 comment)
https://gerrit.osmocom.org/#/c/2604/1/src/osmo-gsm-tester.py
File src/osmo-gsm-tester.py:
Line 216: log.log_exn()
> could it be an issue: if at some early stage before log object is initializ
'pass' would "swallow" the exception, the 'raise' keeps it alive and p
Hello Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/2604
to look at the new patch set (#2).
fix: exception outside of trial run should be logged and cause nonzero rc
>From an earlier stage of the code, there was still an exception catcher tha
Patch Set 2:
(1 comment)
https://gerrit.osmocom.org/#/c/2600/2/src/osmo_gsm_tester/trial.py
File src/osmo_gsm_tester/trial.py:
Line 77: self.log('Detailed log at', detailed_log)
> join this logging with the one above it?
I considered it, but adding the long abspath to the 'Trial start'
Patch Set 1: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/2594
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I574937dbf31bce49cfb7523f91041c20fecb421e
Gerrit-PatchSet: 1
Gerrit-Project: osmo-gsm-manuals
Gerrit-Branch: maste
Neels Hofmeyr has submitted this change and it was merged.
Change subject: fix PACCH paging: don't return early in case of NULL TBF
..
fix PACCH paging: don't return early in case of NULL TBF
Commit b78a4a6dfef217c538d45949a6ae
Patch Set 3: Code-Review+2
No comments happening, I think we need to resolve it and fix the PCU. I was
going to revert, but lynxis' "looks good to me" encourages me to merge this.
--
To view, visit https://gerrit.osmocom.org/2420
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerri
Neels Hofmeyr has submitted this change and it was merged.
Change subject: OsmoGSMTester: add install docs; fixes and tweaks
..
OsmoGSMTester: add install docs; fixes and tweaks
Change-Id: I574937dbf31bce49cfb7523f91041c20fecb4
Hello Pau Espin Pedrol, Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/2597
to look at the new patch set (#2).
cosmetic: rather have an underscore in pcap_recorder.py
Change-Id: Ib4886cb3fe0cb9c66d170097b98b4d1eb67cab81
---
M src/osmo_gsm_test
Patch Set 2: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/2597
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ib4886cb3fe0cb9c66d170097b98b4d1eb67cab81
Gerrit-PatchSet: 2
Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Neels Hofmeyr has submitted this change and it was merged.
Change subject: cosmetic: rather have an underscore in pcap_recorder.py
..
cosmetic: rather have an underscore in pcap_recorder.py
Change-Id: Ib4886cb3fe0cb9c66d170097b
Patch Set 2: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/2608
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I949849d85d93cb47c839cd738788cf6f9b22fc12
Gerrit-PatchSet: 2
Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Patch Set 1:
(1 comment)
https://gerrit.osmocom.org/#/c/2606/1/src/osmo_gsm_tester/bts_sysmo.py
File src/osmo_gsm_tester/bts_sysmo.py:
Line 50: self.run_dir =
util.Dir(os.path.abspath(self.suite_run.trial.get_run_dir().new_dir(self.name(
> I did something in that line in https:
Patch Set 1:
> I still wonder how can we test this for sure.
> Do we have GAN implementation using those somewhere?
IDK. Harald (as git blame shows) should know better.
--
To view, visit https://gerrit.osmocom.org/2592
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageTy
Patch Set 2: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/2600
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Id5f443f1af0dc774d4d4c588a238b786a83e44d3
Gerrit-PatchSet: 2
Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Patch Set 2: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/2604
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I8085318c91b06a3e8f7d3f8cfdd15a99650666e2
Gerrit-PatchSet: 2
Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Pau Espin Pedrol has abandoned this change.
Change subject: jenkins-run: Provide a link to the latest trial archived
..
Abandoned
--
To view, visit https://gerrit.osmocom.org/2443
To unsubscribe, visit https://gerrit.osmocom.o
Harald Welte has restored this change.
Change subject: stats: Build UDP socket code only if sys/socket.h exists
..
Restored
--
To view, visit https://gerrit.osmocom.org/1607
To unsubscribe, visit https://gerrit.osmocom.org/set
Hello Jenkins Builder, Holger Freyther,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/1607
to look at the new patch set (#2).
stats: Build UDP socket code only if sys/socket.h exists
Change-Id: I6beb4fcc394ed7d3f8dd7a097b6e998245ecf401
---
M include/osmocom/c
Review at https://gerrit.osmocom.org/2622
endian.h: Make it work on 'bare iron' builds (and possibly more platforms)
We can use __BYTE_ORDER__ which seems to be defined by both gcc and
clang/llvm.
Change-Id: Id6821c99e88242126d9697099b1dd92c6212526a
---
M include/osmocom/core/endian.h
1 file c
Review at https://gerrit.osmocom.org/2624
use osmo_{htonl,htons,ntohl,ntohs}() functions all over libosmocore
This gets us one step closer to fixing the embedded build
Change-Id: I3fc2639b6ade9ab138766987eceab7ec9498fdc7
---
M src/gsm/gsm0341.c
M src/gsm/gsm0808.c
M src/gsm/gsm0808_utils.c
M s
Review at https://gerrit.osmocom.org/2626
libosmogb: Convert to use new endian conversion helpers
Change-Id: I5ed17702cd9487e1cab6bae39802aa11fb6a37b0
---
M src/gb/gprs_bssgp.c
M src/gb/gprs_bssgp_bss.c
M src/gb/gprs_bssgp_util.c
M src/gb/gprs_ns.c
M src/gb/gprs_ns_frgre.c
M src/gb/gprs_ns_vty.
Review at https://gerrit.osmocom.org/2627
gsm0808 + ipa: fix compilation on systems without sys/socket.h
Change-Id: I60f5d4982cd96ab29f9924ec11b46bf56fbfc346
---
M include/osmocom/gsm/gsm0808.h
M include/osmocom/gsm/gsm0808_utils.h
M src/gsm/gsm0808_utils.c
M src/gsm/ipa.c
4 files changed, 17 i
Review at https://gerrit.osmocom.org/2628
strrb: Add missing #include to talloc.h
The strrb.c code uses talloc API without using the proper #include.
Change-Id: I4ac485324f40545339d8df6884f1dea319a23648
---
M src/strrb.c
1 file changed, 1 insertion(+), 0 deletions(-)
git pull ssh://gerrit.
Review at https://gerrit.osmocom.org/2623
introduce byteswap.h with osmo_{htonl,ntohl,htons,ntohs}
We need to have an architecture-independend way of endian conversion /
byte swapping functions which will also work on embedded (bare iron)
builds. Let's introduce osmocom/core/bytesawp.h for th
Review at https://gerrit.osmocom.org/2625
tlv: Add tlvp_val{16,32}be() to get TLV values in unaligned host byte order
Change-Id: I3749e1e4974742bc8c32722e0e998775b5394829
---
M include/osmocom/gsm/tlv.h
1 file changed, 23 insertions(+), 0 deletions(-)
git pull ssh://gerrit.osmocom.org:29418
Review at https://gerrit.osmocom.org/2621
src/logging.c: Fix EMBEDDED build
We don't enable/build _file_output in EMBEDDED builds, so we shouldn'r
refer to that symbol.
Change-Id: I491aa8ee9d20a96cbb2814700475afe03eb99c9e
---
M src/logging.c
1 file changed, 1 insertion(+), 1 deletion(-)
gi
Patch Set 1: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/2628
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I4ac485324f40545339d8df6884f1dea319a23648
Gerrit-PatchSet: 1
Gerrit-Project: libosmocore
Gerrit-Branch: master
Ger
Patch Set 1: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/2625
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I3749e1e4974742bc8c32722e0e998775b5394829
Gerrit-PatchSet: 1
Gerrit-Project: libosmocore
Gerrit-Branch: master
Ger
Patch Set 1: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/2627
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I60f5d4982cd96ab29f9924ec11b46bf56fbfc346
Gerrit-PatchSet: 1
Gerrit-Project: libosmocore
Gerrit-Branch: master
Ger
Patch Set 1: Code-Review-1
documentation missing
--
To view, visit https://gerrit.osmocom.org/2623
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ibc0cc1e36d4ed63a35cf8ceff3af0f26e5ac7a3d
Gerrit-PatchSet: 1
Gerrit-Project: libosmocore
Ge
Patch Set 1: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/2622
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Id6821c99e88242126d9697099b1dd92c6212526a
Gerrit-PatchSet: 1
Gerrit-Project: libosmocore
Gerrit-Branch: master
Ger
Patch Set 1: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/2621
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I491aa8ee9d20a96cbb2814700475afe03eb99c9e
Gerrit-PatchSet: 1
Gerrit-Project: libosmocore
Gerrit-Branch: master
Ger
Patch Set 1: Code-Review+1
--
To view, visit https://gerrit.osmocom.org/2624
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I3fc2639b6ade9ab138766987eceab7ec9498fdc7
Gerrit-PatchSet: 1
Gerrit-Project: libosmocore
Gerrit-Branch: master
Ger
Patch Set 1: Code-Review+1
--
To view, visit https://gerrit.osmocom.org/2626
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I5ed17702cd9487e1cab6bae39802aa11fb6a37b0
Gerrit-PatchSet: 1
Gerrit-Project: libosmocore
Gerrit-Branch: master
Ger
Hello Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/2623
to look at the new patch set (#2).
introduce byteswap.h with osmo_{htonl,ntohl,htons,ntohs}
We need to have an architecture-independend way of endian conversion /
byte swapping function
Harald Welte has submitted this change and it was merged.
Change subject: endian.h: Make it work on 'bare iron' builds (and possibly more
platforms)
..
endian.h: Make it work on 'bare iron' builds (and possibly more platforms)
Harald Welte has submitted this change and it was merged.
Change subject: src/logging.c: Fix EMBEDDED build
..
src/logging.c: Fix EMBEDDED build
We don't enable/build _file_output in EMBEDDED builds, so we shouldn'r
refer to th
Patch Set 2: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/2623
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ibc0cc1e36d4ed63a35cf8ceff3af0f26e5ac7a3d
Gerrit-PatchSet: 2
Gerrit-Project: libosmocore
Gerrit-Branch: master
Ger
Hello Jenkins Builder, Holger Freyther,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/1607
to look at the new patch set (#4).
stats: Disable stats*.c on embedded targets
Change-Id: I6beb4fcc394ed7d3f8dd7a097b6e998245ecf401
---
M include/osmocom/core/stats.h
M
Hello Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/2586
to look at the new patch set (#2).
nitb: Filter ssh packets in pcap
Change-Id: I62a6ae7bd3a84baceb684c26727d2269c86ed023
---
M src/osmo_gsm_tester/osmo_nitb.py
1 file changed, 1 inserti
Review at https://gerrit.osmocom.org/2629
pcap: pass complete filter string to PcapRecorder
Composing the filter in gen_filter() is more complex than we need. We pass the
address and potentially further filter elements separately, and then
gen_filter() has to guess how to combine these. Instead
Patch Set 4:
(1 comment)
https://gerrit.osmocom.org/#/c/1607/4/src/stats.c
File src/stats.c:
Line 245: #endif
Would be nice to get a comment on where this belongs to - similar to other
#endifs in this file.
--
To view, visit https://gerrit.osmocom.org/1607
To unsubscribe, visit https://gerr
Hello Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/2596
to look at the new patch set (#2).
pcap recorder: include vim indenting rules as comment
Change-Id: Idbc4521053610d55c9c32eff950ed8fa68dda171
---
M src/osmo_gsm_tester/pcap_recorder.py
Patch Set 1: Code-Review+1
(3 comments)
(cosmetics)
https://gerrit.osmocom.org/#/c/2585/1/src/osmo_gsm_tester/bts_octphy.py
File src/osmo_gsm_tester/bts_octphy.py:
Line 57: '-i', self.nitb.addr())
(the oct bts is not working and we're not going to make it work at th
pcap: pass complete filter string to PcapRecorder
Composing the filter in gen_filter() is more complex than we need. We pass the
address and potentially further filter elements separately, and then
gen_filter() has to guess how to combine these. Instead, have just a filter
string to use right fro
Neels Hofmeyr has submitted this change and it was merged.
Change subject: log the location of the detailed log file
..
log the location of the detailed log file
The idea is that if someone reads the short log and scratches her
Neels Hofmeyr has submitted this change and it was merged.
Change subject: debug suite: add test that deliberately fails
..
debug suite: add test that deliberately fails
Change-Id: I42ab624f8dd8e91ee62b8eccfa451e01fd3694d0
---
Neels Hofmeyr has submitted this change and it was merged.
Change subject: log: tweak osmo-gsm-tester.py stdout: all_origins=False,
src=False
..
log: tweak osmo-gsm-tester.py stdout: all_origins=False, src=False
Change-Id: I24
Neels Hofmeyr has submitted this change and it was merged.
Change subject: log state dir
..
log state dir
On the log output, clearly show which state dir is used. Since all runs should
use the same state dir, this is important
Neels Hofmeyr has abandoned this change.
Change subject: pcap recorder: filter out SSH
..
Abandoned
-> #2629 and #2586
--
To view, visit https://gerrit.osmocom.org/2595
To unsubscribe, visit https://gerrit.osmocom.org/setting
Neels Hofmeyr has submitted this change and it was merged.
Change subject: pcap recorder: include vim indenting rules as comment
..
pcap recorder: include vim indenting rules as comment
Change-Id: Idbc4521053610d55c9c32eff950ed
Patch Set 2: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/2596
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Idbc4521053610d55c9c32eff950ed8fa68dda171
Gerrit-PatchSet: 2
Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Neels Hofmeyr has submitted this change and it was merged.
Change subject: contrib: rethink jenkins jobs with build slave approach
..
contrib: rethink jenkins jobs with build slave approach
Previously, the idea was to run a tes
Neels Hofmeyr has submitted this change and it was merged.
Change subject: checkout dependencies from script
..
checkout dependencies from script
The jenkins jobs used to have multiple source repositories configured, but it
is
Neels Hofmeyr has submitted this change and it was merged.
Change subject: cosmetic: contrib: add 'build-' to script names
..
cosmetic: contrib: add 'build-' to script names
Distinguish the scripts that build binaries from the
Neels Hofmeyr has submitted this change and it was merged.
Change subject: fix: exception outside of trial run should be logged and cause
nonzero rc
..
fix: exception outside of trial run should be logged and cause nonzero rc
Neels Hofmeyr has submitted this change and it was merged.
Change subject: fix name of local var, to not shadow 'suite' module
..
fix name of local var, to not shadow 'suite' module
This caused "UnboundLocalError: local variabl
Patch Set 2: Code-Review+1 Verified+1
I'm working on pcu-related issue anyway so I've decided to give it a quick try:
works as good as master so I think it's safe to merge it.
--
To view, visit https://gerrit.osmocom.org/1753
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-Me
Patch Set 2: Code-Review+1
since I hijacked this patch of yours, please add your review to the patch
despite being in your name.
--
To view, visit https://gerrit.osmocom.org/2586
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I62a6ae7bd
Patch Set 2: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/2624
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I3fc2639b6ade9ab138766987eceab7ec9498fdc7
Gerrit-PatchSet: 2
Gerrit-Project: libosmocore
Gerrit-Branch: master
Ger
Patch Set 2: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/2626
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I5ed17702cd9487e1cab6bae39802aa11fb6a37b0
Gerrit-PatchSet: 2
Gerrit-Project: libosmocore
Gerrit-Branch: master
Ger
Harald Welte has submitted this change and it was merged.
Change subject: gsm0808 + ipa: fix compilation on systems without sys/socket.h
..
gsm0808 + ipa: fix compilation on systems without sys/socket.h
Change-Id: I60f5d4982cd9
Harald Welte has submitted this change and it was merged.
Change subject: libosmogb: Convert to use new endian conversion helpers
..
libosmogb: Convert to use new endian conversion helpers
Change-Id: I5ed17702cd9487e1cab6bae398
Harald Welte has submitted this change and it was merged.
Change subject: tlv: Add tlvp_val{16,32}be() to get TLV values in unaligned
host byte order
..
tlv: Add tlvp_val{16,32}be() to get TLV values in unaligned host byte orde
Harald Welte has submitted this change and it was merged.
Change subject: introduce byteswap.h with osmo_{htonl,ntohl,htons,ntohs}
..
introduce byteswap.h with osmo_{htonl,ntohl,htons,ntohs}
We need to have an architecture-inde
Harald Welte has submitted this change and it was merged.
Change subject: strrb: Add missing #include to talloc.h
..
strrb: Add missing #include to talloc.h
The strrb.c code uses talloc API without using the proper #include.
C
Harald Welte has submitted this change and it was merged.
Change subject: use osmo_{htonl,htons,ntohl,ntohs}() functions all over
libosmocore
..
use osmo_{htonl,htons,ntohl,ntohs}() functions all over libosmocore
This gets us
Patch Set 1: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/2611
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic36eafbb9868b2ec9ae2173f8d1b651b976ccf88
Gerrit-PatchSet: 1
Gerrit-Project: libosmo-sccp
Gerrit-Branch: master
Ge
Patch Set 2: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/2629
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Iff7ddf51d3bf0189ce07b488a3dcdcfce6907aba
Gerrit-PatchSet: 2
Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Harald Welte has submitted this change and it was merged.
Change subject: remove pcu own bitvector implementation
..
remove pcu own bitvector implementation
The osmocore bitvec is exact the same, but use a pointer instead of
a
Neels Hofmeyr has submitted this change and it was merged.
Change subject: jenkins-run: clean up also upon error
..
jenkins-run: clean up also upon error
Change-Id: I949849d85d93cb47c839cd738788cf6f9b22fc12
---
M contrib/jenkin
Neels Hofmeyr has submitted this change and it was merged.
Change subject: trial: always use abspath
..
trial: always use abspath
No actual failure known, but using the abspath will prevent further problems
when running a binar
Patch Set 2: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/1753
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Id8f797631d89aa12b6e48efb2dc153a3e2f059f7
Gerrit-PatchSet: 2
Gerrit-Project: osmo-pcu
Gerrit-Branch: master
Gerrit
Patch Set 2: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/2586
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I62a6ae7bd3a84baceb684c26727d2269c86ed023
Gerrit-PatchSet: 2
Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Neels Hofmeyr has submitted this change and it was merged.
Change subject: pcap: pass complete filter string to PcapRecorder
..
pcap: pass complete filter string to PcapRecorder
Composing the filter in gen_filter() is more comp
Neels Hofmeyr has submitted this change and it was merged.
Change subject: nitb: Filter ssh packets in pcap
..
nitb: Filter ssh packets in pcap
Change-Id: I62a6ae7bd3a84baceb684c26727d2269c86ed023
---
M src/osmo_gsm_tester/osmo
Patch Set 1: Code-Review+2
> You're probably right. I still wonder how can we test this for
> sure. Do we have GAN implementation using those somewhere?
There is some (long abendoned) UMA/GAN code in openbsc.git branch
laforge/ganc-2012 branch.
--
To view, visit https://gerrit.osmocom.org/25
Patch Set 1: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/2591
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic3d5da00f7ece6dbcd4c999187a5748c9331e60f
Gerrit-PatchSet: 1
Gerrit-Project: libosmocore
Gerrit-Branch: master
Ger
Harald Welte has submitted this change and it was merged.
Change subject: gsm/tlv.h: fix copy-paste error
..
gsm/tlv.h: fix copy-paste error
Change-Id: Id52ddd8358fd4af1ecc71142deb09f1e237021f3
---
M include/osmocom/gsm/tlv.h
1
Harald Welte has submitted this change and it was merged.
Change subject: fsm_test.c: fix unreachable check
..
fsm_test.c: fix unreachable check
Change-Id: Ic3d5da00f7ece6dbcd4c999187a5748c9331e60f
---
M tests/fsm/fsm_test.c
1
1 - 100 of 145 matches
Mail list logo