laforge has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-upf/+/30235 )

Change subject: cosmetic: rename upf_gtp_tun to upf_gtp_tunend
......................................................................

cosmetic: rename upf_gtp_tun to upf_gtp_tunend

Change-Id: I0815012679237838a031e28c0afb98b7e0d184bd
---
M include/osmocom/upf/up_gtp_action.h
M include/osmocom/upf/upf_gtp.h
M src/osmo-upf/upf_gtp.c
3 files changed, 37 insertions(+), 37 deletions(-)

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



diff --git a/include/osmocom/upf/up_gtp_action.h 
b/include/osmocom/upf/up_gtp_action.h
index 5b4d5ee..8e291b7 100644
--- a/include/osmocom/upf/up_gtp_action.h
+++ b/include/osmocom/upf/up_gtp_action.h
@@ -54,7 +54,7 @@
        enum up_gtp_action_kind kind;
        union {
                /* En-/De-capsulate GTP: add/remove a GTP header and forward 
the GTP payload from/to plain IP. */
-               struct upf_gtp_tun_desc tunend;
+               struct upf_gtp_tunend_desc tunend;

                /* Tunnel-map GTP: translate from one TEID to another and 
forward */
                struct upf_nft_tunmap_desc tunmap;
diff --git a/include/osmocom/upf/upf_gtp.h b/include/osmocom/upf/upf_gtp.h
index 1e0df5e..ec0046d 100644
--- a/include/osmocom/upf/upf_gtp.h
+++ b/include/osmocom/upf/upf_gtp.h
@@ -60,14 +60,14 @@
        struct llist_head tunnels;
 };

