> From: Mattias Rönnblom [mailto:mattias.ronnb...@ericsson.com] > Sent: Monday, 11 April 2022 17.16 > > Mark the trylock family of spinlock functions with > __rte_warn_unused_result. > > Acked-by: Bruce Richardson <bruce.richard...@intel.com> > > Signed-off-by: Mattias Rönnblom <mattias.ronnb...@ericsson.com> > --- > lib/eal/include/generic/rte_spinlock.h | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/lib/eal/include/generic/rte_spinlock.h > b/lib/eal/include/generic/rte_spinlock.h > index 40fe49d5ad..73ed4bfbdc 100644 > --- a/lib/eal/include/generic/rte_spinlock.h > +++ b/lib/eal/include/generic/rte_spinlock.h > @@ -97,6 +97,7 @@ rte_spinlock_unlock (rte_spinlock_t *sl) > * @return > * 1 if the lock is successfully taken; 0 otherwise. > */ > +__rte_warn_unused_result > static inline int > rte_spinlock_trylock (rte_spinlock_t *sl); > > @@ -174,6 +175,7 @@ rte_spinlock_unlock_tm(rte_spinlock_t *sl); > * 1 if the hardware memory transaction is successfully started > * or lock is successfully taken; 0 otherwise. > */ > +__rte_warn_unused_result > static inline int > rte_spinlock_trylock_tm(rte_spinlock_t *sl); > > @@ -243,6 +245,7 @@ static inline void > rte_spinlock_recursive_unlock(rte_spinlock_recursive_t *slr) > * @return > * 1 if the lock is successfully taken; 0 otherwise. > */ > +__rte_warn_unused_result > static inline int > rte_spinlock_recursive_trylock(rte_spinlock_recursive_t *slr) > { > int id = rte_gettid(); > @@ -299,6 +302,7 @@ static inline void > rte_spinlock_recursive_unlock_tm( > * 1 if the hardware memory transaction is successfully started > * or lock is successfully taken; 0 otherwise. > */ > +__rte_warn_unused_result > static inline int rte_spinlock_recursive_trylock_tm( > rte_spinlock_recursive_t *slr); > > -- > 2.25.1 >
Acked-by: Morten Brørup <m...@smartsharesystems.com>
