From patchwork Mon Feb 2 15:11:24 2015 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Hans Dedecker X-Patchwork-Id: 435555 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from arrakis.dune.hu (arrakis.dune.hu [78.24.191.176]) (using TLSv1.1 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ozlabs.org (Postfix) with ESMTPS id 8911014010F for ; Tue, 3 Feb 2015 02:13:51 +1100 (AEDT) Received: from arrakis.dune.hu (localhost [127.0.0.1]) by arrakis.dune.hu (Postfix) with ESMTP id E78D428BFC6; Mon, 2 Feb 2015 16:09:46 +0100 (CET) X-Spam-Checker-Version: SpamAssassin 3.3.2 (2011-06-06) on arrakis.dune.hu X-Spam-Level: X-Spam-Status: No, score=-1.5 required=5.0 tests=BAYES_00,FREEMAIL_FROM, T_DKIM_INVALID autolearn=unavailable version=3.3.2 Received: from arrakis.dune.hu (localhost [127.0.0.1]) by arrakis.dune.hu (Postfix) with ESMTP id 07D0328BFC4 for ; Mon, 2 Feb 2015 16:09:40 +0100 (CET) X-policyd-weight: using cached result; rate: -8.5 Received: from mail-we0-f170.google.com (mail-we0-f170.google.com [74.125.82.170]) by arrakis.dune.hu (Postfix) with ESMTPS for ; Mon, 2 Feb 2015 16:09:38 +0100 (CET) Received: by mail-we0-f170.google.com with SMTP id w55so34185957wes.1 for ; Mon, 02 Feb 2015 07:11:56 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=x1K5VMMTNPz23cMgAkSN3BUJ2DEEGjmBUnVkShTNuHg=; b=vCCbRruRKRHvm8w7u6GZ2evdqvBcQBY3SUU9iB4ufzoT94J5rWKyuhSYrO0iYCVZPX 4VcpaiekVRZPbRkHGHzdnMvy1LPMFCJUr5MGcBNwAlUOplZ6ExDUSh+8DIwa6tqzWJ8m SClQfKOXXLKJTZhT7LTXdTPEs0KW+stLNFG1Xrmu8BW14Mo1ZVcCMPgwrzXIiCba9VjF 86pG+Tk6iGjwbDByQZDuSdmxrJBvLpbfLodYKLGI7ZXYi8uv3X/wttJRspsW2VLs8x/J FM0yfZ8d1oRLH3bwWQk/yGZlVoGdOxwHr6nFh+KuiSoYr2f09DyB/spalN/DBY9/59+f 0kuA== X-Received: by 10.180.20.226 with SMTP id q2mr25991469wie.28.1422889916690; Mon, 02 Feb 2015 07:11:56 -0800 (PST) Received: from cplx43.eu.thmulti.com ([141.11.62.7]) by mx.google.com with ESMTPSA id bv16sm3691382wib.0.2015.02.02.07.11.55 (version=TLSv1.2 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Mon, 02 Feb 2015 07:11:55 -0800 (PST) From: Hans Dedecker To: openwrt-devel@lists.openwrt.org Date: Mon, 2 Feb 2015 16:11:24 +0100 Message-Id: <1422889884-3096-3-git-send-email-dedeckeh@gmail.com> X-Mailer: git-send-email 1.9.1 In-Reply-To: <1422889884-3096-1-git-send-email-dedeckeh@gmail.com> References: <1422889884-3096-1-git-send-email-dedeckeh@gmail.com> Cc: Hans Dedecker , Jean-Francois Remy , cyrus@openwrt.org Subject: [OpenWrt-Devel] [PATCH 3/3] netifd: Add option to configure base_reachable_time_ms for each device X-BeenThere: openwrt-devel@lists.openwrt.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: OpenWrt Development List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , MIME-Version: 1.0 Errors-To: openwrt-devel-bounces@lists.openwrt.org Sender: "openwrt-devel" The UCI parameter neighreachabletime allows to control the hardware address to IP mapping lifetime in the neighbour table for both IPv4 and IPv6 Signed-off-by: Hans Dedecker Signed-off-by: Jean-Francois Remy --- device.c | 14 ++++++++++++++ device.h | 22 +++++++++++++--------- system-linux.c | 40 ++++++++++++++++++++++++++++++++++++++++ 3 files changed, 67 insertions(+), 9 deletions(-) diff --git a/device.c b/device.c index 54638e7..556d9fc 100644 --- a/device.c +++ b/device.c @@ -43,6 +43,7 @@ static const struct blobmsg_policy dev_attrs[__DEV_ATTR_MAX] = { [DEV_ATTR_ACCEPTLOCAL] = { .name = "acceptlocal", .type = BLOBMSG_TYPE_BOOL }, [DEV_ATTR_IGMPVERSION] = { .name = "igmpversion", .type = BLOBMSG_TYPE_INT32 }, [DEV_ATTR_MLDVERSION] = { .name = "mldversion", .type = BLOBMSG_TYPE_INT32 }, + [DEV_ATTR_NEIGHREACHABLETIME] = { .name = "neighreachabletime", .type = BLOBMSG_TYPE_INT32 }, }; const struct uci_blob_param_list device_attr_list = { @@ -162,6 +163,10 @@ device_merge_settings(struct device *dev, struct device_settings *n) n->acceptlocal = s->flags & DEV_OPT_ACCEPTLOCAL ? s->acceptlocal : os->acceptlocal; n->igmpversion = s->flags & DEV_OPT_IGMPVERSION ? s->igmpversion : os->igmpversion; n->mldversion = s->flags & DEV_OPT_MLDVERSION ? s->mldversion : os->mldversion; + n->neigh4reachabletime = s->flags & DEV_OPT_NEIGHREACHABLETIME ? + s->neigh4reachabletime : os->neigh4reachabletime; + n->neigh6reachabletime = s->flags & DEV_OPT_NEIGHREACHABLETIME ? + s->neigh6reachabletime : os->neigh6reachabletime; n->flags = s->flags | os->flags; } @@ -233,6 +238,11 @@ device_init_settings(struct device *dev, struct blob_attr **tb) DPRINTF("Failed to resolve mldversion: %d\n", blobmsg_get_u32(cur)); } + if ((cur = tb[DEV_ATTR_NEIGHREACHABLETIME])) { + s->neigh6reachabletime = s->neigh4reachabletime = blobmsg_get_u32(cur); + s->flags |= DEV_OPT_NEIGHREACHABLETIME; + } + device_set_disabled(dev, disabled); } @@ -794,6 +804,10 @@ device_dump_status(struct blob_buf *b, struct device *dev) blobmsg_add_u32(b, "igmpversion", st.igmpversion); if (st.flags & DEV_OPT_MLDVERSION) blobmsg_add_u32(b, "mldversion", st.mldversion); + if (st.flags & DEV_OPT_NEIGHREACHABLETIME) { + blobmsg_add_u32(b, "neigh4reachabletime", st.neigh4reachabletime); + blobmsg_add_u32(b, "neigh6reachabletime", st.neigh6reachabletime); + } } s = blobmsg_open_table(b, "statistics"); diff --git a/device.h b/device.h index ca4cc11..6418f16 100644 --- a/device.h +++ b/device.h @@ -37,6 +37,7 @@ enum { DEV_ATTR_ACCEPTLOCAL, DEV_ATTR_IGMPVERSION, DEV_ATTR_MLDVERSION, + DEV_ATTR_NEIGHREACHABLETIME, __DEV_ATTR_MAX, }; @@ -65,15 +66,16 @@ struct device_type { }; enum { - DEV_OPT_MTU = (1 << 0), - DEV_OPT_MACADDR = (1 << 1), - DEV_OPT_TXQUEUELEN = (1 << 2), - DEV_OPT_IPV6 = (1 << 3), - DEV_OPT_PROMISC = (1 << 4), - DEV_OPT_RPFILTER = (1 << 5), - DEV_OPT_ACCEPTLOCAL = (1 << 6), - DEV_OPT_IGMPVERSION = (1 << 7), - DEV_OPT_MLDVERSION = (1 << 8), + DEV_OPT_MTU = (1 << 0), + DEV_OPT_MACADDR = (1 << 1), + DEV_OPT_TXQUEUELEN = (1 << 2), + DEV_OPT_IPV6 = (1 << 3), + DEV_OPT_PROMISC = (1 << 4), + DEV_OPT_RPFILTER = (1 << 5), + DEV_OPT_ACCEPTLOCAL = (1 << 6), + DEV_OPT_IGMPVERSION = (1 << 7), + DEV_OPT_MLDVERSION = (1 << 8), + DEV_OPT_NEIGHREACHABLETIME = (1 << 9), }; /* events broadcasted to all users of a device */ @@ -125,6 +127,8 @@ struct device_settings { bool acceptlocal; unsigned int igmpversion; unsigned int mldversion; + unsigned int neigh4reachabletime; + unsigned int neigh6reachabletime; }; /* diff --git a/system-linux.c b/system-linux.c index 4737fa6..fb2ee29 100644 --- a/system-linux.c +++ b/system-linux.c @@ -285,6 +285,16 @@ static void system_set_mldversion(struct device *dev, const char *val) system_set_dev_sysctl("/proc/sys/net/ipv6/conf/%s/force_mld_version", dev->ifname, val); } +static void system_set_neigh4reachabletime(struct device *dev, const char *val) +{ + system_set_dev_sysctl("/proc/sys/net/ipv4/neigh/%s/base_reachable_time_ms", dev->ifname, val); +} + +static void system_set_neigh6reachabletime(struct device *dev, const char *val) +{ + system_set_dev_sysctl("/proc/sys/net/ipv6/neigh/%s/base_reachable_time_ms", dev->ifname, val); +} + static int system_get_sysctl(const char *path, char *buf, const size_t buf_sz) { int fd = -1, ret = -1; @@ -343,6 +353,18 @@ static int system_get_mldversion(struct device *dev, char *buf, const size_t buf dev->ifname, buf, buf_sz); } +static int system_get_neigh4reachabletime(struct device *dev, char *buf, const size_t buf_sz) +{ + return system_get_dev_sysctl("/proc/sys/net/ipv4/neigh/%s/base_reachable_time_ms", + dev->ifname, buf, buf_sz); +} + +static int system_get_neigh6reachabletime(struct device *dev, char *buf, const size_t buf_sz) +{ + return system_get_dev_sysctl("/proc/sys/net/ipv6/neigh/%s/base_reachable_time_ms", + dev->ifname, buf, buf_sz); +} + // Evaluate netlink messages static int cb_rtnl_event(struct nl_msg *msg, void *arg) { @@ -1017,6 +1039,16 @@ system_if_get_settings(struct device *dev, struct device_settings *s) s->mldversion = strtoul(buf, NULL, 0); s->flags |= DEV_OPT_MLDVERSION; } + + if (!system_get_neigh4reachabletime(dev, buf, sizeof(buf))) { + s->neigh4reachabletime = strtoul(buf, NULL, 0); + s->flags |= DEV_OPT_NEIGHREACHABLETIME; + } + + if (!system_get_neigh6reachabletime(dev, buf, sizeof(buf))) { + s->neigh6reachabletime = strtoul(buf, NULL, 0); + s->flags |= DEV_OPT_NEIGHREACHABLETIME; + } } void @@ -1072,6 +1104,14 @@ system_if_apply_settings(struct device *dev, struct device_settings *s, unsigned snprintf(buf, sizeof(buf), "%d", s->mldversion); system_set_mldversion(dev, buf); } + if (s->flags & DEV_OPT_NEIGHREACHABLETIME & apply_mask) { + char buf[12]; + + snprintf(buf, sizeof(buf), "%d", s->neigh4reachabletime); + system_set_neigh4reachabletime(dev, buf); + snprintf(buf, sizeof(buf), "%d", s->neigh6reachabletime); + system_set_neigh6reachabletime(dev, buf); + } } int system_if_up(struct device *dev)