pespin submitted this change.

View Change

Approvals: osmith: Looks good to me, approved laforge: Looks good to me, but someone else must approve Jenkins Builder: Verified
Move g_ggsn_list declaration to ggsn.c

The llist belongs to the data domain, not the vty.

Change-Id: I0a93aacbdecc283dd4f6b32892430ebdb0b94e50
---
M ggsn/ggsn.c
M ggsn/ggsn_vty.c
2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/ggsn/ggsn.c b/ggsn/ggsn.c
index 8fa0f74..650150a 100644
--- a/ggsn/ggsn.c
+++ b/ggsn/ggsn.c
@@ -60,6 +60,8 @@
#include "ggsn.h"
#include "../gtp/gtp_internal.h"

+LLIST_HEAD(g_ggsn_list);
+
static int ggsn_tun_fd_cb(struct osmo_fd *fd, unsigned int what);
static int cb_tun_ind(struct tun_t *tun, void *pack, unsigned len);

diff --git a/ggsn/ggsn_vty.c b/ggsn/ggsn_vty.c
index 68d2cdc..13036d8 100644
--- a/ggsn/ggsn_vty.c
+++ b/ggsn/ggsn_vty.c
@@ -50,8 +50,6 @@
#define IFCONFIG_STR "GGSN-based interface configuration\n"
#define GGSN_STR "Gateway GPRS Support NODE (GGSN)\n"

-LLIST_HEAD(g_ggsn_list);
-
enum ggsn_vty_node {
GGSN_NODE = _LAST_OSMOVTY_NODE + 1,
APN_NODE,

To view, visit change 38456. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-MessageType: merged
Gerrit-Project: osmo-ggsn
Gerrit-Branch: master
Gerrit-Change-Id: I0a93aacbdecc283dd4f6b32892430ebdb0b94e50
Gerrit-Change-Number: 38456
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>