Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/818040/?format=api
{ "id": 818040, "url": "http://patchwork.ozlabs.org/api/patches/818040/?format=api", "web_url": "http://patchwork.ozlabs.org/project/netdev/patch/20170925032941.14586-13-tom@quantonium.net/", "project": { "id": 7, "url": "http://patchwork.ozlabs.org/api/projects/7/?format=api", "name": "Linux network development", "link_name": "netdev", "list_id": "netdev.vger.kernel.org", "list_email": "netdev@vger.kernel.org", "web_url": null, "scm_url": null, "webscm_url": null, "list_archive_url": "", "list_archive_url_format": "", "commit_url_format": "" }, "msgid": "<20170925032941.14586-13-tom@quantonium.net>", "list_archive_url": null, "date": "2017-09-25T03:29:41", "name": "[v3,net-next,12/12] gtp: Allow configuring GTP interface as standalone", "commit_ref": null, "pull_url": null, "state": "changes-requested", "archived": true, "hash": "c2f0c7209bcc9d79c1d1c52c8ddb36ce5ffb2b28", "submitter": { "id": 72064, "url": "http://patchwork.ozlabs.org/api/people/72064/?format=api", "name": "Tom Herbert", "email": "tom@quantonium.net" }, "delegate": { "id": 34, "url": "http://patchwork.ozlabs.org/api/users/34/?format=api", "username": "davem", "first_name": "David", "last_name": "Miller", "email": "davem@davemloft.net" }, "mbox": "http://patchwork.ozlabs.org/project/netdev/patch/20170925032941.14586-13-tom@quantonium.net/mbox/", "series": [ { "id": 4865, "url": "http://patchwork.ozlabs.org/api/series/4865/?format=api", "web_url": "http://patchwork.ozlabs.org/project/netdev/list/?series=4865", "date": "2017-09-25T03:29:29", "name": "gtp: Additional feature support - Part I", "version": 3, "mbox": "http://patchwork.ozlabs.org/series/4865/mbox/" } ], "comments": "http://patchwork.ozlabs.org/api/patches/818040/comments/", "check": "pending", "checks": "http://patchwork.ozlabs.org/api/patches/818040/checks/", "tags": {}, "related": [], "headers": { "Return-Path": "<netdev-owner@vger.kernel.org>", "X-Original-To": "patchwork-incoming@ozlabs.org", "Delivered-To": "patchwork-incoming@ozlabs.org", "Authentication-Results": [ "ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=vger.kernel.org\n\t(client-ip=209.132.180.67; helo=vger.kernel.org;\n\tenvelope-from=netdev-owner@vger.kernel.org;\n\treceiver=<UNKNOWN>)", "ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=quantonium-net.20150623.gappssmtp.com\n\theader.i=@quantonium-net.20150623.gappssmtp.com\n\theader.b=\"0S6MdoT3\"; dkim-atps=neutral" ], "Received": [ "from vger.kernel.org (vger.kernel.org [209.132.180.67])\n\tby ozlabs.org (Postfix) with ESMTP id 3y0qNX2gDfz9t4B\n\tfor <patchwork-incoming@ozlabs.org>;\n\tMon, 25 Sep 2017 13:30:36 +1000 (AEST)", "(majordomo@vger.kernel.org) by vger.kernel.org via listexpand\n\tid S933120AbdIYDae (ORCPT <rfc822;patchwork-incoming@ozlabs.org>);\n\tSun, 24 Sep 2017 23:30:34 -0400", "from mail-pf0-f176.google.com ([209.85.192.176]:47512 \"EHLO\n\tmail-pf0-f176.google.com\" rhost-flags-OK-OK-OK-OK) by vger.kernel.org\n\twith ESMTP id S933044AbdIYDa1 (ORCPT\n\t<rfc822;netdev@vger.kernel.org>); Sun, 24 Sep 2017 23:30:27 -0400", "by mail-pf0-f176.google.com with SMTP id u12so3144514pfl.4\n\tfor <netdev@vger.kernel.org>; Sun, 24 Sep 2017 20:30:27 -0700 (PDT)", "from localhost.localdomain (c-73-162-13-107.hsd1.ca.comcast.net.\n\t[73.162.13.107]) by smtp.gmail.com with ESMTPSA id\n\tj2sm9112907pgn.26.2017.09.24.20.30.24\n\t(version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128);\n\tSun, 24 Sep 2017 20:30:25 -0700 (PDT)" ], "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=quantonium-net.20150623.gappssmtp.com; s=20150623;\n\th=from:to:cc:subject:date:message-id:in-reply-to:references;\n\tbh=SIIbcZrGbo9vQ9tcpBRscAQ3X42DLFB/thBEL5E/EhY=;\n\tb=0S6MdoT39pRfoRgR1uO9yEd9eN2iX8jny9UsFx90Cjxy156NpHwXf5mwVa342tVG5M\n\tpJ2ls55DNjjHnbL95PILoEe8y4TXvLgsT83cwwRU4tDvhofPGlZsnZOUeI/5d4OY0hIA\n\tJye36m5RBUBYizXbTvKr/SXXVunoXWU6GfXlIO3E1rGVCW5Q57fmw61pJKkLifg8Vvpl\n\tgDcdGrM6X//KsaT+UnzvULKj9tQQqK/h0rhs5apPgCllEeEGLpMsv1l9bIaLwA9aBe2Z\n\tQjnLG5XO6atINkkCJxhNeCJyPD+URN0w9F9Xca0a+xbq0o3eboLxW3EPARty4nQ1orGB\n\tbfxw==", "X-Google-DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to\n\t:references;\n\tbh=SIIbcZrGbo9vQ9tcpBRscAQ3X42DLFB/thBEL5E/EhY=;\n\tb=maWgktbZSSqJiZ1eaOsh2Rgrh5MFzAozUZoTFt5Yefz1+4dfIoOyWHwbH5H4FOxx7R\n\tgdROEl+r3V46Ku+O9ClKEDzhxih7eePBjv+eUCZWpy1QkkeH+rR/3XeZ5l8GTDDg72hu\n\th5zN7kGBVJEuxjVYSmViifAvuieSLzTH1vk1ut4km+3LxfqZ58UEl3xXMLKKsGOwiIvU\n\tP/DbOjKEy3964cryf1ly8BC1TI4VyhoIejzsnkBULWwTDB7J9lnYtQisdCJWT15xIQVa\n\tx4oo3ehRHgO9NNL4Ti/er9hUEMfFztZQjgRde2nBpihf8ij8zSrs/cQIRFm5ZMBhLoh/\n\t+0Yg==", "X-Gm-Message-State": "AHPjjUi6015QVVTnP0bdRhGZ2DiPHb1h2GeR/jURDlLsi89SHwZC6VIW\n\thpQcFfovXFIjUL2A9mxoR04OWg==", "X-Google-Smtp-Source": "AOwi7QB1Wi9d78DmUIY+rLhLDFmAqAzIQeyI3xxpPUTJtWrRHAj3hvpbLLVu1Y73z10W8FwVP+LKJg==", "X-Received": "by 10.101.66.70 with SMTP id d6mr6219152pgq.169.1506310226816;\n\tSun, 24 Sep 2017 20:30:26 -0700 (PDT)", "From": "Tom Herbert <tom@quantonium.net>", "To": "davem@davemloft.net", "Cc": "pablo@netfilter.org, laforge@gnumonks.org, aschultz@tpip.net,\n\tnetdev@vger.kernel.org, rohit@quantonium.net,\n\tTom Herbert <tom@quantonium.net>", "Subject": "[PATCH v3 net-next 12/12] gtp: Allow configuring GTP interface as\n\tstandalone", "Date": "Sun, 24 Sep 2017 20:29:41 -0700", "Message-Id": "<20170925032941.14586-13-tom@quantonium.net>", "X-Mailer": "git-send-email 2.11.0", "In-Reply-To": "<20170925032941.14586-1-tom@quantonium.net>", "References": "<20170925032941.14586-1-tom@quantonium.net>", "Sender": "netdev-owner@vger.kernel.org", "Precedence": "bulk", "List-ID": "<netdev.vger.kernel.org>", "X-Mailing-List": "netdev@vger.kernel.org" }, "content": "Add new configuration of GTP interfaces that allow specifying a port to\nlisten on (as opposed to having to get sockets from a userspace control\nplane). This allows GTP interfaces to be configured and the data path\ntested without requiring a GTP-C daemon.\n\nSigned-off-by: Tom Herbert <tom@quantonium.net>\n---\n drivers/net/gtp.c | 215 ++++++++++++++++++++++++++++++++++++-----------\n include/uapi/linux/gtp.h | 5 ++\n 2 files changed, 169 insertions(+), 51 deletions(-)", "diff": "diff --git a/drivers/net/gtp.c b/drivers/net/gtp.c\nindex 1c580df4cfc5..dc1fcd3034af 100644\n--- a/drivers/net/gtp.c\n+++ b/drivers/net/gtp.c\n@@ -93,6 +93,9 @@ struct gtp_dev {\n \tstruct sock\t\t*sk0;\n \tstruct sock\t\t*sk1u;\n \n+\tstruct socket\t\t*sock0;\n+\tstruct socket\t\t*sock1u;\n+\n \tstruct net_device\t*dev;\n \n \tunsigned int\t\trole;\n@@ -451,26 +454,33 @@ static void gtp_encap_destroy(struct sock *sk)\n \t}\n }\n \n-static void gtp_encap_disable_sock(struct sock *sk)\n+static void gtp_encap_release(struct gtp_dev *gtp)\n {\n-\tif (!sk)\n-\t\treturn;\n+\tif (gtp->sk0) {\n+\t\tif (gtp->sock0) {\n+\t\t\tudp_tunnel_sock_release(gtp->sock0);\n+\t\t\tgtp->sock0 = NULL;\n+\t\t} else {\n+\t\t\tgtp_encap_destroy(gtp->sk0);\n+\t\t}\n \n-\tgtp_encap_destroy(sk);\n-}\n+\t\tgtp->sk0 = NULL;\n+\t}\n \n-static void gtp_encap_disable(struct gtp_dev *gtp)\n-{\n-\tgtp_encap_disable_sock(gtp->sk0);\n-\tgtp_encap_disable_sock(gtp->sk1u);\n+\tif (gtp->sk1u) {\n+\t\tif (gtp->sock1u) {\n+\t\t\tudp_tunnel_sock_release(gtp->sock1u);\n+\t\t\tgtp->sock1u = NULL;\n+\t\t} else {\n+\t\t\tgtp_encap_destroy(gtp->sk1u);\n+\t\t}\n+\n+\t\tgtp->sk1u = NULL;\n+\t}\n }\n \n static int gtp_dev_init(struct net_device *dev)\n {\n-\tstruct gtp_dev *gtp = netdev_priv(dev);\n-\n-\tgtp->dev = dev;\n-\n \tdev->tstats = netdev_alloc_pcpu_stats(struct pcpu_sw_netstats);\n \tif (!dev->tstats)\n \t\treturn -ENOMEM;\n@@ -482,7 +492,8 @@ static void gtp_dev_uninit(struct net_device *dev)\n {\n \tstruct gtp_dev *gtp = netdev_priv(dev);\n \n-\tgtp_encap_disable(gtp);\n+\tgtp_encap_release(gtp);\n+\n \tfree_percpu(dev->tstats);\n }\n \n@@ -751,6 +762,8 @@ static void gtp_link_setup(struct net_device *dev)\n \t\t\t\t sizeof(struct udphdr) +\n \t\t\t\t sizeof(struct gtp0_header);\n \n+\tgtp->dev = dev;\n+\n \tgro_cells_init(>p->gro_cells, dev);\n }\n \n@@ -764,13 +777,19 @@ static int gtp_newlink(struct net *src_net, struct net_device *dev,\n \t\t struct netlink_ext_ack *extack)\n {\n \tunsigned int role = GTP_ROLE_GGSN;\n+\tbool have_fd, have_ports;\n \tbool is_ipv6 = false;\n \tstruct gtp_dev *gtp;\n \tstruct gtp_net *gn;\n \tint hashsize, err;\n \n-\tif (!data[IFLA_GTP_FD0] && !data[IFLA_GTP_FD1])\n+\thave_fd = !!data[IFLA_GTP_FD0] || !!data[IFLA_GTP_FD1];\n+\thave_ports = !!data[IFLA_GTP_PORT0] || !!data[IFLA_GTP_PORT1];\n+\n+\tif (!(have_fd ^ have_ports)) {\n+\t\t/* Either got fd(s) or port(s) */\n \t\treturn -EINVAL;\n+\t}\n \n \tif (data[IFLA_GTP_ROLE]) {\n \t\trole = nla_get_u32(data[IFLA_GTP_ROLE]);\n@@ -831,7 +850,7 @@ static int gtp_newlink(struct net *src_net, struct net_device *dev,\n out_hashtable:\n \tgtp_hashtable_free(gtp);\n out_encap:\n-\tgtp_encap_disable(gtp);\n+\tgtp_encap_release(gtp);\n \treturn err;\n }\n \n@@ -840,7 +859,7 @@ static void gtp_dellink(struct net_device *dev, struct list_head *head)\n \tstruct gtp_dev *gtp = netdev_priv(dev);\n \n \tgro_cells_destroy(>p->gro_cells);\n-\tgtp_encap_disable(gtp);\n+\tgtp_encap_release(gtp);\n \tgtp_hashtable_free(gtp);\n \tlist_del_rcu(>p->list);\n \tunregister_netdevice_queue(dev, head);\n@@ -851,6 +870,8 @@ static const struct nla_policy gtp_policy[IFLA_GTP_MAX + 1] = {\n \t[IFLA_GTP_FD1]\t\t\t= { .type = NLA_U32 },\n \t[IFLA_GTP_PDP_HASHSIZE]\t\t= { .type = NLA_U32 },\n \t[IFLA_GTP_ROLE]\t\t\t= { .type = NLA_U32 },\n+\t[IFLA_GTP_PORT0]\t\t= { .type = NLA_U16 },\n+\t[IFLA_GTP_PORT1]\t\t= { .type = NLA_U16 },\n };\n \n static int gtp_validate(struct nlattr *tb[], struct nlattr *data[],\n@@ -949,11 +970,35 @@ static void gtp_hashtable_free(struct gtp_dev *gtp)\n \tkfree(gtp->tid_hash);\n }\n \n-static struct sock *gtp_encap_enable_socket(int fd, int type,\n-\t\t\t\t\t struct gtp_dev *gtp,\n-\t\t\t\t\t bool is_ipv6)\n+static int gtp_encap_enable_sock(struct socket *sock, int type,\n+\t\t\t\t struct gtp_dev *gtp)\n {\n \tstruct udp_tunnel_sock_cfg tuncfg = {NULL};\n+\n+\tswitch (type) {\n+\tcase UDP_ENCAP_GTP0:\n+\t\ttuncfg.encap_rcv = gtp0_udp_encap_recv;\n+\t\tbreak;\n+\tcase UDP_ENCAP_GTP1U:\n+\t\ttuncfg.encap_rcv = gtp1u_udp_encap_recv;\n+\t\tbreak;\n+\tdefault:\n+\t\tpr_debug(\"Unknown encap type %u\\n\", type);\n+\t\treturn -EINVAL;\n+\t}\n+\n+\ttuncfg.sk_user_data = gtp;\n+\ttuncfg.encap_type = type;\n+\ttuncfg.encap_destroy = gtp_encap_destroy;\n+\n+\tsetup_udp_tunnel_sock(sock_net(sock->sk), sock, &tuncfg);\n+\n+\treturn 0;\n+}\n+\n+static struct sock *gtp_encap_enable_fd(int fd, int type, struct gtp_dev *gtp,\n+\t\t\t\t\tbool is_ipv6)\n+{\n \tstruct socket *sock;\n \tstruct sock *sk;\n \tint err;\n@@ -986,60 +1031,128 @@ static struct sock *gtp_encap_enable_socket(int fd, int type,\n \tsk = sock->sk;\n \tsock_hold(sk);\n \n-\tswitch (type) {\n-\tcase UDP_ENCAP_GTP0:\n-\t\ttuncfg.encap_rcv = gtp0_udp_encap_recv;\n-\t\tbreak;\n-\tcase UDP_ENCAP_GTP1U:\n-\t\ttuncfg.encap_rcv = gtp1u_udp_encap_recv;\n-\t\tbreak;\n-\tdefault:\n-\t\tpr_debug(\"Unknown encap type %u\\n\", type);\n-\t\tsk = ERR_PTR(-EINVAL);\n-\t\tgoto out_sock;\n-\t}\n-\n-\ttuncfg.sk_user_data = gtp;\n-\ttuncfg.encap_type = type;\n-\ttuncfg.encap_destroy = gtp_encap_destroy;\n-\n-\tsetup_udp_tunnel_sock(sock_net(sock->sk), sock, &tuncfg);\n+\terr = gtp_encap_enable_sock(sock, type, gtp);\n+\tif (err < 0)\n+\t\tsk = ERR_PTR(err);\n \n out_sock:\n \tsockfd_put(sock);\n \treturn sk;\n }\n \n+static struct socket *gtp_create_sock(struct net *net, bool ipv6,\n+\t\t\t\t __be16 port, u32 flags)\n+{\n+\tstruct socket *sock;\n+\tstruct udp_port_cfg udp_conf;\n+\tint err;\n+\n+\tmemset(&udp_conf, 0, sizeof(udp_conf));\n+\n+#if GTP_IPV6\n+\tif (ipv6) {\n+\t\tudp_conf.family = AF_INET6;\n+\t\tudp_conf.ipv6_v6only = 1;\n+\t} else {\n+\t\tudp_conf.family = AF_INET;\n+\t}\n+#else\n+\tudp_conf.family = AF_INET;\n+#endif\n+\n+\tudp_conf.local_udp_port = port;\n+\n+\t/* Open UDP socket */\n+\terr = udp_sock_create(net, &udp_conf, &sock);\n+\tif (err)\n+\t\treturn ERR_PTR(err);\n+\n+\treturn sock;\n+}\n+\n static int gtp_encap_enable(struct gtp_dev *gtp, struct nlattr *data[],\n \t\t\t bool is_ipv6)\n {\n+\tint err;\n+\n+\tstruct socket *sock0 = NULL, *sock1u = NULL;\n \tstruct sock *sk0 = NULL, *sk1u = NULL;\n \n \tif (data[IFLA_GTP_FD0]) {\n \t\tu32 fd0 = nla_get_u32(data[IFLA_GTP_FD0]);\n \n-\t\tsk0 = gtp_encap_enable_socket(fd0, UDP_ENCAP_GTP0, gtp,\n-\t\t\t\t\t is_ipv6);\n-\t\tif (IS_ERR(sk0))\n-\t\t\treturn PTR_ERR(sk0);\n+\t\tsk0 = gtp_encap_enable_fd(fd0, UDP_ENCAP_GTP0, gtp, is_ipv6);\n+\t\tif (IS_ERR(sk0)) {\n+\t\t\terr = PTR_ERR(sk0);\n+\t\t\tsk0 = NULL;\n+\t\t\tgoto out_err;\n+\t\t}\n+\t} else if (data[IFLA_GTP_PORT0]) {\n+\t\t__be16 port = nla_get_u16(data[IFLA_GTP_PORT0]);\n+\n+\t\tsock0 = gtp_create_sock(dev_net(gtp->dev), is_ipv6, port, 0);\n+\t\tif (IS_ERR(sock0)) {\n+\t\t\terr = PTR_ERR(sock0);\n+\t\t\tsock0 = NULL;\n+\t\t\tgoto out_err;\n+\t\t}\n+\n+\t\terr = gtp_encap_enable_sock(sock0, UDP_ENCAP_GTP0, gtp);\n+\t\tif (err)\n+\t\t\tgoto out_err;\n \t}\n \n \tif (data[IFLA_GTP_FD1]) {\n \t\tu32 fd1 = nla_get_u32(data[IFLA_GTP_FD1]);\n \n-\t\tsk1u = gtp_encap_enable_socket(fd1, UDP_ENCAP_GTP1U, gtp,\n-\t\t\t\t\t is_ipv6);\n+\t\tsk1u = gtp_encap_enable_fd(fd1, UDP_ENCAP_GTP1U, gtp, is_ipv6);\n \t\tif (IS_ERR(sk1u)) {\n-\t\t\tif (sk0)\n-\t\t\t\tgtp_encap_disable_sock(sk0);\n-\t\t\treturn PTR_ERR(sk1u);\n+\t\t\terr = PTR_ERR(sk1u);\n+\t\t\tsk1u = NULL;\n+\t\t\tgoto out_err;\n+\t\t}\n+\t} else if (data[IFLA_GTP_PORT1]) {\n+\t\t__be16 port = nla_get_u16(data[IFLA_GTP_PORT1]);\n+\n+\t\tsock1u = gtp_create_sock(dev_net(gtp->dev), is_ipv6, port, 0);\n+\t\tif (IS_ERR(sock1u)) {\n+\t\t\terr = PTR_ERR(sock1u);\n+\t\t\tsock1u = NULL;\n+\t\t\tgoto out_err;\n \t\t}\n+\n+\t\terr = gtp_encap_enable_sock(sock1u, UDP_ENCAP_GTP1U, gtp);\n+\t\tif (err)\n+\t\t\tgoto out_err;\n+\t}\n+\n+\tif (sock0) {\n+\t\tgtp->sock0 = sock0;\n+\t\tgtp->sk0 = sock0->sk;\n+\t} else {\n+\t\tgtp->sk0 = sk0;\n \t}\n \n-\tgtp->sk0 = sk0;\n-\tgtp->sk1u = sk1u;\n+\tif (sock1u) {\n+\t\tgtp->sock1u = sock1u;\n+\t\tgtp->sk1u = sock1u->sk;\n+\t} else {\n+\t\tgtp->sk1u = sk1u;\n+\t}\n \n \treturn 0;\n+\n+out_err:\n+\tif (sk0)\n+\t\tgtp_encap_destroy(sk0);\n+\tif (sk1u)\n+\t\tgtp_encap_destroy(sk1u);\n+\tif (sock0)\n+\t\tudp_tunnel_sock_release(sock0);\n+\tif (sock1u)\n+\t\tudp_tunnel_sock_release(sock1u);\n+\n+\treturn err;\n }\n \n static struct gtp_dev *gtp_find_dev(struct net *src_net, struct nlattr *nla[])\n@@ -1624,8 +1737,8 @@ static const struct genl_ops gtp_genl_ops[] = {\n };\n \n static struct genl_family gtp_genl_family __ro_after_init = {\n-\t.name\t\t= \"gtp\",\n-\t.version\t= 0,\n+\t.name\t\t= GTP_GENL_NAME,\n+\t.version\t= GTP_GENL_VERSION,\n \t.hdrsize\t= 0,\n \t.maxattr\t= GTPA_MAX,\n \t.netnsok\t= true,\ndiff --git a/include/uapi/linux/gtp.h b/include/uapi/linux/gtp.h\nindex 8eec519fa754..0da18aa88be8 100644\n--- a/include/uapi/linux/gtp.h\n+++ b/include/uapi/linux/gtp.h\n@@ -9,6 +9,11 @@ enum gtp_genl_cmds {\n \tGTP_CMD_MAX,\n };\n \n+/* NETLINK_GENERIC related info\n+ */\n+#define GTP_GENL_NAME\t\t\"gtp\"\n+#define GTP_GENL_VERSION\t0\n+\n enum gtp_version {\n \tGTP_V0 = 0,\n \tGTP_V1,\n", "prefixes": [ "v3", "net-next", "12/12" ] }