Re: [PATCH net] ipv6: no need to return rt->dst.error if it is not null entry.

2017-07-26 Thread Roopa Prabhu
On Wed, Jul 26, 2017 at 12:00 PM, David Ahern wrote: > On 7/26/17 12:55 PM, Roopa Prabhu wrote: >> On Wed, Jul 26, 2017 at 11:49 AM, David Ahern wrote: >>> On 7/26/17 12:27 PM, Roopa Prabhu wrote: >>>> agreed...so looks like the check in v3 should be >>>&

Re: [PATCHv4 net] ipv6: no need to check rt->dst.error when get route info

2017-07-27 Thread Roopa Prabhu
t; unreachable 2004::/64 dev lo metric 1024 error -113 pref medium > > Fixes: 18c3a61c4264 ("net: ipv6: RTM_GETROUTE: return matched fib...") > Signed-off-by: Hangbin Liu > --- Acked-by: Roopa Prabhu

[PATCH net-next v2 0/3] ethtool: support for forward error correction mode setting on a link

2017-07-27 Thread Roopa Prabhu
From: Roopa Prabhu Forward Error Correction (FEC) modes i.e Base-R and Reed-Solomon modes are introduced in 25G/40G/100G standards for providing good BER at high speeds. Various networking devices which support 25G/40G/100G provides ability to manage supported FEC modes and the lack of FEC

[PATCH net-next v2 2/3] cxgb4: core hardware/firmware support for Forward Error Correction on a link

2017-07-27 Thread Roopa Prabhu
From: Casey Leedom Signed-off-by: Casey Leedom --- drivers/net/ethernet/chelsio/cxgb4/t4_hw.c | 152 ++--- 1 file changed, 117 insertions(+), 35 deletions(-) diff --git a/drivers/net/ethernet/chelsio/cxgb4/t4_hw.c b/drivers/net/ethernet/chelsio/cxgb4/t4_hw.c index db41

[PATCH net-next v2 3/3] cxgb4: ethtool forward error correction management support

2017-07-27 Thread Roopa Prabhu
From: Casey Leedom Signed-off-by: Casey Leedom --- drivers/net/ethernet/chelsio/cxgb4/cxgb4_ethtool.c | 100 + 1 file changed, 100 insertions(+) diff --git a/drivers/net/ethernet/chelsio/cxgb4/cxgb4_ethtool.c b/drivers/net/ethernet/chelsio/cxgb4/cxgb4_ethtool.c index 26eb0

[PATCH net-next v2 1/3] net: ethtool: add support for forward error correction modes

2017-07-27 Thread Roopa Prabhu
FEC modes i.e. None (No FEC mode), RS, BaseR/FC are defined so that users can configure these fec modes for supported and advertising fields as part of link autonegotiation. Signed-off-by: Vidya Sagar Ravipati Signed-off-by: Dustin Byford Signed-off-by: Roopa Prabhu --- include/linux

Re: [PATCH net-next v2 0/3] ethtool: support for forward error correction mode setting on a link

2017-07-28 Thread Roopa Prabhu
On Thu, Jul 27, 2017 at 7:33 PM, Jakub Kicinski wrote: > On Thu, 27 Jul 2017 16:47:25 -0700, Roopa Prabhu wrote: >> From: Roopa Prabhu >> >> Forward Error Correction (FEC) modes i.e Base-R >> and Reed-Solomon modes are introduced in 25G/40G/100G standards >> for

Re: [PATCH net] ipv6: no need to return rt->dst.error if it is not null entry.

2017-07-28 Thread Roopa Prabhu
On Fri, Jul 28, 2017 at 8:10 AM, David Ahern wrote: > On 7/27/17 10:56 PM, Cong Wang wrote: >> On Wed, Jul 26, 2017 at 11:49 AM, David Ahern wrote: >>> On 7/26/17 12:27 PM, Roopa Prabhu wrote: >>>> agreed...so looks like the check in v3 should be >>>&

Re: [PATCH net] ipv6: no need to return rt->dst.error if it is not null entry.

2017-07-28 Thread Roopa Prabhu
On Fri, Jul 28, 2017 at 10:39 AM, David Ahern wrote: > On 7/28/17 11:13 AM, Roopa Prabhu wrote: >> for fibmatch, my original intent was to return with an error code. >> This is similar >> to the ipv4 behavior. One option is to keep the check in there and put >> the &#x

[PATCH net-next v2] bridge: fdb add and delete tracepoints

2017-08-28 Thread Roopa Prabhu
From: Roopa Prabhu A few useful tracepoints to trace bridge forwarding database updates. Signed-off-by: Roopa Prabhu --- v2 - address comments from Florian include/trace/events/bridge.h | 98 + net/bridge/br_fdb.c |7 +++ net/core/net

Re: [PATCH net-next v2] bridge: fdb add and delete tracepoints

2017-08-29 Thread Roopa Prabhu
On Tue, Aug 29, 2017 at 6:36 AM, Andrew Lunn wrote: > On Mon, Aug 28, 2017 at 09:22:48PM -0700, Roopa Prabhu wrote: >> From: Roopa Prabhu >> >> A few useful tracepoints to trace bridge forwarding >> database updates. > > Hi Roopa > > Once accepted, it would

Re: [PATCH net-next v2] bridge: fdb add and delete tracepoints

2017-08-29 Thread Roopa Prabhu
On Tue, Aug 29, 2017 at 9:46 AM, Florian Fainelli wrote: > On 08/28/2017 09:22 PM, Roopa Prabhu wrote: >> From: Roopa Prabhu >> >> A few useful tracepoints to trace bridge forwarding >> database updates. >> >> Signed-off-by: Roopa Prabhu > > Revie

[PATCH net-next v3] bridge: fdb add and delete tracepoints

2017-08-29 Thread Roopa Prabhu
From: Roopa Prabhu A few useful tracepoints to trace bridge forwarding database updates. Signed-off-by: Roopa Prabhu --- v2: address comments from florian v3: remove stray character '=' in print (pointed out by florian) include/trace/events/bridg

