diff options
| author | Akinobu Mita <akinobu.mita@gmail.com> | 2011-08-02 12:32:01 +0100 |
|---|---|---|
| committer | Alasdair G Kergon <agk@redhat.com> | 2011-08-02 12:32:01 +0100 |
| commit | c8f543e0786785d8c7118fc2878e42bc0193a88d (patch) | |
| tree | b8984adc44c3a15f1bee56f3b6b9ff49f8ed4d2c | |
| parent | 936688d7eb0f39be96c5791be1a04994cc8d6aa0 (diff) | |
dm log: clean up bit little endian bitops
Using __test_and_{set,clear}_bit_le() with ignoring its return value
can be replaced with __{set,clear}_bit_le().
This also removes unnecessary casts.
Signed-off-by: Akinobu Mita <akinobu.mita@gmail.com>
Signed-off-by: Alasdair G Kergon <agk@redhat.com>
| -rw-r--r-- | drivers/md/dm-log.c | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/drivers/md/dm-log.c b/drivers/md/dm-log.c index 5f06fb687408..306ce123d6b4 100644 --- a/drivers/md/dm-log.c +++ b/drivers/md/dm-log.c @@ -257,20 +257,20 @@ struct log_c { */ static inline int log_test_bit(uint32_t *bs, unsigned bit) { - return test_bit_le(bit, (unsigned long *) bs) ? 1 : 0; + return test_bit_le(bit, bs) ? 1 : 0; } static inline void log_set_bit(struct log_c *l, uint32_t *bs, unsigned bit) { - __test_and_set_bit_le(bit, (unsigned long *) bs); + __set_bit_le(bit, bs); l->touched_cleaned = 1; } static inline void log_clear_bit(struct log_c *l, uint32_t *bs, unsigned bit) { - __test_and_clear_bit_le(bit, (unsigned long *) bs); + __clear_bit_le(bit, bs); l->touched_dirtied = 1; } @@ -745,8 +745,7 @@ static int core_get_resync_work(struct dm_dirty_log *log, region_t *region) return 0; do { - *region = find_next_zero_bit_le( - (unsigned long *) lc->sync_bits, + *region = find_next_zero_bit_le(lc->sync_bits, lc->region_count, lc->sync_search); lc->sync_search = *region + 1; |