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

libblock: Remove superfluous volatile qualifier

Browse Source 
All these variables are protected by the bdbuf cache mutex.
This commit is contained in:
Sebastian Huber
2012-02-28 15:59:04 +01:00
parent a3170f70b6
commit c42b03f498
2 changed files with 9 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
cpukit/libblock/include/rtems/bdbuf.h
View File

@@ -321,13 +321,13 @@ typedef struct rtems_bdbuf_buffer
unsigned char* buffer; /**< Pointer to the buffer memory area */
volatile rtems_bdbuf_buf_state state; /**< State of the buffer. */
rtems_bdbuf_buf_state state; /**< State of the buffer. */
volatile uint32_t waiters; /**< The number of threads waiting on this
uint32_t waiters; /**< The number of threads waiting on this
* buffer. */
rtems_bdbuf_group* group; /**< Pointer to the group of BDs this BD is
* part of. */
volatile uint32_t hold_timer; /**< Timer to indicate how long a buffer
uint32_t hold_timer; /**< Timer to indicate how long a buffer
* has been held in the cache modified. */
int references; /**< Allow reference counting by owner. */

12
cpukit/libblock/src/bdbuf.c
View File

@@ -74,7 +74,7 @@ typedef struct rtems_bdbuf_swapout_worker
* idle. */
rtems_id id; /**< The id of the task so we can wake
* it. */
volatile bool enabled; /**< The worker is enabled. */
bool enabled; /**< The worker is enabled. */
rtems_bdbuf_swapout_transfer transfer; /**< The transfer data for this
* thread. */
} rtems_bdbuf_swapout_worker;
@@ -83,7 +83,7 @@ typedef struct rtems_bdbuf_swapout_worker
* Buffer waiters synchronization.
*/
typedef struct rtems_bdbuf_waiters {
volatile unsigned count;
unsigned count;
rtems_id sema;
} rtems_bdbuf_waiters;
@@ -93,7 +93,7 @@ typedef struct rtems_bdbuf_waiters {
typedef struct rtems_bdbuf_cache
{
rtems_id swapout; /**< Swapout task ID */
volatile bool swapout_enabled; /**< Swapout is only running if
bool swapout_enabled; /**< Swapout is only running if
* enabled. Set to false to kill the
* swap out task. It deletes itself. */
rtems_chain_control swapout_workers; /**< The work threads for the swapout
@@ -111,9 +111,9 @@ typedef struct rtems_bdbuf_cache
rtems_id lock; /**< The cache lock. It locks all
* cache data, BD and lists. */
rtems_id sync_lock; /**< Sync calls block writes. */
volatile bool sync_active; /**< True if a sync is active. */
volatile rtems_id sync_requester; /**< The sync requester. */
volatile dev_t sync_device; /**< The device to sync and
bool sync_active; /**< True if a sync is active. */
rtems_id sync_requester; /**< The sync requester. */
dev_t sync_device; /**< The device to sync and
* BDBUF_INVALID_DEV not a device
* sync. */