diff options
author | Paul Jakma <paul.jakma@sun.com> | 2006-10-15 23:33:50 +0000 |
---|---|---|
committer | Paul Jakma <paul.jakma@sun.com> | 2006-10-15 23:33:50 +0000 |
commit | 98954844ae56d142e96341d5dff959ec5518111e (patch) | |
tree | 8f411f4fec908605fb4e5ea423874c6a08d82b54 /lib | |
parent | 965b83f9cc287ca36f5a4152c2ac39007c5bc6d8 (diff) |
[lib] Handle upgrade from SUNWzebra to Quagga for 'interface fooX:Y' commands
2006-10-14 Paul Jakma <paul.jakma@sun.com>
* if.c: (general) Handle upgrades from SUNWzebra, which tried
to track each logical interface as a seperate struct
interface, to Quagga, which assigns only one struct interface
per ifindex.
(if_sunwzebra_get) Try decompose a logical interface name
(fooX:Y) to the 'primary' name (fooX), for Solaris.
(interface_cmd) Use if_sunwzebra_get on Solaris.
Diffstat (limited to 'lib')
-rw-r--r-- | lib/ChangeLog | 10 | ||||
-rw-r--r-- | lib/if.c | 48 |
2 files changed, 56 insertions, 2 deletions
diff --git a/lib/ChangeLog b/lib/ChangeLog index 00ba44e0..f076e2f0 100644 --- a/lib/ChangeLog +++ b/lib/ChangeLog @@ -1,3 +1,13 @@ +2006-10-14 Paul Jakma <paul.jakma@sun.com> + + * if.c: (general) Handle upgrades from SUNWzebra, which tried + to track each logical interface as a seperate struct + interface, to Quagga, which assigns only one struct interface + per ifindex. + (if_sunwzebra_get) Try decompose a logical interface name + (fooX:Y) to the 'primary' name (fooX), for Solaris. + (interface_cmd) Use if_sunwzebra_get on Solaris. + 2006-09-26 Pierre-Yves Ritschard <pierre-yves@spootnik.org> * smux.c: (smux_open,smux_trap,smux_register) Fix various @@ -501,9 +501,49 @@ DEFUN (no_interface_desc, return CMD_SUCCESS; } + +#ifdef SUNOS_5 +/* Need to handle upgrade from SUNWzebra to Quagga. SUNWzebra created + * a seperate struct interface for each logical interface, so config + * file may be full of 'interface fooX:Y'. Solaris however does not + * expose logical interfaces via PF_ROUTE, so trying to track logical + * interfaces can be fruitless, for that reason Quagga only tracks + * the primary IP interface. + * + * We try accomodate SUNWzebra by: + * - looking up the interface name, to see whether it exists, if so + * its useable + * - for protocol daemons, this could only because zebra told us of + * the interface + * - for zebra, only because it learnt from kernel + * - if not: + * - search the name to see if it contains a sub-ipif / logical interface + * seperator, the ':' char. If it does: + * - text up to that char must be the primary name - get that name. + * if not: + * - no idea, just get the name in its entirety. + */ +static struct interface * +if_sunwzebra_get (const char *name, size_t nlen) +{ + struct interface *ifp; + size_t seppos = 0; - -/* See also wrapper function zebra_interface() in zebra/interface.c */ + if ( (ifp = if_lookup_by_name_len(name, nlen)) != NULL) + return ifp; + + /* hunt the primary interface name... */ + while (seppos < nlen && name[seppos] != ':') + seppos++; + + /* Wont catch seperator as last char, e.g. 'foo0:' but thats invalid */ + if (seppos < nlen) + return if_get_by_name_len (name, seppos); + else + return if_get_by_name_len (name, nlen); +} +#endif /* SUNOS_5 */ + DEFUN (interface, interface_cmd, "interface IFNAME", @@ -521,7 +561,11 @@ DEFUN (interface, return CMD_WARNING; } +#ifdef SUNOS_5 + ifp = if_sunwzebra_get (argv[0], sl); +#else ifp = if_get_by_name_len(argv[0], sl); +#endif /* SUNOS_5 */ vty->index = ifp; vty->node = INTERFACE_NODE; |