RTOS/rtems
1
0
Fork 0
forked from Imagelibrary/rtems
Code Pull Requests Activity

libio: Add assert to rtems_libio_iop_drop()

Browse Source 
This assert helps to detect an invalid reference counting in RTEMS_DEBUG
configurations.

Update #3132.
This commit is contained in:
Sebastian Huber
2017-11-15 13:21:01 +01:00
parent 62119d21da
commit d4b99ae3f8
1 changed files with 23 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

23
cpukit/libcsupport/include/rtems/libio_.h
View File

@@ -29,6 +29,7 @@
#include <rtems.h> #include <rtems.h>
#include <rtems/libio.h> #include <rtems/libio.h>
#include <rtems/seterr.h> #include <rtems/seterr.h>
#include <rtems/score/assert.h>
#ifdef __cplusplus #ifdef __cplusplus
extern "C" { extern "C" {
@@ -181,11 +182,33 @@ static inline unsigned int rtems_libio_iop_hold( rtems_libio_t *iop )
*/ */
static inline void rtems_libio_iop_drop( rtems_libio_t *iop ) static inline void rtems_libio_iop_drop( rtems_libio_t *iop )
{ {
#if defined(RTEMS_DEBUG)
unsigned int flags;
bool success;
flags = _Atomic_Load_uint( &iop->flags, ATOMIC_ORDER_RELAXED );
do {
unsigned int desired;
_Assert( flags >= LIBIO_FLAGS_REFERENCE_INC );
desired = flags - LIBIO_FLAGS_REFERENCE_INC;
success = _Atomic_Compare_exchange_uint(
&iop->flags,
&flags,
desired,
ATOMIC_ORDER_RELEASE,
ATOMIC_ORDER_RELAXED
);
} while ( !success );
#else
_Atomic_Fetch_sub_uint( _Atomic_Fetch_sub_uint(
&iop->flags, &iop->flags,
LIBIO_FLAGS_REFERENCE_INC, LIBIO_FLAGS_REFERENCE_INC,
ATOMIC_ORDER_RELEASE ATOMIC_ORDER_RELEASE
); );
#endif
} }
/* /*