[PATCH net-next] bridge: add tracepoint in br_fdb_update

2017-08-30 Thread Roopa Prabhu
From: Roopa Prabhu This extends bridge fdb table tracepoints to also cover learned fdb entries in the br_fdb_update path. Note that unlike other tracepoints I have moved this to when the fdb is modified because this is in the datapath and can generate a lot of noise in the trace output

Re: [PATCH net-next] bridge: add tracepoint in br_fdb_update

2017-08-31 Thread Roopa Prabhu
On Thu, Aug 31, 2017 at 5:38 AM, Jesper Dangaard Brouer wrote: > On Wed, 30 Aug 2017 22:18:13 -0700 > Roopa Prabhu wrote: > >> From: Roopa Prabhu >> >> This extends bridge fdb table tracepoints to also cover >> learned fdb entries in the br_fdb_update path. Note

Re: [RFC net-next] net: sch_clsact: add support for global per-netns classifier mode

2017-09-05 Thread Roopa Prabhu
On Tue, Sep 5, 2017 at 7:07 AM, Jiri Pirko wrote: > Tue, Sep 05, 2017 at 02:48:21PM CEST, niko...@cumulusnetworks.com wrote: >>Hi all, >>This RFC adds a new mode for clsact which designates a device's egress >>classifier as global per netns. The packets that are not classified for >>a particular d

Re: [RFC net-next] net: sch_clsact: add support for global per-netns classifier mode

2017-09-05 Thread Roopa Prabhu
On Tue, Sep 5, 2017 at 11:18 AM, Cong Wang wrote: > On Tue, Sep 5, 2017 at 5:48 AM, Nikolay Aleksandrov > wrote: >> Hi all, >> This RFC adds a new mode for clsact which designates a device's egress >> classifier as global per netns. The packets that are not classified for >> a particular device w

Re: [RFC net-next] net: sch_clsact: add support for global per-netns classifier mode

2017-09-05 Thread Roopa Prabhu
On Tue, Sep 5, 2017 at 3:45 PM, Daniel Borkmann wrote: > On 09/06/2017 12:01 AM, Roopa Prabhu wrote: >> >> On Tue, Sep 5, 2017 at 11:18 AM, Cong Wang >> wrote: >>> >>> On Tue, Sep 5, 2017 at 5:48 AM, Nikolay Aleksandrov >>> wrote: >>>>

Re: [RFC net-next] net: sch_clsact: add support for global per-netns classifier mode

2017-09-05 Thread Roopa Prabhu
On Tue, Sep 5, 2017 at 3:25 PM, Jamal Hadi Salim wrote: > On 17-09-05 06:01 PM, Roopa Prabhu wrote: > >> >> yes, like Nikolay says we have been discussing this as well. Nikolay's >> patch is a cleaver and most importantly non-invasive >> way today given the an

Re: [RFC net-next] net: sch_clsact: add support for global per-netns classifier mode

2017-09-06 Thread Roopa Prabhu
On Wed, Sep 6, 2017 at 12:24 AM, Jiri Pirko wrote: > Wed, Sep 06, 2017 at 06:04:17AM CEST, ro...@cumulusnetworks.com wrote: >>On Tue, Sep 5, 2017 at 3:45 PM, Daniel Borkmann wrote: >>> On 09/06/2017 12:01 AM, Roopa Prabhu wrote: >>>> >>>> On Tue, Sep

Re: [PATCH v2 rfc 0/8] IGMP snooping for local traffic

2017-09-06 Thread Roopa Prabhu
On Tue, Sep 5, 2017 at 5:47 PM, Andrew Lunn wrote: >> The third and last issue will be explained in a followup email. > > Hi DSA hackers > > So there is the third issue. It affects just DSA, but it possible > affects all DSA drivers. > > This patchset broken broadcast with the Marvell drivers. It

Re: [PATCH iproute2 3/3] bridge: request vlans along with link information

2017-09-09 Thread Roopa Prabhu
On Fri, Sep 8, 2017 at 2:52 PM, Roman Mashak wrote: > Signed-off-by: Roman Mashak > --- > bridge/link.c | 16 +--- > 1 file changed, 13 insertions(+), 3 deletions(-) > > diff --git a/bridge/link.c b/bridge/link.c > index 60200f1..9e4206f 100644 > --- a/bridge/link.c > +++ b/bridge/li

Re: [PATCH iproute2 3/3] bridge: request vlans along with link information

2017-09-09 Thread Roopa Prabhu
On Sat, Sep 9, 2017 at 10:23 AM, Jamal Hadi Salim wrote: > On 17-09-09 12:24 PM, Roopa Prabhu wrote: >> >> On Fri, Sep 8, 2017 at 2:52 PM, Roman Mashak wrote: >>> >>> Signed-off-by: Roman Mashak >>> --- >>> bridge/link.c | 16 +++

Re: [PATCH iproute2 3/3] bridge: request vlans along with link information

2017-09-09 Thread Roopa Prabhu
On Sat, Sep 9, 2017 at 11:15 AM, Nikolay Aleksandrov wrote: > On 09/09/17 20:23, Jamal Hadi Salim wrote: >> On 17-09-09 12:24 PM, Roopa Prabhu wrote: >>> On Fri, Sep 8, 2017 at 2:52 PM, Roman Mashak wrote: >>>> Signed-off-by: Roman Mashak >>>> --- >

Re: [RFC PATCH 0/9] ethtool netlink interface (WiP)

2017-12-12 Thread Roopa Prabhu
On Mon, Dec 11, 2017 at 5:53 AM, Michal Kubecek wrote: > This is still work in progress and only a very small part of the ioctl > interface is reimplemented but I would like to get some comments before > the patchset becomes too big and changing things becomes too tedious. > > The interface used f

[PATCH RFC net-next 3/4] ipv4: route: dissect flow in input path if fib rules need it

