From: Patrick McHardy <kaber@trash.net>
To: Krzysztof Oledzki <olel@ans.pl>
Cc: Jarek Poplawski <jarkao2@gmail.com>, netdev@vger.kernel.org
Subject: Re: [PATCH] Fix routing tables with id > 255 for legacy software
Date: Tue, 03 Jun 2008 15:10:13 +0200 [thread overview]
Message-ID: <484542B5.4070401@trash.net> (raw)
In-Reply-To: <Pine.LNX.4.64.0806031449540.3438@bizon.gios.gov.pl>
Krzysztof Oledzki wrote:
> On Tue, 3 Jun 2008, Patrick McHardy wrote:
>
>> Well, if people already use table 253, I guess they might care.
>
> Not really as if pople use a FRA_TABLE aware application they should not
> notice any difference.
In that case not of course.
>> I'm not convinced this is any better than overflowing.
>
> But if they use FRA_TABLE unaware application than overflowing means
> mismatching all:
> - N*256 table as RT_TABLE_UNSPEC
> - N*256+253 tables as RT_TABLE_DEFAULT
> - N*256+254 tables as RT_TABLE_MAIN
> - N*256+255 tables as RT_TABLE_LOCAL
>
> And as I just find out, when it happens is quite unexpected and can
> really hurt. :(
>
>> And I still don't see any other way to handle this properly.
>
> Exactly. So that's why I came with above solution, similar to AS_TRANSIT
> idea used in BGP to handle 16bit -> 32bit ASN transformation.
I think the proper solution is what I wrote in the changelog
entry: fix userspace applications when using extended table
IDs.
Your patch makes it more predictable, so I'm not completely
opposed, but still its just a workaround.
next prev parent reply other threads:[~2008-06-03 13:10 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-06-02 22:09 [PATCH] Fix routing tables with id > 255 for legacy software Krzysztof Oledzki
2008-06-03 8:08 ` Jarek Poplawski
2008-06-03 10:37 ` Krzysztof Oledzki
2008-06-03 11:33 ` Jarek Poplawski
2008-06-03 11:47 ` Jarek Poplawski
2008-06-03 12:43 ` Patrick McHardy
2008-06-03 13:03 ` Krzysztof Oledzki
2008-06-03 13:10 ` Patrick McHardy [this message]
2008-06-03 15:15 ` Krzysztof Oledzki
2008-06-03 17:49 ` Jarek Poplawski
2008-06-03 17:58 ` Patrick McHardy
2008-06-03 18:29 ` Jarek Poplawski
2008-06-03 18:29 ` Krzysztof Oledzki
2008-06-03 18:42 ` Jarek Poplawski
2008-06-03 18:28 ` Patrick McHardy
2008-06-03 18:39 ` Krzysztof Oledzki
2008-06-03 20:28 ` Patrick McHardy
2008-06-03 23:23 ` Patrick McHardy
2008-06-10 22:45 ` David Miller
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=484542B5.4070401@trash.net \
--to=kaber@trash.net \
--cc=jarkao2@gmail.com \
--cc=netdev@vger.kernel.org \
--cc=olel@ans.pl \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.