[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <CAJieiUgkO5x71KDAVvcF1nhA7Du9RnHqxTtTSOOh9JMGehOuyg@mail.gmail.com>
Date: Thu, 23 Apr 2020 12:46:10 -0700
From: Roopa Prabhu <roopa@...ulusnetworks.com>
To: David Ahern <dsahern@...il.com>
Cc: David Miller <davem@...emloft.net>,
netdev <netdev@...r.kernel.org>,
Nikolay Aleksandrov <nikolay@...ulusnetworks.com>,
Benjamin Poirier <bpoirier@...ulusnetworks.com>
Subject: Re: [PATCH net-next 0/2] nexthop: sysctl to skip route notifications
on nexthop changes
On Thu, Apr 23, 2020 at 8:10 AM David Ahern <dsahern@...il.com> wrote:
>
> On 4/23/20 8:48 AM, Roopa Prabhu wrote:
> >
> >
> > On Wed, Apr 22, 2020, 10:21 PM Roopa Prabhu <roopa@...ulusnetworks.com
> > <mailto:roopa@...ulusnetworks.com>> wrote:
> >
> > From: Roopa Prabhu <roopa@...ulusnetworks.com
> > <mailto:roopa@...ulusnetworks.com>>
> >
> > Route notifications on nexthop changes exists for backward
> > compatibility. In systems which have moved to the new
> > nexthop API, these route update notifications cancel
> > the performance benefits provided by the new nexthop API.
> > This patch adds a sysctl to disable these route notifications
> >
> > We have discussed this before. Maybe its time ?
> >
> > Roopa Prabhu (2):
> > ipv4: add sysctl to skip route notify on nexthop changes
> > ipv6: add sysctl to skip route notify on nexthop changes
> >
> >
> >
> > Will update the series with some self test results with sysctl on later
> > today. Expect v2
> >
>
> and I have some changes for you. :-)
>
> There's a bit more to a "all of the processes know about nexthops; give
> me best performance" switch. I will send you a starter patch - the one I
> used for the LPC 2019 talk.
okay, thanks. :) .
Powered by blists - more mailing lists