[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <18088.30757.76332.243475@robur.slu.se>
Date: Thu, 26 Jul 2007 12:32:05 +0200
From: Robert Olsson <Robert.Olsson@...a.slu.se>
To: David Miller <davem@...emloft.net>
Cc: shemminger@...ux-foundation.org, bunk@...sta.de,
Robert.Olsson@...a.slu.se, akpm@...ux-foundation.org,
paulmck@...ux.vnet.ibm.com, mingo@...e.hu, josh@...nel.org,
netdev@...r.kernel.org
Subject: Re: [RFT] fib_trie: cleanup
David Miller writes:
> From: Stephen Hemminger <shemminger@...ux-foundation.org>
> Date: Thu, 26 Jul 2007 09:46:48 +0100
>
> > Try this out:
> > * replace macro's with inlines
> > * get rid of places doing multiple evaluations of NODE_PARENT
>
> No objections from me.
>
> Robert?
Fine it looks cleaner and compiles... thanks Stephen.
BTW
I think might be possible improve functional/performance parts too. It's the
list handling I'm thinking of.
I've observed that in most cases there is only one leaf_info in the leaf list.
In 96% of current Internet prefixes in the router I just looked into (below).
So optimizing for this case could be an idea. Some variant were the leaf holds
the leaf_info data direct could be worth testing.
Cheers.
--ro
Main:
Aver depth: 2.57
Max depth: 6
Leaves: 215131
Internal nodes: 52394
1: 27662 2: 9982 3: 8510 4: 3625 5: 1684 6: 626 7: 240 8: 64 16: 1
Pointers: 427924
Null ptrs: 160400
Total size: 7102 kB
ip route list | wc -l
224649
215131/224649 = 96%
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists