OpenVZ Forum


Home » Mailing lists » Devel » [PATCH 2.6.25] IPv4: thresholds in fib_trie.c are not modified, make them const
[PATCH 2.6.25] IPv4: thresholds in fib_trie.c are not modified, make them const [message #24937] Wed, 12 December 2007 11:05 Go to next message
den is currently offline  den
Messages: 494
Registered: December 2005
Senior Member
IPv4: thresholds in fib_trie.c are not modified, make them const

Signed-off-by: Denis V. Lunev <den@openvz.org>
diff --git a/net/ipv4/fib_trie.c b/net/ipv4/fib_trie.c
index 45a6e4d..57a2c46 100644
--- a/net/ipv4/fib_trie.c
+++ b/net/ipv4/fib_trie.c
@@ -299,10 +299,10 @@ static inline void check_tnode(const struct tnode *tn)
 	WARN_ON(tn && tn->pos+tn->bits > 32);
 }
 
-static int halve_threshold = 25;
-static int inflate_threshold = 50;
-static int halve_threshold_root = 15;
-static int inflate_threshold_root = 25;
+static const int halve_threshold = 25;
+static const int inflate_threshold = 50;
+static const int halve_threshold_root = 15;
+static const int inflate_threshold_root = 25;
 
 static void __alias_free_mem(struct rcu_head *head)
 {
Re: [PATCH 2.6.25] IPv4: thresholds in fib_trie.c are not modified, make them const [message #24987 is a reply to message #24937] Wed, 12 December 2007 18:58 Go to previous messageGo to next message
davem is currently offline  davem
Messages: 463
Registered: February 2006
Senior Member
From: "Denis V. Lunev" <den@openvz.org>
Date: Wed, 12 Dec 2007 14:05:55 +0300

> -static int halve_threshold = 25;
> -static int inflate_threshold = 50;
> -static int halve_threshold_root = 15;
> -static int inflate_threshold_root = 25;

halve_threshold_root and inflate_threshold_root have different
values in the net-2.6.25 tree.

What tree are you patching against?  These were changed way
back in:

commit 965ffea43d4ebe8cd7b9fee78d651268dd7d23c5
Author: Robert Olsson <robert.olsson@its.uu.se>
Date:   Mon Mar 19 16:29:58 2007 -0700

    [IPV4]: fib_trie root node settings
    
    The threshold for root node can be more aggressive set to get
    better tree compression. The new setting mekes the root grow
    from 16 to 19 bits and substansial improvemnt in Aver depth
    this with the current table of 214393 prefixes
    
    But really the dynamic resize should need more investigation
    both in terms convergence and performance and maybe it should
    be possible to change...
    
    Maybe just for the brave to start with or we may have to back
    this out.

Please don't waste my time like this.  Always submit patches
against the current tree, even if you "think" the files
you are patching haven't changed.
Re: [PATCH 2.6.25] IPv4: thresholds in fib_trie.c are not modified, make them const [message #25027 is a reply to message #24987] Thu, 13 December 2007 07:32 Go to previous message
den is currently offline  den
Messages: 494
Registered: December 2005
Senior Member
David Miller wrote:
> From: "Denis V. Lunev" <den@openvz.org>
> Date: Wed, 12 Dec 2007 14:05:55 +0300
> 
>> -static int halve_threshold = 25;
>> -static int inflate_threshold = 50;
>> -static int halve_threshold_root = 15;
>> -static int inflate_threshold_root = 25;
> 
> halve_threshold_root and inflate_threshold_root have different
> values in the net-2.6.25 tree.

oops. this was accidentally changed during porting of the code from the
one old tree and committed by mistake. I'll resend correct patch in a minute

> 
> What tree are you patching against?  These were changed way
> back in:
Previous Topic: [RFC] [PATCH -mm] reiser4: replace uid==0 check with capability
Next Topic: [PATCH 2.6.25] [IPV4] Reduce whitespaces in fib_rules.h.
Goto Forum:
  


Current Time: Wed Sep 10 22:30:38 GMT 2025

Total time taken to generate the page: 0.10386 seconds