dm thin metadata: remove in-core 'read_only' flag
authorMike Snitzer <snitzer@redhat.com>
Thu, 23 Apr 2015 19:06:27 +0000 (15:06 -0400)
committerMike Snitzer <snitzer@redhat.com>
Fri, 29 May 2015 18:18:59 +0000 (14:18 -0400)
Leverage the block manager's read_only flag instead of duplicating it;
access with new dm_bm_is_read_only() method.

Signed-off-by: Mike Snitzer <snitzer@redhat.com>
drivers/md/dm-thin-metadata.c
drivers/md/persistent-data/dm-block-manager.c
drivers/md/persistent-data/dm-block-manager.h

index 79f694120ddf067f327b3c05dda5e090af4a1c51..cb6dd055053d59da06e1e50ebe597522046d6e0e 100644 (file)
@@ -184,7 +184,6 @@ struct dm_pool_metadata {
        uint64_t trans_id;
        unsigned long flags;
        sector_t data_block_size;
-       bool read_only:1;
 
        /*
         * Set if a transaction has to be aborted but the attempt to roll back
@@ -836,7 +835,6 @@ struct dm_pool_metadata *dm_pool_metadata_open(struct block_device *bdev,
        init_rwsem(&pmd->root_lock);
        pmd->time = 0;
        INIT_LIST_HEAD(&pmd->thin_devices);
-       pmd->read_only = false;
        pmd->fail_io = false;
        pmd->bdev = bdev;
        pmd->data_block_size = data_block_size;
@@ -880,7 +878,7 @@ int dm_pool_metadata_close(struct dm_pool_metadata *pmd)
                return -EBUSY;
        }
 
-       if (!pmd->read_only && !pmd->fail_io) {
+       if (!dm_bm_is_read_only(pmd->bm) && !pmd->fail_io) {
                r = __commit_transaction(pmd);
                if (r < 0)
                        DMWARN("%s: __commit_transaction() failed, error = %d",
@@ -1739,7 +1737,6 @@ int dm_pool_resize_metadata_dev(struct dm_pool_metadata *pmd, dm_block_t new_cou
 void dm_pool_metadata_read_only(struct dm_pool_metadata *pmd)
 {
        down_write(&pmd->root_lock);
-       pmd->read_only = true;
        dm_bm_set_read_only(pmd->bm);
        up_write(&pmd->root_lock);
 }
@@ -1747,7 +1744,6 @@ void dm_pool_metadata_read_only(struct dm_pool_metadata *pmd)
 void dm_pool_metadata_read_write(struct dm_pool_metadata *pmd)
 {
        down_write(&pmd->root_lock);
-       pmd->read_only = false;
        dm_bm_set_read_write(pmd->bm);
        up_write(&pmd->root_lock);
 }
index 087411c95ffcb94805c4b3c860683615583244ab..4d6c9b689eaa73d4fccbe44991fc04913e413a8c 100644 (file)
@@ -609,6 +609,12 @@ void dm_bm_prefetch(struct dm_block_manager *bm, dm_block_t b)
        dm_bufio_prefetch(bm->bufio, b, 1);
 }
 
+bool dm_bm_is_read_only(struct dm_block_manager *bm)
+{
+       return bm->read_only;
+}
+EXPORT_SYMBOL_GPL(dm_bm_is_read_only);
+
 void dm_bm_set_read_only(struct dm_block_manager *bm)
 {
        bm->read_only = true;
index 1b95dfc1778640efc74114b2ee382a36d11ad9f1..84330f59886dbc272e894e044e2236296a5119f7 100644 (file)
@@ -123,6 +123,7 @@ void dm_bm_prefetch(struct dm_block_manager *bm, dm_block_t b);
  * Additionally you should not use dm_bm_unlock_move, however no error will
  * be returned if you do.
  */
+bool dm_bm_is_read_only(struct dm_block_manager *bm);
 void dm_bm_set_read_only(struct dm_block_manager *bm);
 void dm_bm_set_read_write(struct dm_block_manager *bm);