2018-02-11 Thread Roopa Prabhu
From: Roopa Prabhu Dissect flow in fwd path if fib rules require it. Controlled by a flag to avoid penatly for the common case. Flag is set when fib rules with sport, dport and proto match that require flow dissect are installed. Also passes the dissected hash keys to the multipath hash function

[PATCH RFC net-next 1/4] ipv4: fib_rules: support match on sport, dport and ip proto

2018-02-11 Thread Roopa Prabhu
From: Roopa Prabhu Add support to match on src port, dst port and ip protocol. Signed-off-by: Roopa Prabhu --- include/uapi/linux/fib_rules.h | 3 +++ net/ipv4/fib_rules.c | 46 -- 2 files changed, 47 insertions(+), 2 deletions(-) diff --git

[PATCH RFC net-next 0/4] fib_rules: support sport, dport and proto match

2018-02-11 Thread Roopa Prabhu
From: Roopa Prabhu This series extends fib rule match support to include sport, dport and ip proto match (to complete the 5-tuple match support). Common use-cases of Policy based routing in the data center require 5-tuple match. The last 2 patches in the series add a call to flow dissect in the

[PATCH RFC net-next 4/4] ipv6: route: dissect flow in input path if fib rules need it

2018-02-11 Thread Roopa Prabhu
From: Roopa Prabhu Dissect flow in fwd path if fib rules require it. Controlled by a flag to avoid penatly for the common case. Flag is set when fib rules with sport, dport and proto match that require flow dissect are installed. Also passes the dissected hash keys to the multipath hash function

[PATCH RFC net-next 2/4] ipv6: fib6_rules: support for match on sport, dport and ip proto

2018-02-11 Thread Roopa Prabhu
From: Roopa Prabhu Add support to match on src port, dst port and ip protocol. Signed-off-by: Roopa Prabhu --- net/ipv6/fib6_rules.c | 49 ++--- 1 file changed, 46 insertions(+), 3 deletions(-) diff --git a/net/ipv6/fib6_rules.c b/net/ipv6

Re: [PATCH RFC net-next 4/4] ipv6: route: dissect flow in input path if fib rules need it

2018-02-12 Thread Roopa Prabhu
On Sun, Feb 11, 2018 at 11:44 PM, Ido Schimmel wrote: > On Sun, Feb 11, 2018 at 02:26:29PM -0800, Roopa Prabhu wrote: >> From: Roopa Prabhu >> >> Dissect flow in fwd path if fib rules require it. Controlled by >> a flag to avoid penatly for the common case. Flag is

Re: [PATCH RFC net-next 1/4] ipv4: fib_rules: support match on sport, dport and ip proto

2018-02-13 Thread Roopa Prabhu
On Mon, Feb 12, 2018 at 4:03 PM, David Miller wrote: > From: Eric Dumazet > Date: Mon, 12 Feb 2018 13:54:59 -0800 > >> We had project/teams using different routing tables for each vlan they >> setup :/ > > Indeed, people use FIB rules and think they can scale in software. As > currently implemen

Re: [PATCH RFC net-next 1/4] ipv4: fib_rules: support match on sport, dport and ip proto

2018-02-13 Thread Roopa Prabhu
On Tue, Feb 13, 2018 at 8:34 AM, David Miller wrote: > From: Roopa Prabhu > Date: Tue, 13 Feb 2018 08:19:35 -0800 > >> - Regardless of this series, I think we should optimize ip rules or >> have a new implementation of policy based routing. happy to hear about >

Re: [PATCH iproute2-next 1/8] iplink: Return from function instead of calling exit()

2018-02-20 Thread Roopa Prabhu
On Tue, Feb 20, 2018 at 12:33 PM, Stephen Hemminger wrote: > On Tue, 20 Feb 2018 13:27:21 -0700 > David Ahern wrote: > >> On 2/20/18 1:17 PM, Serhey Popovych wrote: >> > Stephen Hemminger wrote: >> >> On Tue, 20 Feb 2018 21:39:51 +0200 >> >> Serhey Popovych wrote: >> >> >> >>> Signed-off-by: Ser

Re: [PATCH iproute2-next 1/8] iplink: Return from function instead of calling exit()

2018-02-20 Thread Roopa Prabhu
On Tue, Feb 20, 2018 at 12:49 PM, David Ahern wrote: > On 2/20/18 1:44 PM, Roopa Prabhu wrote: >> On Tue, Feb 20, 2018 at 12:33 PM, Stephen Hemminger >> wrote: >>> On Tue, 20 Feb 2018 13:27:21 -0700 >>> David Ahern wrote: >>> >>>> On

[PATCH iproute2 net-next] ss: print skmeminfo for packet sockets

2018-02-21 Thread Roopa Prabhu
From: Roopa Prabhu before: $ss --packet -p -m p_raw0 0*:eth0 users:(("lldpd",pid=2240,fd=11)) after: $ss --packet -p -m p_raw0 0*:eth0 users:(("lldpd",pid=2240,fd=11))

Re: [RFC PATCH net-next 02/12] selftests: forwarding: Add a test for FDB learning

2018-01-17 Thread Roopa Prabhu
On Wed, Jan 17, 2018 at 1:01 PM, Jiri Pirko wrote: > Wed, Jan 17, 2018 at 09:48:54PM CET, dsah...@gmail.com wrote: >>On 1/15/18 11:18 AM, Ido Schimmel wrote: >>> diff --git a/tools/testing/selftests/forwarding/lib.sh >>> b/tools/testing/selftests/forwarding/lib.sh >>> index bb423371f4de..264bf0af

Re: [RFC PATCH net-next 02/12] selftests: forwarding: Add a test for FDB learning

