IB: Use DEFINE_SPINLOCK() for static spinlocks
Rather than just defining static spinlock_t variables and then initializing them later in init functions, simply define them with DEFINE_SPINLOCK() and remove the calls to spin_lock_init(). This cleans up the source a tad and also shrinks the compiled code; eg on x86-64: add/remove: 0/0 grow/shrink: 0/3 up/down: 0/-40 (-40) function old new delta ib_uverbs_init 336 326 -10 ib_mad_init_module 147 137 -10 ib_sa_init 123 103 -20 Signed-off-by: Roland Dreier <rolandd@cisco.com>
This commit is contained in:
parent
e07cccf404
commit
6276e08a9b
@ -51,8 +51,7 @@ static struct list_head ib_mad_port_list;
|
|||||||
static u32 ib_mad_client_id = 0;
|
static u32 ib_mad_client_id = 0;
|
||||||
|
|
||||||
/* Port list lock */
|
/* Port list lock */
|
||||||
static spinlock_t ib_mad_port_list_lock;
|
static DEFINE_SPINLOCK(ib_mad_port_list_lock);
|
||||||
|
|
||||||
|
|
||||||
/* Forward declarations */
|
/* Forward declarations */
|
||||||
static int method_in_use(struct ib_mad_mgmt_method_table **method,
|
static int method_in_use(struct ib_mad_mgmt_method_table **method,
|
||||||
@ -2984,8 +2983,6 @@ static int __init ib_mad_init_module(void)
|
|||||||
{
|
{
|
||||||
int ret;
|
int ret;
|
||||||
|
|
||||||
spin_lock_init(&ib_mad_port_list_lock);
|
|
||||||
|
|
||||||
ib_mad_cache = kmem_cache_create("ib_mad",
|
ib_mad_cache = kmem_cache_create("ib_mad",
|
||||||
sizeof(struct ib_mad_private),
|
sizeof(struct ib_mad_private),
|
||||||
0,
|
0,
|
||||||
@ -3021,4 +3018,3 @@ static void __exit ib_mad_cleanup_module(void)
|
|||||||
|
|
||||||
module_init(ib_mad_init_module);
|
module_init(ib_mad_init_module);
|
||||||
module_exit(ib_mad_cleanup_module);
|
module_exit(ib_mad_cleanup_module);
|
||||||
|
|
||||||
|
@ -109,10 +109,10 @@ static struct ib_client sa_client = {
|
|||||||
.remove = ib_sa_remove_one
|
.remove = ib_sa_remove_one
|
||||||
};
|
};
|
||||||
|
|
||||||
static spinlock_t idr_lock;
|
static DEFINE_SPINLOCK(idr_lock);
|
||||||
static DEFINE_IDR(query_idr);
|
static DEFINE_IDR(query_idr);
|
||||||
|
|
||||||
static spinlock_t tid_lock;
|
static DEFINE_SPINLOCK(tid_lock);
|
||||||
static u32 tid;
|
static u32 tid;
|
||||||
|
|
||||||
#define PATH_REC_FIELD(field) \
|
#define PATH_REC_FIELD(field) \
|
||||||
@ -1077,9 +1077,6 @@ static int __init ib_sa_init(void)
|
|||||||
{
|
{
|
||||||
int ret;
|
int ret;
|
||||||
|
|
||||||
spin_lock_init(&idr_lock);
|
|
||||||
spin_lock_init(&tid_lock);
|
|
||||||
|
|
||||||
get_random_bytes(&tid, sizeof tid);
|
get_random_bytes(&tid, sizeof tid);
|
||||||
|
|
||||||
ret = ib_register_client(&sa_client);
|
ret = ib_register_client(&sa_client);
|
||||||
|
@ -73,7 +73,7 @@ DEFINE_IDR(ib_uverbs_cq_idr);
|
|||||||
DEFINE_IDR(ib_uverbs_qp_idr);
|
DEFINE_IDR(ib_uverbs_qp_idr);
|
||||||
DEFINE_IDR(ib_uverbs_srq_idr);
|
DEFINE_IDR(ib_uverbs_srq_idr);
|
||||||
|
|
||||||
static spinlock_t map_lock;
|
static DEFINE_SPINLOCK(map_lock);
|
||||||
static struct ib_uverbs_device *dev_table[IB_UVERBS_MAX_DEVICES];
|
static struct ib_uverbs_device *dev_table[IB_UVERBS_MAX_DEVICES];
|
||||||
static DECLARE_BITMAP(dev_map, IB_UVERBS_MAX_DEVICES);
|
static DECLARE_BITMAP(dev_map, IB_UVERBS_MAX_DEVICES);
|
||||||
|
|
||||||
@ -836,8 +836,6 @@ static int __init ib_uverbs_init(void)
|
|||||||
{
|
{
|
||||||
int ret;
|
int ret;
|
||||||
|
|
||||||
spin_lock_init(&map_lock);
|
|
||||||
|
|
||||||
ret = register_chrdev_region(IB_UVERBS_BASE_DEV, IB_UVERBS_MAX_DEVICES,
|
ret = register_chrdev_region(IB_UVERBS_BASE_DEV, IB_UVERBS_MAX_DEVICES,
|
||||||
"infiniband_verbs");
|
"infiniband_verbs");
|
||||||
if (ret) {
|
if (ret) {
|
||||||
|
Loading…
Reference in New Issue
Block a user