Change in osmo-sgsn[master]: Use OSMO_FD_* instead of deprecated BSC_FD_*

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

pespin gerrit-no-reply at lists.osmocom.org
Sun May 10 14:57:01 UTC 2020


pespin has submitted this change. ( https://gerrit.osmocom.org/c/osmo-sgsn/+/18168 )

Change subject: Use OSMO_FD_* instead of deprecated BSC_FD_*
......................................................................

Use OSMO_FD_* instead of deprecated BSC_FD_*

New define is available since libosmocore 1.1.0, and we already require
1.2.0, so no need to update dependenices.
Let's change it to avoid people re-using old BSC_FD_* symbols when
copy-pasting somewhere else.

Change-Id: Iaebd049e383b02204a12f39cc6c932a53d25fd72
---
M src/gprs/sgsn_ares.c
M src/gtphub/gtphub.c
M src/sgsn/sgsn_libgtp.c
3 files changed, 13 insertions(+), 13 deletions(-)

Approvals:
  Jenkins Builder: Verified
  fixeria: Looks good to me, approved



diff --git a/src/gprs/sgsn_ares.c b/src/gprs/sgsn_ares.c
index ba168d6..38e31e1 100644
--- a/src/gprs/sgsn_ares.c
+++ b/src/gprs/sgsn_ares.c
@@ -51,8 +51,8 @@
 	LOGP(DGPRS, LOGL_DEBUG, "C-ares fd(%d) ready(%d)\n", fd->fd, what);
 
 	ares_process_fd(sgsn->ares_channel,
-			(what & BSC_FD_READ) ? fd->fd : ARES_SOCKET_BAD,
-			(what & BSC_FD_WRITE) ? fd->fd : ARES_SOCKET_BAD);
+			(what & OSMO_FD_READ) ? fd->fd : ARES_SOCKET_BAD,
+			(what & OSMO_FD_WRITE) ? fd->fd : ARES_SOCKET_BAD);
 	osmo_ares_reschedule(sgsn);
 	return 0;
 }
@@ -120,14 +120,14 @@
 
 update_fd:
 	if (read)
-		ufd->fd.when |= BSC_FD_READ;
+		ufd->fd.when |= OSMO_FD_READ;
 	else
-		ufd->fd.when &= ~BSC_FD_READ;
+		ufd->fd.when &= ~OSMO_FD_READ;
 
 	if (write)
-		ufd->fd.when |= BSC_FD_WRITE;
+		ufd->fd.when |= OSMO_FD_WRITE;
 	else
-		ufd->fd.when &= ~BSC_FD_WRITE;
+		ufd->fd.when &= ~OSMO_FD_WRITE;
 
 	osmo_ares_reschedule(sgsn);
 }
diff --git a/src/gtphub/gtphub.c b/src/gtphub/gtphub.c
index ca5857b..7b355bf 100644
--- a/src/gtphub/gtphub.c
+++ b/src/gtphub/gtphub.c
@@ -853,7 +853,7 @@
 		return -1;
 	}
 
-	ofd->when = BSC_FD_READ;
+	ofd->when = OSMO_FD_READ;
 	ofd->cb = cb;
 	ofd->data = data;
 	ofd->priv_nr = ofd_id;
@@ -1948,7 +1948,7 @@
 	LOG(LOGL_DEBUG, "=== reading from SGSN (%s)\n",
 	    gtphub_plane_idx_names[plane_idx]);
 
-	if (!(what & BSC_FD_READ))
+	if (!(what & OSMO_FD_READ))
 		return 0;
 
 	struct gtphub *hub = from_sgsns_ofd->data;
@@ -1979,7 +1979,7 @@
 	OSMO_ASSERT(plane_idx < GTPH_PLANE_N);
 	LOG(LOGL_DEBUG, "=== reading from GGSN (%s)\n",
 	    gtphub_plane_idx_names[plane_idx]);
-	if (!(what & BSC_FD_READ))
+	if (!(what & OSMO_FD_READ))
 		return 0;
 
 	struct gtphub *hub = from_ggsns_ofd->data;
diff --git a/src/sgsn/sgsn_libgtp.c b/src/sgsn/sgsn_libgtp.c
index c20c8dc..7b46476 100644
--- a/src/sgsn/sgsn_libgtp.c
+++ b/src/sgsn/sgsn_libgtp.c
@@ -733,7 +733,7 @@
 	struct sgsn_instance *sgi = fd->data;
 	int rc;
 
-	if (!(what & BSC_FD_READ))
+	if (!(what & OSMO_FD_READ))
 		return 0;
 
 	switch (fd->priv_nr) {
@@ -774,7 +774,7 @@
 	sgi->gtp_fd0.fd = gsn->fd0;
 	sgi->gtp_fd0.priv_nr = 0;
 	sgi->gtp_fd0.data = sgi;
-	sgi->gtp_fd0.when = BSC_FD_READ;
+	sgi->gtp_fd0.when = OSMO_FD_READ;
 	sgi->gtp_fd0.cb = sgsn_gtp_fd_cb;
 	rc = osmo_fd_register(&sgi->gtp_fd0);
 	if (rc < 0)
@@ -783,7 +783,7 @@
 	sgi->gtp_fd1c.fd = gsn->fd1c;
 	sgi->gtp_fd1c.priv_nr = 1;
 	sgi->gtp_fd1c.data = sgi;
-	sgi->gtp_fd1c.when = BSC_FD_READ;
+	sgi->gtp_fd1c.when = OSMO_FD_READ;
 	sgi->gtp_fd1c.cb = sgsn_gtp_fd_cb;
 	rc = osmo_fd_register(&sgi->gtp_fd1c);
 	if (rc < 0) {
@@ -794,7 +794,7 @@
 	sgi->gtp_fd1u.fd = gsn->fd1u;
 	sgi->gtp_fd1u.priv_nr = 2;
 	sgi->gtp_fd1u.data = sgi;
-	sgi->gtp_fd1u.when = BSC_FD_READ;
+	sgi->gtp_fd1u.when = OSMO_FD_READ;
 	sgi->gtp_fd1u.cb = sgsn_gtp_fd_cb;
 	rc = osmo_fd_register(&sgi->gtp_fd1u);
 	if (rc < 0) {

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

Gerrit-Project: osmo-sgsn
Gerrit-Branch: master
Gerrit-Change-Id: Iaebd049e383b02204a12f39cc6c932a53d25fd72
Gerrit-Change-Number: 18168
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin at sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <axilirator at gmail.com>
Gerrit-Reviewer: pespin <pespin at sysmocom.de>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20200510/a329fd35/attachment.htm>


More information about the gerrit-log mailing list