Skip to content

Commit

Permalink
Add a potential future problem I can see with metric calculation and …
Browse files Browse the repository at this point in the history
…interfaces that change often.

Signed-off-by: Robin H. Johnson <[email protected]>
  • Loading branch information
robbat2 committed Mar 2, 2012
1 parent 0fa164d commit 0dc9431
Showing 1 changed file with 14 additions and 0 deletions.
14 changes: 14 additions & 0 deletions TODO
Original file line number Diff line number Diff line change
Expand Up @@ -5,3 +5,17 @@
- oldnet[bridging]: Review setting of bridge configuration on dynamic interface add

- Document rc-depend binary.

- _ifindex is not a reliable means of calculating metrics:
_ifindex is used for calculating metrics for new devices but has a major
problem: Since it's only the nth entry in /proc/net/dev
And devices may be removed from that file, and reordered, you won't always
get the same result.
If you do:
- add eth0 - _ifindex (eth0=0)
- add vlan1 - _ifindex (eth0=0,vlan1=1)
- add vlan2 - _ifindex (eth0=0,vlan1=1,vlan2=2)
- rem vlan1 - _ifindex (eth0=0,vlan2=1)
- add vlan3 - _ifindex (eth0=0,vlan2=1,vlan3=2)
Now your routing table has entries for both vlan2 and vlan3 with a metric of 2.

0 comments on commit 0dc9431

Please sign in to comment.