* [PATCH net-next v3] mpls: support for dead routes
@ 2015-11-03 14:49 ` Roopa Prabhu
2015-11-03 15:15 ` Robert Shearman
2015-11-03 15:25 ` [PATCH] mpls: fix semicolon.cocci warnings kbuild test robot
0 siblings, 2 replies; 8+ messages in thread
From: Roopa Prabhu @ 2015-11-03 14:49 UTC (permalink / raw)
To: ebiederm, rshearma; +Cc: davem, netdev
From: Roopa Prabhu <roopa@cumulusnetworks.com>
Adds support for RTNH_F_DEAD and RTNH_F_LINKDOWN flags on mpls
routes due to link events. Also adds code to ignore dead
routes during route selection
Signed-off-by: Roopa Prabhu <roopa@cumulusnetworks.com>
---
Dave, I know you are only taking bug fixes currently. This patch
is borderline a bug fix because Eric thinks it is critical for
mpls multipath routes. I can sure resubmit it as a bug fix against net
when it is time if you did prefer that. Thanks!
RFC to v1:
Addressed a few comments from Eric and Robert:
- remove support for weighted nexthops
- Use rt_nhn_alive in the rt structure to keep count of alive
routes.
What i have not done is: sort nexthops on link events.
I am not comfortable recreating or sorting nexthops on
every carrier change. This leaves scope for optimizing in the future
v1 to v2:
Fix dead nexthop checks as suggested by dave
v2 to v3:
Fix duplicated argument reported by kbuild test robot
net/mpls/af_mpls.c | 189 ++++++++++++++++++++++++++++++++++++++++++++--------
net/mpls/internal.h | 3 +
2 files changed, 165 insertions(+), 27 deletions(-)
diff --git a/net/mpls/af_mpls.c b/net/mpls/af_mpls.c
index c70d750..8054904 100644
--- a/net/mpls/af_mpls.c
+++ b/net/mpls/af_mpls.c
@@ -96,22 +96,15 @@ bool mpls_pkt_too_big(const struct sk_buff *skb, unsigned int mtu)
}
EXPORT_SYMBOL_GPL(mpls_pkt_too_big);
-static struct mpls_nh *mpls_select_multipath(struct mpls_route *rt,
- struct sk_buff *skb, bool bos)
+static u32 mpls_multipath_hash(struct mpls_route *rt,
+ struct sk_buff *skb, bool bos)
{
struct mpls_entry_decoded dec;
struct mpls_shim_hdr *hdr;
bool eli_seen = false;
int label_index;
- int nh_index = 0;
u32 hash = 0;
- /* No need to look further into packet if there's only
- * one path
- */
- if (rt->rt_nhn == 1)
- goto out;
-
for (label_index = 0; label_index < MAX_MP_SELECT_LABELS && !bos;
label_index++) {
if (!pskb_may_pull(skb, sizeof(*hdr) * label_index))
@@ -165,9 +158,37 @@ static struct mpls_nh *mpls_select_multipath(struct mpls_route *rt,
}
}
- nh_index = hash % rt->rt_nhn;
+ return hash;
+}
+
+static struct mpls_nh *mpls_select_multipath(struct mpls_route *rt,
+ struct sk_buff *skb, bool bos)
+{
+ u32 hash = 0;
+ int nh_index;
+ int n = 0;
+
+ /* No need to look further into packet if there's only
+ * one path
+ */
+ if (rt->rt_nhn == 1)
+ goto out;
+
+ if (rt->rt_nhn_alive <= 0)
+ return NULL;
+
+ hash = mpls_multipath_hash(rt, skb, bos);
+ nh_index = hash % rt->rt_nhn_alive;
+ for_nexthops(rt) {
+ if (nh->nh_flags & (RTNH_F_DEAD | RTNH_F_LINKDOWN))
+ continue;
+ if (n == nh_index)
+ return nh;
+ n++;
+ } endfor_nexthops(rt);
+
out:
- return &rt->rt_nh[nh_index];
+ return &rt->rt_nh[0];
}
static bool mpls_egress(struct mpls_route *rt, struct sk_buff *skb,
@@ -365,6 +386,7 @@ static struct mpls_route *mpls_rt_alloc(int num_nh, u8 max_alen)
GFP_KERNEL);
if (rt) {
rt->rt_nhn = num_nh;
+ rt->rt_nhn_alive = num_nh;
rt->rt_max_alen = max_alen_aligned;
}
@@ -536,6 +558,15 @@ static int mpls_nh_assign_dev(struct net *net, struct mpls_route *rt,
RCU_INIT_POINTER(nh->nh_dev, dev);
+ if (!netif_carrier_ok(dev))
+ nh->nh_flags |= RTNH_F_LINKDOWN;
+
+ if (!(dev->flags & IFF_UP))
+ nh->nh_flags |= RTNH_F_DEAD;
+
+ if (nh->nh_flags & (RTNH_F_DEAD | RTNH_F_LINKDOWN))
+ rt->rt_nhn_alive--;
+
return 0;
errout:
@@ -577,7 +608,7 @@ errout:
}
static int mpls_nh_build(struct net *net, struct mpls_route *rt,
- struct mpls_nh *nh, int oif,
+ struct mpls_nh *nh, int oif, int hops,
struct nlattr *via, struct nlattr *newdst)
{
int err = -ENOMEM;
@@ -681,8 +712,8 @@ static int mpls_nh_build_multi(struct mpls_route_config *cfg,
goto errout;
err = mpls_nh_build(cfg->rc_nlinfo.nl_net, rt, nh,
- rtnh->rtnh_ifindex, nla_via,
- nla_newdst);
+ rtnh->rtnh_ifindex, rtnh->rtnh_hops,
+ nla_via, nla_newdst);
if (err)
goto errout;
@@ -875,34 +906,100 @@ free:
return ERR_PTR(err);
}
-static void mpls_ifdown(struct net_device *dev)
+static void mpls_ifdown(struct net_device *dev, int event)
{
struct mpls_route __rcu **platform_label;
struct net *net = dev_net(dev);
- struct mpls_dev *mdev;
unsigned index;
+ int dead;
platform_label = rtnl_dereference(net->mpls.platform_label);
for (index = 0; index < net->mpls.platform_labels; index++) {
struct mpls_route *rt = rtnl_dereference(platform_label[index]);
+
if (!rt)
continue;
+ dead = 0;
for_nexthops(rt) {
+ if ((event == NETDEV_DOWN &&
+ (nh->nh_flags & RTNH_F_DEAD)) ||
+ (event == NETDEV_CHANGE &&
+ (nh->nh_flags & RTNH_F_LINKDOWN))) {
+ dead++;
+ continue;
+ }
+
if (rtnl_dereference(nh->nh_dev) != dev)
continue;
- nh->nh_dev = NULL;
+ switch (event) {
+ case NETDEV_DOWN:
+ case NETDEV_UNREGISTER:
+ nh->nh_flags |= RTNH_F_DEAD;
+ /* fall through */
+ case NETDEV_CHANGE:
+ nh->nh_flags |= RTNH_F_LINKDOWN;
+ rt->rt_nhn_alive--;
+ break;
+ }
+ if (event == NETDEV_UNREGISTER) {
+ nh->nh_dev = NULL;
+ dead = rt->rt_nhn;
+ break;
+ }
+ dead++;
} endfor_nexthops(rt);
+
+ if (dead == rt->rt_nhn) {
+ switch (event) {
+ case NETDEV_DOWN:
+ case NETDEV_UNREGISTER:
+ rt->rt_flags |= RTNH_F_DEAD;
+ /* fall through */
+ case NETDEV_CHANGE:
+ rt->rt_flags |= RTNH_F_LINKDOWN;
+ rt->rt_nhn_alive = 0;
+ break;
+ }
+ }
}
- mdev = mpls_dev_get(dev);
- if (!mdev)
- return;
+ return;
+}
+
+static void mpls_ifup(struct net_device *dev, unsigned int nh_flags)
+{
+ struct mpls_route __rcu **platform_label;
+ struct net *net = dev_net(dev);
+ unsigned index;
+ int alive;
+
+ platform_label = rtnl_dereference(net->mpls.platform_label);
+ for (index = 0; index < net->mpls.platform_labels; index++) {
+ struct mpls_route *rt = rtnl_dereference(platform_label[index]);
+
+ if (!rt)
+ continue;
+ alive = 0;
+ for_nexthops(rt) {
+ struct net_device *nh_dev =
+ rtnl_dereference(nh->nh_dev);
- mpls_dev_sysctl_unregister(mdev);
+ if (!(nh->nh_flags & nh_flags)) {
+ alive++;
+ continue;
+ }
+ if (nh_dev != dev)
+ continue;
+ alive++;
+ nh->nh_flags &= ~nh_flags;
+ } endfor_nexthops(rt);
- RCU_INIT_POINTER(dev->mpls_ptr, NULL);
+ if (alive > 0)
+ rt->rt_flags &= ~nh_flags;
+ rt->rt_nhn_alive = alive;
+ }
- kfree_rcu(mdev, rcu);
+ return;
}
static int mpls_dev_notify(struct notifier_block *this, unsigned long event,
@@ -910,9 +1007,9 @@ static int mpls_dev_notify(struct notifier_block *this, unsigned long event,
{
struct net_device *dev = netdev_notifier_info_to_dev(ptr);
struct mpls_dev *mdev;
+ unsigned int flags;
- switch(event) {
- case NETDEV_REGISTER:
+ if (event == NETDEV_REGISTER) {
/* For now just support ethernet devices */
if ((dev->type == ARPHRD_ETHER) ||
(dev->type == ARPHRD_LOOPBACK)) {
@@ -920,10 +1017,39 @@ static int mpls_dev_notify(struct notifier_block *this, unsigned long event,
if (IS_ERR(mdev))
return notifier_from_errno(PTR_ERR(mdev));
}
- break;
+ return NOTIFY_OK;
+ }
+ mdev = mpls_dev_get(dev);
+ if (!mdev)
+ return NOTIFY_OK;
+
+ switch (event) {
+ case NETDEV_DOWN:
+ mpls_ifdown(dev, event);
+ break;
+ case NETDEV_UP:
+ flags = dev_get_flags(dev);
+ if (flags & (IFF_RUNNING | IFF_LOWER_UP))
+ mpls_ifup(dev, RTNH_F_DEAD | RTNH_F_LINKDOWN);
+ else
+ mpls_ifup(dev, RTNH_F_DEAD);
+ break;
+ case NETDEV_CHANGE:
+ flags = dev_get_flags(dev);
+ if (flags & (IFF_RUNNING | IFF_LOWER_UP))
+ mpls_ifup(dev, RTNH_F_DEAD | RTNH_F_LINKDOWN);
+ else
+ mpls_ifdown(dev, event);
+ break;
case NETDEV_UNREGISTER:
- mpls_ifdown(dev);
+ mpls_ifdown(dev, event);
+ mdev = mpls_dev_get(dev);
+ if (mdev) {
+ mpls_dev_sysctl_unregister(mdev);
+ RCU_INIT_POINTER(dev->mpls_ptr, NULL);
+ kfree_rcu(mdev, rcu);
+ }
break;
case NETDEV_CHANGENAME:
mdev = mpls_dev_get(dev);
@@ -1237,6 +1363,10 @@ static int mpls_dump_route(struct sk_buff *skb, u32 portid, u32 seq, int event,
dev = rtnl_dereference(nh->nh_dev);
if (dev && nla_put_u32(skb, RTA_OIF, dev->ifindex))
goto nla_put_failure;
+ if (nh->nh_flags & RTNH_F_LINKDOWN)
+ rtm->rtm_flags |= RTNH_F_LINKDOWN;
+ if (nh->nh_flags & RTNH_F_DEAD)
+ rtm->rtm_flags |= RTNH_F_DEAD;
} else {
struct rtnexthop *rtnh;
struct nlattr *mp;
@@ -1253,6 +1383,11 @@ static int mpls_dump_route(struct sk_buff *skb, u32 portid, u32 seq, int event,
dev = rtnl_dereference(nh->nh_dev);
if (dev)
rtnh->rtnh_ifindex = dev->ifindex;
+ if (nh->nh_flags & RTNH_F_LINKDOWN)
+ rtnh->rtnh_flags |= RTNH_F_LINKDOWN;
+ if (nh->nh_flags & RTNH_F_DEAD)
+ rtnh->rtnh_flags |= RTNH_F_DEAD;
+
if (nh->nh_labels && nla_put_labels(skb, RTA_NEWDST,
nh->nh_labels,
nh->nh_label))
diff --git a/net/mpls/internal.h b/net/mpls/internal.h
index bde52ce..4f9bf2b 100644
--- a/net/mpls/internal.h
+++ b/net/mpls/internal.h
@@ -41,6 +41,7 @@ enum mpls_payload_type {
struct mpls_nh { /* next hop label forwarding entry */
struct net_device __rcu *nh_dev;
+ unsigned int nh_flags;
u32 nh_label[MAX_NEW_LABELS];
u8 nh_labels;
u8 nh_via_alen;
@@ -70,10 +71,12 @@ struct mpls_nh { /* next hop label forwarding entry */
*/
struct mpls_route { /* next hop label forwarding entry */
struct rcu_head rt_rcu;
+ unsigned int rt_flags;
u8 rt_protocol;
u8 rt_payload_type;
u8 rt_max_alen;
unsigned int rt_nhn;
+ unsigned int rt_nhn_alive;
struct mpls_nh rt_nh[0];
};
--
1.9.1
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH net-next v3] mpls: support for dead routes
2015-11-03 14:49 ` Roopa Prabhu
@ 2015-11-03 15:15 ` Robert Shearman
2015-11-03 15:25 ` [PATCH] mpls: fix semicolon.cocci warnings kbuild test robot
1 sibling, 0 replies; 8+ messages in thread
From: Robert Shearman @ 2015-11-03 15:15 UTC (permalink / raw)
To: Roopa Prabhu, ebiederm; +Cc: davem, netdev
On 03/11/15 14:49, Roopa Prabhu wrote:
> From: Roopa Prabhu <roopa@cumulusnetworks.com>
>
> Adds support for RTNH_F_DEAD and RTNH_F_LINKDOWN flags on mpls
> routes due to link events. Also adds code to ignore dead
> routes during route selection
>
> Signed-off-by: Roopa Prabhu <roopa@cumulusnetworks.com>
> ---
> Dave, I know you are only taking bug fixes currently. This patch
> is borderline a bug fix because Eric thinks it is critical for
> mpls multipath routes. I can sure resubmit it as a bug fix against net
> when it is time if you did prefer that. Thanks!
>
> RFC to v1:
> Addressed a few comments from Eric and Robert:
> - remove support for weighted nexthops
> - Use rt_nhn_alive in the rt structure to keep count of alive
> routes.
> What i have not done is: sort nexthops on link events.
> I am not comfortable recreating or sorting nexthops on
> every carrier change. This leaves scope for optimizing in the future
>
> v1 to v2:
> Fix dead nexthop checks as suggested by dave
>
> v2 to v3:
> Fix duplicated argument reported by kbuild test robot
FYI, I've sent out comments to v2 and all but one of them also apply to
this version too.
Thanks,
Rob
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH] mpls: fix semicolon.cocci warnings
2015-11-03 14:49 ` Roopa Prabhu
2015-11-03 15:15 ` Robert Shearman
@ 2015-11-03 15:25 ` kbuild test robot
2015-11-03 16:17 ` David Miller
1 sibling, 1 reply; 8+ messages in thread
From: kbuild test robot @ 2015-11-03 15:25 UTC (permalink / raw)
To: Roopa Prabhu; +Cc: kbuild-all, ebiederm, rshearma, davem, netdev
net/mpls/af_mpls.c:722:22-23: Unneeded semicolon
Remove unneeded semicolon.
Generated by: scripts/coccinelle/misc/semicolon.cocci
CC: Roopa Prabhu <roopa@cumulusnetworks.com>
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
---
af_mpls.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
--- a/net/mpls/af_mpls.c
+++ b/net/mpls/af_mpls.c
@@ -719,9 +719,9 @@ static int mpls_nh_build_multi(struct mp
rtnh = rtnh_next(rtnh, &remaining);
nhs++;
- } endfor_nexthops(rt);
+ } endfor_nexthops(rt)
- rt->rt_nhn = nhs;
+ rt->rt_nhn = nhs;
return 0;
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH net-next v3] mpls: support for dead routes
2015-11-03 14:49 ` Roopa Prabhu
@ 2015-11-03 15:25 kbuild test robot
2015-11-03 14:49 ` Roopa Prabhu
1 sibling, 1 reply; 8+ messages in thread
From: kbuild test robot @ 2015-11-03 15:25 UTC (permalink / raw)
To: Roopa Prabhu; +Cc: kbuild-all, ebiederm, rshearma, davem, netdev
Hi Roopa,
[auto build test WARNING on net-next/master]
[also WARNING on: next-20151103]
url: https://github.com/0day-ci/linux/commits/Roopa-Prabhu/mpls-support-for-dead-routes/20151103-225234
base: https://github.com/0day-ci/linux Roopa-Prabhu/mpls-support-for-dead-routes/20151103-225234
coccinelle warnings: (new ones prefixed by >>)
>> net/mpls/af_mpls.c:722:22-23: Unneeded semicolon
Please review and possibly fold the followup patch.
---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] mpls: fix semicolon.cocci warnings
2015-11-03 15:25 ` [PATCH] mpls: fix semicolon.cocci warnings kbuild test robot
@ 2015-11-03 16:17 ` David Miller
2015-11-04 2:01 ` [kbuild-all] " Fengguang Wu
0 siblings, 1 reply; 8+ messages in thread
From: David Miller @ 2015-11-03 16:17 UTC (permalink / raw)
To: lkp; +Cc: roopa, kbuild-all, ebiederm, rshearma, netdev
From: kbuild test robot <lkp@intel.com>
Date: Tue, 3 Nov 2015 23:25:39 +0800
> net/mpls/af_mpls.c:722:22-23: Unneeded semicolon
>
>
> Remove unneeded semicolon.
>
> Generated by: scripts/coccinelle/misc/semicolon.cocci
>
> CC: Roopa Prabhu <roopa@cumulusnetworks.com>
> Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
> ---
>
> af_mpls.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> --- a/net/mpls/af_mpls.c
> +++ b/net/mpls/af_mpls.c
> @@ -719,9 +719,9 @@ static int mpls_nh_build_multi(struct mp
>
> rtnh = rtnh_next(rtnh, &remaining);
> nhs++;
> - } endfor_nexthops(rt);
> + } endfor_nexthops(rt)
>
> - rt->rt_nhn = nhs;
> + rt->rt_nhn = nhs;
This new indentation of "rt->rt_nhn = nhs;" is not correct.
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [kbuild-all] [PATCH] mpls: fix semicolon.cocci warnings
2015-11-03 16:17 ` David Miller
@ 2015-11-04 2:01 ` Fengguang Wu
2015-11-04 6:50 ` Julia Lawall
0 siblings, 1 reply; 8+ messages in thread
From: Fengguang Wu @ 2015-11-04 2:01 UTC (permalink / raw)
To: David Miller; +Cc: netdev, roopa, rshearma, kbuild-all, ebiederm, Julia Lawall
On Tue, Nov 03, 2015 at 11:17:36AM -0500, David Miller wrote:
> From: kbuild test robot <lkp@intel.com>
> Date: Tue, 3 Nov 2015 23:25:39 +0800
>
> > net/mpls/af_mpls.c:722:22-23: Unneeded semicolon
> >
> >
> > Remove unneeded semicolon.
> >
> > Generated by: scripts/coccinelle/misc/semicolon.cocci
> >
> > CC: Roopa Prabhu <roopa@cumulusnetworks.com>
> > Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
> > ---
> >
> > af_mpls.c | 4 ++--
> > 1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > --- a/net/mpls/af_mpls.c
> > +++ b/net/mpls/af_mpls.c
> > @@ -719,9 +719,9 @@ static int mpls_nh_build_multi(struct mp
> >
> > rtnh = rtnh_next(rtnh, &remaining);
> > nhs++;
> > - } endfor_nexthops(rt);
> > + } endfor_nexthops(rt)
> >
> > - rt->rt_nhn = nhs;
> > + rt->rt_nhn = nhs;
>
> This new indentation of "rt->rt_nhn = nhs;" is not correct.
Yes. CC Julia for coccinelle.
Thanks,
Fengguang
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [kbuild-all] [PATCH] mpls: fix semicolon.cocci warnings
2015-11-04 2:01 ` [kbuild-all] " Fengguang Wu
@ 2015-11-04 6:50 ` Julia Lawall
2015-11-04 14:57 ` David Miller
0 siblings, 1 reply; 8+ messages in thread
From: Julia Lawall @ 2015-11-04 6:50 UTC (permalink / raw)
To: Fengguang Wu; +Cc: David Miller, netdev, roopa, rshearma, kbuild-all, ebiederm
On Wed, 4 Nov 2015, Fengguang Wu wrote:
> On Tue, Nov 03, 2015 at 11:17:36AM -0500, David Miller wrote:
> > From: kbuild test robot <lkp@intel.com>
> > Date: Tue, 3 Nov 2015 23:25:39 +0800
> >
> > > net/mpls/af_mpls.c:722:22-23: Unneeded semicolon
> > >
> > >
> > > Remove unneeded semicolon.
> > >
> > > Generated by: scripts/coccinelle/misc/semicolon.cocci
> > >
> > > CC: Roopa Prabhu <roopa@cumulusnetworks.com>
> > > Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
> > > ---
> > >
> > > af_mpls.c | 4 ++--
> > > 1 file changed, 2 insertions(+), 2 deletions(-)
> > >
> > > --- a/net/mpls/af_mpls.c
> > > +++ b/net/mpls/af_mpls.c
> > > @@ -719,9 +719,9 @@ static int mpls_nh_build_multi(struct mp
> > >
> > > rtnh = rtnh_next(rtnh, &remaining);
> > > nhs++;
> > > - } endfor_nexthops(rt);
> > > + } endfor_nexthops(rt)
> > >
> > > - rt->rt_nhn = nhs;
> > > + rt->rt_nhn = nhs;
> >
> > This new indentation of "rt->rt_nhn = nhs;" is not correct.
>
> Yes. CC Julia for coccinelle.
I will look into it, but Coccinelle is probably thrown off by there being
what looks like a function call on the same line as a close brace.
I saw that quite a lot of other calls to endfor_nexthops also have
semicolons.
The whole idea of a macro that declares variables some of which are then
used in the code under the macro also seems quite unpleasant. Other
iterators don't do this, and so they don't need the end marker.
julia
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [kbuild-all] [PATCH] mpls: fix semicolon.cocci warnings
2015-11-04 6:50 ` Julia Lawall
@ 2015-11-04 14:57 ` David Miller
0 siblings, 0 replies; 8+ messages in thread
From: David Miller @ 2015-11-04 14:57 UTC (permalink / raw)
To: julia.lawall; +Cc: lkp, netdev, roopa, rshearma, kbuild-all, ebiederm
From: Julia Lawall <julia.lawall@lip6.fr>
Date: Wed, 4 Nov 2015 07:50:05 +0100 (CET)
> The whole idea of a macro that declares variables some of which are then
> used in the code under the macro also seems quite unpleasant. Other
> iterators don't do this, and so they don't need the end marker.
I'm open minded about cleaning these iterators up.
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2015-11-04 14:57 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-11-03 15:25 [PATCH net-next v3] mpls: support for dead routes kbuild test robot
2015-11-03 14:49 ` Roopa Prabhu
2015-11-03 15:15 ` Robert Shearman
2015-11-03 15:25 ` [PATCH] mpls: fix semicolon.cocci warnings kbuild test robot
2015-11-03 16:17 ` David Miller
2015-11-04 2:01 ` [kbuild-all] " Fengguang Wu
2015-11-04 6:50 ` Julia Lawall
2015-11-04 14:57 ` David Miller
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).