diff options
author | Allan Stephens <allan.stephens@windriver.com> | 2008-04-16 21:21:16 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2008-04-16 21:21:16 -0400 |
commit | 4784b7c348779e957c82ba638ba2ada5ad8502d8 (patch) | |
tree | 635e99ff046b7f9519c515ee7e6edc844dced058 /net/tipc/ref.h | |
parent | dd9e0dda66ba38a2ddd1405ac279894260dc5c36 (diff) |
[TIPC]: Remove inlining of reference table locking routines
This patch converts the TIPC reference table locking routines
into non-inlined routines, since they are mainly called from
non-performance critical areas of TIPC and the added code
footprint incurred through inlining can no longer be justified.
Signed-off-by: Allan Stephens <allan.stephens@windriver.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/tipc/ref.h')
-rw-r--r-- | net/tipc/ref.h | 89 |
1 files changed, 4 insertions, 85 deletions
diff --git a/net/tipc/ref.h b/net/tipc/ref.h index 38f3a7f4a78d..7e3798ea93b9 100644 --- a/net/tipc/ref.h +++ b/net/tipc/ref.h | |||
@@ -2,7 +2,7 @@ | |||
2 | * net/tipc/ref.h: Include file for TIPC object registry code | 2 | * net/tipc/ref.h: Include file for TIPC object registry code |
3 | * | 3 | * |
4 | * Copyright (c) 1991-2006, Ericsson AB | 4 | * Copyright (c) 1991-2006, Ericsson AB |
5 | * Copyright (c) 2005, Wind River Systems | 5 | * Copyright (c) 2005-2006, Wind River Systems |
6 | * All rights reserved. | 6 | * All rights reserved. |
7 | * | 7 | * |
8 | * Redistribution and use in source and binary forms, with or without | 8 | * Redistribution and use in source and binary forms, with or without |
@@ -37,95 +37,14 @@ | |||
37 | #ifndef _TIPC_REF_H | 37 | #ifndef _TIPC_REF_H |
38 | #define _TIPC_REF_H | 38 | #define _TIPC_REF_H |
39 | 39 | ||
40 | /** | ||
41 | * struct reference - TIPC object reference entry | ||
42 | * @object: pointer to object associated with reference entry | ||
43 | * @lock: spinlock controlling access to object | ||
44 | * @data: reference value associated with object (or link to next unused entry) | ||
45 | */ | ||
46 | |||
47 | struct reference { | ||
48 | void *object; | ||
49 | spinlock_t lock; | ||
50 | union { | ||
51 | u32 next_plus_upper; | ||
52 | u32 reference; | ||
53 | } data; | ||
54 | }; | ||
55 | |||
56 | /** | ||
57 | * struct tipc_ref_table - table of TIPC object reference entries | ||
58 | * @entries: pointer to array of reference entries | ||
59 | * @index_mask: bitmask for array index portion of reference values | ||
60 | * @first_free: array index of first unused object reference entry | ||
61 | * @last_free: array index of last unused object reference entry | ||
62 | */ | ||
63 | |||
64 | struct ref_table { | ||
65 | struct reference *entries; | ||
66 | u32 index_mask; | ||
67 | u32 first_free; | ||
68 | u32 last_free; | ||
69 | }; | ||
70 | |||
71 | extern struct ref_table tipc_ref_table; | ||
72 | |||
73 | int tipc_ref_table_init(u32 requested_size, u32 start); | 40 | int tipc_ref_table_init(u32 requested_size, u32 start); |
74 | void tipc_ref_table_stop(void); | 41 | void tipc_ref_table_stop(void); |
75 | 42 | ||
76 | u32 tipc_ref_acquire(void *object, spinlock_t **lock); | 43 | u32 tipc_ref_acquire(void *object, spinlock_t **lock); |
77 | void tipc_ref_discard(u32 ref); | 44 | void tipc_ref_discard(u32 ref); |
78 | 45 | ||
79 | 46 | void *tipc_ref_lock(u32 ref); | |
80 | /** | 47 | void tipc_ref_unlock(u32 ref); |
81 | * tipc_ref_lock - lock referenced object and return pointer to it | 48 | void *tipc_ref_deref(u32 ref); |
82 | */ | ||
83 | |||
84 | static inline void *tipc_ref_lock(u32 ref) | ||
85 | { | ||
86 | if (likely(tipc_ref_table.entries)) { | ||
87 | struct reference *r = | ||
88 | &tipc_ref_table.entries[ref & tipc_ref_table.index_mask]; | ||
89 | |||
90 | spin_lock_bh(&r->lock); | ||
91 | if (likely(r->data.reference == ref)) | ||
92 | return r->object; | ||
93 | spin_unlock_bh(&r->lock); | ||
94 | } | ||
95 | return NULL; | ||
96 | } | ||
97 | |||
98 | /** | ||
99 | * tipc_ref_unlock - unlock referenced object | ||
100 | */ | ||
101 | |||
102 | static inline void tipc_ref_unlock(u32 ref) | ||
103 | { | ||
104 | if (likely(tipc_ref_table.entries)) { | ||
105 | struct reference *r = | ||
106 | &tipc_ref_table.entries[ref & tipc_ref_table.index_mask]; | ||
107 | |||
108 | if (likely(r->data.reference == ref)) | ||
109 | spin_unlock_bh(&r->lock); | ||
110 | else | ||
111 | err("tipc_ref_unlock() invoked using obsolete reference\n"); | ||
112 | } | ||
113 | } | ||
114 | |||
115 | /** | ||
116 | * tipc_ref_deref - return pointer referenced object (without locking it) | ||
117 | */ | ||
118 | |||
119 | static inline void *tipc_ref_deref(u32 ref) | ||
120 | { | ||
121 | if (likely(tipc_ref_table.entries)) { | ||
122 | struct reference *r = | ||
123 | &tipc_ref_table.entries[ref & tipc_ref_table.index_mask]; | ||
124 | |||
125 | if (likely(r->data.reference == ref)) | ||
126 | return r->object; | ||
127 | } | ||
128 | return NULL; | ||
129 | } | ||
130 | 49 | ||
131 | #endif | 50 | #endif |