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 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).