* RTM_DELROUTE on interface going down - when to send?
@ 2013-12-17 11:05 Michal Kubecek
2013-12-17 12:11 ` Kristian Evensen
0 siblings, 1 reply; 4+ messages in thread
From: Michal Kubecek @ 2013-12-17 11:05 UTC (permalink / raw)
To: netdev
Hello,
one of our customers noticed that when an interface is switched down, no
RTM_DELROUTE message is sent for IPv4 routes being deleted (but there
are RTM_NEWROUTE messages when the interface is switched up). This is
because rather than deleting the routes with fib_table_delete(), they
are flagged with RTNH_F_DEAD in fib_sync_down_dev() and then deleted in
trie_flush_list().
Before I start with the patch, I would like to ask: would it be more
appropriate to send the message when the route is marked as dead or when
it is actually deleted from the trie?
Michal Kubecek
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: RTM_DELROUTE on interface going down - when to send?
2013-12-17 11:05 RTM_DELROUTE on interface going down - when to send? Michal Kubecek
@ 2013-12-17 12:11 ` Kristian Evensen
2013-12-17 14:01 ` Michal Kubecek
0 siblings, 1 reply; 4+ messages in thread
From: Kristian Evensen @ 2013-12-17 12:11 UTC (permalink / raw)
To: Michal Kubecek; +Cc: Network Development
Hi Michal,
On Tue, Dec 17, 2013 at 12:05 PM, Michal Kubecek <mkubecek@suse.cz> wrote:
>
> Before I start with the patch, I would like to ask: would it be more
> appropriate to send the message when the route is marked as dead or when
> it is actually deleted from the trie?
I noticed the same thing a few months ago and submitted a patch
(http://www.spinics.net/lists/netdev/msg254106.html). However, it was
not applied because it would cause too much traffic on large-scale
routers (see davem's reply here
http://www.spinics.net/lists/netdev/msg254186.html).
-Kristian
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: RTM_DELROUTE on interface going down - when to send?
2013-12-17 12:11 ` Kristian Evensen
@ 2013-12-17 14:01 ` Michal Kubecek
2013-12-18 9:16 ` Nicolas Dichtel
0 siblings, 1 reply; 4+ messages in thread
From: Michal Kubecek @ 2013-12-17 14:01 UTC (permalink / raw)
To: Kristian Evensen; +Cc: Network Development
On Tue, Dec 17, 2013 at 01:11:32PM +0100, Kristian Evensen wrote:
> On Tue, Dec 17, 2013 at 12:05 PM, Michal Kubecek <mkubecek@suse.cz> wrote:
> >
> > Before I start with the patch, I would like to ask: would it be more
> > appropriate to send the message when the route is marked as dead or when
> > it is actually deleted from the trie?
>
> I noticed the same thing a few months ago and submitted a patch
> (http://www.spinics.net/lists/netdev/msg254106.html). However, it was
> not applied because it would cause too much traffic on large-scale
> routers (see davem's reply here
> http://www.spinics.net/lists/netdev/msg254186.html).
Thank you for the pointer. However, I agree that the inconsistencies
(IPv4 vs. IPv6, RTM_DELROUTE vs. RTM_NEWROUTE) are really unfortunate.
Michal Kubecek
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: RTM_DELROUTE on interface going down - when to send?
2013-12-17 14:01 ` Michal Kubecek
@ 2013-12-18 9:16 ` Nicolas Dichtel
0 siblings, 0 replies; 4+ messages in thread
From: Nicolas Dichtel @ 2013-12-18 9:16 UTC (permalink / raw)
To: Michal Kubecek, Kristian Evensen; +Cc: Network Development
Le 17/12/2013 15:01, Michal Kubecek a écrit :
> On Tue, Dec 17, 2013 at 01:11:32PM +0100, Kristian Evensen wrote:
>> On Tue, Dec 17, 2013 at 12:05 PM, Michal Kubecek <mkubecek@suse.cz> wrote:
>>>
>>> Before I start with the patch, I would like to ask: would it be more
>>> appropriate to send the message when the route is marked as dead or when
>>> it is actually deleted from the trie?
>>
>> I noticed the same thing a few months ago and submitted a patch
>> (http://www.spinics.net/lists/netdev/msg254106.html). However, it was
>> not applied because it would cause too much traffic on large-scale
>> routers (see davem's reply here
>> http://www.spinics.net/lists/netdev/msg254186.html).
>
> Thank you for the pointer. However, I agree that the inconsistencies
> (IPv4 vs. IPv6, RTM_DELROUTE vs. RTM_NEWROUTE) are really unfortunate.
Sure. Note that this fix is regularly proposed, another pointer:
http://patchwork.ozlabs.org/patch/195516/
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2013-12-18 9:17 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-12-17 11:05 RTM_DELROUTE on interface going down - when to send? Michal Kubecek
2013-12-17 12:11 ` Kristian Evensen
2013-12-17 14:01 ` Michal Kubecek
2013-12-18 9:16 ` Nicolas Dichtel
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).