Re: [PATCH] Prefix List against 2.5.70 (re-done)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




1. is it okay to have another hook for garbbig prefix list?
   Userspace application can get such information via
   - routing table
   - interface flag

2. is the "managed" flags etc, which is per interface variable, really NEWROUTE information?
It is NOT L2 thing, but it is per-link information.
I think it is NEWLINK thing.


What I'm thinking is:

 - fix "ADDRCONF" flag in route information
 - manage / other flags via NEWLINK message
(- No new interface to get prefix itself.)


Well, there are two reason that I can see to not do so (ADDRCONF flag is already
fixed in earlier patch) :

-  With the latest submission, the actual code to get the prefix list itself
   is very small, the top level inet6_dump_fib uses either the dump_node or
   the dump_prefix, the latter being the new function added. This is the whole
   user interface, 50 odd lines of code with comments.
-  If I understood your point about using interface flag and routing table,
   you are suggesting that the user can make look at rttable and get the prefix
   entries by make checks (it is non-trivial, eg the address should not LL or MC,
   there should be no nexthop and it should be added via an RA, etc). However,
   having a user interface makes it easier to get the prefix list without
   significant bloat to the kernel, and the user doesn't have to make a lot of
   checks to get the system prefixes. I don't see much gain from this approach.

About your point about the managed flag, I think it is a per interface flag
that gets returned when a request for getting flags on that interface is made.
That's why I have made it per interface as part of a GETLNKFLAGS operation.
I don't understand why you think it is NEWLINK thing (not sure what you mean by
that), since it is a flag information on your existing device that a RA is
advertising. I want to get this information not on receipt of an RA, but when
a request is made.

Thanks,

- KK

-
: send the line "unsubscribe linux-net" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Netdev]     [Ethernet Bridging]     [Linux 802.1Q VLAN]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Git]     [Bugtraq]     [Yosemite News and Information]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux PCI]     [Linux Admin]     [Samba]

  Powered by Linux