-struct upf_gtp_tun_desc {
+struct upf_gtp_tunend_desc {
        uint32_t local_teid;
        uint32_t remote_teid;
        struct osmo_sockaddr ue_addr;
        struct osmo_sockaddr gtp_remote_addr;
 };

-int upf_gtp_tun_desc_cmp(const struct upf_gtp_tun_desc *a, const struct 
upf_gtp_tun_desc *b);
+int upf_gtp_tunend_desc_cmp(const struct upf_gtp_tunend_desc *a, const struct 
upf_gtp_tunend_desc *b);

 int upf_gtp_genl_ensure_open();
 void upf_gtp_genl_close();
@@ -77,9 +77,9 @@
 struct upf_gtp_dev *upf_gtp_dev_find_by_name(const char *name);
 struct upf_gtp_dev *upf_gtp_dev_first();

-int upf_gtp_dev_tunnel_add(struct upf_gtp_dev *dev, const struct 
upf_gtp_tun_desc *t);
-bool upf_gtp_dev_is_tunnel_active(struct upf_gtp_dev *dev, const struct 
upf_gtp_tun_desc *t);
-int upf_gtp_dev_tunnel_del(struct upf_gtp_dev *dev, const struct 
upf_gtp_tun_desc *t);
+int upf_gtp_dev_tunnel_add(struct upf_gtp_dev *dev, const struct 
upf_gtp_tunend_desc *t);
+bool upf_gtp_dev_is_tunnel_active(struct upf_gtp_dev *dev, const struct 
upf_gtp_tunend_desc *t);
+int upf_gtp_dev_tunnel_del(struct upf_gtp_dev *dev, const struct 
upf_gtp_tunend_desc *t);

 int upf_gtp_dev_to_str_buf(char *buf, size_t buflen, const struct upf_gtp_dev 
*dev);
 char *upf_gtp_dev_to_str_c(void *ctx, const struct upf_gtp_dev *dev);
diff --git a/src/osmo-upf/upf_gtp.c b/src/osmo-upf/upf_gtp.c
index 880e587..6794a25 100644
--- a/src/osmo-upf/upf_gtp.c
+++ b/src/osmo-upf/upf_gtp.c
@@ -39,7 +39,7 @@
 #include <osmocom/upf/upf_gtpu_echo.h>

 #define LOG_GTP_TUN(TUN, LEVEL, FMT, ARGS...) \
-       LOGP(DGTP, LEVEL, "%s: " FMT, upf_gtp_tun_to_str_c(OTC_SELECT, (TUN)), 
##ARGS)
+       LOGP(DGTP, LEVEL, "%s: " FMT, upf_gtp_tunend_to_str_c(OTC_SELECT, 
(TUN)), ##ARGS)

 int upf_gtp_dev_to_str_buf(char *buf, size_t buflen, const struct upf_gtp_dev 
*dev)
 {
@@ -287,15 +287,15 @@
        return 0;
 }

-struct upf_gtp_tun {
+struct upf_gtp_tunend {
        struct llist_head entry;

        struct upf_gtp_dev *dev;
-       struct upf_gtp_tun_desc desc;
+       struct upf_gtp_tunend_desc desc;
        bool active;
 };

-static int upf_gtp_tun_to_str_buf(char *buf, size_t buflen, const struct 
upf_gtp_tun *tun)
+static int upf_gtp_tunend_to_str_buf(char *buf, size_t buflen, const struct 
upf_gtp_tunend *tun)
 {
        struct osmo_strbuf sb = { .buf = buf, .len = buflen };
        OSMO_STRBUF_PRINTF(sb, "%s:tun{TEID=l:0x%x,r:0x%x UE=", tun->dev->name, 
tun->desc.local_teid,
@@ -307,35 +307,35 @@
        return sb.chars_needed;
 }

-static char *upf_gtp_tun_to_str_c(void *ctx, const struct upf_gtp_tun *tun)
+static char *upf_gtp_tunend_to_str_c(void *ctx, const struct upf_gtp_tunend 
*tun)
 {
-       OSMO_NAME_C_IMPL(ctx, 64, "ERROR", upf_gtp_tun_to_str_buf, tun)
+       OSMO_NAME_C_IMPL(ctx, 64, "ERROR", upf_gtp_tunend_to_str_buf, tun)
 }

-static int upf_gtp_tun_deactivate(struct upf_gtp_tun *tun);
+static int upf_gtp_tunend_deactivate(struct upf_gtp_tunend *tun);

-static int upf_gtp_tun_destruct(struct upf_gtp_tun *tun)
+static int upf_gtp_tunend_destruct(struct upf_gtp_tunend *tun)
 {
        if (tun->active)
-               upf_gtp_tun_deactivate(tun);
+               upf_gtp_tunend_deactivate(tun);
        llist_del(&tun->entry);
        return 0;
 }

-static struct upf_gtp_tun *upf_gtp_tun_alloc(struct upf_gtp_dev *dev, const 
struct upf_gtp_tun_desc *desc)
+static struct upf_gtp_tunend *upf_gtp_tunend_alloc(struct upf_gtp_dev *dev, 
const struct upf_gtp_tunend_desc *desc)
 {
-       struct upf_gtp_tun *tun = talloc(dev, struct upf_gtp_tun);
+       struct upf_gtp_tunend *tun = talloc(dev, struct upf_gtp_tunend);
        OSMO_ASSERT(tun);
-       *tun = (struct upf_gtp_tun){
+       *tun = (struct upf_gtp_tunend){
                .dev = dev,
                .desc = *desc,
        };
        llist_add(&tun->entry, &dev->tunnels);
-       talloc_set_destructor(tun, upf_gtp_tun_destruct);
+       talloc_set_destructor(tun, upf_gtp_tunend_destruct);
        return tun;
 }

-static struct gtp_tunnel *upf_gtp_tun_to_gtp_tunnel(struct upf_gtp_tun *tun)
+static struct gtp_tunnel *upf_gtp_tunend_to_gtp_tunnel(struct upf_gtp_tunend 
*tun)
 {
        struct gtp_tunnel *t;

@@ -355,7 +355,7 @@
        return t;
 }

-int upf_gtp_tun_activate(struct upf_gtp_tun *tun)
+int upf_gtp_tunend_activate(struct upf_gtp_tunend *tun)
 {
        int rc;
        struct gtp_tunnel *t;
@@ -363,7 +363,7 @@
        if (tun->active)
                return -EALREADY;

-       t = upf_gtp_tun_to_gtp_tunnel(tun);
+       t = upf_gtp_tunend_to_gtp_tunnel(tun);
        if (!t)
                return -ENOTSUP;

@@ -381,37 +381,37 @@
        return rc;
 }

-static struct upf_gtp_tun *upf_gtp_dev_tunnel_find(struct upf_gtp_dev *dev, 
const struct upf_gtp_tun_desc *tun_desc)
+static struct upf_gtp_tunend *upf_gtp_dev_tunnel_find(struct upf_gtp_dev *dev, 
const struct upf_gtp_tunend_desc *tun_desc)
 {
-       struct upf_gtp_tun *tun;
+       struct upf_gtp_tunend *tun;
        llist_for_each_entry(tun, &dev->tunnels, entry) {
-               if (upf_gtp_tun_desc_cmp(tun_desc, &tun->desc))
+               if (upf_gtp_tunend_desc_cmp(tun_desc, &tun->desc))
                        continue;
                return tun;
        }
        return NULL;
 }

-int upf_gtp_dev_tunnel_add(struct upf_gtp_dev *dev, const struct 
upf_gtp_tun_desc *tun_desc)
+int upf_gtp_dev_tunnel_add(struct upf_gtp_dev *dev, const struct 
upf_gtp_tunend_desc *tun_desc)
 {
-       struct upf_gtp_tun *tun;
+       struct upf_gtp_tunend *tun;
        tun = upf_gtp_dev_tunnel_find(dev, tun_desc);
        if (!tun)
-               tun = upf_gtp_tun_alloc(dev, tun_desc);
+               tun = upf_gtp_tunend_alloc(dev, tun_desc);
        if (tun->active)
                return 0;
-       return upf_gtp_tun_activate(tun);
+       return upf_gtp_tunend_activate(tun);
 }

-int upf_gtp_dev_tunnel_del(struct upf_gtp_dev *dev, const struct 
upf_gtp_tun_desc *tun_desc)
+int upf_gtp_dev_tunnel_del(struct upf_gtp_dev *dev, const struct 
upf_gtp_tunend_desc *tun_desc)
 {
-       struct upf_gtp_tun *tun;
+       struct upf_gtp_tunend *tun;
        int rc;
        tun = upf_gtp_dev_tunnel_find(dev, tun_desc);
        if (!tun)
                return 0;
        if (tun->active) {
-               rc = upf_gtp_tun_deactivate(tun);
+               rc = upf_gtp_tunend_deactivate(tun);
                if (rc)
                        return rc;
        }
@@ -419,7 +419,7 @@
        return 0;
 }

-static int upf_gtp_tun_deactivate(struct upf_gtp_tun *tun)
+static int upf_gtp_tunend_deactivate(struct upf_gtp_tunend *tun)
 {
        int rc;
        struct gtp_tunnel *t;
@@ -429,7 +429,7 @@
                return -EINVAL;
        }

-       t = upf_gtp_tun_to_gtp_tunnel(tun);
+       t = upf_gtp_tunend_to_gtp_tunnel(tun);
        if (!t)
                return -EINVAL;

@@ -445,9 +445,9 @@

 static int upf_gtp_dev_destruct(struct upf_gtp_dev *dev)
 {
-       struct upf_gtp_tun *t;
+       struct upf_gtp_tunend *t;
        /* Destruct and clean up all active tunnels before deleting the device 
*/
-       while ((t = llist_first_entry_or_null(&dev->tunnels, struct 
upf_gtp_tun, entry)))
+       while ((t = llist_first_entry_or_null(&dev->tunnels, struct 
upf_gtp_tunend, entry)))
                talloc_free(t);
        llist_del(&dev->entry);
        /* osmo_fd_close() is a noop if ofd.fd == -1 */
@@ -458,7 +458,7 @@
        return 0;
 }

-int upf_gtp_tun_desc_cmp(const struct upf_gtp_tun_desc *a, const struct 
upf_gtp_tun_desc *b)
+int upf_gtp_tunend_desc_cmp(const struct upf_gtp_tunend_desc *a, const struct 
upf_gtp_tunend_desc *b)
 {
        int r;


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

Gerrit-Project: osmo-upf
Gerrit-Branch: master
Gerrit-Change-Id: I0815012679237838a031e28c0afb98b7e0d184bd
Gerrit-Change-Number: 30235
Gerrit-PatchSet: 2
Gerrit-Owner: neels <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-MessageType: merged

Reply via email to