netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
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.

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