2018-01-17 Thread Roopa Prabhu
On Wed, Jan 17, 2018 at 2:46 PM, Roopa Prabhu wrote: > On Wed, Jan 17, 2018 at 1:01 PM, Jiri Pirko wrote: >> Wed, Jan 17, 2018 at 09:48:54PM CET, dsah...@gmail.com wrote: >>>On 1/15/18 11:18 AM, Ido Schimmel wrote: >>>> diff --git a/tools/testing/selftests/forwardin

[PATCH net-next] net: link_watch: mark bonding link events urgent

2018-01-21 Thread Roopa Prabhu
From: Roopa Prabhu It takes 1sec for bond link down notification to hit user-space when all slaves of the bond go down. 1sec is too long for protocol daemons in user-space relying on bond link notification to failover/recover (eg: multichassis lag implementations in user-space). Since the link

Re: [PATCH net-next] net: link_watch: mark bonding link events urgent

2018-01-22 Thread Roopa Prabhu
On Mon, Jan 22, 2018 at 12:13 AM, Jiri Pirko wrote: > Mon, Jan 22, 2018 at 07:07:53AM CET, ro...@cumulusnetworks.com wrote: >>From: Roopa Prabhu >> >>It takes 1sec for bond link down notification to hit user-space >>when all slaves of the bond go down. 1sec is too lon

[PATCH net-next v2] net: link_watch: mark bonding link events urgent

2018-01-22 Thread Roopa Prabhu
From: Roopa Prabhu It takes 1sec for bond link down notification to hit user-space when all slaves of the bond go down. 1sec is too long for protocol daemons in user-space relying on bond notification to recover (eg: multichassis lag implementations in user-space). Since the link event code

Re: [PATCH net-next] net/ipv6: Do not allow route add with a device that is down

2018-01-23 Thread Roopa Prabhu
On Tue, Jan 23, 2018 at 7:13 PM, David Ahern wrote: > IPv6 allows routes to be installed when the device is not up (admin up). > Worse, it does not mark it as LINKDOWN. IPv4 does not allow it and really > there is no reason for IPv6 to allow it, so check the flags and deny if > device is admin dow

Re: [PATCH v3 net-next] net/ipv6: Do not allow route add with a device that is down

2018-01-25 Thread Roopa Prabhu
ice is admin down. > > Signed-off-by: David Ahern Reviewed-by: Roopa Prabhu > --- > v3 > - moved err=-ENETDOWN under the if check per Eric's request > - left the up check using dev->flags for consistency with IPv4 > and that it is used more often in ipv4 and ivp6 code

Re: [PATCH net-next v2 0/3] ethtool: support for forward error correction mode setting on a link

2017-07-28 Thread Roopa Prabhu
On Fri, Jul 28, 2017 at 9:46 AM, Jakub Kicinski wrote: > On Fri, 28 Jul 2017 07:53:01 -0700, Roopa Prabhu wrote: >> On Thu, Jul 27, 2017 at 7:33 PM, Jakub Kicinski wrote: >> > On Thu, 27 Jul 2017 16:47:25 -0700, Roopa Prabhu wrote: >> >> From: Roopa Prabhu >&g

Re: [PATCH net-next v2 0/3] ethtool: support for forward error correction mode setting on a link

2017-07-28 Thread Roopa Prabhu
On Fri, Jul 28, 2017 at 10:29 AM, Andrew Lunn wrote: > On Fri, Jul 28, 2017 at 09:46:20AM -0700, Jakub Kicinski wrote: >> On Fri, 28 Jul 2017 07:53:01 -0700, Roopa Prabhu wrote: >> > On Thu, Jul 27, 2017 at 7:33 PM, Jakub Kicinski wrote: >> > > On Thu, 27 Jul 201

Re: [PATCH RFC, iproute2] tc/mirred: Extend the mirred/redirect action to accept additional traffic class parameter

2017-08-02 Thread Roopa Prabhu
On Mon, Jul 31, 2017 at 5:40 PM, Amritha Nambiar wrote: > The Mirred/redirect action is extended to accept a traffic > class on the device in addition to the device's ifindex. > > Usage: mirred > > Example: > # tc qdisc add dev eth0 ingress > > # tc filter add dev eth0 protocol ip parent :

[PATCH net-next] lwtunnel: replace EXPORT_SYMBOL with EXPORT_SYMBOL_GPL

2017-08-04 Thread Roopa Prabhu
From: Roopa Prabhu Signed-off-by: Roopa Prabhu --- net/core/lwtunnel.c | 26 +- 1 file changed, 13 insertions(+), 13 deletions(-) diff --git a/net/core/lwtunnel.c b/net/core/lwtunnel.c index d9cb353..8693ff8 100644 --- a/net/core/lwtunnel.c +++ b/net/core/lwtunnel.c

Re: [PATCH net-next] lwtunnel: replace EXPORT_SYMBOL with EXPORT_SYMBOL_GPL

2017-08-04 Thread Roopa Prabhu
On Fri, Aug 4, 2017 at 3:25 PM, Andrew Lunn wrote: > On Fri, Aug 04, 2017 at 03:23:37PM -0700, Roopa Prabhu wrote: >> From: Roopa Prabhu >> >> Signed-off-by: Roopa Prabhu >> --- >> net/core/lwtunnel.c | 26 +- >> 1 file

[PATCH net-next v2] lwtunnel: replace EXPORT_SYMBOL with EXPORT_SYMBOL_GPL

2017-08-04 Thread Roopa Prabhu
From: Roopa Prabhu Signed-off-by: Roopa Prabhu --- v2 - fixed a incorrect replace net/core/lwtunnel.c | 26 +- 1 file changed, 13 insertions(+), 13 deletions(-) diff --git a/net/core/lwtunnel.c b/net/core/lwtunnel.c index d9cb353..435f35f 100644 --- a/net/core

Re: Driver profiles RFC

2017-08-08 Thread Roopa Prabhu
On Tue, Aug 8, 2017 at 6:15 AM, Arkadi Sharshevsky wrote: > Drivers may require driver specific information during the init stage. > For example, memory based shared resource which should be segmented for > different ASIC processes, such as FDB and LPM lookups. > > The current mlxsw implementation

