Attention is currently required from: fixeria.
neels has posted comments on this change. ( https://gerrit.osmocom.org/c/libosmocore/+/35004?usp=email )
Change subject: msgb: fix doxygen docs for msgb_pull_u{8,16,32): end -> front
......................................................................
Patch Set 1: Code-Review+2
(1 comment)
Patchset:
PS1:
("from the start" would sound better to me)
--
To view, visit https://gerrit.osmocom.org/c/libosmocore/+/35004?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: Ie9851683fa9cc88cd7f6b63ad708b4f78c8f37c9
Gerrit-Change-Number: 35004
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: neels <nhofmeyr(a)sysmocom.de>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Comment-Date: Mon, 13 Nov 2023 02:56:22 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
Attention is currently required from: lynxis lazus.
Hello Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/erlang/osmo_dia2gsup/+/35006?usp=email
to look at the new patch set (#2).
The following approvals got outdated and were removed:
Verified-1 by Jenkins Builder
Change subject: WIP: diameter: add support for Cx (VoLTE)
......................................................................
WIP: diameter: add support for Cx (VoLTE)
Change-Id: Icd623563c495fb6da4216bb6f74bc31873dc7d6e
---
M dia/diameter_3gpp_ts29_229.dia
A dia/diameter_3gpp_ts29_229_cx.dia
A dia/diameter_etsi_es283_035.dia
A dia/diameter_rfc4740.dia
A dia/diameter_rfc8583.dia
A src/osmo_cx2gsup.erl
A src/osmo_cx2gsup_cb.erl
M src/osmo_dia2gsup_sup.erl
8 files changed, 990 insertions(+), 12 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/erlang/osmo_dia2gsup refs/changes/06/35006/2
--
To view, visit https://gerrit.osmocom.org/c/erlang/osmo_dia2gsup/+/35006?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: erlang/osmo_dia2gsup
Gerrit-Branch: master
Gerrit-Change-Id: Icd623563c495fb6da4216bb6f74bc31873dc7d6e
Gerrit-Change-Number: 35006
Gerrit-PatchSet: 2
Gerrit-Owner: lynxis lazus <lynxis(a)fe80.eu>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Attention: lynxis lazus <lynxis(a)fe80.eu>
Gerrit-MessageType: newpatchset
fixeria has uploaded this change for review. ( https://gerrit.osmocom.org/c/libosmocore/+/35004?usp=email )
Change subject: msgb: fix doxygen docs for msgb_pull_u{8,16,32): end -> front
......................................................................
msgb: fix doxygen docs for msgb_pull_u{8,16,32): end -> front
Change-Id: Ie9851683fa9cc88cd7f6b63ad708b4f78c8f37c9
---
M include/osmocom/core/msgb.h
1 file changed, 12 insertions(+), 3 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/libosmocore refs/changes/04/35004/1
diff --git a/include/osmocom/core/msgb.h b/include/osmocom/core/msgb.h
index 5c58c84..b05d8b6 100644
--- a/include/osmocom/core/msgb.h
+++ b/include/osmocom/core/msgb.h
@@ -442,7 +442,7 @@
/*! remove uint8 from front of message
* \param[in] msgb message buffer
- * \returns 8bit value taken from end of msgb
+ * \returns 8bit value taken from the front of msgb
*/
static inline uint8_t msgb_pull_u8(struct msgb *msgb)
{
@@ -452,7 +452,7 @@
/*! remove uint16 from front of message
* \param[in] msgb message buffer
- * \returns 16bit value taken from end of msgb
+ * \returns 16bit value taken from the front of msgb
*/
static inline uint16_t msgb_pull_u16(struct msgb *msgb)
{
@@ -462,7 +462,7 @@
/*! remove uint32 from front of message
* \param[in] msgb message buffer
- * \returns 32bit value taken from end of msgb
+ * \returns 32bit value taken from the front of msgb
*/
static inline uint32_t msgb_pull_u32(struct msgb *msgb)
{
--
To view, visit https://gerrit.osmocom.org/c/libosmocore/+/35004?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: Ie9851683fa9cc88cd7f6b63ad708b4f78c8f37c9
Gerrit-Change-Number: 35004
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-MessageType: newchange
fixeria has uploaded this change for review. ( https://gerrit.osmocom.org/c/libosmocore/+/35005?usp=email )
Change subject: core: add parity calculation functions for uint{32,16,8}_t
......................................................................
core: add parity calculation functions for uint{32,16,8}_t
Change-Id: I2d6265eac5ce062276a40f4968b129d8702c6117
---
M include/osmocom/core/bits.h
1 file changed, 48 insertions(+), 0 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/libosmocore refs/changes/05/35005/1
diff --git a/include/osmocom/core/bits.h b/include/osmocom/core/bits.h
index b1b8040..4b80036 100644
--- a/include/osmocom/core/bits.h
+++ b/include/osmocom/core/bits.h
@@ -119,4 +119,43 @@
return (in << shift) | (in >> (16 - shift));
}
+/*! Calculate parity for the given 32-bit unsigned integer.
+ * \param[in] x a 32-bit unsigned integer.
+ * \returns 1 for even parity (even number of bits set to 1);
+ * 0 for odd parity (odd number of bits set to 1). */
+static inline uint8_t osmo_parity_u32(uint32_t x)
+{
+ x ^= x >> 16;
+ x ^= x >> 8;
+ x ^= x >> 4;
+ x ^= x >> 2;
+ x ^= x >> 1;
+ return (~x) & 1;
+}
+
+/*! Calculate parity for the given 16-bit unsigned integer.
+ * \param[in] x a 16-bit unsigned integer.
+ * \returns 1 for even parity (even number of bits set to 1);
+ * 0 for odd parity (odd number of bits set to 1). */
+static inline uint8_t osmo_parity_u16(uint16_t x)
+{
+ x ^= x >> 8;
+ x ^= x >> 4;
+ x ^= x >> 2;
+ x ^= x >> 1;
+ return (~x) & 1;
+}
+
+/*! Calculate parity for the given 8-bit unsigned integer.
+ * \param[in] x a 8-bit unsigned integer.
+ * \returns 1 for even parity (even number of bits set to 1);
+ * 0 for odd parity (odd number of bits set to 1). */
+static inline uint8_t osmo_parity_u8(uint8_t x)
+{
+ x ^= x >> 4;
+ x ^= x >> 2;
+ x ^= x >> 1;
+ return (~x) & 1;
+}
+
/*! @} */
--
To view, visit https://gerrit.osmocom.org/c/libosmocore/+/35005?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I2d6265eac5ce062276a40f4968b129d8702c6117
Gerrit-Change-Number: 35005
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-MessageType: newchange
Attention is currently required from: keith.
fixeria has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-pcu/+/35003?usp=email )
Change subject: TBF status: Fix VTY output text
......................................................................
Patch Set 3: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/c/osmo-pcu/+/35003?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-pcu
Gerrit-Branch: master
Gerrit-Change-Id: Ic906ca9d02811cb96e4530af43fbc3769f6afce7
Gerrit-Change-Number: 35003
Gerrit-PatchSet: 3
Gerrit-Owner: keith <keith(a)rhizomatica.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Attention: keith <keith(a)rhizomatica.org>
Gerrit-Comment-Date: Sun, 12 Nov 2023 06:39:45 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
Hello Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/osmo-pcu/+/35003?usp=email
to look at the new patch set (#3).
Change subject: TBF status: Fix VTY output text
......................................................................
TBF status: Fix VTY output text
The vty command 'show bts pdch' had the UL and DL TBF count reversed.
This patch corrects that.
Change-Id: Ic906ca9d02811cb96e4530af43fbc3769f6afce7
---
M src/pcu_vty_functions.cpp
1 file changed, 13 insertions(+), 1 deletion(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-pcu refs/changes/03/35003/3
--
To view, visit https://gerrit.osmocom.org/c/osmo-pcu/+/35003?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-pcu
Gerrit-Branch: master
Gerrit-Change-Id: Ic906ca9d02811cb96e4530af43fbc3769f6afce7
Gerrit-Change-Number: 35003
Gerrit-PatchSet: 3
Gerrit-Owner: keith <keith(a)rhizomatica.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-MessageType: newpatchset