Patch Set 2: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/2368
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I399e2986c9d8cb5b3dd31673a6b4bf63070a4770
Gerrit-PatchSet: 2
Gerrit-Project: libosmo-netif
Gerrit-Branch: master
Patch Set 2: Code-Review+1
Hmm.. I would prefer to add the comments in another patch. Try to keep patches
minimal and on a single topic.
--
To view, visit https://gerrit.osmocom.org/2446
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id:
Patch Set 2: Code-Review+1
In two years we won't know what "latest Ubuntu" is. Always better to put a
version number in there. Stretch won't have it either.
--
To view, visit https://gerrit.osmocom.org/2400
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Patch Set 7:
(1 comment)
https://gerrit.osmocom.org/#/c/2286/7/openbsc/src/libcommon/gsm_data_shared.c
File openbsc/src/libcommon/gsm_data_shared.c:
Line 70: const struct value_string bts_variant_names[_NUM_BTS_VARIANT + 1] = {
BikeShedding:
* bts_variant.. they are all osmo-bts variants.
Harald Welte has submitted this change and it was merged.
Change subject: test: Add a manual test to simulate DTMF handling in
osmo-sip-connector
..
test: Add a manual test to simulate DTMF handling in osmo-sip-connector
Add
Patch Set 1: Code-Review+2 Verified+1
--
To view, visit https://gerrit.osmocom.org/2452
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I7f62efbc62455e6fcb1f091afb5fa120099834f0
Gerrit-PatchSet: 1
Gerrit-Project: mncc-python
Harald Welte has submitted this change and it was merged.
Change subject: mncc: Make it possible to build a MNCC server for testing
..
mncc: Make it possible to build a MNCC server for testing
For manual testing the
Patch Set 1: Verified+1
--
To view, visit https://gerrit.osmocom.org/2451
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I8387fe1687557c6a1a26ff1e0cc9dbff3087aa82
Gerrit-PatchSet: 1
Gerrit-Project: mncc-python
Gerrit-Branch: master
Patch Set 1: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/2451
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I8387fe1687557c6a1a26ff1e0cc9dbff3087aa82
Gerrit-PatchSet: 1
Gerrit-Project: mncc-python
Gerrit-Branch: master
Patch Set 2:
> (1 comment)
No, of course. SSE3 support is mandatory for accelerated Viterbi
implementation only (viterbi_sse.c). If SSE3 isn't supported by
target CPU, then not-accelerated Viterbi implementation will be
used (viterbi_sse.c).
BTW: I forgot to change commit author :(
--
To
Patch Set 2:
(1 comment)
https://gerrit.osmocom.org/#/c/2454/2//COMMIT_MSG
Commit Message:
Line 11: is the minimal requirement. SSE4.1 and AVX2 are used if available.
Do you mean this patch introduces a mandatory requirement for SSE3 capable
CPUs? if so, it is not acceptable.
--
To view,
Patch Set 7: Code-Review+2
deferring to Tom's judgement here, but generally it looks fine to me, and if
there are issues remaining, they can be adressed later on.
--
To view, visit https://gerrit.osmocom.org/1628
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType:
Hello Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/2454
to look at the new patch set (#2).
core/conv: add x86 SSE support for Viterbi decoder
Fast convolutional decoding is provided through x86 intrinsic based
SSE operations. SSE3, found on
Review at https://gerrit.osmocom.org/2453
configure.ac: add SIMD detection capabilities
This change adds a new configure flag, which triggers a check
whether compiller supports some SIMD (Single Instruction,
Multiple Data) instructions. The check macro is based on
the AX_EXT from
Review at https://gerrit.osmocom.org/2454
core/conv: add x86 SSE support for Viterbi decoder
Fast convolutional decoding is provided through x86 intrinsic based
SSE operations. SSE3, found on virtually all modern x86 processors,
is the minimal requirement. SSE4.1 and AVX2 are used if
Patch Set 3: Code-Review+1
--
To view, visit https://gerrit.osmocom.org/2098
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I3df4b8db9692016115edbe2247beeec090715687
Gerrit-PatchSet: 3
Gerrit-Project: osmo-trx
Gerrit-Branch: master
Patch Set 3: Code-Review+1
--
To view, visit https://gerrit.osmocom.org/2101
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Iacc83fd668c31644e0efb3e18962cf2870ed1daf
Gerrit-PatchSet: 3
Gerrit-Project: osmo-trx
Gerrit-Branch: master
Patch Set 4: Code-Review+1
(2 comments)
https://gerrit.osmocom.org/#/c/2102/4/config/ax_ext.m4
File config/ax_ext.m4:
Line 22: # HAVE_MMX / HAVE_SSE / HAVE_SSE2 / HAVE_SSE3 / HAVE_SSSE3 /
HAVE_SSE4.1 / HAVE_SSE4.2 / HAVE_AVX
Now we have only HAVE_SSE3 and HAVE_SSE4.1, right?
So, it would
18 matches
Mail list logo