Re: Driver profiles RFC

2017-08-11 Thread Roopa Prabhu
On Wed, Aug 9, 2017 at 4:43 AM, Arkadi Sharshevsky wrote: > > > On 08/08/2017 07:08 PM, Roopa Prabhu wrote: >> On Tue, Aug 8, 2017 at 6:15 AM, Arkadi Sharshevsky >> wrote: >>> [snip] >>> User Configuration >>> -- >>> Suc

Re: [PATCH 1/2] mpls: add handlers

2017-08-11 Thread Roopa Prabhu
On Fri, Aug 11, 2017 at 5:34 AM, David Lamparter wrote: > On Thu, Aug 10, 2017 at 10:28:36PM +0200, Amine Kherbouche wrote: >> Mpls handler allows creation/deletion of mpls routes without using >> rtnetlink. When an incoming mpls packet matches this route, the saved >> function handler is called.

Re: [PATCH 2/2] drivers: add vpls support

2017-08-11 Thread Roopa Prabhu
On Fri, Aug 11, 2017 at 5:55 AM, David Lamparter wrote: > On Thu, Aug 10, 2017 at 10:28:37PM +0200, Amine Kherbouche wrote: >> This commit introduces the support of VPLS virtual device, that allows >> performing L2VPN multipoint to multipoint communication over MPLS PSN. >> >> VPLS device encap r

Re: [PATCH net-next v2] vxlan: change vxlan_[config_]validate() to use netlink_ext_ack for error reporting

2017-08-11 Thread Roopa Prabhu
On Thu, Aug 10, 2017 at 2:16 PM, Girish Moodalbail wrote: > The kernel log is not where users expect error messages for netlink > requests; as we have extended acks now, we can replace pr_debug() with > NL_SET_ERR_MSG_ATTR(). > > Signed-off-by: Matthias Schiffer > Signed-off-by: Girish Moodalbail

Re: [PATCH net-next v2] vxlan: change vxlan_[config_]validate() to use netlink_ext_ack for error reporting

