netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: David Miller <davem@davemloft.net>
To: jiri@resnulli.us
Cc: roopa@cumulusnetworks.com, eric.dumazet@gmail.com,
	idosch@idosch.org, netdev@vger.kernel.org, jiri@mellanox.com,
	mlxsw@mellanox.com, dsa@cumulusnetworks.com,
	nikolay@cumulusnetworks.com, andy@greyhouse.net,
	vivien.didelot@savoirfairelinux.com, andrew@lunn.ch,
	f.fainelli@gmail.com, alexander.h.duyck@intel.com,
	kuznet@ms2.inr.ac.ru, jmorris@namei.org, yoshfuji@linux-ipv6.org,
	kaber@trash.net, idosch@mellanox.com
Subject: Re: [PATCH net-next v2] ipv4: fib: Replay events when registering FIB notifier
Date: Wed, 02 Nov 2016 11:26:18 -0400 (EDT)	[thread overview]
Message-ID: <20161102.112618.743617399868353432.davem@davemloft.net> (raw)
In-Reply-To: <20161102073502.GB1713@nanopsycho.orion>

From: Jiri Pirko <jiri@resnulli.us>
Date: Wed, 2 Nov 2016 08:35:02 +0100

> How do you imagine this mode should looks like? Could you draw me some
> example?

Well, first of all, there is no reason we can't provide a mechanism by
which the driver can request and obtain a FIB dump.

And it can be designed in a way to be preemptible or at least not
require RTNL to be held during the entire operation.  Sequence
counters or similar can be used to make sure that if the table changes
mid-dump due to RTNL being dropped, the dump can be rewound and
restarted.

  reply	other threads:[~2016-11-02 15:26 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-10-31 21:13 [PATCH net-next v2] ipv4: fib: Replay events when registering FIB notifier idosch
2016-10-31 21:24 ` Eric Dumazet
2016-10-31 22:57   ` Ido Schimmel
2016-11-01 14:19     ` Eric Dumazet
2016-11-01 15:14       ` Roopa Prabhu
2016-11-01 15:36         ` David Miller
2016-11-02  7:35           ` Jiri Pirko
2016-11-02 15:26             ` David Miller [this message]
2016-11-01 17:03         ` Ido Schimmel
2016-11-02  2:13           ` Roopa Prabhu
2016-11-02  7:20             ` Jiri Pirko
2016-11-02 13:29               ` Roopa Prabhu
2016-11-02 13:44                 ` Ido Schimmel
2016-11-02 13:48                 ` Jiri Pirko
2016-11-02 14:35                   ` Roopa Prabhu
2016-11-02 14:43                     ` Jiri Pirko
2016-11-01 15:44       ` Ido Schimmel

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20161102.112618.743617399868353432.davem@davemloft.net \
    --to=davem@davemloft.net \
    --cc=alexander.h.duyck@intel.com \
    --cc=andrew@lunn.ch \
    --cc=andy@greyhouse.net \
    --cc=dsa@cumulusnetworks.com \
    --cc=eric.dumazet@gmail.com \
    --cc=f.fainelli@gmail.com \
    --cc=idosch@idosch.org \
    --cc=idosch@mellanox.com \
    --cc=jiri@mellanox.com \
    --cc=jiri@resnulli.us \
    --cc=jmorris@namei.org \
    --cc=kaber@trash.net \
    --cc=kuznet@ms2.inr.ac.ru \
    --cc=mlxsw@mellanox.com \
    --cc=netdev@vger.kernel.org \
    --cc=nikolay@cumulusnetworks.com \
    --cc=roopa@cumulusnetworks.com \
    --cc=vivien.didelot@savoirfairelinux.com \
    --cc=yoshfuji@linux-ipv6.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).