diff options
| author | Eric Dumazet <dada1@cosmosbay.com> | 2008-01-12 21:27:41 -0800 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2008-01-28 15:01:58 -0800 |
| commit | 112d8cfcbf4f5ef0cf669cb5864f1206972076d6 (patch) | |
| tree | a47c7a31adb9fc95f47688717d4639ef832d8974 /net/ipv4/fib_trie.c | |
| parent | b9aed45507b657abab0b88da2c9b509a9dc462b1 (diff) | |
| download | linux-stable-112d8cfcbf4f5ef0cf669cb5864f1206972076d6.tar.gz linux-stable-112d8cfcbf4f5ef0cf669cb5864f1206972076d6.tar.bz2 linux-stable-112d8cfcbf4f5ef0cf669cb5864f1206972076d6.zip | |
[FIB]: Reduce text size of net/ipv4/fib_trie.o
In struct tnode, we use two fields of 5 bits for 'pos' and 'bits'.
Switching to plain 'unsigned char' (8 bits) take the same space
because of compiler alignments, and reduce text size by 435 bytes
on i386.
On i386 :
$ size net/ipv4/fib_trie.o.before_patch net/ipv4/fib_trie.o
text data bss dec hex filename
13714 4 64 13782 35d6 net/ipv4/fib_trie.o.before
13279 4 64 13347 3423 net/ipv4/fib_trie.o
Signed-off-by: Eric Dumazet <dada1@cosmosbay.com>
Acked-by: Stephen Hemminger <stephen.hemminger@vyatta.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ipv4/fib_trie.c')
| -rw-r--r-- | net/ipv4/fib_trie.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/net/ipv4/fib_trie.c b/net/ipv4/fib_trie.c index 3e94a4dddb0c..350904c233ee 100644 --- a/net/ipv4/fib_trie.c +++ b/net/ipv4/fib_trie.c @@ -118,8 +118,8 @@ struct leaf_info { struct tnode { t_key key; unsigned long parent; - unsigned short pos:5; /* 2log(KEYLENGTH) bits needed */ - unsigned short bits:5; /* 2log(KEYLENGTH) bits needed */ + unsigned char pos; /* 2log(KEYLENGTH) bits needed */ + unsigned char bits; /* 2log(KEYLENGTH) bits needed */ unsigned short full_children; /* KEYLENGTH bits needed */ unsigned short empty_children; /* KEYLENGTH bits needed */ struct rcu_head rcu; |