Skip to content

Commit 42bb59e

Browse files
committed
compiler-capability-analysis: Rename __cond_lock() to __cond_acquire()
Just like the pairing of attribute __acquires() with a matching function-like macro __acquire(), the attribute __cond_acquires() should have a matching function-like macro __cond_acquire(). To be consistent, rename __cond_lock() to __cond_acquire(). Signed-off-by: Marco Elver <[email protected]>
1 parent 4d2076f commit 42bb59e

File tree

11 files changed

+24
-24
lines changed

11 files changed

+24
-24
lines changed

drivers/net/wireless/intel/iwlwifi/iwl-trans.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1136,7 +1136,7 @@ void iwl_trans_set_bits_mask(struct iwl_trans *trans, u32 reg,
11361136
bool _iwl_trans_grab_nic_access(struct iwl_trans *trans);
11371137

11381138
#define iwl_trans_grab_nic_access(trans) \
1139-
__cond_lock(nic_access, \
1139+
__cond_acquire(nic_access, \
11401140
likely(_iwl_trans_grab_nic_access(trans)))
11411141

11421142
void __releases(nic_access)

drivers/net/wireless/intel/iwlwifi/pcie/internal.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -560,7 +560,7 @@ void iwl_trans_pcie_free_pnvm_dram_regions(struct iwl_dram_regions *dram_regions
560560

561561
bool __iwl_trans_pcie_grab_nic_access(struct iwl_trans *trans);
562562
#define _iwl_trans_pcie_grab_nic_access(trans) \
563-
__cond_lock(nic_access_nobh, \
563+
__cond_acquire(nic_access_nobh, \
564564
likely(__iwl_trans_pcie_grab_nic_access(trans)))
565565

566566
void iwl_trans_pcie_check_product_reset_status(struct pci_dev *pdev);

include/linux/compiler-capability-analysis.h

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@
1515
# define __releases(x) __attribute__((context(x,1,0)))
1616
# define __acquire(x) __context__(x,1)
1717
# define __release(x) __context__(x,-1)
18-
# define __cond_lock(x, c) ((c) ? ({ __acquire(x); 1; }) : 0)
18+
# define __cond_acquire(x, c) ((c) ? ({ __acquire(x); 1; }) : 0)
1919

2020
#else /* !__CHECKER__ */
2121

@@ -25,7 +25,7 @@
2525
# define __releases(x)
2626
# define __acquire(x) (void)0
2727
# define __release(x) (void)0
28-
# define __cond_lock(x, c) (c)
28+
# define __cond_acquire(x, c) (c)
2929

3030
#endif /* __CHECKER__ */
3131

include/linux/mm.h

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2738,7 +2738,7 @@ static inline pte_t *get_locked_pte(struct mm_struct *mm, unsigned long addr,
27382738
spinlock_t **ptl)
27392739
{
27402740
pte_t *ptep;
2741-
__cond_lock(*ptl, ptep = __get_locked_pte(mm, addr, ptl));
2741+
__cond_acquire(*ptl, ptep = __get_locked_pte(mm, addr, ptl));
27422742
return ptep;
27432743
}
27442744

@@ -3029,7 +3029,7 @@ static inline pte_t *__pte_offset_map(pmd_t *pmd, unsigned long addr,
30293029
{
30303030
pte_t *pte;
30313031

3032-
__cond_lock(RCU, pte = ___pte_offset_map(pmd, addr, pmdvalp));
3032+
__cond_acquire(RCU, pte = ___pte_offset_map(pmd, addr, pmdvalp));
30333033
return pte;
30343034
}
30353035
static inline pte_t *pte_offset_map(pmd_t *pmd, unsigned long addr)
@@ -3044,7 +3044,7 @@ static inline pte_t *pte_offset_map_lock(struct mm_struct *mm, pmd_t *pmd,
30443044
{
30453045
pte_t *pte;
30463046

3047-
__cond_lock(RCU, __cond_lock(*ptlp,
3047+
__cond_acquire(RCU, __cond_acquire(*ptlp,
30483048
pte = __pte_offset_map_lock(mm, pmd, addr, ptlp)));
30493049
return pte;
30503050
}

include/linux/rwlock.h

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -49,8 +49,8 @@ do { \
4949
* regardless of whether CONFIG_SMP or CONFIG_PREEMPT are set. The various
5050
* methods are defined as nops in the case they are not required.
5151
*/
52-
#define read_trylock(lock) __cond_lock(lock, _raw_read_trylock(lock))
53-
#define write_trylock(lock) __cond_lock(lock, _raw_write_trylock(lock))
52+
#define read_trylock(lock) __cond_acquire(lock, _raw_read_trylock(lock))
53+
#define write_trylock(lock) __cond_acquire(lock, _raw_write_trylock(lock))
5454

5555
#define write_lock(lock) _raw_write_lock(lock)
5656
#define read_lock(lock) _raw_read_lock(lock)

include/linux/rwlock_rt.h

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,7 @@ static __always_inline void read_lock_irq(rwlock_t *rwlock)
5555
flags = 0; \
5656
} while (0)
5757

58-
#define read_trylock(lock) __cond_lock(lock, rt_read_trylock(lock))
58+
#define read_trylock(lock) __cond_acquire(lock, rt_read_trylock(lock))
5959

6060
static __always_inline void read_unlock(rwlock_t *rwlock)
6161
{
@@ -111,7 +111,7 @@ static __always_inline void write_lock_irq(rwlock_t *rwlock)
111111
flags = 0; \
112112
} while (0)
113113

114-
#define write_trylock(lock) __cond_lock(lock, rt_write_trylock(lock))
114+
#define write_trylock(lock) __cond_acquire(lock, rt_write_trylock(lock))
115115

116116
#define write_trylock_irqsave(lock, flags) \
117117
({ \

include/linux/sched/signal.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -741,7 +741,7 @@ static inline struct sighand_struct *lock_task_sighand(struct task_struct *task,
741741
struct sighand_struct *ret;
742742

743743
ret = __lock_task_sighand(task, flags);
744-
(void)__cond_lock(&task->sighand->siglock, ret);
744+
(void)__cond_acquire(&task->sighand->siglock, ret);
745745
return ret;
746746
}
747747

include/linux/spinlock.h

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -212,7 +212,7 @@ static inline void do_raw_spin_unlock(raw_spinlock_t *lock) __releases(lock)
212212
* various methods are defined as nops in the case they are not
213213
* required.
214214
*/
215-
#define raw_spin_trylock(lock) __cond_lock(lock, _raw_spin_trylock(lock))
215+
#define raw_spin_trylock(lock) __cond_acquire(lock, _raw_spin_trylock(lock))
216216

217217
#define raw_spin_lock(lock) _raw_spin_lock(lock)
218218

@@ -284,7 +284,7 @@ static inline void do_raw_spin_unlock(raw_spinlock_t *lock) __releases(lock)
284284
#define raw_spin_unlock_bh(lock) _raw_spin_unlock_bh(lock)
285285

286286
#define raw_spin_trylock_bh(lock) \
287-
__cond_lock(lock, _raw_spin_trylock_bh(lock))
287+
__cond_acquire(lock, _raw_spin_trylock_bh(lock))
288288

289289
#define raw_spin_trylock_irq(lock) \
290290
({ \
@@ -499,21 +499,21 @@ static inline int rwlock_needbreak(rwlock_t *lock)
499499
*/
500500
extern int _atomic_dec_and_lock(atomic_t *atomic, spinlock_t *lock);
501501
#define atomic_dec_and_lock(atomic, lock) \
502-
__cond_lock(lock, _atomic_dec_and_lock(atomic, lock))
502+
__cond_acquire(lock, _atomic_dec_and_lock(atomic, lock))
503503

504504
extern int _atomic_dec_and_lock_irqsave(atomic_t *atomic, spinlock_t *lock,
505505
unsigned long *flags);
506506
#define atomic_dec_and_lock_irqsave(atomic, lock, flags) \
507-
__cond_lock(lock, _atomic_dec_and_lock_irqsave(atomic, lock, &(flags)))
507+
__cond_acquire(lock, _atomic_dec_and_lock_irqsave(atomic, lock, &(flags)))
508508

509509
extern int _atomic_dec_and_raw_lock(atomic_t *atomic, raw_spinlock_t *lock);
510510
#define atomic_dec_and_raw_lock(atomic, lock) \
511-
__cond_lock(lock, _atomic_dec_and_raw_lock(atomic, lock))
511+
__cond_acquire(lock, _atomic_dec_and_raw_lock(atomic, lock))
512512

513513
extern int _atomic_dec_and_raw_lock_irqsave(atomic_t *atomic, raw_spinlock_t *lock,
514514
unsigned long *flags);
515515
#define atomic_dec_and_raw_lock_irqsave(atomic, lock, flags) \
516-
__cond_lock(lock, _atomic_dec_and_raw_lock_irqsave(atomic, lock, &(flags)))
516+
__cond_acquire(lock, _atomic_dec_and_raw_lock_irqsave(atomic, lock, &(flags)))
517517

518518
int __alloc_bucket_spinlocks(spinlock_t **locks, unsigned int *lock_mask,
519519
size_t max_size, unsigned int cpu_mult,

include/linux/spinlock_rt.h

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -123,13 +123,13 @@ static __always_inline void spin_unlock_irqrestore(spinlock_t *lock,
123123
}
124124

125125
#define spin_trylock(lock) \
126-
__cond_lock(lock, rt_spin_trylock(lock))
126+
__cond_acquire(lock, rt_spin_trylock(lock))
127127

128128
#define spin_trylock_bh(lock) \
129-
__cond_lock(lock, rt_spin_trylock_bh(lock))
129+
__cond_acquire(lock, rt_spin_trylock_bh(lock))
130130

131131
#define spin_trylock_irq(lock) \
132-
__cond_lock(lock, rt_spin_trylock(lock))
132+
__cond_acquire(lock, rt_spin_trylock(lock))
133133

134134
#define spin_trylock_irqsave(lock, flags) \
135135
({ \

kernel/time/posix-timers.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,7 @@ static struct k_itimer *__lock_timer(timer_t timer_id, unsigned long *flags);
6363

6464
#define lock_timer(tid, flags) \
6565
({ struct k_itimer *__timr; \
66-
__cond_lock(&__timr->it_lock, __timr = __lock_timer(tid, flags)); \
66+
__cond_acquire(&__timr->it_lock, __timr = __lock_timer(tid, flags)); \
6767
__timr; \
6868
})
6969

0 commit comments

Comments
 (0)