2017-08-11 Thread Roopa Prabhu
On Fri, Aug 11, 2017 at 9:39 AM, Jiri Benc wrote: > On Fri, 11 Aug 2017 09:19:34 -0700, Roopa Prabhu wrote: >> > if (tb[IFLA_ADDRESS]) { >> > if (nla_len(tb[IFLA_ADDRESS]) != ETH_ALEN) { >> > - pr_debug("in

Re: [PATCH 2/2] drivers: add vpls support

2017-08-12 Thread Roopa Prabhu
On Sat, Aug 12, 2017 at 6:40 AM, Amine Kherbouche wrote: > > > On 11/08/2017 17:14, Roopa Prabhu wrote: >> >> On Fri, Aug 11, 2017 at 5:55 AM, David Lamparter >> wrote: >>> >>> On Thu, Aug 10, 2017 at 10:28:37PM +0200, Amine Kherbouche wrote: >>

Re: [PATCH 1/2] mpls: add handlers

2017-08-12 Thread Roopa Prabhu
On Sat, Aug 12, 2017 at 6:35 AM, Amine Kherbouche wrote: > > > On 11/08/2017 16:37, Roopa Prabhu wrote: >> >> On Fri, Aug 11, 2017 at 5:34 AM, David Lamparter >> wrote: >>> >>> On Thu, Aug 10, 2017 at 10:28:36PM +0200, Amine Kherbouche wrote: >&

Re: [PATCH RFC net-next 3/6] net: Introduce IFF_LWT_NETDEV flag

2017-05-08 Thread Roopa Prabhu
On Mon, May 8, 2017 at 5:57 PM, David Ahern wrote: > On 5/8/17 1:11 PM, David Miller wrote: >> From: Johannes Berg >> Date: Mon, 08 May 2017 10:55:12 +0200 >> >>> +static inline bool netif_is_lwd(struct net_device *dev) +{ + return !!(dev->priv_flags & IFF_LWT_NETDEV); +} >>

[PATCH net-next 4/8] net: ipv4: Convert inet_rtm_getroute to rcu versions of route lookup

2017-05-24 Thread Roopa Prabhu
. In converting from ip_route_output_key to ip_route_output_key_hash_rcu the xfrm_lookup_route in ip_route_output_flow is dropped since flowi4_proto is not set for a route get request. Signed-off-by: David Ahern Signed-off-by: Roopa Prabhu --- net/ipv4/route.c | 25 +++-- 1

[PATCH net-next 6/8] net: ipv4: add new RTM_F_FIB_MATCH flag for use with RTM_GETROUTE

2017-05-24 Thread Roopa Prabhu
From: Roopa Prabhu This flag when specified will return matched fib result in response to a RTM_GETROUTE query. Signed-off-by: Roopa Prabhu --- include/uapi/linux/rtnetlink.h | 1 + 1 file changed, 1 insertion(+) diff --git a/include/uapi/linux/rtnetlink.h b/include/uapi/linux/rtnetlink.h

[PATCH net-next 7/8] net: ipv4: RTM_GETROUTE: return matched fib result when requested

2017-05-24 Thread Roopa Prabhu
From: Roopa Prabhu This patch adds support to return matched fib result when RTM_F_FIB_MATCH flag is specified in RTM_GETROUTE request. This is useful for user-space applications/controllers wanting to query a matching route. Signed-off-by: Roopa Prabhu --- net/ipv4/route.c | 13

[PATCH net-next 5/8] net: ipv4: Save trie prefix to fib lookup result

2017-05-24 Thread Roopa Prabhu
From: David Ahern Prefix is needed for returning matching route spec on get route request. Signed-off-by: David Ahern Signed-off-by: Roopa Prabhu --- include/net/ip_fib.h | 1 + net/ipv4/fib_trie.c | 1 + 2 files changed, 2 insertions(+) diff --git a/include/net/ip_fib.h b/include/net

[PATCH net-next 3/8] net: ipv4: Remove event arg to rt_fill_info

2017-05-24 Thread Roopa Prabhu
From: David Ahern rt_fill_info has 1 caller with the event set to RTM_NEWROUTE. Given that remove the arg and use RTM_NEWROUTE directly in rt_fill_info. Signed-off-by: David Ahern Signed-off-by: Roopa Prabhu --- net/ipv4/route.c | 7 +++ 1 file changed, 3 insertions(+), 4 deletions

[PATCH net-next 8/8] net: ipv6: RTM_GETROUTE: return matched fib result when requested

2017-05-24 Thread Roopa Prabhu
From: Roopa Prabhu This patch adds support to return matched fib result when RTM_F_FIB_MATCH flag is specified in RTM_GETROUTE request. This is useful for user-space applications/controllers wanting to query a matching route. Signed-off-by: Roopa Prabhu --- net/ipv6/route.c | 36

[PATCH net-next 0/8] net: extend RTM_GETROUTE to return fib result

2017-05-24 Thread Roopa Prabhu
From: Roopa Prabhu This series adds a new RTM_F_FIB_MATCH flag to return matched fib result with RTM_GETROUTE. This is useful for applications and protocols in userspace wanting to query the selected route. examples (with patched iproute2): ipv4: $ip route show default via 192.168.0.2 dev

[PATCH net-next 1/8] net: ipv4: refactor __ip_route_output_key_hash

2017-05-24 Thread Roopa Prabhu
under control remove the leading underscores from the name. _rcu is added to the name of the new helper indicating it is called with the rcu read lock held. Signed-off-by: David Ahern Signed-off-by: Roopa Prabhu --- include/net/route.h | 7 ++- net/ipv4/icmp.c | 2 +- net/ipv4/route.c

[PATCH net-next 2/8] net: ipv4: refactor ip_route_input_noref

2017-05-24 Thread Roopa Prabhu
Signed-off-by: Roopa Prabhu --- include/net/route.h | 3 +++ net/ipv4/route.c| 66 ++--- 2 files changed, 40 insertions(+), 29 deletions(-) diff --git a/include/net/route.h b/include/net/route.h index 5a92347..8d209ad 100644 --- a/include/net

Re: [PATCH net-next 1/8] net: ipv4: refactor __ip_route_output_key_hash

2017-05-24 Thread Roopa Prabhu
On Wed, May 24, 2017 at 6:10 PM, David Ahern wrote: > On 5/24/17 1:33 PM, Rosen, Rami wrote: >> Hi, Rupa /David Ahern, >> >> First, thanks for this patch set! >> >> Second, it seems to me that something might be incorrect here. >> >> You have these additions in this patch (1/8): >> ... >> +struct

Re: [PATCH net-next 8/8] net: ipv6: RTM_GETROUTE: return matched fib result when requested

2017-05-25 Thread Roopa Prabhu
On Wed, May 24, 2017 at 7:35 PM, David Ahern wrote: > Since you have to do a v2 ... > > On 5/24/17 12:19 PM, Roopa Prabhu wrote: >> @@ -3622,6 +3623,7 @@ static int inet6_rtm_getroute(struct sk_buff *in_skb, >> struct nlmsghdr *nlh, >> memset(&fl6

[PATCH net-next v2 0/8] net: extend RTM_GETROUTE to return fib result

2017-05-25 Thread Roopa Prabhu
From: Roopa Prabhu This series adds a new RTM_F_FIB_MATCH flag to return matched fib result with RTM_GETROUTE. This is useful for applications and protocols in userspace wanting to query the selected route. examples (with patched iproute2): ipv4: $ip route show default via 192.168.0.2 dev

[PATCH net-next v2 1/8] net: ipv4: refactor __ip_route_output_key_hash

2017-05-25 Thread Roopa Prabhu
control remove the leading underscores from the name and add _rcu to the name of the new helper indicating it is called with the rcu read lock held. Signed-off-by: David Ahern Signed-off-by: Roopa Prabhu --- include/net/route.h | 9 ++--- net/ipv4/icmp.c | 2 +- net/ipv4/route.c

[PATCH net-next v2 2/8] net: ipv4: refactor ip_route_input_noref

2017-05-25 Thread Roopa Prabhu
Signed-off-by: Roopa Prabhu --- include/net/route.h | 3 +++ net/ipv4/route.c| 66 ++--- 2 files changed, 40 insertions(+), 29 deletions(-) diff --git a/include/net/route.h b/include/net/route.h index 89e4028..08e689f 100644 --- a/include/net

[PATCH net-next v2 3/8] net: ipv4: Remove event arg to rt_fill_info

2017-05-25 Thread Roopa Prabhu
From: David Ahern rt_fill_info has 1 caller with the event set to RTM_NEWROUTE. Given that remove the arg and use RTM_NEWROUTE directly in rt_fill_info. Signed-off-by: David Ahern Signed-off-by: Roopa Prabhu --- net/ipv4/route.c | 7 +++ 1 file changed, 3 insertions(+), 4 deletions

[PATCH net-next v2 8/8] net: ipv6: RTM_GETROUTE: return matched fib result when requested

2017-05-25 Thread Roopa Prabhu
From: Roopa Prabhu This patch adds support to return matched fib result when RTM_F_FIB_MATCH flag is specified in RTM_GETROUTE request. This is useful for user-space applications/controllers wanting to query a matching route. Signed-off-by: Roopa Prabhu --- net/ipv6/route.c | 34

[PATCH net-next v2 6/8] net: ipv4: add new RTM_F_FIB_MATCH flag for use with RTM_GETROUTE

2017-05-25 Thread Roopa Prabhu
From: Roopa Prabhu This flag when specified will return matched fib result in response to a RTM_GETROUTE query. Signed-off-by: Roopa Prabhu --- include/uapi/linux/rtnetlink.h | 1 + 1 file changed, 1 insertion(+) diff --git a/include/uapi/linux/rtnetlink.h b/include/uapi/linux/rtnetlink.h

[PATCH net-next v2 7/8] net: ipv4: RTM_GETROUTE: return matched fib result when requested

2017-05-25 Thread Roopa Prabhu
From: Roopa Prabhu This patch adds support to return matched fib result when RTM_F_FIB_MATCH flag is specified in RTM_GETROUTE request. This is useful for user-space applications/controllers wanting to query a matching route. Signed-off-by: Roopa Prabhu --- net/ipv4/route.c | 13

[PATCH net-next v2 5/8] net: ipv4: Save trie prefix to fib lookup result

2017-05-25 Thread Roopa Prabhu
From: David Ahern Prefix is needed for returning matching route spec on get route request. Signed-off-by: David Ahern Signed-off-by: Roopa Prabhu --- include/net/ip_fib.h | 1 + net/ipv4/fib_trie.c | 1 + 2 files changed, 2 insertions(+) diff --git a/include/net/ip_fib.h b/include/net

[PATCH net-next v2 4/8] net: ipv4: Convert inet_rtm_getroute to rcu versions of route lookup

2017-05-25 Thread Roopa Prabhu
. In converting from ip_route_output_key to ip_route_output_key_hash_rcu the xfrm_lookup_route in ip_route_output_flow is dropped since flowi4_proto is not set for a route get request. Signed-off-by: David Ahern Signed-off-by: Roopa Prabhu --- net/ipv4/route.c | 21 + 1 file

Re: [PATCH net-next v2 0/8] net: extend RTM_GETROUTE to return fib result

2017-05-26 Thread Roopa Prabhu
On Fri, May 26, 2017 at 11:18 AM, David Miller wrote: > From: Roopa Prabhu > Date: Thu, 25 May 2017 10:42:32 -0700 > >> This series adds a new RTM_F_FIB_MATCH flag to return matched fib result >> with RTM_GETROUTE. This is useful for applications and protocols in >>

Re: vxlan: use after free error

2017-05-28 Thread Roopa Prabhu
vial fix like this > makes sense. > Am I missing something? Thanks for the report and analysis...this looks similar to a rarely reproducible crash we hit on the 4.1 kernel. I am glad you have a KASAN recipe to reproduce. Your analysis aligns with ours. I have ported our patch to latest ne

Re: vxlan: use after free error

2017-05-29 Thread Roopa Prabhu
On Sun, May 28, 2017 at 11:28 PM, Mark Bloch wrote: > Hi Roopa, > > On 29/05/2017 05:50, Roopa Prabhu wrote: >> On Sun, May 28, 2017 at 3:49 AM, Mark Bloch wrote: [snip] >> >> From: Balakrishnan Raman >> >> Date: Sun, 28 May 2017 19:34:25 -0700 >&g

[PATCH net] mpls: fix clearing of dead nh_flags on link up

2017-05-30 Thread Roopa Prabhu
From: Roopa Prabhu recent fixes to use WRITE_ONCE for nh_flags on link up, accidently ended up leaving the deadflags on a nh. This patch fixes the WRITE_ONCE to use freshly evaluated nh_flags. Fixes: 39eb8cd17588 ("net: mpls: rt_nhn_alive and nh_flags should be accessed using READ

Re: [PATCH net-next v2 4/8] net: ipv4: Convert inet_rtm_getroute to rcu versions of route lookup

2017-05-31 Thread Roopa Prabhu
On Wed, May 31, 2017 at 2:48 PM, David Ahern wrote: > On 5/31/17 2:11 PM, John Fastabend wrote: >>> @@ -2721,14 +2724,14 @@ static int inet_rtm_getroute(struct sk_buff >>> *in_skb, struct nlmsghdr *nlh, >>> skb->protocol = htons(ETH_P_IP); >>> skb->dev= dev; >>

[PATCH net-next] ipv4: route: restore skb_dst_set in inet_rtm_getroute

2017-05-31 Thread Roopa Prabhu
From: Roopa Prabhu recent updates to inet_rtm_getroute dropped skb_dst_set in inet_rtm_getroute. This patch restores it because it is needed to release the dst correctly. Fixes: 3765d35ed8b9 ("net: ipv4: Convert inet_rtm_getroute to rcu versions of route lookup") Reported-by: John

Re: [net,V2] vxlan: fix use-after-free on deletion

2017-06-01 Thread Roopa Prabhu
ng it. > > We add vxlan_vs_del_dev() to match vxlan_vs_add_dev() and call > it from vxlan_stop() to match the call from vxlan_open(). > > Fixes: 56ef9c909b40 ("vxlan: Move socket initialization to within rtnl scope") > Acked-by: Jiri Benc > Tested-by: Roi Dayan > Signed-off-by: Mark Bloch Acked-by: Roopa Prabhu

[PATCH iproute2] iproute: extend route get to return matching fib route

2017-06-01 Thread Roopa Prabhu
From: Roopa Prabhu Uses newly introduced RTM_GETROUTE flag RTM_F_FIB_MATCH to return a matching fib route. Introduces 'fibmatch' keyword to ip route get. ipv4: $ip route show default via 192.168.0.2 dev eth0 10.0.14.0/24 nexthop via 172.16.0.3 dev dummy0 weight 1

Re: [PATCH net] net: bridge: fix a null pointer dereference in br_afspec

2017-06-05 Thread Roopa Prabhu
; > Reported-by: Gustavo A. R. Silva > Fixes: efa5356b0d97 ("bridge: per vlan dst_metadata netlink support") > Signed-off-by: Nikolay Aleksandrov Acked-by: Roopa Prabhu Thanks Nikolay.

Re: [PATCH net] net: ipv6: Release route when device is unregistering

2017-06-07 Thread Roopa Prabhu
registering and release the > route for that case. > > Fixes: a1a22c12060e4 ("net: ipv6: Keep nexthop of multipath route on admin > down") > Reported-by: Roopa Prabhu > Signed-off-by: David Ahern > --- Acked-by: Roopa Prabhu This patch fixes the issue for me. Thanks David.

[PATCH net-next] vxlan: dont migrate permanent fdb entries during learn

2017-06-11 Thread Roopa Prabhu
From: Roopa Prabhu This patch fixes vxlan_snoop to not move permanent fdb entries on learn events. This is consistent with the bridge fdb handling of permanent entries. Signed-off-by: Roopa Prabhu --- drivers/net/vxlan.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a

Re: [PATCH net-next] vxlan: dont migrate permanent fdb entries during learn

2017-06-11 Thread Roopa Prabhu
On Sun, Jun 11, 2017 at 4:04 PM, David Miller wrote: > From: Roopa Prabhu > Date: Sun, 11 Jun 2017 15:51:22 -0700 > >> From: Roopa Prabhu >> >> This patch fixes vxlan_snoop to not move permanent fdb entries >> on learn events. This is consistent with the br

[PATCH net-next v2] vxlan: dont migrate permanent fdb entries during learn

2017-06-11 Thread Roopa Prabhu
From: Roopa Prabhu This patch fixes vxlan_snoop to not move permanent fdb entries on learn events. This is consistent with the bridge fdb handling of permanent entries. Fixes: 26a41ae60438 ("vxlan: only migrate dynamic FDB entries") Signed-off-by: Roopa Prabhu --- v2 - added

Re: [PATCH net] net: bridge: fix returning of vlan range op errors

2017-10-19 Thread Roopa Prabhu
er vlan dst_metadata netlink support") > Signed-off-by: Nikolay Aleksandrov > --- Acked-by: Roopa Prabhu thanks

[PATCH iproute2] bridge: fdb: print NDA_SRC_VNI if available

2017-10-26 Thread Roopa Prabhu
From: Roopa Prabhu Signed-off-by: Roopa Prabhu --- bridge/fdb.c | 9 + 1 file changed, 9 insertions(+) diff --git a/bridge/fdb.c b/bridge/fdb.c index e5cebf9..081233a 100644 --- a/bridge/fdb.c +++ b/bridge/fdb.c @@ -204,6 +204,15 @@ int print_fdb(const struct sockaddr_nl *who, struct

[PATCH iproute2 net-next 0/2] bridge: support for vlan to tunnel mapping

2017-10-26 Thread Roopa Prabhu
From: Roopa Prabhu This series adds the required options to iplink and bridge commands to enable per vlan tunnel info on a bridge port provided by netlink attributes IFLA_BRPORT_VLAN_TUNNEL (flag) and IFLA_BRIDGE_VLAN_TUNNEL_INFO enable vlan-tunnel mapping on a bridge port: $ip link set dev

[PATCH iproute2 net-next 2/2] bridge: vlan: support for per vlan tunnel info

2017-10-26 Thread Roopa Prabhu
From: Roopa Prabhu This patch uses kernel bridge vlan attribute IFLA_BRIDGE_VLAN_TUNNEL_INFO to set/delete/show per vlan tunnel info. $bridge vlan add dev vxlan0 vid 2000 tunnel_info id 2000 $bridge vlan add dev vxlan0 vid 1000-1001 tunnel_info id 2000-2001 $bridge vlan tunnelshow portvlan

[PATCH iproute2 net-next 1/2] iplink: bridge: support bridge port vlan_tunnel attribute

2017-10-26 Thread Roopa Prabhu
From: Roopa Prabhu This config maps to IFLA_BRPORT_VLAN_TUNNEL bridge port netlink flag attribute. This flag enables vlan to tunnel mapping on a bridge port. It is off by default. set vlan_tunnel attribute on bridge port vxlan0: $ip link set dev vxlan0 type bridge_slave vlan_tunnel on $ip link

[PATCH iproute2 net-next v2 0/2] bridge: support for vlan to tunnel mapping

2017-10-27 Thread Roopa Prabhu
From: Roopa Prabhu This series adds the required options to iplink and bridge commands to enable per vlan tunnel info on a bridge port provided by netlink attributes IFLA_BRPORT_VLAN_TUNNEL (flag) and IFLA_BRIDGE_VLAN_TUNNEL_INFO enable vlan-tunnel mapping on a bridge port: $ip link set dev

[PATCH iproute2 net-next v2 1/2] iplink: bridge: support bridge port vlan_tunnel attribute

2017-10-27 Thread Roopa Prabhu
From: Roopa Prabhu This config maps to IFLA_BRPORT_VLAN_TUNNEL bridge port netlink flag attribute. This flag enables vlan to tunnel mapping on a bridge port. It is off by default. set vlan_tunnel attribute on bridge port vxlan0: $ip link set dev vxlan0 type bridge_slave vlan_tunnel on $ip link

[PATCH iproute2 net-next v2 2/2] bridge: vlan: support for per vlan tunnel info

2017-10-27 Thread Roopa Prabhu
From: Roopa Prabhu This patch uses kernel bridge vlan attribute IFLA_BRIDGE_VLAN_TUNNEL_INFO to set/delete/show per vlan tunnel info. $bridge vlan add dev vxlan0 vid 2000 tunnel_info id 2000 $bridge vlan add dev vxlan0 vid 1000-1001 tunnel_info id 2000-2001 $bridge vlan tunnelshow portvlan

Re: [PATCH net-next] net: bridge: add neigh_suppress to bridge port policies

2017-10-30 Thread Roopa Prabhu
On Mon, Oct 30, 2017 at 3:56 AM, Nikolay Aleksandrov wrote: > Add an entry for IFLA_BRPORT_NEIGH_SUPPRESS to bridge port policies. > > Fixes: 821f1b21cabb ("bridge: add new BR_NEIGH_SUPPRESS port flag to suppress > arp and nd flood") > Signed-off-by: Nikolay Aleksandrov

<    1   2   3   4   5   6   7   8   9   >